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

pillar: containerd: Do not add hooks when no interfaces are configured #4174

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

rene
Copy link
Contributor

@rene rene commented Aug 28, 2024

The hooks for veth.sh and dhcpcd scripts don't need to be added to OCI spec when no network interfaces are configured for the container.

This will be required for bare-metal container Edge Applications.

The hooks for veth.sh and dhcpcd scripts don't need to be added to OCI
spec when no network interfaces are configured for the container.

Signed-off-by: Renê de Souza Pinto <rene@renesp.com.br>
Copy link
Member

@OhmSpectator OhmSpectator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rene, so, we skip setting the DHCP client for the domain because we do not support updating the list of interfaces in runtime, right? Once the domain is started, we never change the list of network interfaces, yep?
I'm also curious: how did you get the idea of this fix?

Anyway, looks fine.

@rene
Copy link
Contributor Author

rene commented Aug 29, 2024

@rene, so, we skip setting the DHCP client for the domain because we do not support updating the list of interfaces in runtime, right? Once the domain is started, we never change the list of network interfaces, yep? I'm also curious: how did you get the idea of this fix?

Anyway, looks fine.

Correct @OhmSpectator , the issue arose when I tried to deploy a native container without any network interface attached/configured to it. The Edge App was a NVIDIA CUDA container from dockerhub.

@OhmSpectator
Copy link
Member

I guess this change should not affect any tests (do we have anything for apps without network interfaces at all?), but I have seen some of them red several times... And I'm not sure how to interpret it. Restarting again...

@OhmSpectator OhmSpectator merged commit f05be98 into lf-edge:master Aug 29, 2024
47 of 51 checks passed
@rene rene deleted the fix-add-hooks branch September 3, 2024 11:37
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.

3 participants