Skip to content

Add configurable overlayfs path#4505

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
ashrayjain:aj/configurable-root
Aug 27, 2020
Merged

Add configurable overlayfs path#4505
dmcgowan merged 1 commit intocontainerd:masterfrom
ashrayjain:aj/configurable-root

Conversation

@ashrayjain
Copy link
Copy Markdown
Contributor

@ashrayjain ashrayjain commented Aug 26, 2020

Fixes #4500

This allows configuring the location of the overlayfs snapshotter by
adding the following in config.toml

[plugins]
  [plugins.overlayfs]
    root_path = "/custom_location"

This is useful to isolate disk i/o for overlayfs from the rest of
containerd and prevent containers saturating disk i/o from negatively
affecting containerd operations and cause timeouts.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 26, 2020

Build succeeded.

@ashrayjain ashrayjain force-pushed the aj/configurable-root branch 2 times, most recently from 47edab8 to 246c14f Compare August 26, 2020 14:52
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 26, 2020

Build succeeded.

@AkihiroSuda
Copy link
Copy Markdown
Member

Looks good but needs gofmt

This allows configuring the location of the overlayfs snapshotter by
adding the following in config.toml
```
[plugins]
  [plugins.overlayfs]
    root_path = "/custom_location"
```

This is useful to isolate disk i/o for overlayfs from the rest of
containerd and prevent containers saturating disk i/o from negatively
affecting containerd operations and cause timeouts.

Signed-off-by: Ashray Jain <[email protected]>
@ashrayjain ashrayjain force-pushed the aj/configurable-root branch from 9945b4b to 5ed177a Compare August 26, 2020 15:08
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 26, 2020

Build succeeded.

@ashrayjain
Copy link
Copy Markdown
Contributor Author

@AkihiroSuda thanks for the review. is this ready to merge?

Copy link
Copy Markdown
Contributor

@Zyqsempai Zyqsempai left a comment

Choose a reason for hiding this comment

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

LG

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit 1a89feb into containerd:master Aug 27, 2020
@ashrayjain ashrayjain deleted the aj/configurable-root branch August 27, 2020 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow configuring location of overlayfs

4 participants