| opendevreview | Arnaud Morin proposed openstack/oslo.messaging master: Add QManager to amqp driver https://review.opendev.org/c/openstack/oslo.messaging/+/889425 | 09:07 |
|---|---|---|
| opendevreview | Arnaud Morin proposed openstack/oslo.messaging master: Add an option to use rabbitmq stream for fanout queues https://review.opendev.org/c/openstack/oslo.messaging/+/890825 | 09:07 |
| *** dkehn_ is now known as dkehn | 14:24 | |
| opendevreview | Merged openstack/oslo.log master: Bump bandit https://review.opendev.org/c/openstack/oslo.log/+/883663 | 17:13 |
| opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Only allow safe cxt fields allowed in notification https://review.opendev.org/c/openstack/oslo.messaging/+/891096 | 18:33 |
| JayF | I assume there's a message somewhere around ^ about a change to oslo messaging | 18:39 |
| JayF | oslo cores please read that change and https://bugs.launchpad.net/oslo.messaging/+bug/2030976 | 18:39 |
| opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Only allow safe cxt fields allowed in notification https://review.opendev.org/c/openstack/oslo.messaging/+/891096 | 21:26 |
| opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Only allow safe context fields in notifications https://review.opendev.org/c/openstack/oslo.messaging/+/891096 | 21:28 |
| JayF | TheJulia: dansmith: ^ I have not tested it yet in devstack; but I have a lot of confidence this does what we need; primarily based on how it impacted unit tests. | 21:28 |
| opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Only allow safe context fields in notifications https://review.opendev.org/c/openstack/oslo.messaging/+/891096 | 21:31 |
| dansmith | JayF: just saw your comment about specificity after I committed my comment about specificity | 21:41 |
| JayF | our brains must be weirdly in sync | 21:42 |
| JayF | that's exactly the key I imagined | 21:42 |
| JayF | user_auth_token vs service_auth_token and the like | 21:42 |
| JayF | blah | 21:42 |
| opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Only allow safe context fields in notifications https://review.opendev.org/c/openstack/oslo.messaging/+/891096 | 22:17 |
| JayF | fixes specificity complaints and I moved the sanitization to its own method | 22:17 |
| JayF | I have to change focus for the rest of the day; will prioritize testing this in ironic+nova+devstack tomorrow | 22:20 |
| opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Only allow safe context fields in notifications https://review.opendev.org/c/openstack/oslo.messaging/+/891096 | 22:22 |
| opendevreview | Jay Faulkner proposed openstack/oslo.messaging master: Only allow safe context fields in notifications https://review.opendev.org/c/openstack/oslo.messaging/+/891096 | 22:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!