Skip to content

Podman Build use absolute filepath#12853

Merged
openshift-merge-robot merged 1 commit intocontainers:mainfrom
cdoern:buildRelative
Jan 14, 2022
Merged

Podman Build use absolute filepath#12853
openshift-merge-robot merged 1 commit intocontainers:mainfrom
cdoern:buildRelative

Conversation

@cdoern
Copy link
Copy Markdown
Contributor

@cdoern cdoern commented Jan 13, 2022

podman build always finds the abs path but was never using it for the containerFile path. This was causing the remote client to be given a relative path that does not exist. Switch to evaluating and using absolute paths only.

resolves #12841

Signed-off-by: cdoern cdoern@redhat.com

podman build always finds the abs path but was never using it for the containerfile path. This
was causing the remote client to be given a relative path that does not exist. Switch to evaluating and using absolute paths only.

resolves containers#12841

Signed-off-by: cdoern <cdoern@redhat.com>
@cdoern
Copy link
Copy Markdown
Contributor Author

cdoern commented Jan 13, 2022

@baude this fixes the issue, let me know if this breaks any other use cases and I can reevaluate.

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Jan 14, 2022

LGTM
@containers/podman-maintainers PTAL
@vrothberg @giuseppe @flouthoc PTAL

@umohnani8
Copy link
Copy Markdown
Member

LGTM

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Jan 14, 2022

/approve
/lgtm

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

openshift-ci bot commented Jan 14, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cdoern, rhatdan

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

The pull request process is described here

Details 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 Jan 14, 2022
@openshift-merge-robot openshift-merge-robot merged commit 9686216 into containers:main Jan 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.

podman build with relative path to dockerfile in subdirectory fails with "no such file or directory" on Mac

4 participants