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

systemd: Don't bring network down when using routed networking #758

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

aivanise
Copy link

@aivanise aivanise commented Aug 2, 2023

Do not bring the networking down on container startup when using routed nic together with NetworkManager, as in this case networking is configured externally by lxd.

Signed-off-by: Aleksandar Ivanisevic aleksandar@ivanisevic.de

@monstermunchkin
Copy link
Member

Thanks for this!

Could you please sign-off your commit, and add a proper commit description?

@aivanise
Copy link
Author

aivanise commented Aug 3, 2023

Thanks for this!

Could you please sign-off your commit, and add a proper commit description?

My google-fu is failing me, how do I sign off an already submitted pull request? Or do I have to submit a new one?

@monstermunchkin
Copy link
Member

You need to run git commit -s --amend which signs-off your commit and lets you edit the message. After that you need to run git push -f (force push).

nic together with NetworkManager, as in this case networking is
configured externally by lxd.

fixes lxc/lxc-ci#776

Signed-off-by: Aleksandar Ivanišević <alex@2e-systems.com>
@stgraber stgraber changed the title fixes https://github.com/lxc/lxc-ci/issues/776 systemd: Don't bring network down when using routed networking Aug 3, 2023
@stgraber stgraber merged commit 8f87993 into lxc:main Aug 3, 2023
8 checks passed
@aivanise
Copy link
Author

aivanise commented Aug 3, 2023

If you don't mind me asking, how long until images in images: are going to be built with this included?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants