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

podman create: building local pause image: do not read ignore files #13530

Merged
merged 1 commit into from
Mar 17, 2022

Conversation

vrothberg
Copy link
Member

Make sure to ignore local {container,docker}ignore files when building a
local pause image. Otherwise, we may mistakenly not be able to copy
catatonit into the build container.

Fixes: #13529
Signed-off-by: Valentin Rothberg [email protected]

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 16, 2022
@vrothberg
Copy link
Member Author

@edsantiago updated. I tried making up by consolidating places using the local pause image.

@vrothberg
Copy link
Member Author

@containers/podman-maintainers PTAL

/hold
... as I want a final head nod from Ed.

Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 16, 2022
Copy link
Member

@edsantiago edsantiago left a comment

Choose a reason for hiding this comment

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

LGTM, and the test now fails (as it should) in unpatched podman, but a few suggestions inline.

Thank you!

test/system/helpers.bash Outdated Show resolved Hide resolved
test/system/helpers.bash Outdated Show resolved Hide resolved
test/system/070-build.bats Outdated Show resolved Hide resolved
test/system/070-build.bats Outdated Show resolved Hide resolved
Make sure to ignore local {container,docker}ignore files when building a
local pause image.  Otherwise, we may mistakenly not be able to copy
catatonit into the build container.

Fixes: containers#13529
Signed-off-by: Valentin Rothberg <[email protected]>
Copy link
Member

@edsantiago edsantiago left a comment

Choose a reason for hiding this comment

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

/lgtm
/hold

@vrothberg
Copy link
Member Author

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 17, 2022
@edsantiago
Copy link
Member

/lgtm again (because my first one was in a review comment, and I discovered this week that github review comments are treated differently from PR comments)

@edsantiago edsantiago added the lgtm Indicates that a PR is ready to be merged. label Mar 17, 2022
@openshift-merge-robot openshift-merge-robot merged commit e59f71d into containers:main Mar 17, 2022
@edsantiago
Copy link
Member

(Nope, my slash-lgtm had no effect; I had to manually use clickybuttons to add the label. githubstatus claims that all is well, but earlier this morning things were very broken, so it's possible that something is still broken).

@vrothberg vrothberg deleted the fix-13529 branch March 18, 2022 07:54
@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.

podman-pause building fails because of .dockerignore
4 participants