Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vote: Accept PR#18407 for backfit to 3.0 as a policy exception. #52

Closed
romen opened this issue May 31, 2022 · 9 comments
Closed

Vote: Accept PR#18407 for backfit to 3.0 as a policy exception. #52

romen opened this issue May 31, 2022 · 9 comments
Assignees
Labels
rejected The policy change proposal was rejected by an OTC vote

Comments

@romen
Copy link
Member

romen commented May 31, 2022

The topic of this vote is: Accept PR#18407 for backfit to 3.0 as a policy exception.

The snippet below records the current votes as cast during the weekly OTC meeting (2022-05-31) during which the vote was called.
The vote is still open.

Topic: Accept PR#18407 for backfit to 3.0 as a policy exception.
Proposed by: Nicola
Issue link: https://github.com/openssl/technical-policies/issues/52
Public: yes
Opened: 2022-05-31
Closed: 
Accepted:    (for: , against: , abstained: , not voted: )

   Dmitry     [+1]
   Matt       [-1]
   Pauli      [+1]
   Tim        [ 0]
   Richard    [ 0]
   Shane      [-1]
   Tomas      [ 0]
   Kurt       [  ]
   Matthias   [-1]   (corrected)
   Nicola     [+1]
@romen romen added enhancement New feature or request ready to vote The policy change proposal is ready to be voted on by the OTC discussed The issue/pr was discussed by the OTC labels May 31, 2022
@romen romen self-assigned this May 31, 2022
@tmshort
Copy link
Contributor

tmshort commented May 31, 2022

-0 from @mspncp ?

@t8m
Copy link
Member

t8m commented May 31, 2022

-0 from @mspncp ?

Yes, it is possible to vote -0 0 or +0. This is abstention with inclination against or for vote.

@mspncp
Copy link
Contributor

mspncp commented May 31, 2022

-0 from @mspncp ?

@tmshort please also note that I like your proposed change and consider it an improvement. I'm just inclined against backporting it because I think we should not lightheartedly undermine our own stable-release-updates policy, which we published only recently:

grafik

But you are right, -0 was an indecisive vote. So instead of leaving the decision up to Kurt who hasn't voted yet, I changed my mind and vote explicitly against it:

@mspncp
Copy link
Contributor

mspncp commented May 31, 2022

Vote: [-1] (corrected)

@mspncp
Copy link
Contributor

mspncp commented May 31, 2022

...instead of leaving the decision up to Kurt who hasn't voted yet...

Well, the vote still depends on @kroeckx's outstanding vote. Which proves that the ability to count properly is not a strict requirement for becoming a mathematician ;-)

@kroeckx
Copy link
Member

kroeckx commented Jun 6, 2022

Voting -1

@romen
Copy link
Member Author

romen commented Jun 6, 2022

-0 from @mspncp ?

@tmshort please also note that I like your proposed change and consider it an improvement. I'm just inclined against backporting it because I think we should not lightheartedly undermine our own stable-release-updates policy, which we published only recently:

grafik

But you are right, -0 was an indecisive vote. So instead of leaving the decision up to Kurt who hasn't voted yet, I changed my mind and vote explicitly against it:

Although openssl/openssl#18407 adds exactly 0 new tests or test cases, so we are not really voting an exception for that part of the policy, rather because we are considering this a new feature (not in the library or binaries, but in the test suite).

I plan to close the vote tomorrow (unless someone else does it sooner) in case there are changes of opinions.

@tmshort
Copy link
Contributor

tmshort commented Jun 7, 2022

Kurt voted -1; the sum() is -1, so the vote does not pass. I will close #18407 since it has already been merged to master.

@romen
Copy link
Member Author

romen commented Jul 1, 2022

I forgot to record this vote as closed, fixing it now:

Topic: Accept PR#18407 for backfit to 3.0 as a policy exception.
Proposed by: Nicola
Issue link: https://github.com/openssl/technical-policies/issues/52
Public: yes
Opened: 2022-05-31
Closed: 2022-06-07
Accepted: no (for: 3, against: 4, abstained: 3, not voted: 0)

   Dmitry     [+1]
   Matt       [-1]
   Pauli      [+1]
   Tim        [ 0]
   Richard    [ 0]
   Shane      [-1]
   Tomas      [ 0]
   Kurt       [-1]
   Matthias   [-1]
   Nicola     [+1]

@romen romen closed this as completed Jul 1, 2022
@romen romen added rejected The policy change proposal was rejected by an OTC vote and removed ready to vote The policy change proposal is ready to be voted on by the OTC labels Jul 1, 2022
@t8m t8m removed enhancement New feature or request discussed The issue/pr was discussed by the OTC labels Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rejected The policy change proposal was rejected by an OTC vote
Projects
None yet
Development

No branches or pull requests

5 participants