*** mhen_ is now known as mhen | 02:03 | |
tobias-urdin | can i get another +2 core review on taskflow patch https://review.opendev.org/c/openstack/taskflow/+/932843 – thx | 09:14 |
---|---|---|
hberaud[m] | OMW | 09:16 |
tobias-urdin | hberaud[m]: thanks :) | 09:26 |
opendevreview | Tobias Urdin proposed openstack/taskflow master: Add release note for mask keys https://review.opendev.org/c/openstack/taskflow/+/948853 | 09:31 |
tobias-urdin | hberaud[m]: ^per your comment | 09:31 |
hberaud[m] | thx | 09:31 |
opendevreview | Stephen Finucane proposed openstack/oslo.middleware master: typing: Fix issue with HealthCheck.app_factory https://review.opendev.org/c/openstack/oslo.middleware/+/948861 | 10:28 |
opendevreview | Stephen Finucane proposed openstack/oslo.middleware master: typing: Allow application to be None https://review.opendev.org/c/openstack/oslo.middleware/+/948862 | 10:28 |
stephenfin | frickler: I've been out for the past fortnight but am back today. The fix for oslo.middleware is there ^ Did I miss anything else? | 10:28 |
frickler | stephenfin: there's some struggle with oslo.utils for 2024.2, but that's more of an issue with reqs tooling, see https://review.opendev.org/c/openstack/requirements/+/947558 and related patches | 10:47 |
frickler | and of course you missed the great setuptools vs. pbr showdown over the weekend ;) | 10:48 |
frickler | which unearthed some backlog of CI issues for pbr, like the ones that are blocking https://review.opendev.org/c/openstack/pbr/+/940778 but went unnoticed so far. I started some small fixes, but no idea yet what to do about the develop deprecation | 10:51 |
opendevreview | Merged openstack/taskflow master: Add mask keys parameters to failure logging https://review.opendev.org/c/openstack/taskflow/+/932843 | 11:11 |
frickler | stephenfin: ah, and https://bugs.launchpad.net/designate/+bug/2109577 was another setuptools easter egg, fun for a whole bunch of services, too | 11:19 |
stephenfin | Fun! On the setuptools easter egg, can't say I didn't warn people 😅 https://review.opendev.org/c/openstack/governance/+/902807 | 12:02 |
frickler | yes. "we as a community are bad at tracking goal completions" is one of the learnings from this. like the fix for designate had been essentially ready and waiting for months | 13:07 |
opendevreview | Stephen Finucane proposed openstack/oslo.middleware master: typing: Fix issue with HealthCheck.app_factory https://review.opendev.org/c/openstack/oslo.middleware/+/948861 | 16:17 |
opendevreview | Stephen Finucane proposed openstack/oslo.middleware master: typing: Allow application to be None https://review.opendev.org/c/openstack/oslo.middleware/+/948862 | 16:17 |
stephenfin | tkajinam: hberaud[m]: https://review.opendev.org/c/openstack/oslo.middleware/+/948861 should merge sooner rather than later. 6.4.0 has a bug that is (correctly) blocking the u-c bump | 17:16 |
stephenfin | s/that is/and our CI is/ | 17:17 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!