Skip to content
This repository was archived by the owner on Oct 10, 2020. It is now read-only.

Atomic/install.py: Fix what c134ee5 broke#973

Closed
baude wants to merge 1 commit intoprojectatomic:masterfrom
baude:fix_install
Closed

Atomic/install.py: Fix what c134ee5 broke#973
baude wants to merge 1 commit intoprojectatomic:masterfrom
baude:fix_install

Conversation

@baude
Copy link
Member

@baude baude commented Apr 17, 2017

The commit c134ee5 broke the ability for atomic install to pull an image
if it isnt present.

@yuqi-zhang
Copy link
Contributor

Sorry for not catching this case. There is one minor problem with this fix being that remote ostree images still get ignored, i.e. if you do something like "atomic install --system gscrivano/flannel" (which has atomic.type=ostree) it would just say Image gscrivano/flannel not found. This is also the case if I use fqn's. So this fixes the docker part that I broke, but breaks the ostree part I fixed...

I will take another look at this today. Thanks for the fix!

@miabbott
Copy link
Contributor

Confirming what @yuqi-zhang said, this appears to fix the case of atomic install of a remote docker image, but breaks the case of atomic install of remote ostree image.

@baude baude force-pushed the fix_install branch 2 times, most recently from d72726e to eb5573c Compare April 18, 2017 15:47
The commit c134ee5 broke the ability for atomic install to pull an image
if it isnt present.

Also, changed the flow in install.py to use more of the backend
objects when determining the backend, the image object, and the
remote image objects.
@miabbott
Copy link
Contributor

The latest commit works better for the various use cases. 👍

@yuqi-zhang
Copy link
Contributor

LGTM

@baude
Copy link
Member Author

baude commented Apr 19, 2017

@rhatdan shall i merge?

@rhatdan
Copy link
Member

rhatdan commented Apr 20, 2017

@rh-atomic-bot r+ d6a5e48

@rh-atomic-bot
Copy link

⌛ Testing commit d6a5e48 with merge 3c273e4...

@baude
Copy link
Member Author

baude commented Apr 20, 2017

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

☀️ Test successful - status-redhatci
Approved by: rhatdan
Pushing 3c273e4 to master...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants