[dashboard] Yaql types 'validators' support in dynamic UI is broken

Bug #1498253 reported by Vahid Hashemian
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Murano
Fix Released
Medium
Ekaterina Chernova

Bug Description

Steps to create:

1. Use a HOT template with a parameter with specified "allowed_values".

2. Create a HOT package using "murano package-create --template ..."

3. Import the HOT package using "murano package-import ..."

4. In the dashboard, create an environment and add the imported package to the environment. I tried dropping the application onto the environment.

An error would pop up saying "Danger: An error occurred. Please try again later."
The horizon log shows:

...
2015-09-22 01:27:24.173456 TypeError: Error when calling the metaclass bases
2015-09-22 01:27:24.173464 cannot deepcopy this pattern object

The HOT template used, the screenshot of the error, and the full log trace is attached.
I used the latest devstack and murano codebase when hitting this bug.

Revision history for this message
Vahid Hashemian (vahidhashemian) wrote :
Revision history for this message
Vahid Hashemian (vahidhashemian) wrote :
Revision history for this message
Vahid Hashemian (vahidhashemian) wrote :
description: updated
Changed in murano:
milestone: none → mitaka-1
Changed in murano:
status: New → Confirmed
importance: Undecided → Medium
Dmytro Dovbii (ddovbii)
Changed in murano:
assignee: nobody → Dmytro Dovbii (ddovbii)
Changed in murano:
milestone: mitaka-1 → mitaka-2
Revision history for this message
Kevin Tibi (ktibi) wrote :

I can import HOT file with allowed_values but murano not propose a list for my choice when I deploy package.

Dmytro Dovbii (ddovbii)
Changed in murano:
assignee: Dmytro Dovbii (ddovbii) → nobody
Changed in murano:
assignee: nobody → Ekaterina Chernova (efedorova)
Revision history for this message
Ekaterina Chernova (efedorova) wrote : Re: [dashboard] Dynamic UI 'validators' attribute is not working

The main reason for this bug is that dynamic UI does not support 'validators'. Example is here https://github.com/openstack/murano-apps/blob/master/Windows/ActiveDirectory/package/UI/ui.yaml#L75

It is used to apply several validators for the one field or to use yaql expression as a validator

summary: - Error when adding a hot package with constrained parameter to an
- environment
+ [dashboard] Dynamic UI 'validators' attribute is not working
summary: - [dashboard] Dynamic UI 'validators' attribute is not working
+ [dashboard] Yaql types 'validators' support in dynamic UI is broken
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano-dashboard (master)

Fix proposed to branch: master
Review: https://review.openstack.org/266867

Changed in murano:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano-dashboard (master)

Reviewed: https://review.openstack.org/266867
Committed: https://git.openstack.org/cgit/openstack/murano-dashboard/commit/?id=b3bdf5f0aa001851b280dbeaf2c2e3f256ffe18e
Submitter: Jenkins
Branch: master

commit b3bdf5f0aa001851b280dbeaf2c2e3f256ffe18e
Author: Ekaterina Chernova <email address hidden>
Date: Wed Jan 13 15:03:26 2016 +0300

    Restore YAQL validators support

    It was broken after YAQl release since YAQL expression
    object contains recursive reference and deepcopy operation
    failed. So need to copy such objects manually in
    a separate function.

    To prevent such kind of failures,
    test of hot-based package was added.

    Change-Id: If3fde7552d9657055aded73c4967fb6832c17536
    Closes-Bug: #1498253

Changed in murano:
status: In Progress → 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.