Project

General

Profile

Task #2010

Remove salt warnings.

Added by Jan Iversen over 1 year ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
Team - Pool
Start date:
Due date:
% Done:

100%

Estimated time:
Tags:
URL:

Description

Currently a salt run gives:

[WARNING ] Top file merge strategy set to 'merge' and multiple top files found. Top file merging order is undefined; for better results use 'same' option
[WARNING ] State for file: /root/.setup_state/initial - Neither 'source' nor 'contents' nor 'contents_pillar' nor 'contents_grains' was defined, yet 'replace' was set to 'True'. As there is no source to replace the file with, 'replace' has been set to 'False' to avoid reading the file unnecessarily.
[WARNING ] /usr/lib/python2.7/dist-packages/salt/utils/templates.py:73: DeprecationWarning: Starting in 2015.5, cmd.run uses python_shell=False by default, which doesn't support shellisms (pipes, env variables, etc). cmd.run is currently aliased to cmd.shell to prevent breakage. Please switch to cmd.shell or set python_shell=True to avoid breakage in the future, when this aliasing is removed.

[WARNING ] /usr/lib/python2.7/dist-packages/salt/states/git.py:460: DeprecationWarning: The 'force' argument to the git.latest state has been deprecated, please use 'force_clone' instead.

[WARNING ] State for file: /root/.byobu/.always-select - Neither 'source' nor 'contents' nor 'contents_pillar' nor 'contents_grains' was defined, yet 'replace' was set to 'True'. As there is no source to replace the file with, 'replace' has been set to 'False' to avoid reading the file unnecessarily.
[WARNING ] State for file: /etc/postfix/main.cf - Neither 'source' nor 'contents' nor 'contents_pillar' nor 'contents_grains' was defined, yet 'replace' was set to 'True'. As there is no source to replace the file with, 'replace' has been set to 'False' to avoid reading the file unnecessarily.
~

We should change that, so that the run is normally free of warnings.

History

#1 Updated by Florian Effenberger over 1 year ago

Who's gonna look into that and how urgent is this?

#2 Updated by Jan Iversen over 1 year ago

  • Assignee set to Jan Iversen

Not urgent, I will look into it, once floyd is cleaned up.

#3 Updated by Florian Effenberger over 1 year ago

  • Target version set to Pool

#4 Updated by Florian Effenberger 7 months ago

  • Assignee changed from Jan Iversen to Guilhem Moulin

Re-assigning to Guilhem to see if this is still relevant
Guilhem, if so, please prioritize accordingly (given the age of the ticket, probably Pool and Low) :-)

#5 Updated by Guilhem Moulin 7 months ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

Aha, I wasn't aware of this ticket. The warnings also bothered me, and I remember doing some tweaking a while back to get rid of them (not sure what anymore, though :-/)

Also available in: Atom PDF