Mirantis OpenStack

[Murano] Exceptions get muted when occur inside Parallel block

Bug #1498186 reported by Stan Lagun
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Invalid
High
MOS Murano
Mirantis OpenStack 8.0
6.1.x
Won't Fix
High
Stan Lagun
Mirantis OpenStack 6.1-updates
7.0.x
Fix Released
High
Stan Lagun
Mirantis OpenStack 7.0-mu-1
8.0.x
Invalid
High
MOS Murano
Mirantis OpenStack 8.0

Bug Description

If MuranoPL function fails with exception this exception may be silenced under some conditions when the function was invoked from Parallel macro. As a result exception will not be reported to user and deployment will continue as if nothing happened.
As an example agent timeouts that happen within Parallel block will look like if deployment froze for 1 hour and then sucessfuly continued when in reality it is not.

Stan Lagun (slagun)
tags: added: murano
Revision history for this message
Ekaterina Chernova (efedorova) wrote :

Stan, if you think, that this bug has high priority, please change.

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

Not reproduced in Liberty

Serg Melikyan (smelikyan)
summary: - [Murano] xceptions get muted when occur inside Parallel block
+ [Murano] Exceptions get muted when occur inside Parallel block
Revision history for this message
Vitaly Sedelnik (vsedelnik) wrote :

Ekaterina, please review the fix and backport to 7.0 if applicable. If not, please update the status accordingly (Won't Fix or Invalid).

Serg Melikyan (smelikyan)
tags: added: 70mu1-confirmed
Revision history for this message
Ekaterina Chernova (efedorova) wrote :

Fix is too hard, it's not a backport.
 we have no time to do it in mu-1

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

Plans changed, we are working on it

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/murano (openstack-ci/fuel-6.1/2014.2)

Fix proposed to branch: openstack-ci/fuel-6.1/2014.2
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/12902

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/murano (openstack-ci/fuel-7.0/2015.1.0)

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/12903

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/murano (openstack-ci/fuel-7.0/2015.1.0)

Reviewed: https://review.fuel-infra.org/12903
Submitter: Vitaly Sedelnik <email address hidden>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: f6ec58b3d39b3acf871b71fd4267f6750f56eb39
Author: Stan Lagun <email address hidden>
Date: Fri Oct 16 12:46:30 2015

Use helper method in Parallel macro

There is a helper method to do parallel execution that
handles a bug in eventlet library. But Parallel macro was
using its own implementation. Now both pselect function
and Parallel macro has common implementation.

Change-Id: I06f1b1d66672707d5f3bc7934bfcbc1580686529
Closes-Bug: #1498186

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/murano (openstack-ci/fuel-7.0/2015.1.0)

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/13205

Revision history for this message
Anastasia Kuznetsova (akuznetsova) wrote :

Bug verification in 7.0-mu1 blocked by https://bugs.launchpad.net/mos/+bug/1513497

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/murano (openstack-ci/fuel-7.0/2015.1.0)

Reviewed: https://review.fuel-infra.org/13205
Submitter: Vitaly Sedelnik <email address hidden>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: 4cde5144526de9c28c57fe17f23642cf43e9f052
Author: Stan Lagun <email address hidden>
Date: Fri Nov 6 14:56:23 2015

Use helper method in Parallel macro

There is a helper method to do parallel execution that
handles a bug in eventlet library. But Parallel macro was
using its own implementation. Now both pselect function
and Parallel macro has common implementation.

NOTE: This updated version of original commit with fixed pep8

Change-Id: I1f10666ab2eb568d5edc8e75a6bbe256383a42bc
Closes-Bug: #1498186

Revision history for this message
Sergey Shevorakov (sshevorakov) wrote :

assigned correct status, since robot failed to do so

Anastasia Kuznetsova (akuznetsova)
tags: added: on-verification
Anastasia Kuznetsova (akuznetsova)
tags: removed: on-verification
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/murano (openstack-ci/fuel-8.0/liberty)

Fix proposed to branch: openstack-ci/fuel-8.0/liberty
Change author: Stan Lagun <email address hidden>
Review: https://review.fuel-infra.org/14077

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/murano (openstack-ci/fuel-8.0/liberty)

Change abandoned by Kirill Zaitsev <email address hidden> on branch: openstack-ci/fuel-8.0/liberty
Review: https://review.fuel-infra.org/14077
Reason: already in liberty

Revision history for this message
Serg Melikyan (smelikyan) wrote :

Fix proposed for 6.1: https://review.fuel-infra.org/12902

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/murano (openstack-ci/fuel-6.1/2014.2)

Change abandoned by Serg Melikyan <email address hidden> on branch: openstack-ci/fuel-6.1/2014.2
Review: https://review.fuel-infra.org/12902
Reason: Is not accepted by maintenance team

Revision history for this message
Denis Meltsaykin (dmeltsaykin) wrote :

Won't fix for 6.x series as they are unsupported and there was no progress on the issue.

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.