fence-agents rpm package is missing uuid option for fence_virsh

Bug #1405363 reported by Bogdan Dobrelya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Undecided
Unassigned
Nominated for 8.0.x by Aleksandr Didenko
6.0.x
Medium
Sergey Otpuschennikov
6.1.x
Medium
Sergey Otpuschennikov
7.0.x
Medium
Sergey Otpuschennikov

Bug Description

Steps to verify the issue:
Ubuntu:
apt-get install fence-agents
fence_virsh -h

Centos:
yum install fence-agents
fence_virsh -h

Expected:
-U, --uuid UUID of the VM to fence.
OR
fence_virsh with -n option accepts domain uuid as well.
Actual:
It is OK in Ubuntu, but in Centos there is no such option and -n does not accept uuid.

Please rebuild the package
Source: https://git.fedorahosted.org/git/fence-agents.git
The missing commit is https://git.fedorahosted.org/cgit/fence-agents.git/commit/?id=908fd921599b4e494028f3972a71c99c0aed674f

Changed in fuel:
importance: Undecided → Medium
milestone: none → 6.1
status: New → Triaged
assignee: nobody → Fuel OSCI Team (fuel-osci)
Roman Vyalov (r0mikiam)
Changed in fuel:
assignee: Fuel OSCI Team (fuel-osci) → Sergey Otpuschennikov (sotpuschennikov)
description: updated
Changed in fuel:
status: Triaged → In Progress
Revision history for this message
Sergey Otpuschennikov (sotpuschennikov) wrote :

Now we have fence-agents v.3.1.5 for both distr.
Need rebuild package for centos with this patch or update to version rpm package includes UUID support in fence_virsh (4.0.7)
or updates version in both distr ?

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

I believe we must update versions of rpm package for every new build. Can you please elaborate the question?

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

For ubuntu trusty, we should not change the vanilla package, IMHO.

Revision history for this message
Sergey Otpuschennikov (sotpuschennikov) wrote :
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

This build looks ok, there is no -U option, but -n now accepts uuid as well

description: updated
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

It would be nice to have this backport for 6.0.1 as we have an external fencing plugin for 6.0 and it is much simpler to work with virsh agent when it understands uuid.

Revision history for this message
Sergey Otpuschennikov (sotpuschennikov) wrote :
Revision history for this message
Sergey Otpuschennikov (sotpuschennikov) wrote :
Revision history for this message
Roman Vyalov (r0mikiam) wrote :

fix for fuel-lib https://review.openstack.org/#/c/154371. need merge with packages

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

this issue has no deployment impact and it should be addressed automatically then Ubuntu trusty and Centos 7 get supported in Fuel

Changed in fuel:
status: New → Confirmed
milestone: 6.1 → 7.0
Mike Scherbakov (mihgen)
no longer affects: fuel
tags: added: qa-agree-7.0
Revision history for this message
Aleksandr Didenko (adidenko) wrote :

We don't support CentOS-6 in Fuel-7.0 so it's invalid. Nominated to 8.0 to re-check when we have centos support.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers