Hi all, After the concerns on the thread I have tested PR 5297 on CentOS 7 with stock qemu-kvm binaries to reevaluate my vote. I found only one regression around snapshots: taking volume snapshots for running VMs does not work anymore as qemu complains with error: Operation not supported: live disk snapshot not supported with this QEMU binary.
I agree with Paul and Rohit in this case as a drop of support for users on CentOS 7 with stock qemu-kvm, so my vote will be -1 in this case. But would be +1 if we can fix the regression for this specific case as the rest of the operations are working fine. Regards, Nicolas Vazquez From: Rohit Yadav <rohit.ya...@shapeblue.com> Date: Wednesday, 2 March 2022 at 11:06 To: dev@cloudstack.apache.org <dev@cloudstack.apache.org> Subject: Re: [VOTE] CentOS 7 KVM binaries Gabriel, Daniel, the pull request (code change) is new information for me https://github.com/apache/cloudstack/pull/5297 There's also no rule or guidance that people can't discuss on a voting thread, I find the terms used to refer to that "polluting" derogatory. The vote also did not mention the PR and its implications, I'm sure most people on the thread have voted only on doc changes. Thanks Gabriel to sharing the background, it also explains what led Daniel to start the vote. Daniel, on the logistics of voting you may want to read the project bylaws and get familiar with the terms. All, I'll need to find some time and then get back to this thread after reviewing pros and cons of the PR and its implications after some due diligence. Alternatively, I'm happy if some other PMC can do the due diligence, I'll then add my vote based on PMC findings. Regards. ________________________________ From: Daniel Augusto Veronezi Salvador <dvsalvador...@gmail.com> Sent: Wednesday, March 2, 2022 6:00:51 PM To: dev@cloudstack.apache.org <dev@cloudstack.apache.org> Subject: Re: [VOTE] CentOS 7 KVM binaries @Gabriel has summarized everything very well. This proposal only adds a requirement (makes it clear) in our documentation for KVM + CentOS 7 (please, refer to the discussion thread[¹] to understand why it is necessary), which is what everybody (according to the discussion we had in the mailing list) is already applying in production environments. Therefore, we would only make this requirement clear to everybody, which would in turn enable us to move forward with KVM improvements (such as my PR that improves the snapshot process[²]). It (the discussion we had) never addressed dropping support for CentOS 7. Regarding Rohit's comment ("However, assuming Daniel has followed the bylaws and is suggesting this as a technical change that removes support in source code or releases..."), I never suggested a drop of support for CentOS 7. I don't know how that conclusion was derived. To emphasize, this proposal do not address dropping support for CentOS 7. Please, review the discussion thread[¹] and the proposal and reconsider your votes. Best regards, Daniel Salvador [¹] https://lists.apache.org/thread/z7s0774n72v4o9dnl140wvm030bxovjd [²] https://github.com/apache/cloudstack/pull/5297 On 02/03/2022 06:47, Gabriel Bräscher wrote: > I don`t want to "pollute" the vote thread, as we should keep discussions on > the proper thread (Discussion here [1]). > However, we are already in such a situation. I hope that I can add some > context and clarify a few things. > > @Daniel, please correct me if I misunderstood anything or my raised points > are not aligned with the respective Discussion and Vote threads. > Here goes my point of view regarding. > 1. Why this is a technical vote: > 1.1 The PR [2] is not moving forward due to the fact that CentOS stock Qemu > does not support the feature added; > 1.2 it will impact on code being accepted in case of "+1" or blocked in > case the vote does not pass; > 1.3 If this vote does pass, we need to change the PRs tests by adding > "qemu-kvm-ev" package in Marvin CentOS7 test environments. > > 2. Why I don't think this is a drop of support: > 2.1 From what I understood, the proposal does not state that CloudStack > will officially drop support for CentOS7 "stock" packages; > 2.2 those running CentOS7 WITHOUT "qemu-kvm-ev" packages already are not > able to perform some actions and they will never be with the current > "stock" packages; > 2.3 there will be no "backward" compatibility issues for them, as they are > still not able to perform such operations. > 2.4 those running CentOS7 WITHOUT "qemu-kvm-ev" will remain capable of > doing what their distro offers support. > > 3. Why this vote is relevant for MOST of the community > 3.1 Those running Ubuntu or CentOS7 WITH "qemu-kvm-ev" will be able to have > critical and relevant features as users will be able to follow the > instructions on documentation and the PRs blocked will finally be able to > be merged; > 3.2 documentation will be clear and help users to make their own decision > and take any risks into account. > > I want to stress how relevant this is and that most (if not all) users with > CentOS7 would be willing to change their packages in order to have the new > features. > With that Said, I am +1 (binding). > > [1] https://lists.apache.org/thread/z7s0774n72v4o9dnl140wvm030bxovjd > [2] https://github.com/apache/cloudstack/pull/5297 > > On Wed, Mar 2, 2022 at 9:34 AM Rohit Yadav <rohit.ya...@shapeblue.com> > wrote: > >> As I explained in my email, my vote is conditional with the stated >> assumption applicable only if this is a technical vote; Daniel advised in >> his last email that this is a technical decision vote that I assume as >> changes to source code and releases, therefore I had to vote my -1. >> >> If this is still a non-technical vote (as defined in the bylaws [1], i.e. >> decision/vote that does not affect changes in source code or releases) and >> changes are "only limited to the docs" as proposed in the vote then I'm +1. >> >> [1] https://cloudstack.apache.org/bylaws.html >> >> >> Regards. >> >> ________________________________ >> From: Paul Angus <p...@angus.uk.com.INVALID> >> Sent: Wednesday, March 2, 2022 13:31 >> To: dev@cloudstack.apache.org <dev@cloudstack.apache.org> >> Subject: RE: [VOTE] CentOS 7 KVM binaries >> >> The proposal said: >> ----- >> "- On CloudStack's Installation Guide > Host KVM Installation[²], we add a >> section 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')." >> ----- >> >> There's no mention of removing support for anything. If there is an >> intention removing support for existing components as part of this, then I >> agree completely with Rohit's -1 on it. >> >> >> >> Kind Regards >> >> >> Paul Angus >> >> >> >> >> -----Original Message----- >> From: Wei ZHOU <ustcweiz...@gmail.com> >> Sent: Wednesday, March 2, 2022 7:06 AM >> To: dev@cloudstack.apache.org >> Subject: Re: [VOTE] CentOS 7 KVM binaries >> >> oh wait, is there any word saying removing the support for centos7 with >> stock qemu ? >> >> -Wei >> >> On Wed, 2 Mar 2022 at 07:38, Rohit Yadav <rohit.ya...@shapeblue.com> >> wrote: >> >>> I had assumed this was a non-technical discussion/vote where the >>> changes are made in docs on suggested changes to how CloudStack is >>> deployed and used with CentOS7. I assumed this will follow as a doc PR >> to the QIG. >>> Changes to docs aren't normally considered technical as per our >>> project bylaws as they don't impact changes in source code or >>> releases. Three different PMCs have already advised on this thread >>> that voting isn't mandatory for this. >>> >>> However, assuming Daniel has followed the bylaws and is suggesting >>> this as a technical change that removes support in source code or >>> releases, then I oppose such a change. >>> >>> -1 (binding/veto) if we're going to technically remove support for >>> centos7 with stock qemu, that is in source code and >>> packaging/releases. CentOS7 will EOL until 2024 and stock support should >> be supported until then. >>> Regards. >>> >>> >>> >>> >>> ________________________________ >>> From: Daniel Augusto Veronezi Salvador <dvsalvador...@gmail.com> >>> Sent: Wednesday, March 2, 2022 2:31:21 AM >>> To: dev@cloudstack.apache.org <dev@cloudstack.apache.org> >>> Subject: Re: [VOTE] CentOS 7 KVM binaries >>> >>> Rohit, >>> >>> As we are deciding a requirement for deploying ACS + KVM + CentOS 7, I >>> see it as an important technical decision, that is why I started the >>> voting thread. The discussion was made via another thread[¹]; >>> therefore, this vote was created with the intention to summarize the >>> discussion we had and then to officially approve (or not approve) the >> idea discussed. >>> Finally, to emphasize, this is the voting thread, intended to reflect >>> the decision we seem to have agreed upon in the other thread[¹]. I >>> would kindly ask to avoid polluting this thread with discussions not >>> related to the voting itself. Furthermore, as already stated, there is >>> a consensus in the discussion thread; therefore, there is no harm in >>> giving a +1 here. >>> >>> Best regards, >>> Daniel Salvador >>> >>> [¹] https://lists.apache.org/thread/z7s0774n72v4o9dnl140wvm030bxovjd >>> >>> On 01/03/2022 16:56, Rohit Yadav wrote: >>>> (phone issue sent draft accidentally)... where consensus is built >>> without opposition. Therefore this vote thread isn't necessary. >>>> Refer to project bylaws https://cloudstack.apache.org/bylaws.html >>>> >>>> Regards. >>>> ________________________________ >>>> From: Daniel Augusto Veronezi Salvador <dvsalvador...@gmail.com> >>>> Sent: Tuesday, March 1, 2022 5:08:55 PM >>>> To: dev@cloudstack.apache.org <dev@cloudstack.apache.org> >>>> Subject: Re: [VOTE] CentOS 7 KVM binaries >>>> >>>> Hi, Andrija and Paul, >>>> >>>> This is the vote thread, not the discussion one. The goal of this >>>> thread is to account votes to verify the agreement of the community >>>> with the proposed solution that we seem to have in the discussion >>>> thread. For discussions, please refer to the discussion thread[¹]. >>>> The goal is to collect +1 and -1 to show the community agreement >>>> with the proposal that we discussed. >>>> >>>> Best regards, >>>> Daniel Salvador >>>> >>>> >>>> [¹] https://lists.apache.org/thread/z7s0774n72v4o9dnl140wvm030bxovjd >>>> >>>> >>>> On 28/02/2022 20:04, Andrija Panic wrote: >>>>> What Paul said... >>>>> >>>> >>>> >>> >>> > On Mon, 28 Feb 2022 at 22:01, Paul Angus <pau...@apache.org> wrote: >>>>>> A vote really isn't required for this. >>>>>> >>>>>> No one disagrees, so just do it. >>>>>> >>>>>> >>>>>> >>>>>> Kind Regards >>>>>> >>>>>> >>>>>> Paul Angus >>>>>> >>>>>> -----Original Message----- >>>>>> From: Wei ZHOU <ustcweiz...@gmail.com> >>>>>> Sent: Monday, February 28, 2022 4:19 PM >>>>>> To: dev@cloudstack.apache.org >>>>>> Subject: Re: [VOTE] CentOS 7 KVM binaries >>>>>> >>>>>> +1 (binding) >>>>>> >>>>>> Daniel, does this need to be approved by the PMC ? >>>>>> >>>>>> -Wei >>>>>> >>>>>> On Mon, 28 Feb 2022 at 17:08, Daniel Salvador >>>>>> <gutoveron...@apache.org >>>>>> wrote: >>>>>> >>>>>>> Hi all, this is the vote thread that emerged from the thread >>>>>>> "[Discussion] CentOS 7 KVM binaries"[¹]. >>>>>>> >>>>>>> As discussed in the thread, users already install (without any >>>>>>> official guide provided by the community) the qemu-kvm-ev binary >>>>>>> in their environments to run CloudStack + CentOS + KVM with all >> features. >>>>>>> With that said, to solve the situation described in the >>>>>>> discussion thread[¹], I propose the following: >>>>>>> >>>>>>> - On CloudStack's Installation Guide > Host KVM Installation[²], >>>>>>> we add a section 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'). >>>>>>> >>>>>>> For sanity in tallying the vote, can PMC members please be sure >>>>>>> to indicate "(binding)" with their vote? >>>>>>> >>>>>>> [ ] +1 approve >>>>>>> [ ] +0 no opinion >>>>>>> [ ] -1 disapprove (and reason why) >>>>>>> >>>>>>> If this gets approved, I'll open a PR on CloudStack Documentation >>>>>>> repository[⁴]. >>>>>>> >>>>>>> >>>>>>> Best regards, >>>>>>> Daniel Salvador >>>>>>> >>>>>>> >>>>>>> [¹] >>>>>>> https://lists.apache.org/thread/z7s0774n72v4o9dnl140wvm030bxovjd >>>>>>> [²] >>>>>>> >>>>>>> >>> http://docs.cloudstack.apache.org/en/latest/installguide/hypervisor/kv >>>>>>> m.html [³] >>>>>>> >>> http://mirror.centos.org/centos-7/7/virt/x86_64/kvm-common/Packages/q/ >>>>>>> [⁴] https://github.com/apache/cloudstack-documentation >>>>>>>
