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.
If all the devices submitted to pvcreate() are already initialized as
LVM physical volumes and override is True (which is default), pvcreate()
should return True and no futher actions should be made. The
'not cmd[1:]' check which is suited for this scenario is incorrect, as
we previously filled the 'cmd' list with two elements and thus the
condition would be always False. This would cause pvcreate() to call the
pvcreate executable with no arguments if all the submitted devices are
already initialized as LVM PVs.
Fixes#39070
If all the devices submitted to pvcreate() are already initialized as
LVM physical volumes, pvcreate() should return True and no futher
actions should be made.
pvdisplay() would be called by pvcreate() twice: firstly to check
whether a device is already initialized for use by LVM and then to
ensure that the pvcreate executable did its job correctly. The test
replaces pvdisplay() with a mock that always returns True and thus
pvcreate() would think that a specified device is already initialized
and exit. In the other words, instead of testing physical volume
initialization the test simulates a case with all the submitted
devices already initialized.
Fix it by replacing pvdisplay with a mock that returns False on the
first call (thus pvcreate thinks that a device is not a PV yet) and True
on the second call (after the pvcreate executable is called).
'lvm version' produces multiline output whereas test_version and
test_fullversion use mocks with single-line output. Use real-life
'lvm version' output in those mocks instead.
