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

[bitnami/multus-cni] multus broken #27320

Closed
kfox1111 opened this issue Jun 18, 2024 · 13 comments
Closed

[bitnami/multus-cni] multus broken #27320

kfox1111 opened this issue Jun 18, 2024 · 13 comments
Assignees
Labels
multus-cni solved stale 15 days without activity tech-issues The user has a technical issue about an application triage Triage is needed

Comments

@kfox1111
Copy link
Contributor

kfox1111 commented Jun 18, 2024

Name and Version

bitnami/multus-cni 2.1.4

What architecture are you using?

amd64

What steps will reproduce the bug?

install it. delete a multus-cni daemonset pod, look at 00-multus.conf on its host

You get nested type:multus sections in delegates, and things break.

What is the expected behavior?

calico config is delegated into multus config without extra multus delegates.

What do you see instead?

extra multus delegates that break things.

@kfox1111 kfox1111 added the tech-issues The user has a technical issue about an application label Jun 18, 2024
@github-actions github-actions bot added the triage Triage is needed label Jun 18, 2024
@carrodher
Copy link
Member

Thank you for bringing this issue to our attention. We appreciate your involvement! If you're interested in contributing a solution, we welcome you to create a pull request. The Bitnami team is excited to review your submission and offer feedback. You can find the contributing guidelines here.

Your contribution will greatly benefit the community. Feel free to reach out if you have any questions or need assistance.

@kfox1111
Copy link
Contributor Author

I would love to help, but I have no idea how the binaries are produced. Could you please point me at their source and build scripts so I can debug further?

@carrodher
Copy link
Member

That would be nice. The container image script and logic are placed at https://github.com/bitnami/containers/tree/main/bitnami/multus-cni

@kfox1111
Copy link
Contributor Author

I don't follow.
https://github.com/bitnami/containers/blob/main/bitnami/multus-cni/4/debian-12/Dockerfile#L35 just points to what looks like are prebuilt images.

How are they built? Where does the source come from?

I straced the binaries and I don't see them following the code from the multus-cni released tarballs. The binaries are different from the multus-cni project's builds. So, not sure how to debug further.

@carrodher
Copy link
Member

Those intermediate packages are the result of downloading the upstream source code (in this case from https://github.com/k8snetworkplumbingwg/multus-cni/releases) and compiling it, in that sense, the upstream source code is not modified.

Then, that intermediate package is included in the Dockerfile to build the container image and later the Helm chart.

What are the differences you're finding between binaries?

@kfox1111
Copy link
Contributor Author

I hit this issue with the chart:
k8snetworkplumbingwg/multus-cni#353

Was trying to dig through the code to understand why the strace doesn't seem to follow the behavior of the binary doesnt seem to match the code. Then wasn't finding the exact code that the containers were using.

I just found this:
k8snetworkplumbingwg/multus-cni@633985d

which doesn't look like it is in the 4.0.2 release?

@carrodher
Copy link
Member

The issue may not be directly related to the Bitnami container image/Helm chart, but rather to how the application is being utilized, configured in your specific environment, or tied to a specific scenario that is not easy to reproduce on our side.

If you think that's not the case and are interested in contributing a solution, we welcome you to create a pull request. The Bitnami team is excited to review your submission and offer feedback. You can find the contributing guidelines here.

Your contribution will greatly benefit the community. Feel free to reach out if you have any questions or need assistance.

Suppose you have any questions about the application, customizing its content, or technology and infrastructure usage. In that case, we highly recommend that you refer to the forums and user guides provided by the project responsible for the application or technology.

With that said, we'll keep this ticket open until the stale bot automatically closes it, in case someone from the community contributes valuable insights.

@kfox1111
Copy link
Contributor Author

Its not directly related to the chart, no. But it is also a major problem for the chart, as its broken when you helm install it.

It looks like at the moment the thin plugin upstream is broken in their released version. Switching to support the thick plugin may help fix the chart.

Copy link

github-actions bot commented Jul 7, 2024

This Issue has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thanks for the feedback.

@github-actions github-actions bot added the stale 15 days without activity label Jul 7, 2024
Copy link

Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Issue. Do not hesitate to reopen it later if necessary.

@bitnami-bot bitnami-bot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 13, 2024
@kfox1111
Copy link
Contributor Author

still a problem, not solved.

@kfox1111
Copy link
Contributor Author

Could someone bump up the version to the newest release? It should have some fixes for this.

@javsalgar javsalgar changed the title multus broken [bitnami/multus-cni] multus broken Aug 14, 2024
@javsalgar
Copy link
Contributor

Hi,

There is an automated PR that is currently failing, we are checking it #28775

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multus-cni solved stale 15 days without activity tech-issues The user has a technical issue about an application triage Triage is needed
Projects
None yet
Development

No branches or pull requests

4 participants