vyos-cloud-init.git/cloudinit/net/tests, branch 20.4

get_interfaces: don't exclude Open vSwitch bridge/bond members (#608)

2020-10-29T13:38:56+00:00

If an OVS bridge was used as the only/primary interface, the 'init' stage failed with a "Not all expected physical devices present" error, leaving the system with a broken SSH setup. LP: #1898997

DHCP sandboxing failing on noexec mounted /var/tmp (#521)

2020-08-24T13:34:24+00:00

* DHCP sandboxing failing on noexec mounted /var/tmp If /var/tmp is mounted with noexec option the DHCP sandboxing will fail with Permission Denied. This patch simply avoids this error by checking the exec permission updating the dhcp path in negative case. rhbz: https://bugzilla.redhat.com/show_bug.cgi?id=1857309 Signed-off-by: Eduardo Otubo * Replacing with os.* calls * Adding test and removing isfile() useless call. Co-authored-by: Rick Harding

networking: refactor wait_for_physdevs from cloudinit.net (#466)

2020-07-14T17:31:13+00:00

* Refactor `cloudinit.net.wait_for_physdevs` to `cloudinit.distros.networking.Networking.wait_for_physdevs` * Split the Linux-specific `udevadm_settle` call out to a separate abstract `Networking.settle` method; implement it on `LinuxNetworking` and add a `NotImplementedError` implementation to `BSDNetworking` * Modify `wait_for_physdevs`s one callsite to use the new location LP: #1884626

networking: refactor is_physical from cloudinit.net (#457)

2020-06-30T18:19:38+00:00

As the first refactor PR, this also includes the initial structure for tests. LP: #1884619

net/networking: remove unused functions/methods (#453)

2020-06-24T19:22:45+00:00

Namely, is_connected, is_wireless and is_present. None of these are used in the cloud-init codebase, so remove the dead code (instead of refactoring it).

printing the error stream of the dhclient process before killing it (#369)

2020-06-19T13:38:05+00:00

This introduces a way to log the dhclient error stream, and uses it for the Azure datasource (where we have a specific requirement for this data to be logged).

Move subp into its own module. (#416)

2020-06-08T16:49:12+00:00

This was painful, but it finishes a TODO from cloudinit/subp.py. It moves the following from util to subp: ProcessExecutionError subp which target_path I moved subp_blob_in_tempfile into cc_chef, which is its only caller. That saved us from having to deal with it using write_file and temp_utils from subp (which does not import any cloudinit things now). It is arguable that 'target_path' could be moved to a 'path_utils' or something, but in order to use it from subp and also from utils, we had to get it out of utils.

test: fix all flake8 E241 (#403)

2020-06-02T15:24:29+00:00

Remove extra spaces after a ','

cloudinit: drop dependencies on unittest2 and contextlib2 (#322)

2020-04-24T13:26:51+00:00

These libraries provide backports of Python 3's stdlib components to Python 2. As we only support Python 3, we can simply use the stdlib now. This pull request does the following: * removes some unneeded compatibility code for the old spelling of `assertRaisesRegex` * replaces invocations of the Python 2-only `assertItemsEqual` with its new name, `assertCountEqual` * replaces all usage of `unittest2` with `unittest` * replaces all usage of `contextlib2` with `contextlib` * drops `unittest2` and `contextlib2` from requirements files and tox.ini It also rewrites some `test_azure` helpers to use bare asserts. We were seeing a strange error in xenial builds of this branch which appear to be stemming from the AssertionError that pytest produces being _different_ from the standard AssertionError. This means that the modified helpers weren't behaving correctly, because they weren't catching AssertionErrors as one would expect. (I believe this is related, in some way, to https://github.com/pytest-dev/pytest/issues/645, but the only version of pytest where we're affected is so far in the past that it's not worth pursuing it any further as we have a workaround.)

net: ignore 'renderer' key in netplan config (#306)

2020-04-03T17:58:08+00:00

LP: #1870421