| opendevreview | Tony Breeds proposed openstack/election master: Remove stupid think-o https://review.opendev.org/c/openstack/election/+/862379 | 00:12 |
|---|---|---|
| opendevreview | Tony Breeds proposed openstack/election master: Gracefully exit if we can't find a date https://review.opendev.org/c/openstack/election/+/862380 | 00:12 |
| opendevreview | Tony Breeds proposed openstack/election master: Update select_release_end_date() to work cross release https://review.opendev.org/c/openstack/election/+/862381 | 00:12 |
| opendevreview | Tony Breeds proposed openstack/election master: Simplify date conversions https://review.opendev.org/c/openstack/election/+/862382 | 00:12 |
| opendevreview | Tony Breeds proposed openstack/election master: fix release name https://review.opendev.org/c/openstack/election/+/872479 | 00:13 |
| opendevreview | Tony Breeds proposed openstack/election master: Use underscores in setup metadata https://review.opendev.org/c/openstack/election/+/872480 | 00:13 |
| tonyb | Once I can get a clean run on the various pacthes I'll self merge them, as the majority already have +2+W | 00:13 |
| opendevreview | Tony Breeds proposed openstack/election master: fix release name https://review.opendev.org/c/openstack/election/+/872479 | 00:39 |
| opendevreview | Tony Breeds proposed openstack/election master: Remove stupid think-o https://review.opendev.org/c/openstack/election/+/862379 | 00:39 |
| opendevreview | Tony Breeds proposed openstack/election master: Gracefully exit if we can't find a date https://review.opendev.org/c/openstack/election/+/862380 | 00:39 |
| opendevreview | Tony Breeds proposed openstack/election master: Update select_release_end_date() to work cross release https://review.opendev.org/c/openstack/election/+/862381 | 00:39 |
| opendevreview | Tony Breeds proposed openstack/election master: Simplify date conversions https://review.opendev.org/c/openstack/election/+/862382 | 00:39 |
| opendevreview | Tony Breeds proposed openstack/election master: Use underscores in setup metadata https://review.opendev.org/c/openstack/election/+/872480 | 00:39 |
| tonyb | Once those ^^ merge I will rebase the 2 candidate reviews | 00:53 |
| tonyb | and then look at the outstanding reviews from Ian | 00:53 |
| opendevreview | Ghanshyam proposed openstack/election master: Fix fetching the dir name with release vesion https://review.opendev.org/c/openstack/election/+/872483 | 01:00 |
| gmann | tonyb: ^^ this will fix the current issue ? | 01:01 |
| gmann | otehrwise we are going back and forth on these placeholder setup | 01:01 |
| tonyb | No. If release is 'bobcat' to match the rewlease tools the it looks for candidates/bobcat for files. So we need to rename both | 01:02 |
| tonyb | Yes. we'll justy have to wear the back-and-forth for now. | 01:03 |
| tonyb | I lost track of time I wanted to have all of this ready befoer now | 01:03 |
| opendevreview | Ghanshyam proposed openstack/election master: Fix fetching the dir name with release version https://review.opendev.org/c/openstack/election/+/872483 | 01:03 |
| tonyb | the placeholder setup is wrong in other ways which I can fix next week | 01:03 |
| opendevreview | Merged openstack/election master: fix release name https://review.opendev.org/c/openstack/election/+/872479 | 01:03 |
| opendevreview | Merged openstack/election master: Remove stupid think-o https://review.opendev.org/c/openstack/election/+/862379 | 01:03 |
| gmann | tonyb: I am saying do not rename either and let it same as it is and ^^ can fix current issue of dir name | 01:03 |
| opendevreview | Merged openstack/election master: Gracefully exit if we can't find a date https://review.opendev.org/c/openstack/election/+/862380 | 01:03 |
| opendevreview | Merged openstack/election master: Update select_release_end_date() to work cross release https://review.opendev.org/c/openstack/election/+/862381 | 01:03 |
| tonyb | gmann: that isn't enough | 01:04 |
| tonyb | gmann: we'd need to track both bobcat and 2023.2 in the election configureation | 01:04 |
| tonyb | and currently we don't do that | 01:05 |
| gmann | can you point me where it break other than setup_election_config.py ? with above fix | 01:05 |
| gmann | release scripts have release_id in schedule file that we can use so setup_election_config.py can be fixed that way but later also | 01:06 |
| opendevreview | Merged openstack/election master: Simplify date conversions https://review.opendev.org/c/openstack/election/+/862382 | 01:06 |
| opendevreview | Merged openstack/election master: Use underscores in setup metadata https://review.opendev.org/c/openstack/election/+/872480 | 01:06 |
| gmann | I am trying to understand what other than setup_election_config.py is broken and 872483 does not fix? | 01:06 |
| tonyb | gmann: https://opendev.org/openstack/election/src/branch/master/doc/source/_exts/candidates.py#L54 here is one place | 01:07 |
| tonyb | Also without renaming the directory: https://zuul.opendev.org/t/openstack/build/92a742315a834f759b3f86e1a833a765 | 01:08 |
| tonyb | if it were just the setup I wouldn't have to rename things | 01:10 |
| tonyb | gmann: We can potentially use the release_id but we currently don't have one for bobcat | 01:11 |
| gmann | yeah, I am adding that. not sure why it was not added during initial schedule preparation | 01:12 |
| tonyb | gmann: See: https://opendev.org/openstack/releases/src/branch/master/data/series_status.yaml#L2-L7 | 01:12 |
| gmann | it was added for antelope | 01:12 |
| gmann | and missed for bobcat | 01:12 |
| gmann | https://review.opendev.org/c/openstack/releases/+/872484 | 01:14 |
| tonyb | Sure that helps but the bottom line is *today*/*now* we need to use bobcat | 01:15 |
| tonyb | This was a know issue months ago but I left things to languish and lost track of time | 01:16 |
| gmann | anyways, renaming things are merged so no point of fixing the actual issue now and delay the things more | 01:17 |
| gmann | like previous election we are on edge, let's not delay the nomination things. | 01:17 |
| gmann | let's fix it for next election | 01:17 |
| opendevreview | Tony Breeds proposed openstack/election master: Jay Faulkner candidacy for Ironic PTL 2023.2 https://review.opendev.org/c/openstack/election/+/872448 | 01:18 |
| ianychoi[m] | Let's first see whether ^ works or not - if it works with Zuul, then let's use bobcat for this election instead of 2023.2 | 01:23 |
| opendevreview | chzhang8 proposed openstack/election master: Modify reference packages alphabetically https://review.opendev.org/c/openstack/election/+/872139 | 01:28 |
| opendevreview | Merged openstack/election master: Jay Faulkner candidacy for Ironic PTL 2023.2 https://review.opendev.org/c/openstack/election/+/872448 | 01:35 |
| ianychoi[m] | tonyb: for ^, I added one comment like this which is needed to double-check as election official :) "Valid - OSF member and https://review.opendev.org/867838 merged on 2022-12-23 to openstack/bifrost for ironic | 01:37 |
| ianychoi[m] | " | 01:37 |
| ianychoi[m] | Since it works, let me re-change dir name.. | 01:37 |
| ianychoi[m] | Oh already changed :) | 01:37 |
| ianychoi[m] | Now seems that all are sorted | 01:38 |
| gmann | ianychoi[m]: yeah dire name are reverted to 'name' but sometime we should fix it otherwise this is story in every election | 01:38 |
| tonyb | ianychoi[m]: Yup. We can figure out the right way to handle the release_name vs release_id after the election concludes | 01:39 |
| ianychoi[m] | gmann tonyb : fully agree :) | 01:39 |
| tonyb | gmann: Well I did say we'd do it after the code cahanges I posetd in Nov merged ... but that didn't happen | 01:39 |
| tonyb | gmann: where "it" is split up release_name and relese_id | 01:40 |
| gmann | yeah | 01:41 |
| opendevreview | Hao Wang proposed openstack/election master: Add Hao Wang candidacy for Zaqar PTL in Bobcat https://review.opendev.org/c/openstack/election/+/872140 | 01:45 |
| opendevreview | Tony Breeds proposed openstack/election master: propose zhangchi for Zun PTL Candidacy https://review.opendev.org/c/openstack/election/+/872130 | 01:50 |
| opendevreview | Merged openstack/election master: Skip if e-mail in Gerrit is None https://review.opendev.org/c/openstack/election/+/857205 | 01:53 |
| opendevreview | Merged openstack/election master: Add CIVS policy to combined voting kickoff template https://review.opendev.org/c/openstack/election/+/857246 | 01:58 |
| opendevreview | chzhang8 proposed openstack/election master: Modify reference packages alphabetically https://review.opendev.org/c/openstack/election/+/872139 | 02:01 |
| opendevreview | Merged openstack/election master: Add Hao Wang candidacy for Zaqar PTL in Bobcat https://review.opendev.org/c/openstack/election/+/872140 | 02:32 |
| opendevreview | chzhang8 proposed openstack/election master: Modify reference packages alphabetically https://review.opendev.org/c/openstack/election/+/872139 | 02:53 |
| opendevreview | Tony Breeds proposed openstack/election master: propose zhangchi for Zun PTL Candidacy https://review.opendev.org/c/openstack/election/+/872130 | 05:07 |
| opendevreview | Tony Breeds proposed openstack/election master: Update package metadata and tox4 setup https://review.opendev.org/c/openstack/election/+/872510 | 05:40 |
| opendevreview | chzhang8 proposed openstack/election master: zhangchi candidacy for Zun PTL 2023.2 https://review.opendev.org/c/openstack/election/+/872532 | 07:12 |
| opendevreview | chzhang8 proposed openstack/election master: ngchi candidacy for Zun PTL 2023.2 https://review.opendev.org/c/openstack/election/+/872533 | 07:55 |
| opendevreview | chzhang8 proposed openstack/election master: zhangchi candidacy for Zun PTL 2023.2 https://review.opendev.org/c/openstack/election/+/872533 | 07:55 |
| opendevreview | chzhang8 proposed openstack/election master: propose zhangchi for Zun PTL Candidacy https://review.opendev.org/c/openstack/election/+/872130 | 08:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!