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

Fedora 40 Changes: Enable IPv4 Address Conflict Detection by default #1656

Closed
dustymabe opened this issue Jan 24, 2024 · 6 comments
Closed
Assignees

Comments

@dustymabe
Copy link
Member

For the proposed Fedora 40 changes we decided the following change needed action:

Since this has the side effect of:

Enabling ACD will cause an additional delay when bringing up interfaces, because NetworkManager needs first to probe the address. The delay is between 1.5 and 3 seconds, because RFC 5227 requires that the probe interval is randomized. The delay will affect both static and DHCP connections.

We may want to consider the implications of this in Fedora CoreOS.

@dustymabe
Copy link
Member Author

Talked with @bengal about this change. He pointed out that the timeout value was adjusted from the original proposal from 3s down to 200ms. I think this alleviates some concern I had where in situations like cloud environments (tightly controlled, conflicts unlikely) we'd be paying a 1-3s penalty for no reason. 200ms seems much more reasonable.

@jlebon
Copy link
Member

jlebon commented Mar 26, 2024

Copying a convo from Matrix:

jlebon: i guess technically we could disable it since it doesn't really make much sense for most cloud platforms. but we do want it in bare metal and some others (e.g. vmware), so simpler to just leave it on

dustymabe: right. with it being a much smaller timeout value I think I'm ok with the more simple, but slightly less ideal option of just accepting the change as is.

@dustymabe
Copy link
Member Author

I'll close this out then unless you think it warrants meeting discussion. If so just re-open and add the meeting label.

@dustymabe dustymabe added the status/pending-next-release Fixed upstream. Waiting on a next release. label Mar 27, 2024
@dustymabe
Copy link
Member Author

The fix for this went into next stream release 40.20240322.1.0. Please try out the new release and report issues.

@dustymabe dustymabe added status/pending-testing-release Fixed upstream. Waiting on a testing release. and removed status/pending-next-release Fixed upstream. Waiting on a next release. labels Mar 27, 2024
@dustymabe
Copy link
Member Author

The fix for this went into testing stream release 40.20240416.2.0. Please try out the new release and report issues.

@dustymabe dustymabe added status/pending-stable-release Fixed upstream and in testing. Waiting on stable release. and removed status/pending-testing-release Fixed upstream. Waiting on a testing release. labels Apr 23, 2024
@dustymabe
Copy link
Member Author

The fix for this went into stable stream release 40.20240416.3.1.

@dustymabe dustymabe removed the status/pending-stable-release Fixed upstream and in testing. Waiting on stable release. label May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants