upload_cirros doesn't support UTF-8

Bug #1618662 reported by Nikita Koshikov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Invalid
Medium
Maksim Malchuk
Mitaka
Fix Released
Medium
Maksim Malchuk

Bug Description

FUEL 9

Steps to reproduce:
1)Deploy env
2)upload to glance image that contains non-asci character
3)Redeploy-changes

Expected result - deployment should succeed

Actual result - deployment fails:

Here is example of image name:

'Ubuntu 14.04\xc2\xa0for KVM VMs with IPv6'

And this is error from ruby:
ruby /etc/puppet/modules/osnailyfacter/modular/astute/upload_cirros.rb

'ascii' codec can't encode character u'\xa0' in position 4943: ordinal not in range(128)
'ascii' codec can't encode character u'\xa0' in position 4943: ordinal not in range(128)
'ascii' codec can't encode character u'\xa0' in position 4943: ordinal not in range(128)
'ascii' codec can't encode character u'\xa0' in position 4943: ordinal not in range(128)
'ascii' codec can't encode character u'\xa0' in position 4943: ordinal not in range(128)
u.rb:95:in `wait_for_glance': Could not get a list of glance images! (RuntimeError)
        from u.rb:162:in `<main>'

description: updated
tags: added: area-library
Changed in fuel:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Fuel Sustaining (fuel-sustaining-team)
milestone: none → 10.0
Revision history for this message
Maksim Malchuk (mmalchuk) wrote :

Instead of fixing upload_cirros.rb we should backport the puppet version of the task.
https://bugs.launchpad.net/fuel/+bug/1566434

Changed in fuel:
status: Confirmed → Invalid
assignee: Fuel Sustaining (fuel-sustaining-team) → Maksim Malchuk (mmalchuk)
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
tags: added: on-verification
Revision history for this message
ElenaRossokhina (esolomina) wrote :

Verified on 9.2 snapshot#553

tags: removed: on-verification
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.