[Murano] Repeatative HOT package uploads fail

Bug #1450549 reported by Alexander Tivelkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Invalid
High
Stan Lagun
6.1.x
Fix Released
High
Stan Lagun
7.0.x
Invalid
High
MOS Murano

Bug Description

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

Second and subsequent uploads of HOT-based packages fail until murano-api service is restarted

Seems like some yaql-related tags leak into the yaml loader and attempted to be applied to regular yaml files, which causes exceptions.

Changed in mos:
status: New → Confirmed
Stan Lagun (slagun)
tags: added: customer-found
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/6461

ruhe (ruhe)
summary: - Repeatative HOT package uploads fail
+ [Murano] Repeatative HOT package uploads fail
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/murano (openstack-ci/fuel-6.1/2014.2)

Reviewed: https://review.fuel-infra.org/6461
Submitter: Serg Melikyan <email address hidden>
Branch: openstack-ci/fuel-6.1/2014.2

Commit: af693c112b0ff4efdedc4bae9160403c5de1d598
Author: Stan Lagun <email address hidden>
Date: Thu May 7 14:25:07 2015

Fixed YAQL tag leakage to YAML loader

Implicit yaml tag resolver being registered for YaqlYamlLoader (which is
user to load MuranoPL classes as YAML documents with embedded YAQL
statements) was being registered in the default YAML Dumper which is
used for any YAML serialization.

This was leading to inability to load more then one HOT-based package
per API service lifetime: the first package which was uploaded was
causing the leakage of YAQL statements into the global tag resolver.
This global resolver was used for all subsequent load attempts, so plain
yaml documents with Heat templates were attempted to be loaded as
MuranoPL classes.

This patch fixes this behavior by registering the implicit resolver
without helper method (which actually registers the resolvers at default
Dumper).

Change-Id: I9593dd22bbb32b10077b9a3c3bd2b6a1a22b1024
Closes-Bug: #1450549

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

ISO #464
Now multiple upload of hot-based packages is available, user doesn't need to restart murano-api to continue to work with hot-based apps

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/8206

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

Change abandoned by Kirill Zaitsev <email address hidden> on branch: openstack-ci/fuel-7.0/2015.1.0
Review: https://review.fuel-infra.org/8206

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

Fix is in kilo and will be automatically included in MOS 7.0

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.