Murano

Misleading error message during creating new environment in case metadata-repository is not available

Bug #1263463 reported by Timur Sufiev on 2013-12-22

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Murano	Fix Released	Medium	Timur Sufiev	Murano 0.4.1

Bug Description

If Murano Metadata service is not available, an attempt to create a new Service in an Environment will lead to somewhat onfusing error message: 'Error: An error occurred. Please try again later.'

Actually, there is the following error in /var/log/murano/murano-dashboard.log: 'CommunicationError: Error communicating with http://localhost:8084/v1 [Errno 111] Connection refused'. That means that Murano Metadata Service is not available. Should provide more sensible error message in that case.

Tags:

Timur Sufiev (tsufiev-x) on 2013-12-22

Changed in murano:
importance:	Undecided → Medium
milestone:	none → 0.4.1
assignee:	nobody → Timur Sufiev (tsufiev-x)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-12-25: Fix proposed to murano-dashboard (master)

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

Changed in murano:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-12-30: Fix merged to murano-dashboard (master)

Reviewed: https://review.openstack.org/64041
Committed: https://git.openstack.org/cgit/stackforge/murano-dashboard/commit/?id=a6665d34ab98a295939578de7ee99d0d9bdef8b0
Submitter: Jenkins
Branch: master

commit a6665d34ab98a295939578de7ee99d0d9bdef8b0
Author: Timur Sufiev <email address hidden>
Date: Wed Dec 25 18:57:55 2013 +0400

Show more sensible error messages during Service Creation.

    If Metadata Repository is not available, show more concise error
    messages.
    Also refactor wrapper for handling Metadata Repository exceptions,
    thus allowing to throw-away some boilerplate code.

Change-Id: I6acd94bf2ca397941f45bc4c8d9e9fe74678b5ee
Closes-bug: #1263463

Changed in murano:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-13: Fix proposed to murano-dashboard (release-0.4)

Fix proposed to branch: release-0.4
Review: https://review.openstack.org/66301

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-01-13: Fix merged to murano-dashboard (release-0.4)

Reviewed: https://review.openstack.org/66301
Committed: https://git.openstack.org/cgit/stackforge/murano-dashboard/commit/?id=c70a0084564631686228b496d05c54f224e414ff
Submitter: Jenkins
Branch: release-0.4

commit c70a0084564631686228b496d05c54f224e414ff
Author: Igor Yozhikov <email address hidden>
Date: Thu Dec 12 13:52:08 2013 +0400

Cherry-pick the following commits from master:

    * Made improvements for rpmbuild
      I1dd7a54f7ade5414ad07cb8e12d0b018ecbc63c2
    * Return status code instead of detailes
      Id0fe5dcae7ae19dee455a7ae8928e4a0d55fca26
    * Add ability to delete file from service
      I6490ac2a391149a8660d5800b62ce8196e47bf00
    * Add author field to the Compose Service form
      Increment service version after modification
      Forbid to edit version
      Closes bugs: #1262237, #1262239, #1262242
      I5061704c7a3ca96c26d79edfb9f551df3e2f80e2
    * Update instance link formation due to changes in Havana
      Fixes-bug: #1262713
      I56e174f35ef4a60cffe31b0e619b53845a701ad6
    * Add link with stack details in service details
      Ic3e34c9d341309ffd005a5479b2a6ef12b3bfe05
      Implements: blueprint add-stack-link
    * Rename dynamic_ui package and normalize it.
      * Move muranodashboard.environments.services package to
        muranodashboard.dynamic_ui.
      * Move all code from dynamic_ui/__init__.py to dynamic_ui/services.py
        thus having normal empty __init__.py for dynamic_ui package (as in
          other packages).
      I395ee621578cb417314fe4960d74108deb848023
    * Minor refactoring of dynamic_ui package.
      Remove one inner import in dynamic_ui.services and rewrite a bit
      iterate_over_services() function.
      If14022435a5ef269a448d5f568fe0f33421c9f23
    * Do not log exception, if it is Http302 (redirect).
      I3488fc8523ec2e4628a26006c0211d2d468f189a
    * Show more sensible error messages during Service Creation.
      If Metadata Repository is not available, show more concise error
      messages.
      Also refactor wrapper for handling Metadata Repository exceptions,
      thus allowing to throw-away some boilerplate code.
      I6acd94bf2ca397941f45bc4c8d9e9fe74678b5ee
      Closes-bug: #1263463

Change-Id: Ia814eccf6841f7dcc153cf0c3c7c793f0f358023

Reviewed:  https://review.openstack.org/66301
Committed: https://git.openstack.org/cgit/stackforge/murano-dashboard/commit/?id=c70a0084564631686228b496d05c54f224e414ff
Submitter: Jenkins
Branch:    release-0.4

commit c70a0084564631686228b496d05c54f224e414ff
Author: Igor Yozhikov <iyozhikov@mirantis.com>
Date:   Thu Dec 12 13:52:08 2013 +0400

Cherry-pick the following commits from master:
    
    * Made  improvements for rpmbuild
      I1dd7a54f7ade5414ad07cb8e12d0b018ecbc63c2
    * Return status code instead of detailes
      Id0fe5dcae7ae19dee455a7ae8928e4a0d55fca26
    * Add ability to delete file from service
      I6490ac2a391149a8660d5800b62ce8196e47bf00
    * Add author field to the Compose Service form
      Increment service version after modification
      Forbid to edit version
      Closes bugs: #1262237, #1262239, #1262242
      I5061704c7a3ca96c26d79edfb9f551df3e2f80e2
    * Update instance link formation due to changes in Havana
      Fixes-bug: #1262713
      I56e174f35ef4a60cffe31b0e619b53845a701ad6
    * Add link with stack details in service details
      Ic3e34c9d341309ffd005a5479b2a6ef12b3bfe05
      Implements: blueprint add-stack-link
    * Rename dynamic_ui package and normalize it.
      * Move muranodashboard.environments.services package to
        muranodashboard.dynamic_ui.
      * Move all code from dynamic_ui/__init__.py to dynamic_ui/services.py
        thus having normal empty __init__.py for dynamic_ui package (as in
          other packages).
      I395ee621578cb417314fe4960d74108deb848023
    * Minor refactoring of dynamic_ui package.
      Remove one inner import in dynamic_ui.services and rewrite a bit
      iterate_over_services() function.
      If14022435a5ef269a448d5f568fe0f33421c9f23
    * Do not log exception, if it is Http302 (redirect).
      I3488fc8523ec2e4628a26006c0211d2d468f189a
    * Show more sensible error messages during Service Creation.
      If Metadata Repository is not available, show more concise error
      messages.
      Also refactor wrapper for handling Metadata Repository exceptions,
      thus allowing to throw-away some boilerplate code.
      I6acd94bf2ca397941f45bc4c8d9e9fe74678b5ee
      Closes-bug: #1263463
    
    Change-Id: Ia814eccf6841f7dcc153cf0c3c7c793f0f358023

Anastasia Kuznetsova (akuznetsova) on 2014-01-21

tags:

added: verified

Anastasia Kuznetsova (akuznetsova) on 2014-02-12

Changed in murano:
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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