Fuel for OpenStack

fence-agents rpm package is missing uuid option for fence_virsh

Series 6.1.x
Bug #1405363

Bug #1405363 reported by Bogdan Dobrelya on 2014-12-24

6

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Fuel for OpenStack	Invalid	Undecided	Unassigned
Nominated for 8.0.x by Aleksandr Didenko
	6.0.x	Won't Fix	Medium	Sergey Otpuschennikov	Fuel for OpenStack 6.0-updates
	6.1.x	Won't Fix	Medium	Sergey Otpuschennikov	Fuel for OpenStack 6.1
	7.0.x	Invalid	Medium	Sergey Otpuschennikov	Fuel for OpenStack 7.0

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

See original description

Tags:

Bogdan Dobrelya (bogdando) on 2014-12-24

Changed in fuel:
importance:	Undecided → Medium
milestone:	none → 6.1
status:	New → Triaged
assignee:	nobody → Fuel OSCI Team (fuel-osci)

Roman Vyalov (r0mikiam) on 2014-12-24

Changed in fuel:
assignee:	Fuel OSCI Team (fuel-osci) → Sergey Otpuschennikov (sotpuschennikov)

Bogdan Dobrelya (bogdando) on 2014-12-24

description:

updated

Sergey Otpuschennikov (sotpuschennikov) on 2015-02-06

Changed in fuel:
status:	Triaged → In Progress

Revision history for this message

Sergey Otpuschennikov (sotpuschennikov) wrote on 2015-02-06:

#1

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 on 2015-02-06:

#2

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 on 2015-02-06:

#3

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

Revision history for this message

Sergey Otpuschennikov (sotpuschennikov) wrote on 2015-02-06:

#4

https://review.fuel-infra.org/#/c/2959/
Packages fence-agents-*.x86_64 0:4.0.7 has been built in repos
http://osci-obs.vm.mirantis.net:82/centos-fuel-6.1-stable-2959/centos/

Revision history for this message

Bogdan Dobrelya (bogdando) wrote on 2015-02-06:

#5

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 on 2015-02-06:

#7

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 on 2015-02-09:

#8

Added amtterm, dependencies for fence-agents
https://review.fuel-infra.org/#/c/3013/
https://review.fuel-infra.org/#/c/3002/
https://review.fuel-infra.org/#/c/2989/

Added fence-agents, dependencies for fence-agents
https://review.fuel-infra.org/#/c/2954
https://review.fuel-infra.org/#/c/2959/
https://review.fuel-infra.org/#/c/3016/

All packages built in
http://osci-obs.vm.mirantis.net:82/centos-fuel-master/centos/
http://osci-obs.vm.mirantis.net:82/centos-fuel-6.1-stable/centos/
http://osci-obs.vm.mirantis.net:82/centos-fuel-6.0.1-stable/centos/

Revision history for this message

Sergey Otpuschennikov (sotpuschennikov) wrote on 2015-02-10:

#9

Revert commit https://review.fuel-infra.org/#/c/2959/ and https://review.fuel-infra.org/#/c/3016/, because build failed http://jenkins-product.srt.mirantis.net:8080/job/6.1.staging.iso/95

Revision history for this message

Roman Vyalov (r0mikiam) wrote on 2015-02-10:

#10

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

Revision history for this message

Bogdan Dobrelya (bogdando) wrote on 2015-02-12:

#11

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

Sergey Vasilenko (xenolog) on 2015-02-24

Changed in fuel:
status:	New → Confirmed
milestone:	6.1 → 7.0

Mike Scherbakov (mihgen) on 2015-02-27

no longer affects:

fuel

Nastya Urlapova (aurlapova) on 2015-04-19

tags:

added: qa-agree-7.0

Revision history for this message

Aleksandr Didenko (adidenko) wrote on 2015-07-29:

#12

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.

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Related blueprints

Fencing in fuel-library manifests

Remote bug watches

Bug watches keep track of this bug in other bug trackers.