Mirantis OpenStack

Murano responds with 500 for package already exists.

Bug #1436536 reported by Kirill Zaitsev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
High
Kirill Zaitsev
Mirantis OpenStack 6.1

Bug Description

Upstream bug https://bugs.launchpad.net/murano/+bug/1463795

Instead of raising the appropriate 409 conflict, murano engine returns 500 error if package is already registered.

See original description
Tags: murano
Kirill Zaitsev (kzaitsev)
Changed in mos:
assignee: nobody → Kirill Zaitsev (kzaitsev)
status: New → In Progress
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

Need to backport this fix to MOS from upstream

Changed in mos:
importance: Undecided → High
milestone: none → 6.1
status: In Progress → Confirmed
Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1.mira12.git.30c40c5.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1~mira1+git.30c40c5.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /trusty-fuel-6.1-stable-4965/

Kirill Zaitsev (kzaitsev)
Changed in mos:
status: Confirmed → In Progress
Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1.mira12.git.128ecd3.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1~mira1+git.128ecd3.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /trusty-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1.mira12.git.c1b7cbf.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1~mira1+git.c1b7cbf.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /trusty-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1.mira12.git.0a08558.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1~mira1+git.0a08558.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Kirill Zaitsev
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /trusty-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1.mira3.git.774a888.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Serg Melikyan
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2, package release == fuel6.1~mira1+git.774a888.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Serg Melikyan
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /trusty-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2.2, package release == fuel6.1.mira3.git.e0c2711.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Serg Melikyan
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /centos-fuel-6.1-stable-4965/

Revision history for this message
OSCI Robot (oscirobot) wrote :

package murano has been built for project openstack/murano
Package version == 2014.2.2, package release == fuel6.1~mira3+git.e0c2711.cee1e6c

Changeset: https://review.fuel-infra.org/4965
project: openstack/murano
branch: openstack-ci/fuel-6.1/2014.2
author: Kirill Zaitsev
committer: Serg Melikyan
subject: Return 409 status instead of 500 if package already exists
status: patchset-created

Files placed on repository:

NOTE: Changeset is not merged, created temporary package repository.
 repository URL: /trusty-fuel-6.1-stable-4965/

Serg Melikyan (smelikyan)
Changed in mos:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Kuznetsova (akuznetsova) wrote :

Verified in 6.1

{"build_id": "2015-04-13_22-54-31", "ostf_sha": "c2a76a60ec4ebbd78e508216c2e12787bf25e423", "build_number": "310", "release_versions": {"2014.2-6.1": {"VERSION": {"build_id": "2015-04-13_22-54-31", "ostf_sha": "c2a76a60ec4ebbd78e508216c2e12787bf25e423", "build_number": "310", "api": "1.0", "nailgun_sha": "d22c074dec091e5ddd8ea3003c37665058303cd5", "openstack_version": "2014.2-6.1", "production": "docker", "python-fuelclient_sha": "9208ff4a08dcb674ce2df132399a5aa3ddfac21c", "astute_sha": "d96a80b63198a578b2c159edbd76048819039eb0", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "335d3ed09ed79bd37e1f7a90442c4831c8845582", "fuellib_sha": "8b80657e9ceed8d59c2dff1c11e1481c7e69380e"}}}, "auth_required": true, "api": "1.0", "nailgun_sha": "d22c074dec091e5ddd8ea3003c37665058303cd5", "openstack_version": "2014.2-6.1", "production": "docker", "python-fuelclient_sha": "9208ff4a08dcb674ce2df132399a5aa3ddfac21c", "astute_sha": "d96a80b63198a578b2c159edbd76048819039eb0", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "335d3ed09ed79bd37e1f7a90442c4831c8845582", "fuellib_sha": "8b80657e9ceed8d59c2dff1c11e1481c7e69380e"}

Changed in mos:
status: Fix Committed → Fix Released
Kirill Zaitsev (kzaitsev)
description: updated
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.