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: Fix plugin crash due to nil in container info #1247

Merged
merged 9 commits into from Nov 28, 2023

Conversation

quzard
Copy link
Collaborator

@quzard quzard commented Nov 20, 2023

No description provided.

pkg/helper/docker_center.go Outdated Show resolved Hide resolved
pkg/helper/docker_cri_adapter.go Outdated Show resolved Hide resolved
@yyuuttaaoo yyuuttaaoo merged commit 8462532 into alibaba:main Nov 28, 2023
20 checks passed
@yyuuttaaoo yyuuttaaoo changed the title fix dockerContainer nil fix: Fix plugin crash due to nil in container info Nov 28, 2023
@yyuuttaaoo yyuuttaaoo added the bug Something isn't working label Nov 28, 2023
@yyuuttaaoo yyuuttaaoo added this to the v1.8 milestone Nov 28, 2023
yyuuttaaoo pushed a commit that referenced this pull request Dec 7, 2023
* fix nil

* 重试机制

* fix

* add retryTimes

* logger Container Info

* Container Info
@quzard quzard deleted the fix/containerd-restart-core-dump branch February 28, 2024 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants