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

Add support for --chrootdirs #13221

Merged
merged 1 commit into from
Mar 14, 2022
Merged

Conversation

LStandman
Copy link
Contributor

@LStandman LStandman commented Feb 12, 2022

Adds support for optional --chrootdirs=x to specify chroot dirs. Special files such as /etc/hosts will be automatically mounted into x/etc/hosts and so on.

Closes #12691

@LStandman LStandman changed the title Add support for --altrootdirs Add support for --altrootdirs Closes #12691 Feb 12, 2022
@LStandman LStandman changed the title Add support for --altrootdirs Closes #12691 Add support for --altrootdirs Feb 12, 2022
test/e2e/create_test.go Outdated Show resolved Hide resolved
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 25, 2022
@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 25, 2022
@LStandman LStandman changed the title Add support for --altrootdirs Add support for --chrootdirs Feb 25, 2022
@rhatdan
Copy link
Member

rhatdan commented Feb 26, 2022

Looks like you need to gofmt your code.

@LStandman LStandman force-pushed the main branch 8 times, most recently from a090d2e to 4cf907f Compare February 26, 2022 19:29
@LStandman LStandman requested a review from rhatdan February 26, 2022 21:04
@TomSweeneyRedHat
Copy link
Member

Other than the nit from @rhatdan , LGTM

@LStandman
Copy link
Contributor Author

Other than the nit from @rhatdan , LGTM

I incorporated yours and @rhatdan suggested changes.
If I understand correctly I still need the lgtm label before moving on to the approvers. Am I correct?

@rhatdan
Copy link
Member

rhatdan commented Mar 14, 2022

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 14, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 14, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LStandman, 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 openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 14, 2022
@openshift-merge-robot openshift-merge-robot merged commit 219f116 into containers:main Mar 14, 2022
@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 21, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 21, 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. 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.

add option to bind mount (pod managed) /etc/* files to nonstandard locations
5 participants