View Issue Details

IDProjectCategoryView StatusLast Update
000141310000-009: Alarms and Conditionspublic2011-05-25 13:57
ReporterMatthias Damm Assigned ToPaul Hunkar  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Summary0001413: Definition of Comment field of ConditionType is too strict
Description

Part 9 defines for the Comment field in ConditionType:
"Comment contains the last comment provided for a certain state (ConditionBranch). It may have been provided by an AddComment Method or some other Method. The initial value of this Variable is null. If a Method provides as an option the ability to set a Comment, then the value of this variable is reset to null if an optional comment is not provided."

An implementer of an OPC UA Alarms & Conditions server complains that

  • his own system is also providing comments and that an empty comment in an OPC call should not reset the comment field provided by the underlying system and that
  • the initial value can not be Null if his unterlying system provided already a comment
  • underlying systems my provide comments on a different level (object) and the current definition makes it hard to map this functionality to OPC UA
  • empty comments should not reset comments - we should provide a seperate method like removeComment() to force this from a client
TagsNo tags attached.
Commit Version
Fix Due Date

Activities

Randy Armstrong

2010-11-16 17:58

administrator   ~0002134

Remove the restriction on 'initial value'.

Change "provided by an AddComment Method or some other Method" to provided by an AddComment Method or some other mechanism".

Make it clear that an AddComment can set the comment to null. Any other mechanism such as Acknowlegde does not change the the comment if the value provided is null or empty.

Paul Hunkar

2011-03-09 07:22

developer   ~0002355

Added new methods AcknowledgeNoComment and ConfirmNoComment which allow for acknowledging or confirming an alarm without altering an existing comment. (Changing the existing method would result in a functional change that would affect existing systems.) Added text indicating that the comment maybe provided by other mechanisms and that these mechanism may provide an initial comment with the Alarm.

Paul Hunkar

2011-04-05 17:56

developer   ~0002588

In call discussion resulted in "no fix" as a solution (spec will be updated to remove the added text)

The alarm comment is to describe the current state of the alarm, an acknowledge or confirms affects the current state - so an ack or confirm changes this current state and the comment should be wiped or updated.

If a user wants additional behaviour they can always add methods in there custom system, but the system must still handle the alarm Acknowledge and confirm as described in the system.

Paul Hunkar

2011-05-24 13:20

developer   ~0002777

Updated the specification to indicate how the method could be called with out providing a comment and how to call the method with an empty comment to reset the comment field.

Randy Armstrong

2011-05-25 13:57

administrator   ~0002785

Discussed in Walldorf

Issue History

Date Modified Username Field Change
2010-11-16 16:35 Matthias Damm New Issue
2010-11-16 16:39 Matthias Damm Description Updated
2010-11-16 17:58 Randy Armstrong Note Added: 0002134
2010-11-16 17:58 Randy Armstrong Status new => assigned
2010-11-16 17:58 Randy Armstrong Assigned To => Paul Hunkar
2011-03-09 07:22 Paul Hunkar Status assigned => resolved
2011-03-09 07:22 Paul Hunkar Resolution open => fixed
2011-03-09 07:22 Paul Hunkar Note Added: 0002355
2011-04-05 17:56 Paul Hunkar Status resolved => feedback
2011-04-05 17:56 Paul Hunkar Resolution fixed => reopened
2011-04-05 17:56 Paul Hunkar Note Added: 0002588
2011-05-24 13:20 Paul Hunkar Status feedback => resolved
2011-05-24 13:20 Paul Hunkar Resolution reopened => fixed
2011-05-24 13:20 Paul Hunkar Note Added: 0002777
2011-05-25 13:56 Randy Armstrong Issue cloned: 0001657
2011-05-25 13:57 Randy Armstrong Status resolved => closed
2011-05-25 13:57 Randy Armstrong Note Added: 0002785