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

[ci:docs] Update troubleshoot page #6199

Merged
merged 1 commit into from
May 21, 2020

Conversation

rhatdan
Copy link
Member

@rhatdan rhatdan commented May 12, 2020

Add information about podman with SELinux and using container_init_t.

Add informantion about Centos7 as well as RHEL7 init containers not working
on a cgroups V2 system.

Signed-off-by: Daniel J Walsh [email protected]

@rhatdan
Copy link
Member Author

rhatdan commented May 12, 2020

Alternate to #6195

@@ -424,9 +431,9 @@ Choose one of the following:
* Install the fuse-overlayfs package for your Linux Distribution.
* Add `mount_program = "/usr/bin/fuse-overlayfs"` under `[storage.options]` in your `~/.config/containers/storage.conf` file.

### 17) rhel7-init based images don't work with cgroups v2
### 17) RHEL 7 and CentOS 7 based `init` images don't work with cgroups v2
Copy link
Member

Choose a reason for hiding this comment

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

I know this is old, but 'cgroups' -> 'cgroup' here and through out.


The systemd version shipped in rhel7-init doesn't have support for cgroups v2. You'll need at least systemd 230.
The systemd version shipped in RHEL 7 and CentOS 7 doesn't have support for cgroups v2. Cgroup V2 support requires at least systemd 230.
Copy link
Member

Choose a reason for hiding this comment

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

cgroup is supposed to be lower case everywhere. How about the second sentence starts with "Further, cgroup v2". Regardless, "V2" to "v2" for consistency.

@rhatdan rhatdan changed the title Update troubleshoot page [ci:docs] Update troubleshoot page May 12, 2020
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rhatdan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 12, 2020
@rhatdan
Copy link
Member Author

rhatdan commented May 19, 2020

troubleshooting.md Outdated Show resolved Hide resolved
troubleshooting.md Outdated Show resolved Hide resolved
Add information about podman with SELinux and using container_init_t.

Add informantion about Centos7 as well as RHEL7 init containers not working
on a cgroups V2 system.

Signed-off-by: Daniel J Walsh <[email protected]>
@rhatdan rhatdan added documentation Issue or fix is in project documentation kind/documentation Categorizes issue or PR as related to documentation. labels May 19, 2020
@QiWang19
Copy link
Contributor

LGTM

@mheon
Copy link
Member

mheon commented May 21, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 21, 2020
@openshift-merge-robot openshift-merge-robot merged commit 88da5a7 into containers:master May 21, 2020
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. documentation Issue or fix is in project documentation kind/documentation Categorizes issue or PR as related to documentation. lgtm Indicates that a PR is ready to be merged. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants