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
Fixes #8755: Document node properties #209
Fixes #8755: Document node properties #209
Conversation
|
||
---- | ||
|
||
${node.properties[mykey]} |
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.
Please use real-life examples wherever possible. "datacenter=Paris" is a good example (here, just replace "mykey" by "datacenter" twice)
Commit modified |
2d0c296
to
04f588f
Compare
|
||
Node properties can be found in the "properties" tab of each node in Rudder. | ||
|
||
Node properties can be modified using Rudder API, see http://www.rudder-project.org/rudder-api-doc/#api-Nodes-updateNodeProperties |
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.
Does this appear as a link when the docs are built?
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.
If not, please make it a link.
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.
I don't know
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.
I cannot generate the doc at the moment, but it seems to work for other links in the documentation.
Commit modified |
04f588f
to
8707c30
Compare
|
||
---- | ||
|
||
WARNING: currently node properties can not be used inside ${eval } JavaScript blocks, since they are evaluated during poliocy generation and node properties are only made available in the agents. A workaround for this will be provided in a future version of Rudder. |
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.
Oops, I made a typo, sorry: s/poliocy/policy/
This looks good to me apart from that one typo. Please fix and self-merge. |
Commit modified |
8707c30
to
d926ab4
Compare
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/8755