Error: p_neutron-l3-agent is not a valid resource while banning neutron

Bug #1558154 reported by Alexandr Kostrikov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Kyrylo Galanov
Mitaka
Fix Released
High
Kyrylo Galanov

Bug Description

Detailed bug description:
At swarm job [0] there were a failure on scenario:
Check l3-agent rescheduling after l3-agent dies on vxlan

        Scenario:
            1. Revert snapshot with neutron cluster
            2. Create an instance with a key pair
            3. Manually reschedule router from primary controller
               to another one
            4. Stop l3-agent on new node with pcs
            5. Check l3-agent was rescheduled
            6. Check network connectivity from instance via
               dhcp namespace
            7. Run OSTF

        Duration 30m

The error was 'l3 agent wasn't banned, it is still node-6.test.domain.local'

Command to ban resource had failed:
root@node-1:~# pcs resource ban p_neutron-l3-agent node-6.test.domain.local
Error: p_neutron-l3-agent is not a valid resource

The `pcs status` on node-6 is at [1]

Library team, please provide correct command to ban neutron resource in comments and reassign it to Fuel-qa team.

Also, fuel-qa should check return code of pcs commands.

[0] https://product-ci.infra.mirantis.net/job/9.0.system_test.ubuntu.ha_neutron_destructive_vxlan/48/testReport/(root)/neutron_l3_migration_vxlan/
[1] http://paste.openstack.org/show/490744/

tags: added: non-release
Changed in fuel:
status: New → Confirmed
importance: High → Medium
tags: removed: non-release
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Kyrylo Galanov (kgalanov)
tags: removed: area-qa
Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

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

Changed in fuel:
assignee: Kyrylo Galanov (kgalanov) → Fuel QA Team (fuel-qa)
Revision history for this message
Kyrylo Galanov (kgalanov) wrote :

Hi QA team,

Resource names were changed: p_ prefix was removed.
You need to update the tests.
If you look at pacemaker/lib/puppet/provider/service/pacemaker.rb you can see how you can do it the best way.
1. Get all resources list
2. Find a resource by mask /(?:p_)neutron-l3-agent/
3. Execute a command with correct resource name.
4. Make the code reusable for all similar cases

---
 Clone Set: clone_neutron-openvswitch-agent [neutron-openvswitch-agent]
 Clone Set: clone_neutron-l3-agent [neutron-l3-agent]
 Clone Set: clone_neutron-metadata-agent [neutron-metadata-agent]
 Clone Set: clone_neutron-dhcp-agent [neutron-dhcp-agent]

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

Change abandoned by Kyrylo Galanov (<email address hidden>) on branch: master
Review: https://review.openstack.org/296866

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Fuel Library Team (fuel-library)
status: In Progress → Won't Fix
status: Won't Fix → Confirmed
importance: Medium → High
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(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:

actual result

version

expected result

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

tags: added: need-info
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Nastya U, please explain why this bug was moved back without any comment.

tags: added: ha
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Nastya Urlapova (aurlapova)
Revision history for this message
Nastya Urlapova (aurlapova) wrote :

Matt, there is a question in original description:
Library team, please provide correct command to ban neutron resource in comments and reassign it to Fuel-qa team.

+ attached review has -2 https://review.openstack.org/#/c/296866/
so we are still waiting suggestions from fuel-lib team.

Changed in fuel:
assignee: Nastya Urlapova (aurlapova) → Fuel Library Team (fuel-library)
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Nastya, this bug can only be solved by merging https://review.openstack.org/#/c/296866/ I will discuss with Sergii today on that.

Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Kyrylo Galanov (kgalanov)
Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/296866
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=d539686847c9102f2a7a6f326579d920dc66e932
Submitter: Jenkins
Branch: master

commit d539686847c9102f2a7a6f326579d920dc66e932
Author: Kyrylo Galanov <email address hidden>
Date: Thu Mar 24 02:46:51 2016 +0100

    Fix neutron resource name

    Change-Id: I5054f69d237640a39fd11801eefd8fd012384bbf
    Closes-bug: #1558154

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Artem Hrechanychenko (agrechanichenko) wrote :
Changed in fuel:
status: Fix Committed → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/306995

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
milestone: 9.0 → 10.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/mitaka)

Reviewed: https://review.openstack.org/306995
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=671c511f51dfc66760f4c53742c568c6fb3b095d
Submitter: Jenkins
Branch: stable/mitaka

commit 671c511f51dfc66760f4c53742c568c6fb3b095d
Author: Kyrylo Galanov <email address hidden>
Date: Thu Mar 24 02:46:51 2016 +0100

    Fix neutron resource name

    Change-Id: I5054f69d237640a39fd11801eefd8fd012384bbf
    Closes-bug: #1558154
    (cherry picked from commit d539686847c9102f2a7a6f326579d920dc66e932)

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Tatyanka (tatyana-leontovich) wrote :

for mitaka verified on 285 iso

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.