Heat Translator

ToscaParserFailed with alarm tests

Bug #1771106 reported by Nguyen Hai
8
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Heat Translator
Fix Committed
Undecided
Unassigned
tacker
Fix Released
Critical
Cong Phuoc Hoang

Bug Description

Recently, these tests are failed. This may due to ToscaParser issues.

Test Group/Test case
tacker.tests.unit.test_tosca_templates_under_samples.TestSamples:
+ test_list_samples
+ test_alarm_sample
tacker.tests.unit.vnfm.infra_drivers.openstack.test_openstack.TestOpenStack
+ test_create_tosca_alarm_respawn
+ test_create_tosca_with_alarm_monitoring_not_matched
+ test_create_tosca_alarm_scale

There is a change in tosca-parser that affects the monitoring policies:

https://github.com/openstack/tosca-parser/blob/12be9db589794ca1564ed5ef664e9585cee68afc/toscaparser/triggers.py#L21

Affected heat-translator tosca monitoring policies:

https://github.com/openstack/heat-translator/blob/59eda86b88fe327b0c15931ec3e2b7308690f301/translator/hot/tosca/tosca_policies_monitoring.py

See original description
Nguyen Hai (nguyentrihai93-deactivatedaccount)
Changed in tacker:
importance: High → Critical
Revision history for this message
Trinh Nguyen (dangtrinhnt) wrote :

This's caused by the parameters were replaced:

https://github.com/openstack/tosca-parser/blob/master/toscaparser/triggers.py#L25

Trinh Nguyen (dangtrinhnt)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tacker (master)

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

Changed in tacker:
assignee: nobody → Trinh Nguyen (dangtrinhnt)
status: New → In Progress
Revision history for this message
Cong Phuoc Hoang (hoangphuoc) wrote :

This issue happens because of https://review.openstack.org/#/c/522935 is merged. I am fixing this error, with some patches (https://review.openstack.org/#/c/523021 and https://review.openstack.org/#/c/524260)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tacker (master)

Change abandoned by Trinh Nguyen (<email address hidden>) on branch: master
Review: https://review.openstack.org/568634
Reason: Abandon in favor of https://review.openstack.org/#/c/524260

Revision history for this message
Nguyen Hai (nguyentrihai93-deactivatedaccount) wrote :

So we need to fix it soon, it seems critical

Trinh Nguyen (dangtrinhnt)
Changed in heat-translator:
status: New → Confirmed
status: Confirmed → Fix Committed
Revision history for this message
Nguyen Hai (nguyentrihai93-deactivatedaccount) wrote :

Phuoc takes care of this bug: https://review.openstack.org/#/c/524260/

Changed in tacker:
assignee: Trinh Nguyen (dangtrinhnt) → nobody
assignee: nobody → Cong Phuoc Hoang (hoangphuoc)
Nguyen Hai (nguyentrihai93-deactivatedaccount)
Changed in tacker:
status: In Progress → Fix Committed
Yasufumi Ogawa (yasufum)
Changed in tacker:
status: Fix Committed → Fix Released
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.