-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Pass DefaultMountsFile to podman build #9094
Conversation
[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 |
@edsantiago Do you want to throw me a test for this? |
Followup: I pulled your PR, ran my tests against it (with changes as described in my previous comment): it passes. I confirmed that the test is really running by adding |
@rhatdan #9091 is merged. At your convenience feel free to rebase and apply the following diff diff --git a/test/system/070-build.bats b/test/system/070-build.bats
index 9e5e3ee1d..05518d8fc 100644
--- a/test/system/070-build.bats
+++ b/test/system/070-build.bats
@@ -188,8 +188,7 @@ ADD mycmd /bin/mydefaultcmd
RUN chmod 755 /bin/mydefaultcmd
RUN chown 2:3 /bin/mydefaultcmd
-#FIXME FIXME FIXME: enable if/when 'podman build' passes mounts.conf to buildah
-#RUN $CAT_SECRET
+RUN $CAT_SECRET
CMD ["/bin/mydefaultcmd","$s_echo"]
EOF
@@ -201,8 +200,7 @@ EOF
local iid="${lines[-1]}"
# Make sure 'podman build' had the secret mounted
- #FIXME FIXME: enable if/when 'podman build' passes mounts.conf to buildah
- #is "$output" ".*$secret_contents.*" "podman build has /run/secrets mounted"
+ is "$output" ".*$secret_contents.*" "podman build has /run/secrets mounted"
if is_remote; then
ENVHOST="" |
The --default-mounts-file path was not being handled in podman build. This will enable it to use for testing. Signed-off-by: Daniel J Walsh <[email protected]>
Restarted flake, #9026. LGTM. All new tests pass. /lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/hold cancel |
The --default-mounts-file path was not being handled in
podman build. This will enable it to use for testing.
Signed-off-by: Daniel J Walsh [email protected]