opendevreview | Pham Le Gia Dai proposed openstack/cinder master: WIP: Add retention policy support https://review.opendev.org/c/openstack/cinder/+/886447 | 03:02 |
---|---|---|
opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/886333 | 04:35 |
opendevreview | Merged openstack/cinder-specs master: Re-propose extend volume completion action https://review.opendev.org/c/openstack/cinder-specs/+/877230 | 05:13 |
raghavendrat | hi whoami-rajat: are you online ? | 06:24 |
*** amoralej|off is now known as amoralej | 07:43 | |
enriquetaso | Hello, If you have time would you mind reviewing https://review.opendev.org/c/openstack/cinder/+/880965 ? | 08:51 |
enriquetaso | I've rebased and need +W again | 08:52 |
opendevreview | Sofia Enriquez proposed openstack/cinder master: Ceph backup: Fix Incremental Restore Using Wrong Container https://review.opendev.org/c/openstack/cinder/+/788319 | 09:04 |
opendevreview | Christian Rohmann proposed openstack/cinder master: Handle deleted volumes during cleanup of incomplete backups https://review.opendev.org/c/openstack/cinder/+/886584 | 09:43 |
opendevreview | Jakub Skunda proposed openstack/devstack-plugin-ceph master: Drop Fedora support https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/886588 | 10:06 |
opendevreview | Christian Rohmann proposed openstack/cinder master: Handle deleted volumes during cleanup of incomplete backups https://review.opendev.org/c/openstack/cinder/+/886584 | 11:22 |
*** amoralej is now known as amoralej|lunch | 11:45 | |
opendevreview | Sofia Enriquez proposed openstack/cinder master: Add api-ref functional tests for backup actions https://review.opendev.org/c/openstack/cinder/+/886609 | 11:50 |
eharney | has anyone been looking into VolumeMultiattachTests.test_multiattach_data_integrity failures? | 11:55 |
*** amoralej|lunch is now known as amoralej | 12:49 | |
opendevreview | Christian Rohmann proposed openstack/cinder-specs master: Spec to introduce new backup_status field for volume https://review.opendev.org/c/openstack/cinder-specs/+/868761 | 13:17 |
opendevreview | Christian Rohmann proposed openstack/cinder-specs master: Spec to introduce new backup_status field for volume https://review.opendev.org/c/openstack/cinder-specs/+/868761 | 13:27 |
whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 14:00 |
whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd | 14:00 |
opendevreview | Eric Harney proposed openstack/cinder-specs master: Spec to introduce new backup_status field for volume https://review.opendev.org/c/openstack/cinder-specs/+/868761 | 14:16 |
*** chuanm0 is now known as chuanm | 15:04 | |
enriquetaso | okay, nobody is discussing so I'm going to start the bug meeting | 15:08 |
enriquetaso | #startmeeting cinder_bs | 15:10 |
opendevmeet | Meeting started Wed Jun 21 15:10:05 2023 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:10 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:10 |
opendevmeet | The meeting name has been set to 'cinder_bs' | 15:10 |
enriquetaso | #topic Restore RBD Incremental Backup Fails When Using the --container Argument. | 15:10 |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2024484 | 15:10 |
enriquetaso | As mentioned on the main meeting: The CephBackupDriver forces the user to use the default RBD pool in every backup restore when restoring from an incremental backup [1]. This is incorrect since you can specify the container you would like to use when creating a backup [2], and that one should be used instead. | 15:10 |
enriquetaso | fix proposed to master: | 15:11 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/788319 | 15:11 |
*** chuanm5 is now known as chuanm | 15:12 | |
enriquetaso | mnaser, proposed the fix a while ago ^ | 15:13 |
enriquetaso | I've solved the merge conflict, fixed a unit test and added a release note | 15:14 |
enriquetaso | thanks for submitting that fix mnaser! | 15:14 |
enriquetaso | hillpd, whoami-rajat have you resolved you issue with https://bugs.launchpad.net/cinder/+bug/2017575 ? | 15:16 |
enriquetaso | moving on | 15:16 |
enriquetaso | #topic Hitachi: wrong exception when deleted volume is busy. | 15:16 |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2024418 | 15:16 |
enriquetaso | When delete_volume() is called and the deleted volume is busy, | 15:17 |
enriquetaso | Hitachi Cinder driver should be issued the exception with the name ``VolumeIsBusy``, | 15:17 |
enriquetaso | but ``VolumeDriverException`` is issued. | 15:17 |
enriquetaso | User would be confused by wrong exception. | 15:17 |
enriquetaso | fix proposed to master: | 15:17 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/885986 | 15:17 |
whoami-rajat | enriquetaso, I'm not sure if a fix for that was proposed, don't see it on the LP bug | 15:17 |
enriquetaso | oh okay, thanks whoami-rajat ! | 15:18 |
enriquetaso | the fix 885986 looks like does way more things that fixing an error message but I'll review later and leave a comment | 15:18 |
enriquetaso | #topic open discussion | 15:29 |
enriquetaso | okay! that's all | 15:31 |
enriquetaso | thank you! | 15:31 |
enriquetaso | #endmeeting | 15:31 |
opendevmeet | Meeting ended Wed Jun 21 15:31:11 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:31 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-06-21-15.10.html | 15:31 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-06-21-15.10.txt | 15:31 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-06-21-15.10.log.html | 15:31 |
*** amoralej is now known as amoralej|off | 17:19 | |
opendevreview | Pete Zaitcev proposed openstack/cinder master: PowerStore driver - documentation update https://review.opendev.org/c/openstack/cinder/+/874813 | 17:28 |
*** chuanm4 is now known as chuanm | 17:56 | |
opendevreview | Jakub Skunda proposed openstack/devstack-plugin-ceph master: Drop Fedora support https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/886588 | 21:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!