saltstack/salt
1
0
Fork 0
mirror of https://github.com/saltstack/salt.git synced 2025-04-17 10:10:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Change deprecation warning to debug logging

Browse source 
This prevents a confusing warning from being logged to the console (and
to the log file upon each salt-minion daemon restart).

It seems that the intent of the deprecation warning was to use the logic
in `warn_until` to force us to make the final deprecation when the
Magnesium release cycle arrives, but when we do these sort of
deprecations we are typically pro-active and do a grep on the release
codename. So, keeping "Magnesium" in the log message should be
sufficient to remind us of this.
This commit is contained in:
Erik Johnson 2018-05-18 09:59:12 -05:00
parent ceb6e10f87
commit 89b3070d4c
No known key found for this signature in database
GPG key ID: 5E5583C437808F3F
1 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
salt/fileclient.py
View file

@ -1391,14 +1391,12 @@ class RemoteClient(Client):
'''
Return the metadata derived from the master_tops system
'''
salt.utils.versions.warn_until(
'Magnesium',
'The _ext_nodes master function has '
'been renamed to _master_tops. To ensure '
'compatibility when using older Salt masters '
'we continue to pass the function as _ext_nodes.'
log.debug(
'The _ext_nodes master function has been renamed to _master_tops. '
'To ensure compatibility when using older Salt masters we will '
'continue to invoke the function as _ext_nodes until the '
'Magnesium release.'
)
# TODO: Change back to _master_tops
# for Magnesium release
load = {'cmd': '_ext_nodes',