Merge ~raharper/cloud-init:eni-drop-rendering-systemd-link-files into cloud-init:master
Proposed by
Ryan Harper
Status: | Merged |
---|---|
Approved by: | Scott Moser |
Approved revision: | 4a34c10a60eba40cbdc7c9739f5d07e2a45b3c72 |
Merged at revision: | 003c6678e9c873b3b787a814016872b6592f5069 |
Proposed branch: | ~raharper/cloud-init:eni-drop-rendering-systemd-link-files |
Merge into: | cloud-init:master |
Diff against target: |
84 lines (+3/-31) 2 files modified
cloudinit/net/eni.py (+0/-25) tests/unittests/test_net.py (+3/-6) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Scott Moser | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Review via email: mp+324675@code.launchpad.net |
Description of the change
cloudinit.net: remove systemd link file writing from eni renderer
During the network v2 merge, we re-enabled rendering systemd .link files.
This files are not required as cloud-init already has to do interface
renaming due to issues with udevd which may refuse to rename certain
interfaces (such as veth devices in a LXD container). As such, removing
the code altogether.
To post a comment you must log in.
PASSED: Continuous integration, rev:4a34c10a60e ba40cbdc7c9739f 5d07e2a45b3c72 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 411/ /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- amd64/411 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- arm64/411 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- ppc64el/ 411 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=metal- s390x/411 /jenkins. ubuntu. com/server/ job/cloud- init-ci/ nodes=vm- i386/411
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/cloud- init-ci/ 411/rebuild
https:/