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

[simplify-networking] mco: Set OVS links to MACAddressPolicy none #94

Open
wants to merge 4 commits into
base: simplify-networking
Choose a base branch
from

Conversation

RamLavi
Copy link
Collaborator

@RamLavi RamLavi commented Jul 31, 2024

Due to a bug [0] in networkManager, when set to
MACAddressPolicy=persistent links can get reapplied. This can result with unexpected behaviors like the node getting a new IP lease from DHCP.
In order to prevent this, changing the MACAddressPolicy to none on ovs links.
On newer releases the policy is already set to none [1] so this change will only matter in old releases.

[0] https://issues.redhat.com/browse/RHEL-40683
[1] https://fedoraproject.org/wiki/Changes/MAC_Address_Policy_none

Due to a bug [0] in networkManager, when set to
MACAddressPolicy=persistent links can get reapplied. This can result
with unexpected behaviors like the node getting a new IP lease from
DHCP.
In order to prevent this, changing the MACAddressPolicy to none on ovs
links.
On newer releases the policy is already set to none [1] so this change
will only matter in old releases.

[0] https://issues.redhat.com/browse/RHEL-40683
[1] https://fedoraproject.org/wiki/Changes/MAC_Address_Policy_none

Signed-off-by: Ram Lavi <[email protected]>
@RamLavi
Copy link
Collaborator Author

RamLavi commented Jul 31, 2024

@qinqon lets merge this change only after we have confirmation that the proposed solution is working.
/hold

@RamLavi RamLavi changed the title mco: Set OVS links to MACAddressPolicy none [simplify-networking] mco: Set OVS links to MACAddressPolicy none Aug 1, 2024
Former container failed to load due to manifest v1 deprecated.
Signed-off-by: Ram Lavi <[email protected]>
Signed-off-by: Ram Lavi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant