[RESULT][VOTE] CentOS 7 KVM binaries

[Daniel Salvador <gu...@apache.org>]

Hi all,

Thank you all for the engagement and participation.

The vote for CentOS 7 KVM binaries *passes* with
5 PMC + 1 non-PMC votes.


+1 (PMC / binding)
 persons (Gabriel, Nicolas, Rohit, Sven, Wei)

+1 (non binding)
1 person (Slavka)

0
none

-1
none


I'll open a PR on CloudStack Documentation repository[¹] adding a
RECOMMENDATION section on CloudStack's Installation Guide > Host KVM
Installation[²] guiding users to install the qemu-kvm-ev binaries, if
they are using CentOS 7. The packages that we will guide users to
install will be the latest provided by the official CentOS site[³]
(the current latest version is '2.12.0-44.1.el7_8.1.x86_64').

*Note:* PRs addressing KVM changes still have to look at the stock
qemu-kvm, as ACS will keep support of it.


Best regards,
Daniel Salvador


[¹] https://github.com/apache/cloudstack-documentation
[²] http://docs.cloudstack.apache.org/en/latest/installguide/hypervisor/kvm.html
[³] http://mirror.centos.org/centos-7/7/virt/x86_64/kvm-common/Packages/q/


On Fri, Mar 4, 2022 at 12:29 PM Nathan Johnson <na...@nathanjohnson.org>
wrote:

>
> > On Mar 4, 2022, at 9:04 AM, Simon Weller <sw...@ena.com> wrote:
> >
> > We've added features that are only supported on newer versions of
> qemu-kvm packages previously, such as the IOP and bandwidth limits.
> > I'd have to go hunting here, but I think it was done by have a check on
> the qemu-kvm version and if it was later than a certain release, exposing
> the feature.
> >
> > @Nathan Johnson might be able to jump in here, as he may remember how
> that was implemented.
> >
> > -Si
> >
>
> Yes, this is correct.  There are version checks all over the kvm plugin it
> looks like.  Here is an example of one I added:
>
>
> https://github.com/apache/cloudstack/blob/main/plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java#L1113
>
>
>
>