Skip to content

Vendor buildah, imagebuilder, docker/docker, fsouza and more#2627

Closed
TomSweeneyRedHat wants to merge 0 commit intocontainers:masterfrom
TomSweeneyRedHat:dev/tsweeney/bigvendorbuildah
Closed

Vendor buildah, imagebuilder, docker/docker, fsouza and more#2627
TomSweeneyRedHat wants to merge 0 commit intocontainers:masterfrom
TomSweeneyRedHat:dev/tsweeney/bigvendorbuildah

Conversation

@TomSweeneyRedHat
Copy link
Member

Signed-off-by: TomSweeneyRedHat [email protected]

Vendors in Buidah, imagebuilder, fsouza/docker-client, docker/docker and
a few more related. Of particular note, changes to the TweakCapabilities()
function from docker/docker along with the parse.IDMappingOptions() function
from Buildah. Please pay particular attention to the related changes in
the call from libpod to those functions during the review.

Passes baseline tests.

@openshift-ci-robot openshift-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Mar 13, 2019
@vrothberg
Copy link
Member

One dependency seems to be missing:

?? vendor/golang.org/x/sys/windows/registry/

@TomSweeneyRedHat
Copy link
Member Author

repushed, we'll see how long the tests last this time....

@rh-atomic-bot
Copy link
Collaborator

☔ The latest upstream changes (presumably #2625) made this pull request unmergeable. Please resolve the merge conflicts.

@openshift-ci-robot openshift-ci-robot added size/XS and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Mar 13, 2019
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: TomSweeneyRedHat

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

@mheon
Copy link
Member

mheon commented Mar 13, 2019

@TomSweeneyRedHat I think you pushed master to your branch

@mheon
Copy link
Member

mheon commented Mar 13, 2019

Github automerges when there are 0 changes

@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 27, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 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. 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.

5 participants