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
[baremetal] Create and enable overlay mount point for NetworkManager #2017
[baremetal] Create and enable overlay mount point for NetworkManager #2017
Conversation
MCO looks unhappy with this: lstat /etc/NetworkManager/system-connections-merged/.dummy: no such file or directory |
Thanks. I'll take a look at that. |
3409edc
to
b56dcfc
Compare
templates/common/baremetal/units/baremetal-system-connections-mount.yaml
Outdated
Show resolved
Hide resolved
b56dcfc
to
06685b4
Compare
/retitle [baremetal] Create and enable overlay mount point for NetworkManager |
/retest |
This does 4 things for the baremetal platform: 1. Configures NetworkManager to look for it's keyfiles in /etc/NetworkManager/system-connections-merged 2. Creates that directory 3. Adds two temporary directories for systemd-tmpfiles to process. These are the storage and work locations for the overlay. 4. Mount an overlay filesystem at /etc/NetworkManager/system-connections-merged that uses /etc/NetworkManager/system-connections as the lower filesystem, and the temporary directory as the upper. Any changes made outside of a MachineConfig will result in it being ephemeral. This maintains the idea that MCO will own the configuration of a machine before kubelet runs.
06685b4
to
6666e7f
Compare
Implementation of Option C in openshift/enhancements#399 |
/retest |
@bcrochet: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/retest |
1 similar comment
/retest |
/lgtm Looks like all of the comments have been addressed and it's passing ci. |
lgtm, implements what has been described in openshift/enhancements#399 in Option C |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bcrochet, cybertron, sinnykumari The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
18 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
This does 4 things for the baremetal platform:
Any changes made outside of a MachineConfig will result in it being ephemeral. This maintains the idea that MCO will own the configuration of a machine before kubelet runs. MCO changes would go into /etc/NetworkManager/system-connections.
- What I did
- How to verify it
- Description for the changelog
Mount an overlay directory to make NetworkManager changes ephemeral.