Wednesday, 2021-08-25

opendevreviewzhaoleilc proposed openstack/cinder master: Modify erroneous annotation
opendevreviewzhaoleilc proposed openstack/cinder master: Erroneous log location causes a misconception
opendevreviewMerged openstack/cinder stable/wallaby: Support Images API v2.11
opendevreviewGirishIBM proposed openstack/cinder master: [SVF]:Fix multiple lshost calls during attach.
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations
opendevreviewMerged openstack/cinder stable/train: NFS: Update connection info on online snap create
opendevreviewRajat Dhasmana proposed openstack/cinder master: WIP: Add support for qcow2 image optimization in glance cinder config
*** akekane_ is now known as abhishekk08:19
opendevreviewVenkata krishna Thumu proposed openstack/cinder master: [SVF] Update rccg details for mirror volumes
stephenfinMorning o/ Friendly reminder that the remove-sqlalchemy-migrate series is still available and needs another reviewer. Nova closed out the effort last week \o/
whoami-rajatstephenfin, hey, can you take a look at my comment and see if it makes sense?
stephenfinwhoami-rajat: sure09:17
stephenfinwhoami-rajat: Replied. tl;dr: I think it's necessary and I _could_ check to prove it, but I don't think it's not a big issue in any case and it could easily be addressed in a follow-up if necessary09:21
stephenfinso I'd be tempted to plough on, get this merged before M3 and then tidy it up after09:22
stephenfinif that's okay?09:22
whoami-rajatstephenfin, sounds good, +W09:25
whoami-rajatstephenfin, also i had a doubt with the initial refactor patch, the sqlalchemy docs mention the type of import we're currently using, wanted to know if it's required for alembic work?
stephenfinwhoami-rajat: It's certainly not mandatory, no. The reason I did that was because it was done in nova and I wanted to keep things as similar as possible to make reviewing easier (nova people could look at the cinder patches and vice versa)09:34
stephenfinand it was done in nova because we did a lot of refactoring of the nova.db module, and importing modules rather than objects resulted in smaller, more sensible imports and less chance of conflicts as the refactoring happened09:35
stephenfinI _could_ strip it out, but it's going to result in a huge merge conflicts that will be very tedious to resolve. I'll also lose the +2s I have higher up the series. Given it's not wrong and it is kept separate from more functional changes, I'd like to keep it if at all possible09:36
whoami-rajatyeah for cinder we've few imports that doesn't seem to be conflicting much with sa imports so i was confused, although it makes sense to import modules instead of classes for clarity (not sure why sqlalchemy docs doesn't mention it that way) and since smcginnis and eharney already voted so i won't object to it09:38
stephenfin🙏 thanks, I really did not want to do that horrible rebase 😅09:39
stephenfinI suspect the sqlalchemy docs (and sqlalchemy) don't use the "Do not import objects, only modules" rule from hacking like we do, though tbf sqlalchemy is one of the few exceptions carved out for whatever reason
stephenfinanyway, keep the questions coming :)09:41
whoami-rajatstephenfin, np, thanks for your work on this09:43
*** dviroel|ruck|afk is now known as dviroel|ruck10:59
opendevreviewMerged openstack/cinder master: db: Use 'import sqlalchemy as sa' pattern
opendevreviewMerged openstack/cinder master: db: Reorder initial migration
enriquetasorosmaita, do we support stein?13:34
smcginnisStein is currently in extended maintenance:
smcginnisThough not sure if there have been any discussions to move to EOL.13:36
rosmaitaenriquetaso: it's in extended maintenance, so we don't release from it any more, fixes get backported ... maybe13:36
rosmaitai think the oldest EOL branch is pike13:37
rosmaitawe were a bit ahead on that , but i think pike is EOL in most projects at this point (not sure though)13:37
toskybefore EOLing stein, we should EOL queens and rocky13:37
tosky(or all a the same time)13:37
toskyrosmaita: well, I wish it was, but ocata finally transitioned to EOL not long ago (days, maybe a few weeks)13:38
opendevreviewMerged openstack/cinder stable/victoria: Fix PowerFlex volume type conversion
rosmaitatosky: ok, thanks ... at least pike is EOL for cinder!13:38
toskyuhm, no, ocata EOL-ingis still onging (just seen a message on #openstack-release)13:39
rosmaitaok, that's good to know13:48
opendevreviewGorka Eguileor proposed openstack/cinder master: 3PAR: Fix race in concurrent attachments
rosmaitaEarly reminder: today's cinder meeting is on video + #openstack-meeting-alt at 1400 UTC13:56
rosmaitaconnection info:
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna ^^13:56
jungleboyjrosmaita:  Thanks.  Will join as soon as I can.14:01
opendevreviewEric Harney proposed openstack/cinder master: Tests: Remove brick_lvm mocks from all unit tests
venkatakrishnatHi eharney, geguileo, jungleboyj, rosmaita, whoami-rajat:  Could you please review which is to manage Global mirror change volumes on separate storage pools on IBM SVC cinder driver. This patch got one CR +2 and waiting for workflow +2 14:21
*** lbragstad_ is now known as lbragstad14:39
enriquetaso#startmeeting cinder_bs14:54
opendevmeetMeeting started Wed Aug 25 14:54:30 2021 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:54
opendevmeetThe meeting name has been set to 'cinder_bs'14:54
enriquetasoFull list of bugs:14:54
enriquetaso#topic bug_1 [oslo.utils] Usage of human output is deprecated in QemuImgInfo14:54
enriquetasoTakashi proposed a fix for this, please feel free to review it14:55
enriquetaso#topic bug_2 [os-brick] flush failed when both host and vm use lvm the multipath14:55
opendevreviewEric Harney proposed openstack/cinder stable/victoria: LVM: Retry lvextend commands on code 139
opendevreviewEric Harney proposed openstack/cinder stable/victoria: LVM: Use --readonly for lvdisplay in lv_has_snapshot
opendevreviewEric Harney proposed openstack/cinder stable/victoria: LVM: Retry lvdisplay and lvcreate calls on segfault
enriquetaso#topic bug_3 cinder-manage commands crash during db sync15:00
enriquetaso#topic bug_4  Tests: Swift + S3 backup driver unit tests fail w/ RunTimeError 15:04
enriquetaso#topic bug_5 [Stein] Multipathd delivers wrong path to Cinder15:07
enriquetaso#topic bug_6  [Doc] Cinder retype installation is missing nova config 15:08
*** abhishekk is now known as akekane|home15:08
*** akekane|home is now known as abhishekk15:08
opendevreviewMerged openstack/cinder stable/ussuri: PowerMax Driver - Fix for create snapshot
opendevreviewefineshi proposed openstack/os-brick master: flush failed when both host and vm use lvm the multipath
opendevmeetMeeting ended Wed Aug 25 15:15:59 2021 UTC.  Information about MeetBot at . (v 0.1.4)15:15
opendevmeetMinutes (text):
rosmaitathanks, sofia!15:16
opendevreviewEric Harney proposed openstack/cinder master: Change 'host' option from HostAddressOpt to StrOpt
enriquetasothanks you rosmaita 15:16
opendevreviewGorka Eguileor proposed openstack/cinder master: Change the host config option type
sfernandif some core could take a look on netapp feature patches :) We are quite desperate for community reviews.15:20
sfernandAlso,I have some time to work on review so send me a message if needed. I'm not able to give a +2 but can antecipate some issues15:20
geguileoeharney: you were faster than me...15:20
eharneygeguileo: oh oops15:20
geguileoon the string patch, though mine has the release note15:20
geguileoeharney: add the release note and I'll abandon mine15:20
opendevreviewEric Harney proposed openstack/cinder master: Change 'host' option from HostAddressOpt to StrOpt
amar___jungleboyj: rosmaita: Addressed all your comments really need it get merged in xena, CI is also green15:39
opendevreviewGorka Eguileor proposed openstack/cinder master: Update snapshot api microversion docs and tests
opendevreviewGorka Eguileor proposed openstack/cinder master: Fix notifications of migration temp volume
opendevreviewGorka Eguileor proposed openstack/cinder master: Clear OVO history and compatibility
opendevreviewGorka Eguileor proposed openstack/cinder master: Improve quota usage for temporary resources
opendevreviewGorka Eguileor proposed openstack/cinder master: Expose volume & snapshot use_quota field
opendevreviewGorka Eguileor proposed openstack/cinder master: RBD: Don't flatten temporary resources
opendevreviewSofia Enriquez proposed openstack/cinder master: WIP Try regular delete for snapshot or deferred deletion
opendevreviewBrian Rosmaita proposed openstack/cinder stable/victoria: Support Images API v2.11
opendevreviewBrian Rosmaita proposed openstack/python-cinderclient master: Add consumes quota field support
*** dviroel|ruck is now known as dviroel|out22:59

Generated by 2.17.2 by Marius Gedminas - find it at!