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

libcgroup and libcontainers examples and README #1483

Closed
wants to merge 3 commits into from

Conversation

krisnova
Copy link
Contributor

As I have been working on Aurae project I would have found value in some README and examples. I created the things that would have been helpful for me, in the hopes they will make it easy for others to adopt Youki.

Signed-off-by: Kris Nóva kris@nivenly.com

@krisnova krisnova changed the title Introduce libcgroup examples and README updates libcgroup and libcontainers examples and README Jan 17, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jan 17, 2023

Codecov Report

Merging #1483 (f735a88) into main (3f3c052) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1483      +/-   ##
==========================================
- Coverage   68.78%   68.77%   -0.02%     
==========================================
  Files         120      120              
  Lines       13082    13086       +4     
==========================================
+ Hits         8999     9000       +1     
- Misses       4083     4086       +3     

@utam0k
Copy link
Member

utam0k commented Jan 18, 2023

@krisnova Thanks for your contribution! I love your idea, but it looks like that CI got failed. May I ask you to check and fix it? If you need help, please ask me. You may leave the bpf example off for the moment. In that case, it would be helpful if you could create an issue.

@krisnova
Copy link
Contributor Author

Yes sounds good I will send more follow up commits. I will also add more README files where I work and offer other small improvements as I work if this is acceptable to Youki.

Signed-off-by: Kris Nóva <kris@nivenly.com>
Signed-off-by: Kris Nóva <kris@nivenly.com>
Signed-off-by: Kris Nóva <kris@nivenly.com>
};
use anyhow::{bail, Context, Ok, Result};
use nix::mount::MsFlags;
use nix::sched::CloneFlags;
use nix::sys::stat::Mode;
use nix::unistd::setsid;

use log::warn;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hey, it seems this is making clippy fail. Can you remove this and check if it fixes it?

@YJDoc2
Copy link
Collaborator

YJDoc2 commented Jan 23, 2023

Hey @krisnova , The examples and docs are something that we definitely need, but can we not make code changes in this PR then, and keep it just for docs / comment docs?
#1484 seems to extend this in some way (?) or at least a lot of code in that is also present in this. Can we separate the PRs such that one is only for docs and the other for feature intro?

@krisnova
Copy link
Contributor Author

Yes makes sense I will get it cleaned up.

@krisnova krisnova closed this Feb 9, 2023
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.

4 participants