| opendevreview | Merged openstack/election master: Adding Rajat Dhasmana candidacy for Cinder PTL https://review.opendev.org/c/openstack/election/+/828385 | 00:24 |
|---|---|---|
| opendevreview | Merged openstack/election master: Add Andrey Pavlov candidacy for Ec2 API https://review.opendev.org/c/openstack/election/+/828279 | 00:26 |
| opendevreview | Merged openstack/election master: TC Nomination - spotz https://review.opendev.org/c/openstack/election/+/828361 | 00:26 |
| opendevreview | feng.shengqin proposed openstack/election master: Adding FengShengqin candidacy for Zun in Z cycle https://review.opendev.org/c/openstack/election/+/828401 | 04:10 |
| gmann | spotz: as you are running for TC, it is good to remove name from election official list too. | 04:32 |
| ianychoi[m] | gmann: good catch thanks | 04:52 |
| opendevreview | feng.shengqin proposed openstack/election master: Adding FengShengqin candidacy for Zun in Z cycle https://review.opendev.org/c/openstack/election/+/828402 | 06:14 |
| opendevreview | feng.shengqin proposed openstack/election master: Adding FengShengqin candidacy for Zun in Z cycle https://review.opendev.org/c/openstack/election/+/828402 | 06:21 |
| opendevreview | Lucian Petrut proposed openstack/election master: Add Lucian Petrut candidacy for Winstackers https://review.opendev.org/c/openstack/election/+/828466 | 07:41 |
| opendevreview | Michal Nasiadka proposed openstack/election master: Michal Nasiadka candidacy for Kolla PTL (Z) https://review.opendev.org/c/openstack/election/+/828523 | 12:58 |
| spotz__ | gmann I don't think we removed folks last cycle but we can | 13:28 |
| fungi | we traditionally have removed election officials when they're running in any elections being officiated by the group, in order to make it clear they'll be unable to directly subvert any election in their own favor | 13:46 |
| ianychoi[m] | On last cycle, Kendall proposed to remove herself fyi: https://review.opendev.org/c/openstack/election/+/801547 | 15:16 |
| ianychoi[m] | (although I strongly think spotz is helping a lot now - thanks) | 15:18 |
| opendevreview | Lajos Katona proposed openstack/election master: Adding Lajos Katona candidacy for Neutron Z PTL https://review.opendev.org/c/openstack/election/+/828577 | 15:52 |
| opendevreview | Lajos Katona proposed openstack/election master: Adding Lajos Katona candidacy for Neutron Z PTL https://review.opendev.org/c/openstack/election/+/828577 | 15:54 |
| spotz | Up to y'all I'm not looking at patches | 16:02 |
| jungleboyj | I suppose we should remove you just to avoid any questions. I think it is what people have done in the past and I am trying to backfill for you. :-) | 16:03 |
| spotz | No prob. I can do a patch or one of youo can. | 16:06 |
| spotz | Watching Bobba Fett:) | 16:06 |
| ianychoi[m] | Another suggestion: would it be good if we can just change - e.g., from current election officials to election official voluteers, and active election officials are dependent considering their availability and election details? 1) I believe spotz will come back :) and 2) Belmiro also needs to be removed now 3) we need to think for every future election period.. | 16:09 |
| ianychoi[m] | I meant statement from https://governance.openstack.org/election/#election-officials | 16:10 |
| jungleboyj | Could change the wording to indicate it is election volunteers that are available? | 16:14 |
| jungleboyj | FYI: https://review.opendev.org/c/openstack/election/+/828402 The Zun PTL candidate doesn't meet the criteria. So, we will need to discuss how to handle that if there are no other candidates. | 16:14 |
| opendevreview | Merged openstack/election master: Michal Nasiadka candidacy for Kolla PTL (Z) https://review.opendev.org/c/openstack/election/+/828523 | 16:20 |
| opendevreview | Merged openstack/election master: Add Lucian Petrut candidacy for Winstackers https://review.opendev.org/c/openstack/election/+/828466 | 16:21 |
| spotz | So do we want a patch to remove me or a patch to say the EO for a release will be eligible members of the officials? | 16:24 |
| fungi | normally we've (muself included) merged a patch at the start of election activities or shortly before to set the specific list of officials for that election round, as much to provide people with usable contacts as anything, but also at the same time removed everyone except those people from the group in gerrit so they couldn't accidentally +2 or approve a nomination | 16:29 |
| gmann | spotz__: yeah, if not running in election then we do not remove but if running in same election it is better to remove now and after election we will add back. | 16:38 |
| spotz | Ok I'll get that in. | 16:38 |
| gmann | thanks. | 16:39 |
| ianychoi[m] | we can also change from "Our current election officials are:" to | 16:40 |
| ianychoi[m] | Below are the list of volunteers for election officials. Active election officials have election-core access: https://review.opendev.org/admin/groups/208108b62745877449d8e6fce45f44e3cc013e8b,members | 16:40 |
| gmann | jungleboyj: for zun candidate, we will keep it open and after election we will go in PTL appointment by TC if no other candidate. | 16:40 |
| jungleboyj | gmann: Ok, I thought that was the case. | 16:41 |
| ianychoi[m] | now I agree with gmann and would you tell Belmiro if he is interested in the election now? | 16:41 |
| jungleboyj | Thanks for confirming. | 16:41 |
| ianychoi[m] | I will clean up gerrit list now | 16:41 |
| gmann | ianychoi[m]: I am not sure if he will re-run or not, let's wait. I will try to reach out to him | 16:41 |
| gmann | ianychoi[m]: +1 | 16:41 |
| opendevreview | Amy Marrich proposed openstack/election master: Remove Amy from Election Officials https://review.opendev.org/c/openstack/election/+/828590 | 16:41 |
| spotz__ | ok done | 16:42 |
| gmann | ianychoi[m]: if Belmiro not running then it is fine to have him in official | 16:42 |
| gmann | spotz__: thanks again and do not forget to add yourself or I will do after election | 16:42 |
| gmann | ianychoi[m]: +1 on wording the doc for election official | 16:42 |
| spotz__ | gmann: Sounds good | 16:42 |
| ianychoi[m] | Thank you all :) | 16:43 |
| spotz | Belmeiro just emailed he's not running for TC | 16:46 |
| ianychoi[m] | Ah thank you for adding the context, spotz ! | 16:47 |
| ianychoi[m] | jungleboyj: note that CI still returns success with log message - "no current active election" if current time frame is not within election period (my usual working hour starts just after 15 mins from 23:45 UTC but today it seems that I executed recheck early :p ). Investigating manual log messages for *-review is so useful :) | 16:51 |
| fungi | however, one of the jobs checks all the candidates in the branch along with the current nominee, so will start failing for all nominees after the nomination period begins if an invalid nomination was merged as a candidate | 16:54 |
| fungi | so at least it will get noticed and can be reverted | 16:54 |
| jungleboyj | ianychoi[m]: Did I merge one that that was the case for? I thought I double checked that. | 16:55 |
| ianychoi[m] | jungleboyj: No for merge. Just adding a note regarding your +2 on https://review.opendev.org/c/openstack/election/+/828385 | 16:56 |
| ianychoi[m] | (I think you are fully aware of the candidate since you are an Cinder expert) | 16:57 |
| jungleboyj | ianychoi[m]: Oh, yes. Thank you. I learned that yesterday! :-) | 16:57 |
| jungleboyj | Yes, I knew he was valid. | 16:57 |
| ianychoi[m] | You're welcome and it seems that I couldn't well keep track of some messages here thank you too :) | 16:58 |
| jungleboyj | No problem. Lots going on. | 16:59 |
| ianychoi[m] | Ah fungi you're right in case of after nomination period is over thank you! - e.g., https://review.opendev.org/c/openstack/election/+/806339 | 17:00 |
| ianychoi[m] | :) | 17:01 |
| fungi | well, it's after the nomination period starts, but yes | 17:01 |
| fungi | that one you linked was a different problem | 17:02 |
| fungi | i think it's the election-tox-ci-checks-election job which will start failing for everyone during the nomination period (possibly openstack-tox-docs too) if there's an invalid candidate already merged to the directory | 17:03 |
| jungleboyj | I think that is right as I saw in one of the logs I looked at earlier that it also checks the other e-mails that were submitted. | 17:04 |
| ianychoi[m] | Yes, in sum, 1) we need to make sure CI validates after the nomination period starts 2) proper further steps (review, telling to TC) are needed after nomination period is over | 17:05 |
| ianychoi[m] | Ah correct. *-election does sanity check for all the nominations | 17:06 |
| fungi | right, so the main risk is that if you approve candidates early and they're technically invalid for some reason, you can end up blocking all further candidates from merging during the nomination period | 17:11 |
| fungi | though i, for one, don't entirely trust the automated checks (possibly because i wrote them), so i always look at the log from the review job | 17:11 |
| opendevreview | Merged openstack/election master: Adding Lajos Katona candidacy for Neutron Z PTL https://review.opendev.org/c/openstack/election/+/828577 | 18:11 |
| opendevreview | Merged openstack/election master: Remove Amy from Election Officials https://review.opendev.org/c/openstack/election/+/828590 | 19:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!