View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0007142 | NodeSets, XSDs and Generated Code | Api Change | public | 2021-07-27 16:57 | 2021-11-17 18:27 |
Reporter | Jim Luth | Assigned To | Randy Armstrong | ||
Priority | normal | Severity | feature | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Fixed in Version | 1.05.01 RC1 | ||||
Summary | 0007142: OPC UA Part 9. Alarm shelving methods should support "Comment" parameter | ||||
Description | TimedShelve,OneShotShelve and Unshelve methods should support "Comment" parameter. | ||||
Tags | No tags attached. | ||||
Commit Version | |||||
Fix Due Date | |||||
related to | 0004130 | closed | Paul Hunkar | 10000-009: Alarms and Conditions | OPC UA Part 9. Alarm shelving methods should support "Comment" parameter |
|
Other methods should support comment as well: Suppress, Unsuppress, Reset, ... etc. |
|
There is an AddComment method that can be called at any time to add a comment to the alarm. This can be called prior to a shelving method or after the shelving method is invoked, in any case the comment will be stored as part of the event. Changing the signatures of the methods would be a breaking changes and likely not supported by both client and server applications. Is there a reason that this method is not sufficient? |
|
The Signature of AddComment has an EventId. 1.) The first step I guess is to define, when is a condition in an interesting state. At least regarding the OPC UA standard fields. Commenting with the current specification is complicated. The above are just some suggestions, that would make commenting easier. |
|
After UA meeting discussion - add optional comment to all method that do not have a comment field - include description that old system might return errors and what behavior a client should have in this case |
|
decision in UA meeting - add a version 2 of all methods that do not support comments, that includes a comment |
|
Agree in call that addition of new method to statemachine is not a breaking change to statemachine |
|
UPdated specification to add all of the required method as version 2 |
|
Agreed to changes in telecon. |
|
Added new methods to 1.05.01 |
|
Reviewed with Randy -- agreed to close. |
Date Modified | Username | Field | Change |
---|---|---|---|
2021-07-27 16:57 | Jim Luth | New Issue | |
2021-07-27 16:57 | Jim Luth | Status | new => assigned |
2021-07-27 16:57 | Jim Luth | Assigned To | => Randy Armstrong |
2021-07-27 16:57 | Jim Luth | Issue generated from: 0004130 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014702 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014703 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014704 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014705 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014706 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014707 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014708 | |
2021-07-27 16:57 | Jim Luth | Note Added: 0014709 | |
2021-07-27 16:57 | Jim Luth | Relationship added | related to 0004130 |
2021-07-27 16:57 | Jim Luth | Project | 10000-009: Alarms and Conditions => NodeSets, XSDs and Generated Code |
2021-07-27 16:57 | Jim Luth | Category | Spec => Api Change |
2021-07-27 16:59 | Jim Luth | Product Version | => 1.05 |
2021-07-27 16:59 | Jim Luth | Target Version | => 1.05 |
2021-11-16 22:42 | Randy Armstrong | Status | assigned => resolved |
2021-11-16 22:42 | Randy Armstrong | Resolution | open => fixed |
2021-11-16 22:42 | Randy Armstrong | Note Added: 0015330 | |
2021-11-17 18:27 | Jim Luth | Status | resolved => closed |
2021-11-17 18:27 | Jim Luth | Fixed in Version | => 1.05.01 RC1 |
2021-11-17 18:27 | Jim Luth | Note Added: 0015346 |