fence-agents rpm package is missing uuid option for fence_virsh

[Bogdan Dobrelya]

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

[Sergey Otpuschennikov (sotpuschennikov)]

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 ?

[Bogdan Dobrelya (bogdando)]

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

[Bogdan Dobrelya (bogdando)]

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

[Sergey Otpuschennikov (sotpuschennikov)]

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/

[Bogdan Dobrelya (bogdando)]

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

[Bogdan Dobrelya (bogdando)]

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.

[Sergey Otpuschennikov (sotpuschennikov)]

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/

[Sergey Otpuschennikov (sotpuschennikov)]

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

[Roman Vyalov (r0mikiam)]

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

[Bogdan Dobrelya (bogdando)]

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

[Aleksandr Didenko (adidenko)]

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.