[engine] Deploy fails silently if name of deploying environment is not unique

Bug #1314106 reported by Ekaterina Chernova
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Murano
Fix Released
High
Steve McLellan

Bug Description

Steps to reproduce:
1) Create environment with name "one"
2) Add application (Telnet as an example) and deploy the environment
3) Create another environment with the same name"one"
4) Send environment to deploy

Actual result:

Environment has status Ready, but actually deploy is not started because of stack with the same name as the environment name already exists
Here is traceback from murano-engine
http://paste.openstack.org/show/77637/

Timur Sufiev (tsufiev-x)
summary: - Deploy fails silently if name of deploying environment is not unique
+ [engine] Deploy fails silently if name of deploying environment is not
+ unique
Changed in murano:
status: New → Confirmed
importance: Undecided → High
milestone: none → juno-1
Steve McLellan (sjmc7)
Changed in murano:
assignee: nobody → Steve McLellan (sjmc7)
Revision history for this message
Steve McLellan (sjmc7) wrote :

The simplest solution to this would be to disallow duplicate environment names. I'd suggest making tenant_id/environment name unique and then this will get fixed as a side effect. Ekaterina - would that be an OK solution?

Revision history for this message
Ekaterina Chernova (efedorova) wrote :

Yes, that works.

ruhe (ruhe)
Changed in murano:
status: Confirmed → Triaged
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano (master)

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

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

Reviewed: https://review.openstack.org/97347
Committed: https://git.openstack.org/cgit/stackforge/murano/commit/?id=d8dacef34f9afea84b1f5206e4977e9c6e2f2c33
Submitter: Jenkins
Branch: master

commit d8dacef34f9afea84b1f5206e4977e9c6e2f2c33
Author: Steve McLellan <email address hidden>
Date: Mon Jun 2 14:47:09 2014 -0500

    Make environment name unique within a tenant

    Duplicate environment names cause all kinds of problems because of
    naming conventions used. There's no real reason to allow them (heat
    allows duplicate stack names but there was a lot of discussion about
    it, and the reason for having them was nobody wanted to remove them).
    In our case, duplicate environment names cause trouble and there is
    no advantage to them.

    Server will return a 409 Conflict in this instance.

    Change-Id: Ibe694c3dab19d77f6de3aa31844ff3e3cc0d9eb4
    Closes-Bug: 1314106

Changed in murano:
status: In Progress → Fix Committed
ruhe (ruhe)
Changed in murano:
status: Fix Committed → Fix Released
Changed in murano:
milestone: juno-1 → 2014.2
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.