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

vendor latest openshift/imagebuilder #1390

Closed
wants to merge 2 commits into from

Conversation

vrothberg
Copy link
Member

@vrothberg vrothberg commented Mar 7, 2019

openshift/imagebuilder had a dependency on a deprecated package that is
now forked into imagebuilder to allow using more recent versions of
docker/docker.

Also vendor the latest docker/docker to a) demonstrate that imagebuilder
does not force using an old version docker/docker and to b) pull in
fixes and updates to our dependencies on docker/docker.

See openshift/imagebuilder#125 for more details
on the imagebuilder change.

Signed-off-by: Valentin Rothberg rothberg@redhat.com

@vrothberg vrothberg force-pushed the ci-test-imagebuilder branch 5 times, most recently from d6a49c9 to 0dca4b4 Compare March 8, 2019 09:07
openshift/imagebuilder had a dependency on a deprecated package that is
now forked into imagebuilder to allow using more recent versions of
docker/docker.

Also vendor the latest docker/docker to a) demonstrate that imagebuilder
does not force using an old version docker/docker and to b) pull in
fixes and updates to our dependencies on docker/docker.

See openshift/imagebuilder#125 for more details
on the imagebuilder change.

Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
@vrothberg vrothberg changed the title WIP - test github.com/openshift/imagebuilder/pull/125 vendor latest/imagebuilder Mar 12, 2019
@vrothberg vrothberg changed the title vendor latest/imagebuilder vendor latest openshift/imagebuilder Mar 12, 2019
@vrothberg
Copy link
Member Author

The PR has been merged into imagebuilder. We can now use the latest version of docker/docker (if desired). Once this PR is merged, we should take it over to libpod rather quickly to not forget about updating the dependencies.

@TomSweeneyRedHat @rhatdan @nalind PTAL

It looks like the latest `vndr` wants to pull in more files than before.

Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
@TomSweeneyRedHat
Copy link
Member

LGTM and passes baseline tests.

@rhatdan
Copy link
Member

rhatdan commented Mar 12, 2019

@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 3377253 has been approved by rhatdan

@rh-atomic-bot
Copy link
Collaborator

⌛ Testing commit 3377253 with merge beafe55...

rh-atomic-bot pushed a commit that referenced this pull request Mar 12, 2019
It looks like the latest `vndr` wants to pull in more files than before.

Signed-off-by: Valentin Rothberg <rothberg@redhat.com>

Closes: #1390
Approved by: rhatdan
@rh-atomic-bot
Copy link
Collaborator

☀️ Test successful - status-papr, status-travis
Approved by: rhatdan
Pushing beafe55 to master...

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants