| opendevreview | Merged openstack/oslo.messaging master: Add Python3 antelope unit tests https://review.opendev.org/c/openstack/oslo.messaging/+/856643 | 06:17 |
|---|---|---|
| opendevreview | Merged openstack/oslo.vmware master: Add PBM WSDL for vSphere 8.0 https://review.opendev.org/c/openstack/oslo.vmware/+/895245 | 07:12 |
| frickler | stephenfin: hberaud: do you want to wait for other reviews on the oslo.db stuff or - given the release schedule - single/self approve and do a release now so we can du further testing? | 07:37 |
| stephenfin | ship it | 08:40 |
| opendevreview | Merged openstack/oslo.db stable/2023.2: [stable-only] Re-add autocommit support https://review.opendev.org/c/openstack/oslo.db/+/895631 | 08:51 |
| opendevreview | Merged openstack/oslo.db stable/2023.2: [stable-only] Re-add migration helpers, tests https://review.opendev.org/c/openstack/oslo.db/+/895632 | 08:54 |
| opendevreview | Stephen Finucane proposed openstack/oslo.db stable/2023.2: Move sqlalchemy-migrate to doc requirements https://review.opendev.org/c/openstack/oslo.db/+/895780 | 10:16 |
| opendevreview | Stephen Finucane proposed openstack/oslo.db stable/2023.2: [stable-only] Move sqlalchemy-migrate to doc requirements https://review.opendev.org/c/openstack/oslo.db/+/895780 | 10:16 |
| *** ralonsoh_ is now known as ralonsoh | 11:07 | |
| opendevreview | Dr. Jens Harbott proposed openstack/oslo.db stable/2023.2: bindep: Use new mysql-* package names https://review.opendev.org/c/openstack/oslo.db/+/895784 | 11:41 |
| hberaud | frickler: +1 to ship it now | 13:06 |
| frickler | hberaud: can you also review the castellan series? https://review.opendev.org/q/topic:revert-castellan-bobcat | 14:02 |
| hberaud | frickler: already done | 14:04 |
| frickler | oh, I should've checked before pinging, sorry. do you also want to self-approve or whom should this be waiting for? | 14:05 |
| hberaud | I'd prefer a second check, just in case, we could ask Ade Lee (alee) and Grzegorz Grasza (xek) as they are really concerned by these changes, but they are not present in this channel | 14:11 |
| hberaud | frickler: ^ | 14:12 |
| frickler | hberaud: Greg sent a mail earlier regarding barbican where they stated they are on unplanned PTO for the next two weeks | 14:14 |
| frickler | but I can add both as reviewers to the patches anyway | 14:14 |
| hberaud | ah | 14:14 |
| hberaud | okk | 14:14 |
| hberaud | s/ok/ | 14:14 |
| hberaud | if nobody won't review your patches by tomorrow end of the afternoon, then I'll self approve them | 14:16 |
| frickler | reviewers added, thx | 14:18 |
| frickler | hberaud: could you also check https://review.opendev.org/c/openstack/oslo.db/+/895780 please? that should resolve the failure on the release patch | 14:19 |
| hberaud | frickler: as I said in the release patch and on the release channel, I don't think we have to remove this requirement. I think the message simply ask us to propose a minor version rather than a bugfix version, which is required by semver | 14:21 |
| hberaud | normally a new requirement should be reflected by a minor version | 14:21 |
| hberaud | that's not the case in the oslo.db release patch | 14:22 |
| frickler | ah, sorry, being in a meeting in parallel I lag in reading all channels | 14:23 |
| frickler | answered on the other side | 14:24 |
| hberaud | np | 14:26 |
| hberaud | so, I just +2,+W the oslo.db patch | 14:26 |
| hberaud | but I think the version proposed for oslo.db should be at least a minor version in all cases | 14:28 |
| hberaud | and AFAIK even if adding a new requirements to a stable branch is not allowed by our policy, this is not forbidden by the release tooling, its just that the version number proposed here do not respect the semver https://github.com/openstack/releases/blob/74ea15922cdc35619e313470e728e947d3023fc8/openstack_releases/cmds/validate.py#L1097 | 14:31 |
| hberaud | so normally moving to a minor release should fix that in all the cass | 14:32 |
| opendevreview | Merged openstack/oslo.db stable/2023.2: [stable-only] Move sqlalchemy-migrate to doc requirements https://review.opendev.org/c/openstack/oslo.db/+/895780 | 14:37 |
| *** mtreinish_ is now known as mtreinish | 14:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!