saltstack-formulas / salt-formula

Yes, Salt can Salt itself!
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
197 stars 423 forks source link

fix(log_fmt_console): double option in config #517

Closed ze42 closed 2 years ago

ze42 commented 2 years ago

When option was specified, it appeared twice in config, making configuration file buggy, and not loaded.

PR progress checklist (to be filled in by reviewers)


What type of PR is this?

Primary type

Secondary type

Does this PR introduce a BREAKING CHANGE?

No.

Related issues and/or pull requests

Describe the changes you're proposing

log_fmt_console can now be specified without breaking the generated config files.

Pillar / config required to test the proposed changes

salt:
  master:
    log_fmt_console: '%(colorlevel)s %(colormsg)s'
  minion:
    log_fmt_console: '%(colorlevel)s %(colormsg)s'

Debug log showing how the proposed changes work

Documentation checklist

Testing checklist

Additional context

noelmcloughlin commented 2 years ago

Thanks @ze42 and @aboe76 for review

saltstack-formulas-travis commented 2 years ago

:tada: This PR is included in version 1.10.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: