incorrect separator for multiple ARP IP targets
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Netplan |
Fix Released
|
Undecided
|
Unassigned | ||
netplan.io (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bionic |
Fix Released
|
Undecided
|
Unassigned | ||
Disco |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
Users of netplan who need to configure multiple values for ARP IP targets.
[Test case]
Write a valid configuration for a bond with multiple ARP IP targets:
network:
version: 2
renderer: networkd
bond:
bd0:
[...]
parameters:
Verify that the configuration is sucessfully applied, and shows up in the /sys filesystem for the affected device:
/sys/class/
[Regression potential]
This affects only this particular value and generating a valid configuration file for the networkd renderer. The existing configuration file generation is /invalid/, but working as the bad value is ignored by systemd-networkd. This changes the generation of the configuration file only, to write the proper sequence of values for ARPIPTargets=.
---
The separator used to add multiple ARP IP targets for the networkd renderer is ",".
Unfortunately, networkd doesn't allow that, but rather expects whitespace:
(in config_
[...]
for (;;) {
r = extract_
extract_
if (!separators)
Finally:
src/basic/
So; it doesn't handle commas for the list, just whitespace; so let's give it whitespace please :)
description: | updated |
Changed in netplan.io (Ubuntu Disco): | |
status: | New → Fix Committed |
Changed in netplan: | |
status: | Fix Committed → Fix Released |
Status changed to 'Confirmed' because the bug affects multiple users.