-
Notifications
You must be signed in to change notification settings - Fork 73
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 #21238: Rename "classic nodes" to "unix node" #4351
Fixes #21238: Rename "classic nodes" to "unix node" #4351
Conversation
I think this should be validated by a thread in produit. The terminology and semantics should be agreed on and shared throughout the team. |
PR updated with a new commit |
webapp/sources/rudder/rudder-core/src/main/scala/com/normation/rudder/ncf/TechniqueWriter.scala
Outdated
Show resolved
Hide resolved
PR updated with a new commit |
4 similar comments
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
PR updated with a new commit |
2 similar comments
PR updated with a new commit |
PR updated with a new commit |
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.
OK to me, great enhancement. Update of LDAP works as intended
This PR is not mergeable to upper versions. |
fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! fixup! fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! fixup! fixup! fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! fixup! fixup! fixup! fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\" fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! fixup! Fixes #21238: Rename \"classic nodes\" to \"unix node\" Fixes #21238: Rename \"classic nodes\" to \"unix node\"
PR rebased |
bee2cc9
to
cf0fbfa
Compare
OK, merging this PR |
override def toRulesPath = "/dsc" | ||
override def inventoryAgentNames = Set("dsc") | ||
override val inventorySoftwareName = "Rudder agent (DSC)" | ||
override val inventorySoftwareName = "Rudder agent (Windows)" |
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.
this break detection of windows version
https://issues.rudder.io/issues/21238
![image](https://user-images.githubusercontent.com/23410978/176430436-d3e55948-bf8d-4c20-9ef5-e941a5ea9a3f.png)
![image](https://user-images.githubusercontent.com/23410978/176430505-404bee1b-62e6-4f1e-b053-e13acc2dca1b.png)