Fuel for OpenStack

[ostf] HA test 'Check pacemaker status' has failed with zabbix enabled

Bug #1499236 reported by Michael Polenchuk
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Michael Polenchuk
Fuel for OpenStack 8.0
7.0.x
Fix Released
High
Matthew Mosesohn
Fuel for OpenStack 7.0-mu-1
8.0.x
Fix Released
High
Michael Polenchuk
Fuel for OpenStack 8.0

Bug Description

Environment:
env {
  release: "7.0"
  openstack_version: "2015.1.0-7.0"
  build_number: "301"
  plugins: ['zabbix']
}

Steps to reproduce:
* deploy cluster with zabbix plugin enabled
* run ostf HA test "Check pacemaker status"

Expected result:
Test successfully passed.

Actual result:
Test has been failed even though the zabbix resources/services works well.
We've got the false-positive result here.

Logs:
Resource group__zabbix-server allowed to start on the nodes [], but it is not started on any node..

Impact:
The operator will get an incorrect results of Zabbix service status.

Solution:
Zabbix server with its vip defined in the pcmk group, and therefore test gets the wrong status for them.
Add handler for pcmk group in ha_base.py -> class TestPacemakerBase() -> def get_pcs_resources()

Michael Polenchuk (mpolenchuk)
Changed in fuel:
assignee: nobody → Michael Polenchuk (mpolenchuk)
Ksenia Svechnikova (kdemina)
Changed in fuel:
status: New → Confirmed
importance: Undecided → Medium
importance: Medium → High
milestone: none → 8.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-ostf (master)

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

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

Reviewed: https://review.openstack.org/227188
Committed: https://git.openstack.org/cgit/stackforge/fuel-ostf/commit/?id=7027476f0f8749bbdc71efebc299f7f304773580
Submitter: Jenkins
Branch: master

commit 7027476f0f8749bbdc71efebc299f7f304773580
Author: Michael Polenchuk <email address hidden>
Date: Thu Sep 24 12:06:42 2015 +0300

    Add handler for pcmk group

    Handle resource group properly in "Check pacemaker status" HA test.
    Get names of resources inside pacemaker group as well.

    Change-Id: Ibf4044db77ed58d52d332007a76d9d3cde31e168
    Closes-Bug: #1499236

Changed in fuel:
status: In Progress → Fix Committed
Veronica Krayneva (vkrayneva)
tags: added: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-ostf (stable/7.0)

Fix proposed to branch: stable/7.0
Review: https://review.openstack.org/231534

Revision history for this message
Veronica Krayneva (vkrayneva) wrote :

Checked on fuel-kilo-8.0-108-2015-10-02_10-23-00.iso

I built plugin with next metadata (fixed fuel_version: ['7.0'] into fuel_version: [‘8.0'] and fixed version):
..........................................
# Required fuel version
fuel_version: ['8.0']
..........................................
# The plugin is compatible with releases in the list
releases:
  - os: ubuntu
    version: 2015.1.0-8.0
    mode: ['ha']
    deployment_scripts_path: deployment_scripts/
    repository_path: repositories/ubuntu
  - os: centos
    version: 2015.1.0-8.0
    mode: ['ha']
    deployment_scripts_path: deployment_scripts/
    repository_path: repositories/centos
..........................................................

Bug was fixed, ha tests were passed

tags: removed: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-ostf (stable/7.0)

Reviewed: https://review.openstack.org/231534
Committed: https://git.openstack.org/cgit/stackforge/fuel-ostf/commit/?id=b33e3950fda73f5fd8ebe0453121ca7cc0137c40
Submitter: Jenkins
Branch: stable/7.0

commit b33e3950fda73f5fd8ebe0453121ca7cc0137c40
Author: Michael Polenchuk <email address hidden>
Date: Thu Sep 24 12:06:42 2015 +0300

    [stable/7.0]Add handler for pcmk group

    Handle resource group properly in "Check pacemaker status" HA test.
    Get names of resources inside pacemaker group as well.

    Change-Id: Ibf4044db77ed58d52d332007a76d9d3cde31e168
    Closes-Bug: #1499236
    (cherry picked from commit 7027476f0f8749bbdc71efebc299f7f304773580)

Alexander Zatserklyany (zatserklyany)
tags: added: on-verification
Alexander Zatserklyany (zatserklyany)
tags: removed: on-verification
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Revision history for this message
Andrey Sledzinskiy (asledzinskiy) wrote :

verified on 7.0-301 with MU

tags: added: 7mu1-verified
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.