* Catch exception raised from invalid verify_options
This fails the state gracefully in these cases.
Refs: 6b97161293 (r63577507)
* Raise an exception when package verification fails
* Catch failed package verification commands
Also, since pkg.verify is run separately on each targeted package, don't
add errors for each failure, just add one error to what we report.
* Update rpm test_verify test to reflect change from cmd.run to cmd.run_all
cron.file() writes the crontab out to a temporary file before comparing
with the source. Asserting file ownership and permissions in this
context creates sometimes false positives.
This change does not affect highstate without test=True
As also noted in https://github.com/saltstack/salt/issues/33266, sometimes `dmidecode` will output additional commentary text in strings:
```
root@n01:~# dmidecode -s system-manufacturer
# SMBIOS implementations newer than version 2.8 are not
# fully supported by this version of dmidecode.
Supermicro
```
This PR filters those out.
Function is expecting two arguments. But after processing state only one
parameter (it contain two words) is pushed into list of options.
This patch reverts the parsing of "iptables --match-set" to the
behaviour of salt v2014.*.
This is a backport of https://github.com/saltstack/salt/pull/28325Closes#23643
As also noted in https://github.com/saltstack/salt/issues/33266, sometimes `dmidecode` will output additional commentary text in strings:
```
root@n01:~# dmidecode -s system-manufacturer
# SMBIOS implementations newer than version 2.8 are not
# fully supported by this version of dmidecode.
Supermicro
```
This PR filters those out.
* Added support for the agentv2
* Updated os field to match the SD API requirement
* filename is an absolute path on official installations
* Install script URL uses a redirect now, so we should instruct curl to follow them
* Added a missing import
* Updated the settings for the v2 agent
* Fixed pep8 issues
* select v2 agent for the test
Function is expecting two arguments. But after processing state only one
parameter (it contain two words) is pushed into list of options.
This patch reverts the parsing of "iptables --match-set" to the behaviour
of salt v2014.*.
This is a backport of https://github.com/saltstack/salt/pull/28325Closes#23643
* Pass an actual error message when raising exception
* debian_service.py: ignore retcode when checking service's status
This prevents spurious errors from being logged when a nonexistant
service is polled, or the service is not running.
* freebsdservice.py: ignore retcode when checking service's status
This prevents spurious errors from being logged when a nonexistant
service is polled, or the service is not running.
* netbsdservice.py: ignore retcode when checking service's status
This prevents spurious errors from being logged when a nonexistant
service is polled, or the service is not running.
* openbsdservice.py: ignore retcode when checking service's status
This prevents spurious errors from being logged when a nonexistant
service is polled, or the service is not running.
* Fix context clash.
boto_vpc uses both boto and boto3. Both utils for those use the same name for their connection context. If you used states that required both boto and boto3, the get_conn would return the object.
* Add a unit test to the boto context conflict problem
* Fix pylint errors and ensure new test to runs correctly
Merges #32985
* Fix typo
Docs were added for:
- master_tries
- mine_enabled
- mine_return_job
- syndic_failover
- auth_tries: this is an old setting, but docs were not found
in the minion.rst docs. This change should be added to older branches.
* better support for named images, locations, sizes
* better support for named images, locations, sizes
* Tests for Vultr driver
* Add sleep so test instance will be old enough to delete as part of the test
* Lint.
