xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Lars Kurth <lars.kurth@citrix.com>
To: Ian Jackson <Ian.Jackson@citrix.com>
Cc: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
	"win-pv-devel@lists.xenproject.org"
	<win-pv-devel@lists.xenproject.org>,
	"committers@xenproject.org" <committers@xenproject.org>,
	"mirageos-devel@lists.xenproject.org"
	<mirageos-devel@lists.xenproject.org>,
	"xen-api@lists.xenproject.org" <xen-api@lists.xenproject.org>
Subject: Re: [PATCH v3 4/4] Addressed comments on quorum and security team members
Date: Tue, 4 Oct 2016 09:49:35 +0000	[thread overview]
Message-ID: <D4194734.2EC04%lars.kurth__10185.2245083593$1475574684$gmane$org@citrix.com> (raw)
In-Reply-To: <22514.34545.829269.902279@mariner.uk.xensource.com>



On 03/10/2016 17:27, "Ian Jackson" <ian.jackson@eu.citrix.com> wrote:

>Lars Kurth writes ("[PATCH v3 4/4] Addressed comments on quorum and
>security team members"):
>> Main changes
>> Leadership team decisions: express quorum in terms of +1 votes
>> Security Team Members: election
>> Project Wide Decision Making: minor text changes
>
>The resulting series is a little odd because your v3 4/4 patch only
>changes things that are introduced in v3 3/4 and agreed to be probably
>wrong there.  I would have been more usual to fold these changes in,
>at least if the series related to code.

I will merge the two for the next version : hopefully the last
 
>
>> --- a/governance.pandoc
>> +++ b/governance.pandoc
>> @@ -410,18 +410,26 @@ resolution. There is no differentiation between
>>**+1**/ **+2** and
>>  **-1**/**-2**: in other words a **+2** is counted as a vote for, a
>>**-2** as a 
>>  vote against the resolution. The number of votes for and against a
>>resolution 
>>  is called **active vote**. **0** votes **are not counted** as an
>>active vote.
>> --   A **quorum of more than 50% of active votes** is required for a
>>resolution 
>> -to pass. In other words, if the leadership team has 7 members, at
>>least 4 
>> -active votes are required for a resolution to pass.
>> +-   A **quorum of at least 1/3 of +1 votes for a proposal** is
>>required for a 
>> +resolution to pass. In other words, if the leadership team has 7
>>members, at 
>> +least 3 members need to vote for the resolution.
>
>This paragraph should say `positive' rather than `+1', since as
>written it appears to exclude +2.  (Same in the table.)

Agreed


>>  #### Project Lead Elections
>>  
>> @@ -553,10 +568,10 @@ as outlined below.
>>  -   Project leadership team members vote for or against a proposal
>>(there is no 
>>  differentiation between **-1**/**-2** and **+1**/**+2**). A **0** vote
>>is not 
>>  counted as a valid vote.
>> --   A **quorum of more than 50%** of each project's leadership team
>>members is 
>> -required. In other words: if more than half of a project's leadership
>>team 
>> +-   A **quorum of at least 50%** of each project's leadership team
>>members is 
>> +required. In other words: if fewer than half of a project's leadership
>>team 
>>  members do not vote or abstain, the entire sub-project's vote is not
>>counted. 
>> -This avoids situations where only a minority of leadership team
>>members votes, 
>> +This avoids situations where only a minority of leadership team
>>members vote, 
>
>This still has the non-monotonicity problem.
>
>I would suggest to deal with this issue by, when calculating the
>percentage, dividing all the votes by the larger of (a) the number of
>people voting (including `0' votes); (b) one third of the size of the
>project leadership team.
>
>So if only two out of a 10-person leadership team vote, and they both
>votes in favour, that subproject's overall vote is
>  2 / max(10/3, 2)
>which = 2 / max(10/3, 6/3) = 2 / (max(10,6) / 3) = 2 / (10/3)
>      = 2 * (3/10) = 6 / 10 = 0.6 = 60%
>
>I would add a further backstop that a successful resolution must have
>positive votes from at least three (or maybe, two) separate people.

Let me play with this
Originally I was planning on changing the quorum to match the one for
leadership teams for consistency.

Lars

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

  reply	other threads:[~2016-10-04  9:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1474656929-3528-1-git-send-email-lars.kurth@citrix.com>
2016-09-23 18:55 ` [PATCH v3 1/4] Code motion changes to make real patches easier to read Lars Kurth
2016-09-23 18:55 ` [PATCH v3 2/4] Added document containing governance related todo list Lars Kurth
2016-09-23 18:55 ` [PATCH v3 3/4] Significant changes to decision making; some new roles and minor changes Lars Kurth
2016-09-23 18:55 ` [PATCH v3 4/4] Addressed comments on quorum and security team members Lars Kurth
     [not found] ` <1474656929-3528-5-git-send-email-lars.kurth@citrix.com>
2016-10-03 16:27   ` Ian Jackson
2016-10-04  9:49     ` Lars Kurth [this message]
     [not found]     ` <D4194734.2EC04%lars.kurth@citrix.com>
2016-10-04  9:52       ` Ian Jackson
     [not found] ` <1474656929-3528-2-git-send-email-lars.kurth@citrix.com>
2016-10-14 20:12   ` [PATCH v3 1/4] Code motion changes to make real patches easier to read Konrad Rzeszutek Wilk
     [not found] ` <1474656929-3528-3-git-send-email-lars.kurth@citrix.com>
2016-10-14 20:13   ` [PATCH v3 2/4] Added document containing governance related todo list Konrad Rzeszutek Wilk
     [not found] ` <1474656929-3528-4-git-send-email-lars.kurth@citrix.com>
2016-10-14 20:30   ` [PATCH v3 3/4] Significant changes to decision making; some new roles and minor changes Konrad Rzeszutek Wilk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='D4194734.2EC04%lars.kurth__10185.2245083593$1475574684$gmane$org@citrix.com' \
    --to=lars.kurth@citrix.com \
    --cc=Ian.Jackson@citrix.com \
    --cc=committers@xenproject.org \
    --cc=mirageos-devel@lists.xenproject.org \
    --cc=win-pv-devel@lists.xenproject.org \
    --cc=xen-api@lists.xenproject.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).