| gksk | Hi Team , Can any one pls review this minor change related to delete volumes flag support for delete volume group operation | 05:02 |
|---|---|---|
| gksk | https://review.opendev.org/c/openstack/cinder/+/869704 | 05:02 |
| opendevreview | saikrishnagk proposed openstack/cinder master: [SVf] Adding Support for --delete-volumes flag for delete volume group https://review.opendev.org/c/openstack/cinder/+/869704 | 05:10 |
| opendevreview | Kumar Kanishka proposed openstack/cinder master: [SVf] lsportip needs to fetch IPs with "host" flag https://review.opendev.org/c/openstack/cinder/+/869305 | 08:06 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding support for volumegroup snapshots https://review.opendev.org/c/openstack/cinder/+/869292 | 08:19 |
| opendevreview | Rajat Dhasmana proposed openstack/python-cinderclient stable/yoga: DNM: Test functional py39 job on yoga https://review.opendev.org/c/openstack/python-cinderclient/+/870513 | 08:39 |
| *** geguileo is now known as Guest1482 | 09:07 | |
| *** akrog is now known as geguileo | 09:07 | |
| *** elodilles_pto is now known as elodilles | 10:19 | |
| geguileo | whoami-rajat: tosky what was the URL to see the stats on a job run? | 11:48 |
| geguileo | not a single run, but latest runs | 11:49 |
| geguileo | to see if it's a problem with a specific patch or not... | 11:49 |
| geguileo | trying to get https://review.opendev.org/c/openstack/cinder/+/836071 to pass CI, but there are 2 jobs that are failing | 11:50 |
| geguileo | in one fo them I don't even see the c-vol service logs and in the other I don't see any ERROR in the cinder logs... | 11:50 |
| whoami-rajat | geguileo, https://zuul.openstack.org/status this one? | 11:51 |
| geguileo | whoami-rajat: I can't find the stats on failures there | 11:52 |
| geguileo | whoami-rajat: OK, they are in the BUILS tab | 11:53 |
| whoami-rajat | yeah you can find build history of a job there | 11:53 |
| whoami-rajat | the one i shared shows the current running job status | 11:54 |
| geguileo | yeah, that one I knew about, but didn't realize that I had to go to the builds tab to find what I needed | 11:55 |
| whoami-rajat | ack | 11:59 |
| ganso | rosmaita, whoami-rajat: Hi! when you have some time, could you please review those 2 patches: https://review.opendev.org/c/openstack/cinder/+/812685 and https://review.opendev.org/c/openstack/cinder/+/868485 thanks in advance! | 14:43 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Send the correct location URI to the Glance v2 API https://review.opendev.org/c/openstack/cinder/+/845178 | 14:56 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add the clone_across_pools driver capability https://review.opendev.org/c/openstack/cinder/+/847131 | 14:56 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: declare the clone_across_pools capability https://review.opendev.org/c/openstack/cinder/+/847132 | 14:56 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: drop _attach_volume() and _detach_volume() https://review.opendev.org/c/openstack/cinder/+/787296 | 14:56 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool driver: remove the obsolete backup_volume() https://review.opendev.org/c/openstack/cinder/+/803823 | 14:56 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements https://review.opendev.org/c/openstack/cinder/+/843277 | 14:56 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: drop copy_image_to_volume() and copy_volume_to_image() https://review.opendev.org/c/openstack/cinder/+/870550 | 14:56 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: fix the retype volume flow https://review.opendev.org/c/openstack/cinder/+/870551 | 14:56 |
| *** dviroel is now known as dviroel|lunch | 15:03 | |
| *** dviroel|lunch is now known as dviroel | 15:28 | |
| opendevreview | Merged openstack/cinder master: S3 Backup: Remove list of compression algorithms https://review.opendev.org/c/openstack/cinder/+/866588 | 16:30 |
| gksk | Hi team can any one please review the below patch | 16:43 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/869704 | 16:43 |
| geguileo | whoami-rajat: rosmaita patch already has your +2 only missing +W: https://review.opendev.org/c/openstack/cinder/+/836071 | 17:33 |
| whoami-rajat | geguileo, done | 17:34 |
| geguileo | whoami-rajat: thanks! :-) | 17:34 |
| whoami-rajat | np! | 17:34 |
| geguileo | how did we restrict jobs to run only tox3 and not tox4? | 18:24 |
| geguileo | whoami-rajat: rosmaita eharney tosky ^ | 18:24 |
| rosmaita | not sure those changes merged | 18:25 |
| geguileo | rosmaita: mmmm, cinderlib jobs are failing like crazy and it looks to me it's tox4 stuff | 18:25 |
| geguileo | I'll try fixing it | 18:26 |
| rosmaita | geguileo: https://review.opendev.org/q/topic:tox-4-postponed | 18:26 |
| geguileo | rosmaita: thanks | 18:27 |
| rosmaita | geguileo: this is the one that uses tox 4 (master only): https://review.opendev.org/c/openstack/cinderlib/+/866936 | 18:27 |
| rosmaita | oh, it hasn't merged yet | 18:27 |
| sfv880_ | Hello team, I'm sorry to bother you! We are looking for a final +2 and +w on the Infinidat fixes: https://review.opendev.org/c/openstack/cinder/+/852369 and https://review.opendev.org/c/openstack/cinder/+/864287 - Thank you so much! | 18:28 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix DBMS storage plugin https://review.opendev.org/c/openstack/cinderlib/+/870092 | 18:32 |
| opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix tox.ini https://review.opendev.org/c/openstack/cinderlib/+/870668 | 18:32 |
| opendevreview | Merged openstack/cinder master: LVM-nvmet: Use nvmetcli as library instead of CLI https://review.opendev.org/c/openstack/cinder/+/836071 | 19:01 |
| happystacker | hey guys! does anyone of you can explain why do we do a qemu-img convert operation when fetching an image to a volume? | 21:52 |
| happystacker | I'm looking particularly into this bug https://bugs.launchpad.net/cinder/+bug/2002535 | 21:52 |
| *** dviroel is now known as dviroel|out | 23:25 | |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: fix the retype volume flow https://review.opendev.org/c/openstack/cinder/+/870551 | 23:26 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements https://review.opendev.org/c/openstack/cinder/+/843277 | 23:26 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: fix the "rename volume" unit test emulation https://review.opendev.org/c/openstack/cinder/+/870681 | 23:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!