[NO TESTS NEEDED] podman-remote build does not support volumes #9651
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 |
openshift-ci-robot
added
the
approved
rhatdan
changed the title
| @@ -86,7 +86,7 @@ resulting image's configuration. | |||
| #### **--cache-from** | |||
|
|
|||
| Images to utilize as potential cache sources. Podman does not currently support | |||
| caching so this is a NOOP. | |||
| caching so this is a NOOP. (Not available for remote commands) | |||
There was a problem hiding this comment.
Suggest "This option is not available with the remote Podman client"
There was a problem hiding this comment.
We have this comment all over the podman-run and podman-create man page, so I will make the change everywhere. Want to get @TomSweeneyRedHat agreement though so we don't bikeshed.
There was a problem hiding this comment.
@mheon's suggestion SGTM
rhatdan
force-pushed
the
volume
branch
2 times, most recently
from
14b9453 to
37dd5cf
Compare
Remove --volume option from podman-remote since it is not supported, also add information to podman-build man page indicating options not supported over remote connections. Signed-off-by: Daniel J Walsh <[email protected]>
|
LGTM |
|
Hmm, I could have sworn I LGTM'd this earlier, I'll do so now. |
|
/lgtm |
github-actions
bot
added
the
locked - please file new issue/PR
Remove --volume option from podman-remote since it is
not supported, also add information to podman-build man page
indicating options not supported over remote connections.
Signed-off-by: Daniel J Walsh [email protected]