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.

Sign up for GitHub

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

Jump to bottom

Building container fails after removing old image. #5465

Closed
iangneal opened this issue Dec 10, 2017 · 0 comments
Closed

Building container fails after removing old image. #5465

iangneal opened this issue Dec 10, 2017 · 0 comments
Labels
area/volumes kind/bug

Comments

@iangneal
Copy link

I found this issue while working on a fix for issue #4337.

By running the following sequence of commands on this example repository:

git checkout tags/test2_run1
docker-compose build
docker rmi --force dctest_service1
docker-compose build
docker-compose up

Compose fails because the old image is never properly rebuilt from the old container:

WARNING: The Docker Engine you're using is running in swarm mode.

Compose does not use swarm mode to deploy services to multiple nodes in a swarm. All containers will be scheduled on the current node.

To deploy your application across the swarm, use `docker stack deploy`.

Recreating dctest_service1_1 ... error

ERROR: for dctest_service1_1  no such image: sha256:bb41503c2e723478308998afc7df0886fac0811621b70d61b28ca36418a41bb7: No such image: sha256:bb41503c2e723478308998afc7df0886fac0811621b70d61b28ca36418a41bb7

ERROR: for service1  no such image: sha256:bb41503c2e723478308998afc7df0886fac0811621b70d61b28ca36418a41bb7: No such image: sha256:bb41503c2e723478308998afc7df0886fac0811621b70d61b28ca36418a41bb7
ERROR: Encountered errors while bringing up the project.
iangneal added a commit to Rozelette/compose that referenced this issue Dec 10, 2017
@iangneal
Fix docker#5465 by catching a no-image exception in
4b87865 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.
@iangneal iangneal mentioned this issue Dec 10, 2017
Closed
iangneal added a commit to Rozelette/compose that referenced this issue Dec 10, 2017
@iangneal
Fix docker#5465 by catching a no-image exception in
b1721dc 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
1baab93 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>
@iangneal iangneal mentioned this issue Jan 11, 2018
Closed
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
1c5b05f 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
9134e96 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
2a955b6 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
a5b97d5 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
9a465a0 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
31ed61f 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
3cc3078 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
23239f7 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
475c782 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
d0cc1a7 
get_container_data_volumes. Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Signed-off-by: Ian Glen Neal <[email protected]>

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
d360611 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
e2fd741 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
iangneal added a commit to Rozelette/compose that referenced this issue Jan 11, 2018
@iangneal
Fix docker#5465 by catching a no-image exception in
d334e92 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
shin- pushed a commit to shin-/compose that referenced this issue Jan 12, 2018
@iangneal @shin-
Fix docker#5465 by catching a no-image exception in
844b972 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request docker#5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
@shin- shin- mentioned this issue Jan 18, 2018
Merged
shin- pushed a commit that referenced this issue Jan 19, 2018
@iangneal @shin-
Fix #5465 by catching a no-image exception in
7591639 
get_container_data_volumes. The ImageNotFound exception is now bubbled
up to the client, which prompts the user on the desired course of action
(rebuild or abort). Added a case to the unit test to check that
an empty existing image doesn't result in an exception.

Closed old pull request #5466 because I did a rebase and it showed over
300 commits to merge, which I thought was messy.

Signed-off-by: Ian Glen Neal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/volumes kind/bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@shin- @iangneal