Murano with Glare backend does not work with HOT packages

[Alexander Tivelkov]

How to reproduce

0 Configure murano with Glare backend

1 Download HOT template

    wget https://raw.githubusercontent.com/openstack/heat-templates/master/hot/F20/WordPress_Native.yaml -O wordpress.yaml

2 Make murano HOT package

    murano package-create -t wordpress.yaml

3 Import wordpress.zip package to murano via CLI or Dashboard

4 Try to add application to environment

Expected result

Application added to environment

Observed result

Exception.

upstream bug: https://bugs.launchpad.net/murano/+bug/1565805

[Bug Checker Bot (bug-checker)]

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

[Fuel Devops McRobotson (fuel-devops-robot)]

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

Commit: 2246f72b9f8215291909746f5ecb42a0bcf5e242
Author: Alexander Tivelkov <email address hidden>
Date: Wed Apr 13 09:30:08 2016

GetPackageUI API is now called even if Glare is used

Since UI definitions are now always provided by Murano API (even when
the packages themselves are stored in Glare), the muranoclient should
now call the 'get_ui' method of the legacy controller class instead of
the glare one.

Change-Id: Ie8936891bff443cdf5f8b97cb93e63cd781c0696
Depends-on: I4ecfa5e40d5d7c49e2c89f8d369afc127c367eee
Closes-bug: #1565904

[Alex Kholkin (akholkin)]

Issue still reprodused on 390 mos-all.

Via dashboard I can't add application into environment and got this traceback from horizon:
https://paste.mirantis.net/show/2302/

Via CLI I'm able to add application into environment, but can't deploy env, and got this:
https://paste.mirantis.net/show/2303/

I'm using this HOT template https://raw.githubusercontent.com/openstack/heat-templates/master/hot/hello_world.yaml

Package was successfully created, imported, and correctly listed in 'murano package-list' output.

Moving to Confirmed state.

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: 9.0/mitaka
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/21263

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: 9.0/mitaka
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/21264

[Fuel Devops McRobotson (fuel-devops-robot)]

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

Commit: 8352bfdb655e58bc010e5aad685bca2b8502a209
Author: Alexander Tivelkov <email address hidden>
Date: Thu May 26 09:18:30 2016

Forced flush of tmp file to disk on ui retrieval

UI retrieval routine loads the package's temporary file from disk
to properly render its ui. This procedure does not close the file
before reading it, so the write buffers should be flushed first.

This was causing the bug when the UI could not be properly fetched
from the API.

Change-Id: I95da22de2e367b5e1ad314c797d6d0c1768b473d
Partial-bug: #1565904

[Fuel Devops McRobotson (fuel-devops-robot)]

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

Commit: 33cbd112dcf8eef7a755f16f4c423c41c873bef7
Author: Alexander Tivelkov <email address hidden>
Date: Thu May 26 09:19:46 2016

Added package references to generated UIs

Generated UIs of the non-muranoPL packages (HOT, cloudify_tosca and
CSAR) reference the type of object model's object by class FQNs.
However, when these packages are uploaded to glare, their class
contents are not indexed since the client which uploads them has no
knowledge of the generated classes.

To workaround this issue it is proposed to reference objects not just
by class FQNs but to include the package FQN and the version, so the
engine does not have to rely on Glare's indexes.

Change-Id: I5ca0b2f442def431fd08badbcf874ed35d78f92c
Partial-bug: #1565904

[Alex Kholkin (akholkin)]

Verified on #418 ISO
9.0-mos.all

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: mcp/newton
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/33523

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: mcp/newton
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/33609

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: 11.0/ocata
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/33909

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: 11.0/ocata
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/34085

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: mcp/ocata
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/34597

[Fuel Devops McRobotson (fuel-devops-robot)]

Fix proposed to branch: mcp/ocata
Change author: Alexander Tivelkov <email address hidden>
Review: https://review.fuel-infra.org/34789

[Fuel Devops McRobotson (fuel-devops-robot)]

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34597
Reason: The fix was upstreamed and exists in mcp/ocata.

[Fuel Devops McRobotson (fuel-devops-robot)]

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/34085
Reason: 11.0/ocata is obsolete. We use mcp/ocata instead.

[Fuel Devops McRobotson (fuel-devops-robot)]

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

[Fuel Devops McRobotson (fuel-devops-robot)]

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

[Fuel Devops McRobotson (fuel-devops-robot)]

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

[Fuel Devops McRobotson (fuel-devops-robot)]

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