| rm_work[m] | jbernard: any idea on status of https://review.opendev.org/c/openstack/cinder/+/995140 ? | 03:08 |
|---|---|---|
| rm_work[m] | it appears to be merge conflict right now, and has at least one outstanding comment, would love to get your work in (and possibly my follow-up, or someone else's version of it, I really don't mind either way) so I can drop a depends-on for my oslo testing with non-eventlet :) | 03:08 |
| rm_work[m] | also, trying to make sure I understand -- eventlet isn't being made optional in these patches, it's being removed entirely? so we don't need to do any ugly compatibility stuff? | 03:16 |
| rm_work[m] | if there's ANYTHING I can do to move some of this stuff along, I am happy to help -- cinder is one of the last major pieces that is not fully eventlet-free for our deployment (we're running it from master branch) | 06:15 |
| harsh | Received one +2 one the review : 869305: [SVf] lsportip needs to fetch IPs with "host" flag | https://review.opendev.org/c/openstack/cinder/+/869305 | 09:36 |
| harsh | Could someone please add another +2 so that this can merge. Thanks :) | 09:37 |
| opendevreview | Inyong Hong proposed openstack/cinder-specs master: Add spec for incremental backup with CBT optimization https://review.opendev.org/c/openstack/cinder-specs/+/985089 | 13:07 |
| opendevreview | Christian Schwede proposed openstack/cinder master: backup: make num_dependent_backups updates atomic https://review.opendev.org/c/openstack/cinder/+/1000097 | 13:28 |
| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Remove Eventlet https://review.opendev.org/c/openstack/cinder/+/999439 | 13:58 |
| mhen | eharney: are you around | 14:08 |
| mhen | ? | 14:08 |
| rosmaita | reminder: Cinder Festival of Reviews today, 1400-1600 UTC | 14:08 |
| rosmaita | join us in Google meet: https://meet.google.com/ijz-ivqg-bam | 14:08 |
| rosmaita | or go directly to the etherpad: https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:08 |
| rosmaita | jungleboyj rosmaita tosky whoami-rajat m5z e0ne geguileo eharney jbernard sfernand hemna fabiooliveira happystacker sfv880 jayaanan bryanneumann agalica erlon lutimura sp-bmilanov kaisers Anoop_Shukla agalica | 14:08 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Fix lost update race on num_dependent_backups https://review.opendev.org/c/openstack/cinder/+/1000102 | 14:15 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Optimize getting parent backup for new incremental https://review.opendev.org/c/openstack/cinder/+/484729 | 14:16 |
| erlon | Here are the tapermonkey scripts I mentioned in the meeting: | 14:28 |
| erlon | - Hide/Unhide CIs from Gerrit pages: https://gist.github.com/sombrafam/82d1b16426418e5ba24f0c62418cf5ef | 14:28 |
| erlon | - Summarises Gerrit Discussions and patches: https://gist.github.com/sombrafam/915d55c3b23567f48d4c1e44f17a181f | 14:28 |
| erlon | To use them, you need to install the tampermonkey again on Chrome or Firefox, then add those scripts. For the gerrit sumarization, you need to set the script with your open AI or antropic API key. | 14:28 |
| opendevreview | Christian Schwede proposed openstack/cinder master: backup: count dependent backups in the database https://review.opendev.org/c/openstack/cinder/+/1000097 | 14:31 |
| harsh | erlon : are you there? | 14:32 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Optimize getting parent backup for new incremental https://review.opendev.org/c/openstack/cinder/+/484729 | 14:48 |
| erlon | eharney: when you have chance, please give a look on these backports. We need a stable-core to +2 it: https://review.opendev.org/q/Ifd632433ce5759494172b74ce311816e10728d21 | 15:01 |
| harsh | Please check these two reviews as they have been pending for a long time and we need these for the upcoming features. The upcoming reviews are in relation chain with these two: | 15:08 |
| harsh | [IBM SVf Driver] Added support of clone operation for volumegroup | https://review.opendev.org/c/openstack/cinder/+/982617 | 15:08 |
| harsh | [IBM SVf driver] Adding support for individual volume snapshots | https://review.opendev.org/c/openstack/cinder/+/949006 | 15:08 |
| clayg | @simondodsley RE: 1000097: backup: count dependent backups in the database | https://review.opendev.org/c/openstack/cinder/+/1000097 Christian normally works on swift; so anything you can push up that will help get the gate fixed merged sooner would be a DREAM! TY | 15:29 |
| simondodsley | @clayg Sure - I'll work on updating his patch | 15:33 |
| clayg | super hero!!! | 15:34 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: backup: count dependent backups in the database https://review.opendev.org/c/openstack/cinder/+/1000097 | 15:41 |
| simondodsley | @clayg done | 15:44 |
| opendevreview | Merged openstack/cinder master: [IBM SVf Driver] Define meaningful names for error constants https://review.opendev.org/c/openstack/cinder/+/959683 | 16:19 |
| opendevreview | Merged openstack/cinder master: [IBM-Storwize-SVf] Invalid Exception raised due to invalid portset value https://review.opendev.org/c/openstack/cinder/+/874633 | 16:36 |
| opendevreview | Merged openstack/cinder master: [SVf] lsportip needs to fetch IPs with "host" flag https://review.opendev.org/c/openstack/cinder/+/869305 | 16:49 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.openstack.org will be offline momentarily at 18:00 UTC for a patch upgrade (just under an hour from now), but should return within a few minutes. | 17:10 | |
| opendevreview | Merged openstack/cinder stable/2025.2: Fix properties={} being sent to Glance https://review.opendev.org/c/openstack/cinder/+/988260 | 17:18 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily for a patch upgrade, but should return within a few minutes. | 18:02 | |
| opendevreview | Simon Dodsley proposed openstack/cinder master: LVM: add Changed Block Tracking backup support for thin volumes https://review.opendev.org/c/openstack/cinder/+/995738 | 20:34 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!