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

Option --url and --connection should imply --remote. #13357

Conversation

Romain-Geissler-1A
Copy link
Contributor

@Romain-Geissler-1A Romain-Geissler-1A commented Feb 26, 2022

Closes #13242

This is a backport of #13296 in branch "v4.0". I have read in another issue that Red Hat plans on shipping podman 4 in RHEL 8.6 in couple of months, I am a RHEL user and I expect to see this commit part of the next RHEL 8.6 release ;)

I am not sure what is your policy for backport though, nor if I shall go though the paying Red Hat customer portal for these requests, so feel free to decline if I have done it the wrong way.

Closes containers#13242

Signed-off-by: Romain Geissler <romain.geissler@amadeus.com>
@mheon
Copy link
Member

mheon commented Feb 28, 2022

I'm a little iffy on this because it's technically a behavioral change from v4.0, so we can't really include it in a v4.0.1.

@baude @rhatdan @vrothberg Thoughts?

@rhatdan
Copy link
Member

rhatdan commented Feb 28, 2022

Currently if I use these fields without the --remote option, then they do nothing, which I would consider a bug. Users expect something to happen.
Now if I do it with --remote option they continue to work.
So I don't see this as a breaking change but more of a bug fix.

LGTM

@TomSweeneyRedHat
Copy link
Member

The changes LGTM, but RHEL 8.6 is closed to everything except blocker or exception bugs. I would not consider this either as there is a workaround. This could be included for the first z-stream release, RHEL 8.6.0.1, but at this point, we should not merge this here.

@rhatdan
Copy link
Member

rhatdan commented Feb 28, 2022

/approve
/lgtm

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

openshift-ci bot commented Feb 28, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rhatdan, Romain-Geissler-1A

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 Feb 28, 2022
@Romain-Geissler-1A
Copy link
Contributor Author

I expect this hypothetic RHEL 8.6.0.1 would be released a few days/weeks after the first RHEL 8.6 release has been shipped to customers ? If it's just a matter of some days/weeks, that's not a big deal for us.

@openshift-merge-robot openshift-merge-robot merged commit cff448b into containers:v4.0 Feb 28, 2022
@TomSweeneyRedHat
Copy link
Member

@rhatdan this should NOT have been merged IMO....

@mheon
Copy link
Member

mheon commented Mar 1, 2022

@TomSweeneyRedHat If need be, we can start a v4.0.1-rhel branch from the v4.0.1 tag that does not include this.

@rhatdan
Copy link
Member

rhatdan commented Mar 1, 2022

Lets talk about it at watercooler.

TomSweeneyRedHat added a commit to TomSweeneyRedHat/podman that referenced this pull request Mar 2, 2022
Addresses containers#13393 and replaces containers#13357.  Many thanks to
@Romain-Geissler-1A for the initial submission and discussions.

Signed-off-by: tomsweeneyredhat <tsweeney@redhat.com>
@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.

None yet

5 participants