View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002137 | 10000-004: Services | public | 2012-07-26 10:14 | 2012-08-16 18:48 | |
Reporter | Thomas Merk | Assigned To | Paul Hunkar | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.01 | ||||
Fixed in Version | 1.02 | ||||
Summary | 0002137: isOnline parameter in RegisterServer changed semantically | ||||
Description | In 1.01 a manually launched server had to register with isOnline set to false on shutdown. A server implemented with specification 1.01 will still register as offline on shutdown. In that case discovery server returns this server in the FindServers service even if not running and able to accepting connections (it is still a server that needs manual launch). The problem is connected to issue 1796 and 1919. | ||||
Tags | No tags attached. | ||||
Commit Version | |||||
Fix Due Date | |||||
related to | 0002173 | closed | Randy Armstrong | 10000-012: Discovery | isOnline parameter in RegisterServer changed semantically |
|
This particular issue is something that was recently found in the lab, and we also need the clarification. |
|
Discussed in telco today. During the implementation of the LDS and work on the discovery specification the use of the isOnline flag was changed in OPC UA part 12. This happened right after release of OPC UA Part 4 1.01 We changed the specification for 1.02 because all implementations working with the discovery server of the OPC Foundation need to work like described in Part 12 or now in OPC UA Part 4 version 1.02. This was the reason why we did this semantic change since this change was applied to applications working with the LDS since a long time. |
|
I still do not think that the new specification is a good solution. Beside that also servers implementing 1.02 are not able to notify discovery server that the server is shutting down. |
|
I agree with Thomas. I understand the intent and also the desire to maintain backward compatibility, but I would hate to be a vendor receiving calls as to why Clients can't connect to their servers even though their servers are listed as available and causing that customer to waste time troubleshooting; and once word gets out that this is the intended behavior then we'll get a poke in the eye. I realize how akward this is becuase of on-demand servers that are currently offline and how they would not be polling/registering with the LDS to set IsOnline=True.I get that. Part 12 (I'm looking at v1.02 DRAFT 7) discusses the "semaphore file" 4 times, and it is obviously an important concept, but there are no instructions/guidelines about this file, and there are no references for more information. It seems that this file is intended to be what the IsOnline parameter used to be. Is that correct? |
|
A suggestion for change: Combine the isOnline flag with the semaphoreFilePath. The usage of an optional semaphoreFilePath to identify the server instance is not needed since the mandatory serverUri can be used to identify the server instance. semaphoreFilePath: isOnline: Figure 11 would be reverted to the figure in version 1.01. This would still mean a little semantic change - at least offline registration of manually launched servers ist still possible. The usage of semaphore files in version 1.01 is not clearly stated - especially for manually lauched servers. Since the server shall register offline on shutdown a semaphore file is not needed at all for those servers. |
|
We decided in the phone conference on August 7th, 2012 to not change the specification again. |
|
Agreed to reopen issue in telecon 2012-08-14. Paul agreed to edit the changes into the final 1.02 version for publication. The agreement is to separate the use of IsOnline from the use of the semaphore file. |
|
Fixed in version "OPC UA Part 4 - Services 1.02 Specification+PEH.doc" |
|
Pauls edits and some additional edits by me made it into the final 1.02 version. |
Date Modified | Username | Field | Change |
---|---|---|---|
2012-07-26 10:14 | Thomas Merk | New Issue | |
2012-07-26 19:13 |
|
Note Added: 0003924 | |
2012-07-31 16:33 | Matthias Damm | Status | new => assigned |
2012-07-31 16:33 | Matthias Damm | Assigned To | => Matthias Damm |
2012-07-31 16:34 | Matthias Damm | Status | assigned => resolved |
2012-07-31 16:34 | Matthias Damm | Resolution | open => won't fix |
2012-07-31 16:34 | Matthias Damm | Note Added: 0003937 | |
2012-08-01 06:56 | Thomas Merk | Status | resolved => feedback |
2012-08-01 06:56 | Thomas Merk | Resolution | won't fix => reopened |
2012-08-01 06:56 | Thomas Merk | Note Added: 0003938 | |
2012-08-01 15:04 |
|
Note Added: 0003941 | |
2012-08-02 09:26 | Thomas Merk | Note Added: 0003942 | |
2012-08-10 09:10 | Matthias Damm | Status | feedback => resolved |
2012-08-10 09:10 | Matthias Damm | Resolution | reopened => won't fix |
2012-08-10 09:10 | Matthias Damm | Note Added: 0003974 | |
2012-08-16 18:43 | Jim Luth | Assigned To | Matthias Damm => Paul Hunkar |
2012-08-16 18:43 | Jim Luth | Status | resolved => feedback |
2012-08-16 18:43 | Jim Luth | Resolution | won't fix => reopened |
2012-08-16 18:43 | Jim Luth | Note Added: 0003977 | |
2012-08-16 18:44 | Jim Luth | Status | feedback => resolved |
2012-08-16 18:44 | Jim Luth | Fixed in Version | => 1.02 |
2012-08-16 18:44 | Jim Luth | Resolution | reopened => fixed |
2012-08-16 18:44 | Jim Luth | Note Added: 0003978 | |
2012-08-16 18:45 | Jim Luth | Status | resolved => closed |
2012-08-16 18:45 | Jim Luth | Note Added: 0003979 | |
2012-08-16 18:46 | Jim Luth | Issue cloned: 0002173 | |
2012-08-16 18:46 | Jim Luth | Relationship added | related to 0002173 |
2012-08-16 18:48 | Jim Luth | Issue cloned: 0002174 |