(mirror of https://github.com/vyos/vyos-cloud-init.git) https://git.amelek.net/vyos/vyos-cloud-init.git/atom?h=20.4 2020-11-09T17:24:55+00:00 2020-11-09T17:24:55+00:00 dermotbradley dermot_bradley@yahoo.com 2020-11-09T17:24:55+00:00 urn:sha1:57349eb7df1c422d9e9558e54b201c85778997ae Add code so that specifying "wakeonlan: true" actually results in relevant configuration entry appearing in /etc/network/interfaces, Netplan, and sysconfig for RHEL and OpenSuse. Add testcases for the above. 2020-10-29T14:05:42+00:00 Eduardo Otubo otubo@redhat.com 2020-10-29T14:05:42+00:00 urn:sha1:b46e4a8cff667c8441622089cf7d57aeb88220cd The static and static6 subnet types for network_data.json were being ignored by the Openstack handler, this would cause the code to break and not function properly. As of today, if a static6 configuration is chosen, the interface will still eventually be available to receive router advertisements or be set from NetworkManager to wait for them and cycle the interface in negative case. It is safe to assume that if the interface is manually configured to use static ipv6 address, there's no need to wait for router advertisements. This patch will set automatically IPV6_AUTOCONF and IPV6_FORCE_ACCEPT_RA both to "no" in this case. This patch fixes the specific behavior only for RHEL flavor and sysconfig renderer. It also introduces new unit tests for the specific case as well as adjusts some existent tests to be compatible with the new options. This patch also addresses this problem by assigning the appropriate subnet type for each case on the openstack handler. rhbz: #1889635 rhbz: #1889635 Signed-off-by: Eduardo Otubo otubo@redhat.com 2020-10-29T13:38:56+00:00 Lukas Märdian luk@slyon.de 2020-10-29T13:38:56+00:00 urn:sha1:3c432b32de1bdce2699525201396a8bbc6a41f3e 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 2020-10-13T19:42:54+00:00 Anh Vo anhvo@microsoft.com 2020-10-13T19:42:54+00:00 urn:sha1:8ec8c3fc63a59b85888a0b52356b784314a1d4cc 2020-09-15T16:00:00+00:00 Eduardo Otubo otubo@redhat.com 2020-09-15T16:00:00+00:00 urn:sha1:8439b191ec2f336d544cab86dba2860f969cd5b8 Prior to this change, vlans were rendered in sysconfig with 'TYPE=Ethernet', and incorrectly rendered the PHYSDEV based on the name of the vlan device rather than the 'link' provided in the network config. The change here fixes: * rendering of TYPE=Ethernet for a vlan * adds a warning if the configured device name is not supported per the RHEL 7 docs "11.5. Naming Scheme for VLAN Interfaces" LP: #1788915 LP: #1826608 RHBZ: #1861871 2020-08-25T15:21:18+00:00 Paride Legovini paride.legovini@canonical.com 2020-08-25T15:21:18+00:00 urn:sha1:07104504ab5b30efd2d1f7a8c36effe18b8e5fe0 Changes: tox: bump the pylint version to 2.6.0 in the default run Fix pylint 2.6.0 W0707 warnings (raise-missing-from) 2020-08-24T13:34:24+00:00 Eduardo Otubo otubo@redhat.com 2020-08-24T13:34:24+00:00 urn:sha1:db86753f81af73826158c9522f2521f210300e2b * 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 <otubo@redhat.com> * Replacing with os.* calls * Adding test and removing isfile() useless call. Co-authored-by: Rick Harding <rharding@mitechie.com> 2020-07-15T14:26:12+00:00 Daniel Watkins oddbloke@ubuntu.com 2020-07-15T14:26:12+00:00 urn:sha1:4fe576516d65feda17ba78e9265a8e494a195e7b * cloudinit: remove global disable of pylint W0107 and fix errors This includes removing a test class which contained no tests but wasn't detected as empty because of an errant pass statement. * .pylintrc: update disable comment to match arguments 2020-07-14T17:31:13+00:00 Daniel Watkins oddbloke@ubuntu.com 2020-07-14T17:31:13+00:00 urn:sha1:25289087e44c9c74543248519e37ca1f11b8a711 * 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 2020-07-13T16:00:32+00:00 Daniel Watkins oddbloke@ubuntu.com 2020-07-13T16:00:32+00:00 urn:sha1:3cec3881062490727c5fff1b16b53f0176f976f0 This includes a fix to a test that had a string concatenation issue, and so was only testing a prefix of what was intended.