* Apply fix from #38705 to 2016.3 branch
This is a better fix and covers more use cases than the sudo_user one.
* Remove saltenv param from internal state func call
This was probably redundant in the first place, but since state.sls,
state.highstate, etc. accept a saltenv param and the actual state
functions do not, this results in multiple values passed for the saltenv
param.
Remove this argument and let file.get_managed reference __env__
internally.
* Update archive tests to match 2016.11 branch
The IDE entry needs to be a dictionary entry. Without this, it is seen
as a string. This is probably done this way so that in the future,
configurations can be added to the IDE device.
The minion_ids var that was passed into the function was being
overwritten by a list of ALL the minion ids found in the minion cache
Fixes: #38003
ZD1187
* ignore ADMX/ADML files with syntax errors
* narrow except for XML Syntax Error on admx file
* attempt to remove any "encoding='unicode'" from ADMX/ADML files if lxml does not support the encoding
* lint fixes
* fix bonehead typo
handle scenario that gpedit has not been run on a system (Machine and User GroupPolicy sub-folders do not exist)
handle scenario that startup/shutdown script policy file does not exist
* Don't abort pillar.get with merge=True if default is None
This prevents a SaltInvocationError from being raised if the default is
loaded from a import_yaml jinja macro and the file being loaded is empty
(which is an entirely valid use case). In these cases, the loaded
default would be None, which we don't want to cause an exception.
* Add test case for issue 39062
This keeps trimmed output from failing the test. We are still testing
with a specific module lower down in the test, so this doesn't reduce
our test coverage.
This test fails often due to being trimmed, thanks to
salt.utils.dicttrim trimming values > max_event_size.
This commit changes this test so that it runs sys.doc twice, ensuring
that the return from sys.doc (for now) is not trimmed.
* Update rh_ip.py
Add support for the "primary" bonding option in active-backup, balancd-tlb and balance-alb type bonds for RH. This is used to specify a preferred slave for the link.
Remove support for the balance-xor bond type, since it is not supported by the kernel.
Reference: https://www.kernel.org/doc/Documentation/networking/bonding.txt
"The primary option is only valid for active-backup(1), balance-tlb (5) and balance-alb (6) mode."
* Update rh_ip.py
* Update debian_ip.py
Add support for the "primary" bonding option in active-backup, balancd-tlb and balance-alb type bonds for Debian. This is used to specify a preferred slave for the link.
Remove support for the balance-xor bond type, since it is not supported by the kernel.
Reference: https://www.kernel.org/doc/Documentation/networking/bonding.txt
"The primary option is only valid for active-backup(1), balance-tlb (5) and balance-alb (6) mode."
* Update debian_ip.py
Also added an index.rst file for executors, renamed the
azurearm.rst file (Was misspelled with "azureare"), and
removed the xbps-pkg.rst file since that is an invalid
reference.