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

Fix panic error from windows agent if no cni is deployed. #6768

Closed
wants to merge 10 commits into from

Conversation

Dev-Redtronics
Copy link

Proposed Changes

Fixed panic error if no cni is deployed from windows agent.

Types of Changes

Bugfix (Code change)

User-Facing Change

NONE

dereknola and others added 9 commits August 30, 2024 08:30
[release-1.31] Bump multus chart to v4.1.000
As discussed, it doesn't make sense anymore to keep them in the airgap tarball since we don't synchronize the sriov chart with an rke2 release.
[release 1.31] Remove sriov images from airgap tarball
Signed-off-by: Vitor Savian <[email protected]>
@Dev-Redtronics Dev-Redtronics requested a review from a team as a code owner September 9, 2024 12:51
@dereknola
Copy link
Member

Please rebase your code off master (that should resolve you branch conflicts).

@rbrtbnfgl
Copy link
Contributor

I think that you didn't rebase your branch but merge it with the master that's why your PR include different commits and also the conflicts aren't fixed rightly. I'll create an issue to track this and a PR with the fix similar to yours.

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.

6 participants