DLPX-69865 IP configuration is lost if a MAC address changes#21
Merged
Conversation
Contributor
Author
|
With this change applied: |
To make the network configuration more resilient to changes of the MAC address, this change modifies the netplan configuration generated by cloud-init to no longer match on the MAC address. This way, if the MAC address of a NIC changes (which can happen in VM environments), the network configuration on the system should still work as before.
Contributor
Author
|
|
Contributor
Author
|
Also worth noting, this is only half of the solution to address DLPX-69865; I still need to develop a similar change for the configuration we generate via the DE software. I'll open a review for that once I have it ready. |
pzakha
approved these changes
Sep 1, 2020
Contributor
pzakha
left a comment
There was a problem hiding this comment.
This LGTM.
Also, there might be another part to it, that is what to do with engines that were previously configured by cloud-init with MAC address matching. That could be addressed by your other change that tries to re-apply network configuration on each boot (although that could potentially add latency to the boot), or somehow trigger a one-time reconfiguration.
sdimitro
approved these changes
Sep 1, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To make the network configuration more resilient to changes of the MAC
address, this change modifies the netplan configuration generated by
cloud-init to no longer match on the MAC address. This way, if the MAC
address of a NIC changes (which can happen in VM environments), the
network configuration on the system should still work as before.