cleanup upload_glance workarounds that were added when debugging CI issues

Bug #1563981 reported by Alex Schultz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Alex Schultz
Mitaka
Fix Released
High
Alex Schultz
Newton
Fix Released
High
Alex Schultz

Bug Description

While working Bug 1556068, we introduced some additional code into the upload_cirros.rb script. We should remove the additional items if they are no longer used as they did not address the core issue. It would be beneficial to leverage the upstream glance_image provider instead of the custom ruby script we have today.

https://review.openstack.org/#/c/297679/
https://review.openstack.org/#/c/295828/

Changed in fuel:
status: New → Confirmed
Revision history for this message
Alex Schultz (alex-schultz) wrote :

Puppet provider work moved to Bug 1566434 as it introduces way more complexity and it's too close to SCF to consider. This bug should be used to track the cleanup of the extra code that was introduced while attempting to address Bug 1556068

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/301896

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

Reviewed: https://review.openstack.org/301896
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=031df97cb9cccb80cda562a2782aaf2c7fe45cba
Submitter: Jenkins
Branch: master

commit 031df97cb9cccb80cda562a2782aaf2c7fe45cba
Author: Alex Schultz <email address hidden>
Date: Tue Apr 5 14:13:52 2016 -0600

    Remove extra loops from upload_cirros script

    This change removes the extra loops from the upload_cirros script that
    were added while attempting to debug Bug 1556068. These extra loops
    could cause the task to exceed the task timeout and ultimately did not
    address the main issue. This change cleans up the extra code that was
    added for debugging purposes now that the previous bug has been closed.

    Change-Id: I26cfe1d0f6f2f6d8fb2e9fbba2cd964574c7bbef
    Closes-Bug: #1563981

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

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/303477

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

Reviewed: https://review.openstack.org/303477
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=cda2e7bbbf07c85e7df4966554aac45acf8ef422
Submitter: Jenkins
Branch: stable/mitaka

commit cda2e7bbbf07c85e7df4966554aac45acf8ef422
Author: Alex Schultz <email address hidden>
Date: Tue Apr 5 14:13:52 2016 -0600

    Remove extra loops from upload_cirros script

    This change removes the extra loops from the upload_cirros script that
    were added while attempting to debug Bug 1556068. These extra loops
    could cause the task to exceed the task timeout and ultimately did not
    address the main issue. This change cleans up the extra code that was
    added for debugging purposes now that the previous bug has been closed.

    Change-Id: I26cfe1d0f6f2f6d8fb2e9fbba2cd964574c7bbef
    Closes-Bug: #1563981
    (cherry picked from commit 031df97cb9cccb80cda562a2782aaf2c7fe45cba)

tags: added: dev-to-verify
tags: removed: dev-to-verify
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.