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
ncm-directoryservices ncm-fmonagent ncm-mcx: Deprecate with warnings #1051
Conversation
@@ -24,5 +24,7 @@ type component_directoryservices = { | |||
"ldapv3" ? directoryservices_ldap_entry{} | |||
}; | |||
|
|||
bind "/software/components/directoryservices" = component_directoryservices; | |||
|
|||
bind "/software/components/directoryservices" = component_directoryservices with { |
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.
any reason why this is done at the bind, not at the type itself? (not that i care 😄 )
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.
It just feels more appropriate at the point it is actually used.
Is Jenkins okay? It seems like it's taking forever to build PRs at the moment. |
ran out of inodes, we're cleaning up atm |
retest this please |
🥗 ! |
@stdweird I'll let you merge this if you want to do the deprecation in this release cycle. |
Ok. |
c5db4d2
to
7f85a88
Compare
Discussed at March 2017 workshop... no longer deprecates syslogng as it is expected to live on for some time at RAL and MS. |
@stdweird this should be good to merge now |
Throw deprecation if the following components are initialised:
This is the first step towards #1027.