| opendevreview | Dr. Jens Harbott proposed openstack/election master: Add Jens Harbott candidacy for TC 2026.1 https://review.opendev.org/c/openstack/election/+/999917 | 06:39 |
|---|---|---|
| opendevreview | Dr. Jens Harbott proposed openstack/election master: Add Jens Harbott candidacy for TC 2027.1 https://review.opendev.org/c/openstack/election/+/999917 | 07:39 |
| opendevreview | Ivan Anfimov proposed openstack/election master: Add Ivan Anfimov candidacy for Zun PTL https://review.opendev.org/c/openstack/election/+/999951 | 10:03 |
| opendevreview | Ivan Anfimov proposed openstack/election master: Add Ivan Anfimov candidacy for Zun PTL https://review.opendev.org/c/openstack/election/+/999951 | 10:06 |
| opendevreview | Omer Schwartz proposed openstack/election master: Adding Omer Schwartz for Designate 2027.1 PTL https://review.opendev.org/c/openstack/election/+/999964 | 12:29 |
| opendevreview | Juan Larriba proposed openstack/election master: Add Juan Larriba candidacy for Telemetry PTL https://review.opendev.org/c/openstack/election/+/999971 | 12:52 |
| opendevreview | Juan Larriba proposed openstack/election master: Add Juan Larriba candidacy for Telemetry 2027.1 PTL https://review.opendev.org/c/openstack/election/+/999971 | 12:53 |
| mnasiadka | gouthamr: I think https://review.opendev.org/c/openstack/governance/+/995535 covers now 2/3 of votes - should I merge it? | 16:25 |
| opendevreview | Merged openstack/governance master: Add reviewers as Active Contributors https://review.opendev.org/c/openstack/governance/+/995535 | 18:24 |
| gouthamr | mnasiadka: missed pings from you and frickler regarding this | 18:25 |
| gouthamr | thanks for checking! | 18:25 |
| mnasiadka | Just in time for the elections ;) | 18:26 |
| gouthamr | +++ | 18:30 |
| sean-k-mooney | wait reviewer wernt ATC before | 18:46 |
| sean-k-mooney | that expalins why i didnt get election invites to some projects in the past that i expected to get | 18:46 |
| clarkb | note I don't think the change proposed makes all reviewers ATC | 18:47 |
| clarkb | just core reviewers who do a review | 18:47 |
| gouthamr | sean-k-mooney: reviewer is short hand for the colloquial "core reviewer" in this case | 18:47 |
| sean-k-mooney | ya its just +/-2 or +w | 18:47 |
| sean-k-mooney | i guess. i have entirly change the course of how some feature wer eimplemented in neutron via review without writing any code in it as a non core | 18:48 |
| gouthamr | i for one would like PTLs/maintainers to actively mark that sort of contribution with an AC | 18:49 |
| sean-k-mooney | so core vs non core seam a littel arbiary but the current change is an improvement regardless | 18:49 |
| gouthamr | extra-AC* | 18:49 |
| fungi | sean-k-mooney: the driving reason is that some smaller/slower teams have interested ptl volunteers who have approved changes in recent cycles but not authored any new changes that merged | 18:52 |
| sean-k-mooney | i think that is creating extra work for them but they can | 18:52 |
| fungi | so under the existing rules those volunteers don't qualify as ptl candidates, and lead to the teams getting treated by the tc as leaderless for the election | 18:53 |
| sean-k-mooney | fungi: ya that makes sense. | 18:53 |
| gouthamr | did corvus time this: https://review.opendev.org/c/zuul/zuul/+/1000000 | 19:10 |
| opendevreview | Michal Nasiadka proposed openstack/election master: Add Michal Nasiadka candidacy for Magnum/2027.1 https://review.opendev.org/c/openstack/election/+/1000030 | 19:10 |
| clarkb | I don't think so. elodilles was trying to get it with a huge stack of changes and i think corvus got lucky when pushing that change the timing fit in just right around elodilles' changes | 19:10 |
| clarkb | I'm glad that a real change got it and not a get the number change :P | 19:10 |
| gouthamr | :D so cool | 19:10 |
| gouthamr | gibi was trying to as well, by the looks of it: https://review.opendev.org/c/openstack/nova/+/1000001 | 19:11 |
| gouthamr | what fun | 19:11 |
| opendevreview | Michal Nasiadka proposed openstack/election master: Add Michal Nasiadka candidacy for Magnum/2027.1 https://review.opendev.org/c/openstack/election/+/1000030 | 19:16 |
| fungi | serendipitous that change 1 and change 1m were pushed by the same individual almost 14 years apart | 19:16 |
| fungi | fitting, even? | 19:16 |
| sean-k-mooney | im only sad the zuul change was not an optimisation to the gerrit driver | 19:20 |
| * gouthamr shakes fist at GitHub stealing that glory | 19:20 | |
| gouthamr | great fact fungi - sort of fun fact Iād like to read in the openinfra newsletter š | 19:21 |
| fungi | me too. i'm told there are already professional writers spinning a story about it | 19:22 |
| sean-k-mooney | i wonder which came first the millionth chagne or recheck :P | 19:23 |
| sean-k-mooney | in either case gerrit has certenly stoo the test of tiem, if bazar was still the git solution i don think openstack would have come this far | 19:25 |
| sean-k-mooney | looks liek gerrit was adopted 9 months before i start getting invovlved i dont eve recall what was used before | 19:27 |
| opendevreview | Michal Nasiadka proposed openstack/election master: Add Michal Nasiadka PTL candidacy for Kolla/2027.1 https://review.opendev.org/c/openstack/election/+/1000033 | 19:27 |
| opendevreview | Michal Nasiadka proposed openstack/election master: Add Michal Nasiadka PTL candidacy for Magnum/2027.1 https://review.opendev.org/c/openstack/election/+/1000030 | 19:27 |
| fungi | sean-k-mooney: we used bazaar before git/gerrit yes | 19:36 |
| fungi | though it was also (shortly) before i got involved too | 19:36 |
| fungi | it looks like change #1 is somewhat of an anomaly/bug, since we set the floor for change numbers to 1000 early on and the first change after that was actually pushed on 2011-10-20 | 19:42 |
| fungi | my first change (11338) was pushed on 2012-08-14 so not quite a year after our earliest gerrit changes | 19:44 |
| mnaser | my first was 7196 | 19:45 |
| mnaser | my first acutal merged was 7464 | 19:46 |
| sean-k-mooney | oh to keystone fixing an input validation bug no less | 19:48 |
| fungi | i see what's going on with change #1, something reset the datestamps in gerrit (i briefly remember it now) | 19:48 |
| mnaser | sean-k-mooney: the one before that was "implement fixed ip quota system" lol | 19:48 |
| mnaser | no specs, no nothing | 19:48 |
| fungi | the actual git commit for change #1 was made 2011-07-18 | 19:48 |
| sean-k-mooney | fungi: maybe with the migration to notedb or one of the other datamigrations over the years messed with it | 19:50 |
| fungi | no, it was way earlier | 19:51 |
| fungi | gerrit's mysql database originally made extensive use of auto-updating datestamps, and at one point in late 2012 we did a write query to correct something on a bunch of old changes which updated the dates on all those records to the time the query ran, but we didn't notice until much later once it was too late to try to correct them all | 19:52 |
| sean-k-mooney | ah ok | 19:53 |
| sean-k-mooney | i both love and hate that gerrit now uses git as its db | 19:53 |
| fungi | double-edged sword, yes | 19:53 |
| sean-k-mooney | its nice but there arent really good toosl to query that info | 19:53 |
| fungi | and we don't make use of the main feature it was developed to enable (replication between different gerrit sites) | 19:53 |
| clarkb | sean-k-mooney: you do occasionally learn some really fun git internals though | 20:08 |
| clarkb | like how to create raw objects and such | 20:09 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!