Mirantis OpenStack

[murano][glare] importing a package with !yaql tag fails

Bug #1587446 reported by Kirill Zaitsev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Status tracked in 10.0.x
10.0.x
Fix Committed
High
MOS Murano
Mirantis OpenStack 10.0
8.0.x
Invalid
High
MOS Murano
Mirantis OpenStack 8.0-updates
9.x
Fix Released
High
Kirill Zaitsev
Mirantis OpenStack 9.0

Bug Description

When glare is used client has to parse the classes to store inheritance info in glare. When doing so client apparently uses unmodified loader, thus importing a package with "yaql!"

The error is:
Error could not determine a constructor for the tag '!yaql'
  in "Classes/OpenLDAP.yaml", line 86, column 9 occurred while installing package XXX

to reproduce — try importing io.murano.opaas.CiCdEnvironment from a.o.o

Expected result: no errors when importing such packages.

upstream bug: https://bugs.launchpad.net/python-muranoclient/+bug/1586028

See original description
Tags: area-murano
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/python-muranoclient (9.0/mitaka)

Fix proposed to branch: 9.0/mitaka
Change author: Kirill Zaitsev <email address hidden>
Review: https://review.fuel-infra.org/21550

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/python-muranoclient (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/21550
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 6a909b9243682ca77c970d97603e48255dd7ab3a
Author: Kirill Zaitsev <email address hidden>
Date: Thu Jun 2 11:06:55 2016

Use YaqlYamlLoader to load classes

This commit changes the loader, that loads MuranoPL classes during
package upload to YaqlYamlLoader. This would allow importing and parsing
packages that contain '!yaql' tag without any errors.

Change-Id: I196b6394bd7717db8a954a9b3dc26b2eaf643d52
Closes-Bug: #1587446

Revision history for this message
Alex Kholkin (akholkin) wrote :

Verified on #445 ISO
9.0-mos.all

Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

marking 10.0 as fixed in upstream

description: updated
Revision history for this message
Kirill Zaitsev (kzaitsev) wrote :

fix has been backported to the stable/liberty brunch, this can be fetched with the usual merge of fixes from stable branches to 8.0

Revision history for this message
Sergii Rizvan (srizvan) wrote :

Fix for 8.0 has been fetched via sync with stable/liberty. That's why status for 8.0 has been set as 'Invalid'.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/python-muranoclient (mcp/newton)

Fix proposed to branch: mcp/newton
Change author: Kirill Zaitsev <email address hidden>
Review: https://review.fuel-infra.org/33701

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/python-muranoclient (11.0/ocata)

Fix proposed to branch: 11.0/ocata
Change author: Kirill Zaitsev <email address hidden>
Review: https://review.fuel-infra.org/34018

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/python-muranoclient (mcp/ocata)

Fix proposed to branch: mcp/ocata
Change author: Kirill Zaitsev <email address hidden>
Review: https://review.fuel-infra.org/34622

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/python-muranoclient (11.0/ocata)

Change abandoned by Roman Podoliaka <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/34018
Reason: we do not need 11.0/ocata anymore - use mcp/ocata instead

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/python-muranoclient (mcp/ocata)

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34622
Reason: The patch has been upstreamed and is not required for mcp/ocata.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/python-muranoclient (mcp/newton)

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/newton
Review: https://review.fuel-infra.org/33701
Reason: The patch has been upstreamed and is not required for mcp/newton.

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.