Skip to content
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 #5843: Add user defined (via API) node properties #671

Conversation

fanf
Copy link
Member

@fanf fanf commented Nov 28, 2014

No description provided.

@fanf fanf force-pushed the ust_5506/impl_5843/add_node_property_key_value_pair branch from d8ee35b to 2e75b81 Compare November 28, 2014 09:16
@fanf fanf force-pushed the ust_5506/impl_5843/add_node_property_key_value_pair branch from 2e75b81 to e154e33 Compare November 28, 2014 09:18
, isBroken : Boolean
, isSystem : Boolean
, isPolicyServer : Boolean
, creationDate : DateTime
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why to you add the creationDate ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be able to have it in display where we don't need nodeInfo (because we also have inventory)

@ncharles
Copy link
Member

looks ok, aside from the few questions which I would like adressed

ncharles added a commit that referenced this pull request Nov 28, 2014
…y_key_value_pair

Fixes #5843: Add user defined (via API) node properties
@ncharles ncharles merged commit 9596dfe into Normation:branches/rudder/3.0 Nov 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants