| frickler | corvus: seems fine to me, thx | 06:32 |
|---|---|---|
| *** ykarel|away is now known as ykarel | 06:33 | |
| *** amoralej is now known as amoralej|lunch | 11:00 | |
| *** amoralej|lunch is now known as amoralej | 12:20 | |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Uncap test requirements https://review.opendev.org/c/opendev/git-review/+/895939 | 14:24 |
| fungi | frickler: ^ that seemed to work locally, if you have more insight into what led to that stestr situation, i'm happy to add better detail to the commit message | 14:25 |
| opendevreview | Jeremy Stanley proposed opendev/git-review master: Uncap test requirements https://review.opendev.org/c/opendev/git-review/+/895939 | 15:05 |
| clarkb | frickler: fungi https://review.opendev.org/c/openstack/project-config/+/895865 is a quick change to clean up our nodepool dib status dashboard a bit | 15:12 |
| clarkb | we can add more filters too for other old images | 15:12 |
| fungi | seems like a good idea, yeah | 15:13 |
| clarkb | and if we can approve the ptgbot bookworm/py3.11 update before 2000 UTC I've been told diablo_rojo can help double check it all looks good around then | 15:14 |
| clarkb | that is https://review.opendev.org/c/openstack/ptgbot/+/895835 | 15:15 |
| fungi | clarkb: sounds good. i posted a quick question to that change but otherwise lgtm | 15:21 |
| opendevreview | Merged openstack/project-config master: Add the Pure Storage Flashblade charm-manila subordinate https://review.opendev.org/c/openstack/project-config/+/893910 | 15:25 |
| clarkb | fungi: replied. tldr I don't know which version of python did it but it looks like we get more verbose error messages now and need to adjust the mock assertion to match | 15:35 |
| clarkb | fungi: doing a quiick test locally against 3.9 and 3.10 it looks like 3.10 added the extra verbosity | 15:37 |
| clarkb | fungi: https://paste.opendev.org/show/bVSnVQ0vQQkB30IuOsvE/ | 15:38 |
| fungi | got it, thanks. i assumed it was something like that | 15:42 |
| clarkb | the PYTHONTRACEMALLOC resource warnings stuff has also been welcome change for debugging. Add in the performance improvements and recent python releases have been well worth the updates | 15:46 |
| clarkb | I appreciate that there is effort in making existing python codebases quicker and easier to debug after a lot of focus on "rewrite everything to make use of new tools and maybe you'll see improvement" | 15:47 |
| frickler | fungi: iiuc the issue was caused by capping stestr, but not testtools, see https://github.com/mtreinish/stestr/issues/347 | 15:53 |
| fungi | aha | 15:57 |
| fungi | thanks! | 15:57 |
| opendevreview | Merged opendev/git-review master: Uncap test requirements https://review.opendev.org/c/opendev/git-review/+/895939 | 16:24 |
| fungi | infra-root: i've completed preparations for tomorrow's mailing list migration maintenance window. next step will be approving 895205 around 13:30z tomorrow. i can do that if nobody beats me to it | 16:51 |
| clarkb | I probably won't be around until just after 1500 UTC (school stuff before then) | 16:51 |
| clarkb | but I plan to be there | 16:51 |
| fungi | reminder that the maintenance plan is at the bottom of https://etherpad.opendev.org/p/mm3migration (line 211 is where tomorrow starts) | 16:52 |
| fungi | i expect it to go pretty smoothly, this plan incorporates all the fixes/improvements from the first four sites we migrated | 16:53 |
| fungi | chatter on the mm3 ml is that there will be a 3.3.9 rsn | 16:53 |
| fungi | so we'll probably look at upgrading after lists.openstack.org moves | 16:54 |
| clarkb | sounds like a plan | 17:02 |
| clarkb | fungi: ptgbot looks happy do you want to proceed with https://review.opendev.org/c/opendev/gerritbot/+/895831 ? | 17:03 |
| fungi | yep, approved just now | 17:04 |
| clarkb | thanks | 17:04 |
| clarkb | fungi: looks like kopecmartin is happy with the refstack update too, but I'm happy to get the easy irc bots out of the way first | 17:05 |
| fungi | sure | 17:09 |
| opendevreview | Merged opendev/gerritbot master: Update Gerrit bot to python3.11 on bookworm https://review.opendev.org/c/opendev/gerritbot/+/895831 | 17:13 |
| fungi | looks like it's restarted | 17:19 |
| clarkb | quick push something. I actually have a chagne I could write. Give me a few | 17:20 |
| fungi | yeah, i was trying to come up with something, and failing | 17:21 |
| opendevreview | Clark Boylan proposed opendev/gear master: Update gear to python3.11 and Bookworm container https://review.opendev.org/c/opendev/gear/+/895968 | 17:23 |
| clarkb | thoughts on ending maintenance for gear? cc corvus Nothing we deploy these days relies on it | 17:24 |
| corvus | specifically what would that entail? | 17:29 |
| clarkb | corvus: probably announce the intention and see if anyone else wants to maintain it and if not we could either A) completely retire the repo or B) stop building containers for it and maybe reduce zuul jobs to the noop job | 17:31 |
| clarkb | I think the main issue is that we're currently keeping a conatiner image up to date for it and I'm not sure anyone is actually using that. Not maintaining the container image is one less thing we have to update when we rotate our container image to new debian releases and/or new python releases | 17:32 |
| corvus | yeah, i'm fine with dropping the container image. i think if opendev wants to drop it as an official repo (assuming we make such a distinction?) i think that's fine too. i'm not sure about declaring it unmaintained though. i think it's more "finished" than "unmaintained" at this point... | 17:36 |
| corvus | but if the process for that is for opendev to declare it unmaintained and i volunteer to take over "maintenance" personally, that's fine too. | 17:37 |
| clarkb | ya I guess I'm happy to look at a random bugfix or python3.x updates in the future. The main concern today is the container image is largely unused as far as I can tell and it demands the most effort in that repo today so maybe we can just drop that instead | 17:38 |
| corvus | i feel exactly the same way | 17:38 |
| clarkb | cool. In that case I think we can do the bookworm + 3.11 update since I've already written it. Then afterwards pull out the bits for the container image | 17:41 |
| fungi | yeah, i don't see any need to declare anything in particular about it other than "we're not continuing to publish new container images" | 17:41 |
| opendevreview | Clark Boylan proposed opendev/gear master: Update gear to python3.11 and Bookworm container https://review.opendev.org/c/opendev/gear/+/895968 | 18:41 |
| clarkb | fungi: re statusbot I guess I'm happy for that to land whenever. I need to dig something up for lunch and do a school run later today, but I'm generally around | 18:41 |
| fungi | yeah, i figured i'd approve it here shortly | 18:49 |
| fungi | gerritbot seems to be working fine | 18:49 |
| fungi | and in it goes | 18:50 |
| opendevreview | Merged opendev/statusbot master: Move statusbot to python3.11 and Bookworm https://review.opendev.org/c/opendev/statusbot/+/895834 | 18:58 |
| fungi | and it came back | 19:27 |
| fungi | #status xyzzy | 19:27 |
| opendevstatus | fungi: unknown command | 19:27 |
| fungi | and responds | 19:27 |
| fungi | lgtm | 19:28 |
| Clark[m] | Woo | 19:32 |
| fungi | up for refstack or want to wait? | 19:33 |
| fungi | looks like gear has some bitrot | 19:34 |
| fungi | i'm going to transition to doing yardwork shortly but can still check in on things during frequent hydration+shade breaks | 19:35 |
| Clark[m] | I'm happy to do refstack I guess | 19:39 |
| Clark[m] | It's out of the path for openstack releases so may as well | 19:39 |
| Clark[m] | I'm finishing up lunch now | 19:39 |
| fungi | cool, i'll approve it now | 19:43 |
| opendevreview | Clark Boylan proposed opendev/gear master: Update gear to python3.11 and Bookworm container https://review.opendev.org/c/opendev/gear/+/895968 | 20:24 |
| opendevreview | Merged opendev/system-config master: Bump refstack to Bookworm and python3.11 https://review.opendev.org/c/opendev/system-config/+/895830 | 20:25 |
| clarkb | refstack updated its container a few minuts ago and I can still reach the service at https://refstack.openstack.org cc kopecmartin I think it looks good from here but let us know if something looks wrong | 20:36 |
| fungi | yeah, seems to be responding at least | 20:37 |
| clarkb | fungi: bindep is failign at least in part due to python3.5 being unable to find pbr which I think is due to the sni problems with pypi.org. I'm guessing we should drop python3.5 testing at this point | 20:53 |
| clarkb | and python2.7 is failing because virtualenv can't python2.7 anymore (yay) | 20:53 |
| clarkb | corvus: to amend the previous proposal I think we should update thinsg to be python3.6 and newer only and then drop the container image | 20:54 |
| corvus | clarkb: what about leaving the 2.7 declaration but dropping the jobs? since technically nothing has happened since then... though i suppose regardless, the next change merged should drop it anyway, and we might forget to do that... so yeah i think i agree with you. :) | 20:56 |
| corvus | (just leaving how i talked myself around to it there for posterity :) | 20:56 |
| corvus | (in other words, when/if there is another release, it will necessarily be 3.x+ only, so why not drop it now) | 20:56 |
| corvus | (we don't actually have to release it now) | 20:57 |
| Clark[m] | Yes and we don't need to set requires python we can just drop testing. Sorry transitioning to school run now | 20:59 |
| opendevreview | Merged zuul/zuul-jobs master: Pin python-subunit in fetch-subunit-output test https://review.opendev.org/c/zuul/zuul-jobs/+/895715 | 21:07 |
| opendevreview | Merged zuul/zuul-jobs master: Revert "Disable base role testing that runs code on localhost" https://review.opendev.org/c/zuul/zuul-jobs/+/895708 | 21:08 |
| opendevreview | Merged zuul/zuul-jobs master: Unpin stestr and python-subunit in fetch-subunit-output test https://review.opendev.org/c/zuul/zuul-jobs/+/895716 | 21:08 |
| fungi | clarkb: i guess if you want 3.5 to continue working you can port the thing i just deleted from git-review's test-requirements.txt | 21:18 |
| kopecmartin | clarkb: sure, thanks | 22:02 |
| opendevreview | Clark Boylan proposed opendev/gear master: Update gear to python3.11 and Bookworm container https://review.opendev.org/c/opendev/gear/+/895968 | 22:14 |
| clarkb | ok I think ^ will pass CI. If we land that we'll get a published gear container image that is relatively up to date for anyone who might be using it. Then we can announce we'll stop making new ones and drop the job | 22:20 |
| opendevreview | Merged openstack/diskimage-builder master: Remove extra spaces from GRUB_CMDLINE_LINUX_DEFAULT https://review.opendev.org/c/openstack/diskimage-builder/+/895484 | 23:09 |
| opendevreview | Merged openstack/diskimage-builder master: Refactor cloud-init element https://review.opendev.org/c/openstack/diskimage-builder/+/895485 | 23:09 |
| opendevreview | Merged openstack/diskimage-builder master: Set grub timeout style https://review.opendev.org/c/openstack/diskimage-builder/+/895486 | 23:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!