-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
⭐️ new OPC UA resource pack #1224
Conversation
This comment has been minimized.
This comment has been minimized.
ffc07a1
to
8109e21
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Thanks @chris-rock
Only some minor comments.
I tried to test this locally, with the Azure container image, but got an error:
The address is the one used inside the container:
To run the container, I used the command from the README in this PR. Any ideas? |
@czunker Yes, the problem is that the azure opc service returns a resolved url. This essentially highlights that the OPC UA endpoint discovery cannot be trusted for the initial connection. I adjust the code. |
862672e
to
ea73194
Compare
ea73194
to
8243e4a
Compare
Works now:
Thanks @chris-rock |
The new OPC UA resource pack allows you to connect to OPC UA server and to query the tree easily via MQL:
Follow ups that need to be implemented on top of this PR: