Don't run `dockerctl build all` when it passed successfully.

Bug #1333213 reported by Evgeniy L
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Matthew Mosesohn
5.0.x
Fix Committed
High
Matthew Mosesohn

Bug Description

When I run puppet apply on the master it tries to run 'dockerctl build all' again.

During the upgrade we have to run puppet second time on the host system because we need to upgrade dockerctl, fuelclient, etc.
In this case we don't need to run dockerctl again.

One of solutions is to create separate manifest for host system which we will run in case of upgrades.

http://paste.openstack.org/show/84682/

Tags: library
Changed in fuel:
status: New → Triaged
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Matthew Mosesohn (raytrac3r)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/103097

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/103097
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=6abeeb7ae534e691f01311e2c84b694c167d4a69
Submitter: Jenkins
Branch: master

commit 6abeeb7ae534e691f01311e2c84b694c167d4a69
Author: Matthew Mosesohn <email address hidden>
Date: Fri Jun 27 15:26:02 2014 +0400

    Don't build docker containers if any exist

    Skips dockerctl build all in puppet if any
    docker containers (running or stopped) exist.

    Change-Id: Idca4e43f2b714ae953a0755b69b215e9efc4c56a
    Closes-Bug: #1333213

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/103851

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/5.0)

Reviewed: https://review.openstack.org/103851
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=f5e842546aba457fb00c604ecd7eaae1ea0a94ea
Submitter: Jenkins
Branch: stable/5.0

commit f5e842546aba457fb00c604ecd7eaae1ea0a94ea
Author: Matthew Mosesohn <email address hidden>
Date: Fri Jun 27 15:26:02 2014 +0400

    Don't build docker containers if any exist

    Skips dockerctl build all in puppet if any
    docker containers (running or stopped) exist.

    Change-Id: Idca4e43f2b714ae953a0755b69b215e9efc4c56a
    Closes-Bug: #1333213
    (cherry picked from commit 6abeeb7ae534e691f01311e2c84b694c167d4a69)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.