Incorrect work of murano-repository in non-ha mode

Bug #1328490 reported by Anastasia Kuznetsova
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Fuel Library (Deprecated)
Mirantis OpenStack
Fix Released
High
MOS Murano

Bug Description

FUEL 4.1.1 ISO RC-2
QEMU, Neutron GRE, 1 controller, 1 compute, Murano

When user try to perform an action with packages he see error messages in UI (such as in attached screenshot) and traceback in /var/log/murano/murano-dashboard.log
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/lib/python2.7/dist-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/lib/python2.7/dist-packages/muranorepository/api/ha_sync.py", line 33, in wrapper
    request_data = RequestData(request)
  File "/usr/lib/python2.7/dist-packages/muranorepository/api/ha_sync.py", line 65, in __init__
    self.path = copy.copy(req.full_path)
  File "/usr/lib/python2.7/dist-packages/werkzeug/local.py", line 336, in __getattr__
    return getattr(self._get_current_object(), name)
AttributeError: 'Request' object has no attribute 'full_path'

In traceback we see that ha_sync.py was executed in non-ha.

Tags: murano
Revision history for this message
Anastasia Kuznetsova (akuznetsova) wrote :
description: updated
Revision history for this message
Anastasia Kuznetsova (akuznetsova) wrote :
Changed in fuel:
milestone: none → 4.1.2
assignee: nobody → Fuel Library Team (fuel-library)
Changed in fuel:
importance: Undecided → Medium
Changed in fuel:
status: New → Confirmed
Changed in fuel:
importance: Medium → Critical
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

Fix commited to stable/4.1:
https://review.openstack.org/#/c/99066/

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/4.1)

Reviewed: https://review.openstack.org/99066
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=d3cdce20d13a592c98331d47bf47ebae64ab2b27
Submitter: Jenkins
Branch: stable/4.1

commit d3cdce20d13a592c98331d47bf47ebae64ab2b27
Author: Dmitry Ilyin <email address hidden>
Date: Tue Jun 10 18:12:00 2014 +0400

    Fix ha_nodes and data_dir options for murano

    Closes-Bug: 1328490
    Change-Id: Ifd7a171bdab82b97d0d144620452c20444447826

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Kirill Omelchenko (komelchenko) wrote :

Same problem while trying to toggle state and delete service definition.

Revision history for this message
Kirill Omelchenko (komelchenko) wrote :
Revision history for this message
Kirill Omelchenko (komelchenko) wrote :

Using FUEL build 375

{"build_id": "2014-06-26_04-36-50", "mirantis": "yes", "build_number": "375", "nailgun_sha": "bdc747cd70721874d1e8af36181fd4179cce4fce", "ostf_sha": "f4f15b4d98459650c1945b0efc30290a619be824", "fuelmain_sha": "ae77ae1c21a8715a63c7b1ca3f7a7ecadaff68aa", "astute_sha": "55df06b2e84fa5d71a1cc0e78dbccab5db29d968", "release": "4.1.1", "fuellib_sha": "a93544b7d7395043f3e55fec055841b8257be620"}

Changed in fuel:
status: Fix Committed → Confirmed
Changed in mos:
status: New → Confirmed
importance: Undecided → High
Changed in fuel:
importance: Critical → High
Changed in mos:
assignee: nobody → MOS Murano (mos-murano)
milestone: none → 4.1.2
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

We have fix for this issue:
https://review.openstack.org/#/c/99066/

We tested this fix on Express project and it works fine.

Changed in mos:
status: Confirmed → Fix Committed
Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
Kirill Omelchenko (komelchenko) wrote :

Verified on FUEL build 380.

{"build_id": "2014-07-01_00-41-14", "mirantis": "yes", "build_number": "380", "nailgun_sha": "bdc747cd70721874d1e8af36181fd4179cce4fce", "ostf_sha": "f4f15b4d98459650c1945b0efc30290a619be824", "fuelmain_sha": "ae77ae1c21a8715a63c7b1ca3f7a7ecadaff68aa", "astute_sha": "55df06b2e84fa5d71a1cc0e78dbccab5db29d968", "release": "4.1.1", "fuellib_sha": "2a5f7cb14825d619a632213c040477b77f0ad123"}

Changed in mos:
status: Fix Committed → Fix Released
Changed in fuel:
status: Fix Committed → Fix Released
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.