hberaud | JayF: Personally I'm ok to revert the previous o.m fixes (from 2023.3 to yoga). | 08:17 |
---|---|---|
opendevreview | Merged openstack/oslo.context master: Add method for getting redacted copy of context https://review.opendev.org/c/openstack/oslo.context/+/891608 | 10:09 |
opendevreview | Merged openstack/oslo.messaging stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/oslo.messaging/+/894068 | 10:50 |
opendevreview | Stephen Finucane proposed openstack/oslo.context master: pre-commit: Bump dependencies https://review.opendev.org/c/openstack/oslo.context/+/894524 | 11:10 |
opendevreview | Stephen Finucane proposed openstack/oslo.context master: pre-commit: Integrate bandit, mypy https://review.opendev.org/c/openstack/oslo.context/+/894525 | 11:10 |
opendevreview | Merged openstack/oslo.messaging stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/oslo.messaging/+/894070 | 14:03 |
JayF | hberaud: For now; I have patches up for the three broken projects; all but one have landed and backports are in place | 14:24 |
JayF | hberaud: I think at this point we're just dealing with "masakari doesn't have active reviewers" ... so I'm going to send some emails today and try to get that ladned | 14:25 |
JayF | fungi: Are you around to sanity check a thing for me? | 18:59 |
JayF | nevermind, I figured it out | 19:00 |
* JayF might need lunch | 19:00 | |
fungi | JayF: i'm around now, but can't profess to any real degree of sanity. what do you need? | 19:19 |
fungi | oh, you are apparently more sane than me | 19:19 |
JayF | heh | 19:19 |
JayF | nothing like reading the code so much | 19:19 |
JayF | that the indentation levels start running together | 19:19 |
* JayF is in oslo messaging unit test hell trying to get unit tests to pass around actual contexts | 19:20 | |
opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Utilize the new RequestContext redacted_copy method https://review.opendev.org/c/openstack/oslo.messaging/+/894574 | 19:55 |
JayF | https://review.opendev.org/c/openstack/oslo.messaging/+/894574 can't land until the oslo.context change that landed is released, but I'd appreciate some early thoughts | 19:58 |
JayF | it changed a bit more than I expected trying to use real contexts in unit tests; I think we might be wise to take 90%+ of the places we pass around contexts in unit tests and just ... don't (pass None) instead | 19:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!