Closed dth202 closed 3 years ago
@dth202 could you update commit message to comply with https://www.conventionalcommits.org/en/v1.0.0/#summary
:tada: This PR is included in version 1.9.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:
Apologies for the excessive delay, @dth202 -- finally got around to reviewing and finalising this.
Thanks for the prompt, @noelmcloughlin.
Thanks @myii @dth202
https://docs.saltproject.io/en/latest/ref/configuration/minion.html#master-type
That page explains all valid values except "str". What does "str" mean?
That page explains all valid values except "str". What does "str" mean?
@noelmcloughlin Good question, I've never changed this option myself. Here's a little digging down.
https://github.com/saltstack/salt/pull/25703/files
standard
at one point.https://github.com/saltstack/salt/blob/8ce39ddb2594358d47e24628af85a9b260ff1fe1/conf/minion#L39-L47 https://github.com/saltstack/salt/blob/8ce39ddb2594358d47e24628af85a9b260ff1fe1/conf/proxy#L61-L67
Multiple hot masters are configured by setting this value to "str".
https://github.com/saltstack/salt/issues/42803
PR progress checklist (to be filled in by reviewers)
What type of PR is this?
Primary type
[build]
Changes related to the build system[chore]
Changes to the build process or auxiliary tools and libraries such as documentation generation[ci]
Changes to the continuous integration configuration[feat]
A new feature[fix]
A bug fix[perf]
A code change that improves performance[refactor]
A code change that neither fixes a bug nor adds a feature[revert]
A change used to revert a previous commit[style]
Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)Secondary type
[docs]
Documentation changes[test]
Adding missing or correcting existing testsDoes this PR introduce a
BREAKING CHANGE
?No.
Related issues and/or pull requests
Describe the changes you're proposing
Change salt:minion:master_type from 'true' to 'str' in defaults.yaml. The default value for the salt-minion config is str according to salt documentation, but the value in defaults.yaml is true. Setting master_type to true in the salt-minion config causes an error the salt-minion starts
Pillar / config required to test the proposed changes
Debug log showing how the proposed changes work
Documentation checklist
README
(e.g.Available states
).pillar.example
.Testing checklist
state_top
).Additional context