Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

networkmanager: add support for stable-ssid MAC option (LP: #2084234) #524

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

daniloegea
Copy link
Collaborator

802-11-wireless.cloned-mac-address = stable-ssid was added in Network Manager 1.46 and is available through the GUI applet.

Fixes https://bugs.launchpad.net/ubuntu/+source/netplan.io/+bug/2084234

Description

Checklist

  • Runs make check successfully.
  • Retains code coverage (make check-coverage).
  • New/changed keys in YAML format are documented.
  • (Optional) Adds example YAML for new feature.
  • (Optional) Closes an open bug in Launchpad.

@slyon slyon added the stable Might be merged in a stable branch label Oct 14, 2024
Copy link
Collaborator

@slyon slyon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM! And should be fine to be pulled into the pending 1.1.1 stable release.

I left a small remark about the autopkgtest (thanks for providing one!)

tests/integration/wifi.py Show resolved Hide resolved
tests/integration/wifi.py Show resolved Hide resolved
802-11-wireless.cloned-mac-address = stable-ssid was added in Network
Manager 1.46 and is available through the GUI applet.

tests: add integration test for stable-ssid

In order to create a predictable MAC address we need to set a fixed
/etc/machine-id and /var/lib/NetworkManager/secret_key.
Note that this test is currently unstable and skipped by default.
@slyon
Copy link
Collaborator

slyon commented Oct 14, 2024

@daniloegea Regarding the MAC address flakyness, apparently networkd changes to a random MAC address after taking over an interface, changing the original MAC that was assigned when the (test-)interface was created, e.g. via ip link add ....

We tried to tackle that in the past, but maybe we're seeing another instance of this here?? 1413f0e

@daniloegea
Copy link
Collaborator Author

It could be something related to that. I think the previous MAC address is used as one of the inputs for the calculation of the new one...

@daniloegea daniloegea merged commit 83a8d8c into canonical:main Oct 14, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stable Might be merged in a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants