Skip to content

panic when master dev doesn't exist within a container #830

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.

Sign up for GitHub

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

Jump to bottom
Closed
mlguerrero12 opened this issue Feb 16, 2023 · 0 comments · Fixed by #831
Closed

panic when master dev doesn't exist within a container #830

mlguerrero12 opened this issue Feb 16, 2023 · 0 comments · Fixed by #831

Comments

@mlguerrero12
Copy link
Member

macvlan and vlan plugins panic when the master interface within a container does not exist.
mlguerrero12 added a commit to mlguerrero12/plugins that referenced this issue Feb 16, 2023
@mlguerrero12
Fix overwritten error var in getMTUByName
02adbbb 
this prevents the error to be lost which was causing the
panic while accesing a nil var.

Fix containernetworking#830
@mlguerrero12 mlguerrero12 mentioned this issue Feb 16, 2023
Merged
mlguerrero12 added a commit to mlguerrero12/plugins that referenced this issue Feb 16, 2023
@mlguerrero12
Fix overwritten error var in getMTUByName
9ee4d32 
this prevents the error to be lost which was causing the
panic while accesing a nil var.

Fix containernetworking#830

Signed-off-by: Marcelo Guerrero Viveros <[email protected]>
@dcbw dcbw closed this as completed in #831 Feb 20, 2023
maiqueb pushed a commit to maiqueb/plugins that referenced this issue Mar 23, 2023
@mlguerrero12 @maiqueb
Fix overwritten error var in getMTUByName
ebddf4f 
this prevents the error to be lost which was causing the
panic while accesing a nil var.

Fix containernetworking#830

Signed-off-by: Marcelo Guerrero Viveros <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix overwritten error var in getMTUByName mlguerrero12/plugins
1 participant
@mlguerrero12