OpenStack Dashboard (Horizon)

volume backups fail if no container name provided

Bug #1658137 reported by Eric Peterson on 2017-01-20

This bug affects 4 people

Affects		Status	Importance	Assigned to	Milestone
	Cinder	Invalid	Undecided	Unassigned
	OpenStack Dashboard (Horizon)	Fix Released	High	Eric Peterson	OpenStack Dashboard (Horizon) ocata-3 "o-3"

Bug Description

if no container name is provided, horizon ends up passing "" to the cinder client api call. This is an invalid container name - None is the correct value to pass for this to work as desired.

Tags:

Eric Peterson (ericpeterson-l) on 2017-01-23

Changed in horizon:
assignee:	nobody → Eric Peterson (ericpeterson-l)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-01-23: Fix proposed to horizon (master)

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

Changed in horizon:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-01-24: Fix merged to horizon (master)

Reviewed: https://review.openstack.org/424261
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=90a9af2e0b4524b4dec1caafbb6159cb587878d4
Submitter: Jenkins
Branch: master

commit 90a9af2e0b4524b4dec1caafbb6159cb587878d4
Author: Eric Peterson <email address hidden>
Date: Mon Jan 23 10:39:52 2017 -0700

Autogenerate container name correctly for vol backup

    The volume backup code is passing '' as a default container name,
    which will cause cinder/swift to fail. If no container name is provided,
    the default value of None will enable the auto creation of a container correctly.

Change-Id: I7813153c65652ce6e1a5ac03204a8738eefdc64e
Closes-Bug: #1658137

Changed in horizon:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-01-24: Fix proposed to horizon (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/424634

Revision history for this message

Adrien Cunin (adri2000) wrote on 2017-01-24:

There was already a bug report (bug #1603647) as well as a patch WiP.

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-01-26: Fix merged to horizon (stable/newton)

Reviewed: https://review.openstack.org/424634
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=3238dc0738952b54871d214fbddbe1b33ed9fc84
Submitter: Jenkins
Branch: stable/newton

commit 3238dc0738952b54871d214fbddbe1b33ed9fc84
Author: Eric Peterson <email address hidden>
Date: Mon Jan 23 10:39:52 2017 -0700

Autogenerate container name correctly for vol backup

    Change-Id: I7813153c65652ce6e1a5ac03204a8738eefdc64e
    Closes-Bug: #1658137
    (cherry picked from commit 90a9af2e0b4524b4dec1caafbb6159cb587878d4)