Switch glance image uploading from custom ruby scripts to use puppet provider

Bug #1566434 reported by Alex Schultz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Alex Schultz
Mitaka
Fix Committed
Medium
Maksim Malchuk

Bug Description

Related to Bug 1556068 and Bug 1563981, we should be using the puppet glance_image provider to upload images and not the custom ruby scripts that we have today.

This will ensure that the glance images can continue to be managed via LCM in a more consistent fashion and cleans up the duplicate code we have today.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: New → In Progress
description: updated
Revision history for this message
Maksim Malchuk (mmalchuk) wrote :

I think this should be raised to Hight due to: https://bugs.launchpad.net/fuel/+bug/1567870

Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

We passed SCF for 9.0. Moving the medium priority bug to 10.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/301835
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=85e9a608f40d3b507154bdb600f0b93cc015e877
Submitter: Jenkins
Branch: master

commit 85e9a608f40d3b507154bdb600f0b93cc015e877
Author: Alex Schultz <email address hidden>
Date: Tue Apr 5 11:09:49 2016 -0600

    Pass glance image parameters as a hash

    This change updates the deployment serializer for the test vm data to
    pass the glance properties as a hash that can be used by the
    glance_image provider rather than using the glance_properties string
    that is currently in place. The glance_properties string should be
    considered deprecated and anything that uses it should switch to the
    properties hash.

    DocImpact: glance_properties string provided as part of the
    test_vm_image hiera data is deprecated in favor of the properties hash
    provided by this change

    Change-Id: I79a9b20d89ae00a7ceaa24c4ce655cbd16972c30
    Partial-Bug: #1566434

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

Reviewed: https://review.openstack.org/298456
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=300f8c230d4ee2981c2f43df682e0326b1516789
Submitter: Jenkins
Branch: master

commit 300f8c230d4ee2981c2f43df682e0326b1516789
Author: Alex Schultz <email address hidden>
Date: Mon Mar 28 16:35:14 2016 -0600

    Rewrite upload_cirros using puppet

    This change replaces the upload_cirros ruby script with a puppet
    equivalent that will wait for the glance backends and use the
    glance_image provider to upload the image to glance.

    Change-Id: I3bf439539e5db87fd3598e779165934a78ed9570
    Closes-Bug: #1566434

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

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

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

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

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

commit f671a5305675bebc49b63209663eff78cc994a8c
Author: Alex Schultz <email address hidden>
Date: Mon Mar 28 16:35:14 2016 -0600

    Rewrite upload_cirros using puppet

    This change replaces the upload_cirros ruby script with a puppet
    equivalent that will wait for the glance backends and use the
    glance_image provider to upload the image to glance.

    Change-Id: I3bf439539e5db87fd3598e779165934a78ed9570
    Closes-Bug: #1618662
    Closes-Bug: #1566434
    (cherry picked from commit 300f8c230d4ee2981c2f43df682e0326b1516789)

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

Reviewed: https://review.openstack.org/363535
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=f77e0733c78eb970af77812916aa8520a163f6af
Submitter: Jenkins
Branch: stable/mitaka

commit f77e0733c78eb970af77812916aa8520a163f6af
Author: Alex Schultz <email address hidden>
Date: Tue Apr 5 11:09:49 2016 -0600

    Pass glance image parameters as a hash

    This change updates the deployment serializer for the test vm data to
    pass the glance properties as a hash that can be used by the
    glance_image provider rather than using the glance_properties string
    that is currently in place. The glance_properties string should be
    considered deprecated and anything that uses it should switch to the
    properties hash.

    DocImpact: glance_properties string provided as part of the
    test_vm_image hiera data is deprecated in favor of the properties hash
    provided by this change

    Change-Id: I79a9b20d89ae00a7ceaa24c4ce655cbd16972c30
    Partial-Bug: #1618662
    Partial-Bug: #1566434
    (cherry picked from commit 85e9a608f40d3b507154bdb600f0b93cc015e877)

tags: added: in-stable-mitaka
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.