Heat stack_update needs password

Bug #1394505 reported by Ladislav Smola
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
Ladislav Smola

Bug Description

This call needs also password:

https://github.com/openstack/horizon/blob/05ed60b123550c8e05746330ef2f228bdaccda50/openstack_dashboard/api/heat.py#L108

It should be
heat.heatclient(request, password).stacks.update(stack_id, **kwargs)

Otherwise it fails. Depends on configuration of heat, but default configuration needs password for create and update.

Tags: heat
Ladislav Smola (lsmola)
Changed in horizon:
assignee: nobody → Ladislav Smola (lsmola)
tags: added: heat
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
status: New → In Progress
Changed in horizon:
assignee: Ladislav Smola (lsmola) → Lin Hua Cheng (lin-hua-cheng)
Changed in horizon:
milestone: none → kilo-1
importance: Undecided → Medium
Changed in horizon:
assignee: Lin Hua Cheng (lin-hua-cheng) → Radomir Dopieralski (thesheep)
Changed in horizon:
assignee: Radomir Dopieralski (thesheep) → Ladislav Smola (lsmola)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/135914
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=01f5d1e7189964699a227240126097f792b9b658
Submitter: Jenkins
Branch: master

commit 01f5d1e7189964699a227240126097f792b9b658
Author: Ladislav Smola <email address hidden>
Date: Thu Nov 20 10:40:02 2014 +0100

    Stack update call needs password

    Stack update call needs password otherwise it fails.
    Added missing tests.

    Change-Id: I99b0ce460c772862b22fa0216466589893948178
    Closes-bug: #1394505

Changed in horizon:
status: In Progress → Fix Committed
Revision history for this message
Marcos Lobo (marcos-fermin-lobo) wrote :

Is it possible to backport this patch to Icehouse?

Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-1 → 2015.1.0
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.