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

cgroup: fix the issue of crashed when meet unsupported cgroup #414

Merged
merged 1 commit into from Jul 15, 2020

Conversation

lifupan
Copy link
Member

@lifupan lifupan commented Jul 15, 2020

Fix the issue of applying/set to unsupported cgroups.

Fixes: #408

Signed-off-by: fupan.lfp fupan.lfp@antfin.com

@lifupan
Copy link
Member Author

lifupan commented Jul 15, 2020

/test

@lifupan
Copy link
Member Author

lifupan commented Jul 15, 2020

Hi @jodh-intel Please take a look on this PR.

"perf_event" => Some(Box::new(PerfEvent())),
"freezer" => Some(Box::new(Freezer())),
"name=systemd" => Some(Box::new(Named())),
_ => None,
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @lifupan ! Might be worth adding a log call so we can see the agent detected an unusual cgroup:

info!(sl!(), "Found unexpected cgroup"; "cgroup" => name);

Copy link
Member Author

Choose a reason for hiding this comment

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

@jodh-intel Done!

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @lifupan!

Fix the issue of applying/set to unsupported cgroups.

Fixes: kata-containers#408

Signed-off-by: fupan.lfp <fupan.lfp@antfin.com>
@lifupan
Copy link
Member Author

lifupan commented Jul 15, 2020

/test

@yyyeerbo yyyeerbo merged commit dacb2fd into kata-containers:2.0-dev Jul 15, 2020
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.

Rust agent crashes if rdma cgroup present
4 participants