Currently the example top file under "Matching Grains in the Top File" has
node_type grains whose values do not match the names of the states which
they specify. (e.g. the `webserver` node_type specifies the `web` state).
Later these values are templated using jinja templates as if they are identical.
This commit adjusts the values in the example to be identical, so that the
template example follows naturally.
Also added the loop_interval docs to minion.rst and corrected
the default documented (this value is 1 second for the minion
schedule maintenance process, not 60 seconds - the master is 60).
In some cases, minions do not need to return a job for a master,
such as installations where jobs are run asynchronously and a 3rd-party
system evalutes a job cache to determine the results of a job.
* Ensure rh_service not used on CloudLinux 7
Add CloudLinux to RHEL-derived distros excluded from rh_service use in osrelease >= 7
* Fix binary search and replace (#32542)
* Don't return None from eval_master (#32555)
Raise an exception instead. Because eval master if returns should return
a tuple.
* redact passwords and hashes from user.present updates
Fixes#32381
* Better log message on minion restart if master couldn't be reached. (#32576)
* Revert PR #32480 and apply #32314 with fixes / documentation (#32558)
* Revert "Fix loop in maint.flo"
This reverts commit 5196cd6a6e5db3c7b1a47b1740881bbd3e87ea3d.
* Revert "Clear VCS fsbackend and git_pillar locks on master start"
This reverts commit 7e3caa9bae1ac4de62db9924374e35a8b826937e.
* Revert "Add functions to remove VCS fsbackend update locks and git_pillar update/checkout locks"
This reverts commit 4c2db32419022501eae2a695ec488693e043d189.
* prevent eternal gitfs lock due to process crash
* Use salt.utils.fopen() instead of open()
* Make pid locking work for more than just gitfs
Also, make logging more descriptive, to aid in troubleshooting.
* Add git_pillar_global_lock config option default value
* Document proper usage of {gitfs,git_pillar}_global_lock
* Fix comments value in salt.states.pkgrepo example (#32604)
'comments' option adds '#' automatically. Example contains `#http://mirror.centos.org/centos/$releasever/os/$basearch/` string which becomes prefixed with '##' in generated file.
* alphabetize directories for dynamic modules (#32599)
Also add engines and proxy minions to the list.
* Expand on the open-source vs open-core FAQ
* Language clarification.
* Fix some mistakes in the salt-ssh thin shell script (#32583)
* [[ is bash, not compatible with /bin/sh
* check if python command exists before calling it
* Deprecate 'user' and 'group' in state cmd (#32613)
* Remove unused 'group' argument
* Fix unit testing of cmd.mod_run_check without group arg
* Deprecate 'user/group' in cmd.run
* Deprecate 'user'/'group' in cmd.script
* Deprecate 'user' in cmd.wait
* Deprecate 'user'/'group' in cmd.wait_script
* Fix mod_run_check without 'group'
* Push deprecation back one release
* Fix mac_service and mac_system modules (#32587)
* Fix mac_service module
* Add integration tests for new functions
* Start will not enable the service beforehand
* Remove unused variables
* ip_protocol.isdigit() fails when isinstance(ip_protocol, int)
* Fix """Warnings: 'name' is an invalid keyword argument for
'boto_iam.account_policy'. If you were trying to pass additional
data to be used in a template context, please populate 'context'
with 'key: value' pairs. Your approach will work until Salt Carbon
is out. Please update your state files."""
* Add boto_iam.get_all_users() module function
* Add boto_iam.get_all_groups() module function
* Update boto_iam.export_users() to use new get_all_users() function
* Add boto_iam.get_all_roles() module function
* Finally got around to implementing image_name option to boto_ec2.instance_present :)
* Add boto_secgroup.get_all_security_groups() function
Add options public_ip, allocation_id, and allocate_eip to boto_ec2.instance_exists()
Add option release_eip to boto_ec2.instance_absent()
* Boy, I REALLY disagree with PEP about the whole "whitespace after/before braces/brackets/parens" readability thing....
* INFRA-856 - add 'private:<Name tag>' and 'public:<Name tag>' flavors to boto_route53.present() 'value' option to allow route53 to automatically figure out IPs of freshly created instances
* INFRA-856 - add in_states argument so we only get back useful instances
* INFRA-856 - ...aaaaand boto arbitrarily maps PublicIpAddress to ip_address, while mapping EVERY OTHER instance attribute to it's immediate CamelCase->snake_case equivalents. Helpful? Not! Brilliant? NOT!
* Enable blocking on min_party until minimum set of nodes are available
* Fix the comment in min_party for salt-pr-lint-n
* Add test cases to the min_party method
* Fix the lint issues causing build failure
* Change the comment for test_min_party
* CR comments for modules/zk_concurrency.py
* Adding more examples for blocking/min_nodes
* Change the comment formatting
* Removing space at end of line
* Comments changes to rebuild
* i32571-winrepo: `saltenv` is required to lookup the cache file
* `win_pkg` module: fix pep8 errors
* i32571-winrepo: purged/remove also need to know the `saltenv`
* Add minion.restart
* Improve minion.kill for use by minion.restart
+ Add "retcode" status
+ Wait until the process dies (add timeout value)
+ Provide meaningful "comment" information when appropriate
* Add minion.restart: kills minion with minion.kill and then starts a new minion process
+ Can provide a specific ``minion_restart_command`` arg vector in minion configuration
+ If no ``minion_restart_command`` is available then uses ``argv`` of current process
+ If minion is not daemonized then it will kill but not restart - allows for
direct management of minion by init, systemd or other direct process
manager.
* Add EX_TEMPFAIL to exitcodes for timeout situtations (used by minion.kill)
TODO: integration tests
* Appease the pylint gods.
* Tweak the returned data structure for minion.kill.
* Seed the ``minion.kill`` ``killed`` entry with ``None.``
* add basic runner caller to thorium
* add basic wheel module
* only start the reactor if it is configured and add it correctly to the engines
* fix lint and a syntax error I forget to get into the PR
* add break to speed things up a bit
* bah! whitespace!
This is required for test.ping to return True with `multiprocessing: True`
caused by the fact that the ssh connection has to be reconnected each
time the process is forked.