implement the update subcommand(partially)

[knight42]

Ref #147

This PR did several things:

• Update the version of oci-spec to the latest
• Extract a common function create_cgroup_manager which could be used in ps and update command
• Add the update subcommand and make it be abled to changed the pids limit in the container

[knight42]

/cc @utam0k

One thing I am curious is that now we read the cgroup path from the config.json under container root path

youki/crates/youki/src/commands/ps.rs

Line 14 in cbba0bc

let config_absolute_path = container.root.join("config.json");

, but config.json is not copied to the root dir when creating container, so the config.json is missing when I run the ps command, is this intended?

[Furisto]

The CI failure is due to some clippy warnings that need to be fixed

[Furisto]

Also the clippy warnings still need to be fixed and the pnet package seems to be specified twice in the cargo.lock file now.

[codecov-commenter]

Codecov Report

Merging #536 (7332eb4) into main (529ca67) will decrease coverage by 0.58%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #536      +/-   ##
==========================================
- Coverage   61.09%   60.51%   -0.59%     
==========================================
  Files          85       99      +14     
  Lines       12532    12643     +111     
==========================================
- Hits         7657     7651       -6     
- Misses       4875     4992     +117     

[utam0k]

@knight42
Oh I forget to take care of ps command 😭
When this PR #447 merged, we stopped copying config.rs for improving performance.
Now, we use YoukiConfig instead of config.rs

youki/crates/libcontainer/src/container/container_start.rs

Line 43 in 781f9fa

let config = YoukiConfig::load(&self.root)

If you like, I will ask you to fix it. What do you think? Of course, if you are busy, I'll fix it on my end.

[knight42]

@knight42 Oh I forget to take care of ps command 😭 When this PR #447 merged, we stopped copying config.rs for improving performance. Now, we use YoukiConfig instead of config.rs

@utam0k Hi! I believe this issue has been fixed in this PR based on @Furisto 's suggestion here.

@Furisto Would you like to take another look?

[Furisto]

@knight42 Looks good! One last thing: Can you please squash the last 4 commits into one and name it "Address review comments" or something like that? Otherwise LGTM!

[knight42]

@Furisto Hi! I have already squashed the commits.

[utam0k]

Looks good. Thanks!

[Furisto]

@knight42 Can you resolve the merge conflicts?

[knight42]

@Furisto Fixed now.

[utam0k]

@knight42 Awesome!