Use ip instead of ifconfig and route
Bug #925145 reported by
Garrett Holmstrom
This bug affects 4 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
cloud-init |
Fix Released
|
Medium
|
Unassigned | ||
Fedora |
Confirmed
|
Medium
|
Bug Description
ifconfig just changed its output format again, breaking netinfo.py. Since net-tools have been deprecated for some time it makes more sense to switch to iproute as the net-tools maintainers suggest. Attached is a patch for netinfo.py and cc_disable_
Related branches
~james-hogarth/cloud-init:net-tools-deprecation
Rejected
for merging
into
cloud-init:master
- Chad Smith: Disapprove
- Server Team CI bot: Needs Fixing (continuous-integration)
-
Diff: 632 lines (+378/-124)2 files modifiedcloudinit/netinfo.py (+215/-64)
cloudinit/tests/test_netinfo.py (+163/-60)
Changed in fedora: | |
importance: | Unknown → Medium |
status: | Unknown → Confirmed |
Changed in cloud-init: | |
status: | Triaged → Fix Released |
To post a comment you must log in.
Hi,
I see that cloud-init uses net-tools (ifconfig and route) for displaying of network interfaces and routes. You maybe know that net-tools have been deprecated for quite a long time and we (as well as other distributions [1][2]) try to eviscerate its use from as many packages as possible.
I've created 2 patches that replace use of ifconfig and route with 'ip addr' and 'ip route'. I could send them to upstream myself but I see there are several patches in spec file so you have probably more experience with this. Will you do that for me ?
If you decide to apply this patches in Fedora then remove the superflous
'Requires: net-tools' line from spec file.
Thanks.
[1] http:// lists.debian. org/debian- devel/2009/ 03/msg00780. html /bugzilla. novell. com/show_ bug.cgi? id=492665
[2] https:/