Skip to content

update the docs to version=2 #4836

Closed
QFREEE wants to merge 2 commits intocontainerd:mainfrom
QFREEE:master
Closed

update the docs to version=2 #4836
QFREEE wants to merge 2 commits intocontainerd:mainfrom
QFREEE:master

Conversation

@QFREEE
Copy link
Copy Markdown

@QFREEE QFREEE commented Dec 13, 2020

updated the docs to config version=2. #4426

However, We are not sure where to find the exact config struct and descriptions for plugins and timeouts. Can you guide me on where to find descriptions apart from https://github.com/QFREEE/containerd/blob/master/services/server/config/config.go ?

We also found these timeout properties from other configs, however we are not sure where exactly they are coming from.[timeouts] "io.containerd.timeout.shim.cleanup" = "5s" "io.containerd.timeout.shim.load" = "5s" "io.containerd.timeout.shim.shutdown" = "3s" "io.containerd.timeout.task.state" = "2s"

commit history:
QFREEE@7fa35db
QFREEE@4ce9575

@k8s-ci-robot
Copy link
Copy Markdown

Hi @QFREEE. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@QFREEE QFREEE changed the title update the docs update the docs to version=2 Dec 13, 2020
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Dec 13, 2020

Build succeeded.

@kzys
Copy link
Copy Markdown
Member

kzys commented Dec 16, 2020

Your commits must have DCO (Signed-off-by: Your Name <[email protected]> line).

@dmcgowan dmcgowan added this to the 1.5 milestone Feb 26, 2021
@dmcgowan
Copy link
Copy Markdown
Member

@QFREEE are you able to squash your commits and add the sign off. Please reach out if you have question or want any guidance on that!

@dmcgowan dmcgowan added status/needs-update Awaiting contributor update and removed needs-ok-to-test labels Feb 27, 2021
**no_shim** specifies whether to use a shim or not (Default: **false**),
**shim_debug** turns on debugging for the shim (Default: **false**)
- **[plugins.scheduler]** has several options that perform advanced tuning for the scheduler:
- **[plugins.io.containerd.gc.v1.scheduler]** has several options that perform advanced tuning for the scheduler:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The plugin name needs double quotes

## FORMAT

**version**
: The version of the config file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we have some explanation about v1 and v2?

@thaJeztah
Copy link
Copy Markdown
Member

@QFREEE looks like this needs a rebase; could you also have a look at the review comments?

@dmcgowan dmcgowan modified the milestones: 1.5, 1.6 Apr 21, 2021
@AkihiroSuda
Copy link
Copy Markdown
Member

ping @QFREEE

@estesp
Copy link
Copy Markdown
Member

estesp commented Oct 25, 2021

Closing in favor of #6125

@estesp estesp closed this Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/needs-update Awaiting contributor update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants