Functional tests with service and node alerts failed due to incorrect attribute

Bug #1627671 reported by Vitaly Gusev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StackLight
Fix Released
High
Rodion Promyshlennikov

Bug Description

9.1 snapshot #302, StackLight 1.0

Steps to reproduce:
Run one of the following tests:
toolchain_critical_alert_node
toolchain_warning_alert_node
toolchain_warning_alert_service
toolchain_critical_alert_service

Expected result:
Test passed

Actual result:
Test failed with:
AttributeError: 'PluginHelper' object has no attribute 'full_vip_name'
Root cause: full_vip_name was renamed to get_vip_address

Changed in lma-toolchain:
status: New → In Progress
Revision history for this message
Rodion Promyshlennikov (rpromyshlennikov) wrote :

-Root cause: full_vip_name was renamed to get_vip_address
+Root cause: full_vip_name was renamed to get_vip_resource_name

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to stacklight-integration-tests (master)

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

Changed in lma-toolchain:
assignee: LMA-Toolchain Fuel Plugins (mos-lma-toolchain) → Rodion Promyshlennikov (rpromyshlennikov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to stacklight-integration-tests (master)

Reviewed: https://review.openstack.org/380098
Committed: https://git.openstack.org/cgit/openstack/stacklight-integration-tests/commit/?id=86966e30bd56b708a07ee9338dff88c9f3e666f2
Submitter: Jenkins
Branch: master

commit 86966e30bd56b708a07ee9338dff88c9f3e666f2
Author: Rodion Promyshlennikov <email address hidden>
Date: Tue Oct 4 11:46:06 2016 +0300

    Fix tests that checks services' and nodes' alerts

    Fix naming of getting vip name method:
    method "PluginHelper.full_vip_name" was renamed to "PluginHelper.get_vip_resource_name".
    Fix naming of services on nagios: "global-" prefix was added
    for plugins with version >=1.0.
    Fix checking of haproxy backends state:
    removed checks for services,
    that don't have haproxy backend.
    Fixed disk usage percentage to be consistent
    with alerts' criteria.
    Temporary disabled checking of openvswitch-agent,
    because it is managed by pacemaker.
    Removed code duplicity in tests.

    Change-Id: If3b77c595fbb5e4348f5f3ebd6e82b445b01062c
    Closes-Bug: #1627671

Changed in lma-toolchain:
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.