| mattoliver | Tim is sick, and noone else in Swift core has +2/+A rights on stable branches. So we've left +1's.. but someone from the stable branch team would need to +A them. | 14:27 |
|---|---|---|
| fungi | mattoliver: thanks for the heads up, it's not critical so long as the changes are available in gerrit (which they are) | 14:32 |
| fungi | thanks for staying up so late! | 14:32 |
| mattoliver | fungi: kk! nps, thanks for all you do! | 14:34 |
| fungi | it's my pleasure | 14:34 |
| frickler | I've pulled out my stable hat to approve these now | 16:51 |
| fungi | thanks frickler! | 16:56 |
| fungi | it was still on my todo list to figure out who had delegated approval perms for swift's stable branches | 16:56 |
| frickler | looks like Tim and John Dickinson plus stable-maint-core. the latter could use some refreshing, too https://review.opendev.org/admin/groups/2267a5998d4224dd0acf1081eb2ee7b11573b7ea,members | 17:00 |
| clarkb | fungi: speaking of bionic node cleanups any chance you want to weigh in on https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/965396? | 17:47 |
| fungi | approved now | 17:51 |
| opendevreview | Merged openstack/openstack-zuul-jobs master: Convert py27 job to ubuntu jammy https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/965396 | 17:56 |
| frickler | 965400 would also be nice | 17:58 |
| fungi | looking | 17:59 |
| clarkb | fungi: no that was not intentional to remove those two py27 jobs.I thought I had only done 36 and 37 | 18:36 |
| clarkb | fungi: let me replace and drop the nodeset specifier then we can land that | 18:36 |
| clarkb | oh wait actually I know why I did that. its because openstack-tox-py27 is updated to work on jammy but not generic tox-py27 | 18:38 |
| clarkb | fungi: so I think the change is correct as is and if someone wants to fix tox-py27 too that can be done separately | 18:38 |
| clarkb | I left a comment about it on the chaneg but I think we can approve it as is if no one objects to my explanation there | 18:40 |
| fungi | wfm, thanks | 18:40 |
| clarkb | if tox was better about specifying virtualenv caps that work for py27 when it has py27 envs configured then this would be a non issue | 18:41 |
| clarkb | I think that would be a good feature of tox, but I'll leave that as an exercise to interested readers (basically tox should determine which python versions are configured in its config then ensure the version(s) of virtualenv that are installed are compatible with those python versions) | 18:42 |
| fungi | unfortunately, tox and virtualenv shared the same maintainer last i checked, and their position was that everyone should use the latest version of both and stop supporting eol python interpreters, so i doubt a feature like that would ever be accepted | 18:44 |
| opendevreview | Merged openstack/openstack-zuul-jobs master: Drop Bionic based jobs from project templates https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/965400 | 18:50 |
| *** mnaser[m] is now known as mnaser | 20:46 | |
| opendevreview | Merged openstack/pbr master: Run py27 job with default ansible https://review.opendev.org/c/openstack/pbr/+/965397 | 21:40 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!