Tuesday, 2021-08-31

opendevreviewZhaoYixin proposed openstack/cinder master: Fix snapshot status is always backing-up  https://review.opendev.org/c/openstack/cinder/+/80666502:24
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group os-reset_status  https://review.opendev.org/c/openstack/cinder/+/80473503:37
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status  https://review.opendev.org/c/openstack/cinder/+/80403503:43
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77819303:48
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group-snapshot os-reset_status  https://review.opendev.org/c/openstack/cinder/+/80475703:53
opendevreviewTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77398504:05
opendevreviewFernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support  https://review.opendev.org/c/openstack/cinder/+/79966804:08
opendevreviewAndrei proposed openstack/cinder master: JovianDSS: fix code style and naming  https://review.opendev.org/c/openstack/cinder/+/80655904:51
opendevreviewFernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support  https://review.opendev.org/c/openstack/cinder/+/79966804:58
opendevreviewFernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support  https://review.opendev.org/c/openstack/cinder/+/79966804:59
opendevreviewFernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support  https://review.opendev.org/c/openstack/cinder/+/79966805:49
opendevreviewFernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support  https://review.opendev.org/c/openstack/cinder/+/79966805:53
sfernandhello folks :) If someone have some time to work on reviews please take a look at our features for xena release. 05:55
sfernandhttps://blueprints.launchpad.net/cinder/+spec/ontap-storage-assisted-migration05:55
sfernandhttps://blueprints.launchpad.net/cinder/+spec/ontap-revert-to-snapshot05:55
sfernandhttps://blueprints.launchpad.net/cinder/+spec/ontap-report-provisioned-capacity05:55
opendevreviewKazumasa Nomura proposed openstack/cinder master: Hitachi: Add generic volume groups  https://review.opendev.org/c/openstack/cinder/+/78291007:47
opendevreviewVenkata krishna Thumu proposed openstack/cinder master: [SVF] Manage GMCV volumes on separate pools  https://review.opendev.org/c/openstack/cinder/+/78652407:50
opendevreviewAndrei proposed openstack/cinder master: JovianDSS: fix code style and naming  https://review.opendev.org/c/openstack/cinder/+/80655907:58
*** whoami-rajat is now known as Guest587908:10
*** whoami-rajat__ is now known as whoami-rajat08:10
opendevreviewZohar Mamedov proposed openstack/os-brick master: NVMeOF connector support multipath-enabled kernels  https://review.opendev.org/c/openstack/os-brick/+/80668708:34
whoami-rajatwalshh_, hi, around?08:48
*** akekane_ is now known as abhishekk09:02
opendevreviewliujiong proposed openstack/cinder master: Move librbd diff_iterate to a separate thread  https://review.opendev.org/c/openstack/cinder/+/80447009:20
*** bhagyashris_ is now known as bhagyashris10:07
opendevreviewMerged openstack/cinder stable/wallaby: Allow removing NFS snapshots in error status  https://review.opendev.org/c/openstack/cinder/+/80528310:22
opendevreviewMerged openstack/python-cinderclient stable/wallaby: Unset tempest.lib timeout in functional tests  https://review.opendev.org/c/openstack/python-cinderclient/+/80296710:38
opendevreviewMerged openstack/python-cinderclient stable/wallaby: [stable-only] Add missing classes to cinderclient.v3  https://review.opendev.org/c/openstack/python-cinderclient/+/80290410:41
opendevreviewRaghavendra Tilay proposed openstack/cinder master: Nimble: Add Alletra 6k related information  https://review.opendev.org/c/openstack/cinder/+/80672510:53
opendevreviewAndrei proposed openstack/cinder master: JovianDSS: fix iscsi target recovery function  https://review.opendev.org/c/openstack/cinder/+/80619111:15
opendevreviewAndrei proposed openstack/cinder master: JovianDSS: add multy-attach support  https://review.opendev.org/c/openstack/cinder/+/80672611:15
zenkurorosmaita: hi, Ive broke original big patch in 3 peaces https://review.opendev.org/c/openstack/cinder/+/79496211:19
zenkurorosmaita: here is a blueprint with short description https://blueprints.launchpad.net/cinder/+spec/open-e-joviandss-add-multy-attach-support11:19
zenkurorosmaita: since https://review.opendev.org/c/openstack/cinder/+/806191 is a bug fix, Im curious if there are any specific actions I have to do besides posting it11:21
opendevreviewAndrei proposed openstack/cinder master: JovianDSS: fix code style and naming  https://review.opendev.org/c/openstack/cinder/+/80655911:31
opendevreviewSofia Enriquez proposed openstack/cinder stable/victoria: Allow removing NFS snapshots in error status  https://review.opendev.org/c/openstack/cinder/+/80669611:53
opendevreviewVenkata krishna Thumu proposed openstack/cinder master: [SVF] Update rccg details for mirror volumes  https://review.opendev.org/c/openstack/cinder/+/78878211:57
zigoHi there! I just filled this bug report: https://bugs.launchpad.net/cinder/+bug/194220312:05
zigoIt'd be nice if someone implemented this quickly, as it's not a lot of work, and everyone will at some point need it.12:06
zigoIs the Yoga branch already opened for such development, or is this too early for Yoga (and probably too late for Xena) ?!?12:07
zigoDepending on the answer to this last question, I may attempt something ...12:07
opendevreviewliujiong proposed openstack/cinder master: Move librbd diff_iterate to a separate thread  https://review.opendev.org/c/openstack/cinder/+/80447012:09
raghavendratHi, it would be great if someone can have a look at https://review.opendev.org/c/openstack/cinder/+/80579612:14
raghavendratHPE 3PAR: Add HPE Alletra 9k related information12:15
rosmaitazigo: thanks for filing that, i think we want to fix it by using a systemd slice file12:15
raghavendratGot one +2. Need one more approval. Thanks.12:15
opendevreviewBrian Rosmaita proposed openstack/python-cinderclient stable/victoria: Unset tempest.lib timeout in functional tests  https://review.opendev.org/c/openstack/python-cinderclient/+/80669712:18
opendevreviewBrian Rosmaita proposed openstack/python-cinderclient stable/victoria: [stable-only] Add missing classes to cinderclient.v3  https://review.opendev.org/c/openstack/python-cinderclient/+/80669812:19
zigorosmaita: Can you explain how that works?12:22
zigorosmaita: Maybe I can do that at the packaging level?!?12:22
rosmaitazigo: don't think at packaging level, because we will want to write the throttling level configured in cinder into the file ... i think we'll have to dynamically create the file12:24
rosmaitagive me a few minutes to find my notes on this12:24
zigorosmaita: I don't think it's a good idea for a process to write systemd files to the filesystem. I very much prefer if we could just get the new cgset / cgcreate stuff in...12:25
rosmaitawhat is providing cgset/cgcreate?12:26
rosmaitai think that's libcgroup?12:26
opendevreviewVladislav Belogrudov proposed openstack/cinder master: Fix volume caching in PowerFlex driver  https://review.opendev.org/c/openstack/cinder/+/80660512:27
opendevreviewSofia Enriquez proposed openstack/cinder master: RBD: Fix _show_msg_check_clone_v2_api  https://review.opendev.org/c/openstack/cinder/+/77703112:36
rosmaitazigo: left a comment on the bug for you12:38
zigorosmaita: Thanks.12:41
zigoHowever, there's maybe another route. Attempt to use the old style cgcreate / cgset, to detect if we're in v1, and then use v2 style if it isn't working...12:41
zigorosmaita: I don't see how the .slice file is helping in this regard ...12:41
zigoWe'd just drop the calls?12:43
rosmaitawe'd replace the call to cgexec with something like 'systemd run --slice=cinder-something.slice <command>' instead of using cgexec12:45
opendevreviewSofia Enriquez proposed openstack/cinder master: RBD: Fix _show_msg_check_clone_v2_api  https://review.opendev.org/c/openstack/cinder/+/77703113:16
opendevreviewSeena Fallah proposed openstack/cinder master: snapshot: allow revert to any point for rbd driver  https://review.opendev.org/c/openstack/cinder/+/80674013:16
whoami-rajat__walshh_, hi, around?13:26
walshh_Hi13:26
whoami-rajat__rosmaita, hey, can you take a look at my stable backport, this is the only blocking patch for wallaby release https://review.opendev.org/c/openstack/cinder/+/80278113:29
rosmaitalooking13:29
rosmaitawhoami-rajat__: not sure that's the correct way to indicate a squash of 2 cherry-picks in your commit message13:31
whoami-rajat__rosmaita, hmm, i think so you're right, but i can't remember the right way either13:32
rosmaitame neither13:33
rosmaitasofia did a squash fairly recently13:33
rosmaitaenriquetaso: ^^13:33
enriquetasohttps://paste.opendev.org/show/808460/13:38
venkatakrishnatHi jungleboyj, I have addressed your review comments for the patch https://review.opendev.org/c/openstack/cinder/+/786524 Please have a look.13:42
jungleboyjLooking.13:43
jungleboyjvenkatakrishnat:  Done.13:45
venkatakrishnatjungleboyj: Thank you very much for the quick response.13:47
opendevreviewRajat Dhasmana proposed openstack/cinder stable/wallaby: Fix: Schema validation for attachment create API  https://review.opendev.org/c/openstack/cinder/+/80278113:55
opendevreviewRajat Dhasmana proposed openstack/cinder stable/wallaby: Fix: Schema validation for attachment create API  https://review.opendev.org/c/openstack/cinder/+/80278114:11
venkatakrishnatHi eharney, enriquetaso, geguileo, hemna, rosmaita, whoami-rajat: Could you please take a look and review the patch https://review.opendev.org/c/openstack/cinder/+/776307 which is to fix IBM SVC HyperSwap snapshot clone issues.14:13
*** whoami-rajat__ is now known as whoami-rajat14:15
opendevreviewHelen Walsh proposed openstack/cinder master: PowerMax Driver - refactoring feedback  https://review.opendev.org/c/openstack/cinder/+/80675714:24
opendevreviewSeena Fallah proposed openstack/cinder master: snapshot: allow revert to any point for rbd driver  https://review.opendev.org/c/openstack/cinder/+/80674015:43
opendevreviewBrian Rosmaita proposed openstack/cinder master: Update mv version history document for 3.65  https://review.opendev.org/c/openstack/cinder/+/80676915:58
rosmaitageguileo: could use your vote on ^^15:58
amar_jungleboyj: rosmaita: Addressed all your comments https://review.opendev.org/c/openstack/cinder/+/790910 really need it get merged in xena, CI is also green.16:09
opendevreviewSeena Fallah proposed openstack/cinder master: snapshot: allow revert to any point for rbd driver  https://review.opendev.org/c/openstack/cinder/+/80674016:16
opendevreviewSeena Fallah proposed openstack/cinder master: snapshot: allow revert to any point for rbd driver  https://review.opendev.org/c/openstack/cinder/+/80674016:20
*** dviroel|out is now known as dviroel|ruck16:54
opendevreviewAndrei proposed openstack/cinder master: JovianDSS: fix iscsi target recovery function  https://review.opendev.org/c/openstack/cinder/+/80619116:54
geguileorosmaita: LGTM16:58
geguileothanks16:58
opendevreviewSofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails  https://review.opendev.org/c/openstack/cinder/+/80588717:12
venkatakrishnatHi core-reviewers: Could someone please review this patch https://review.opendev.org/c/openstack/cinder/+/788782. CI passed. Simple code-change to update missing RCCG to metadata of a volume. 17:34
enriquetasodoes anyone facing https://paste.opendev.org/show/808477/ when running tox?18:29
manoj_katari@core team:  could you please review https://review.opendev.org/c/openstack/cinder/+/791281 18:30
opendevreviewMerged openstack/cinder master: [SVF] Manage GMCV volumes on separate pools  https://review.opendev.org/c/openstack/cinder/+/78652418:36
opendevreviewBrian Rosmaita proposed openstack/cinder master: Snapshot in-use volumes without force flag  https://review.opendev.org/c/openstack/cinder/+/78956418:45
opendevreviewBrian Rosmaita proposed openstack/cinder master: Snapshot in-use volumes without force flag  https://review.opendev.org/c/openstack/cinder/+/78956418:51
opendevreviewMerged openstack/cinder stable/wallaby: PowerMax Driver - Temporary snapshot enhancements  https://review.opendev.org/c/openstack/cinder/+/79808819:00
opendevreviewMerged openstack/cinder stable/wallaby: PowerMax Driver - Fix for deleting replication group  https://review.opendev.org/c/openstack/cinder/+/79809119:00
opendevreviewMerged openstack/cinder stable/wallaby: PowerMax Driver - Fix for create group from source  https://review.opendev.org/c/openstack/cinder/+/79809419:00
opendevreviewMerged openstack/cinder master: Update mv version history document for 3.65  https://review.opendev.org/c/openstack/cinder/+/80676920:47
opendevreviewMerged openstack/cinder stable/wallaby: Fix: Schema validation for attachment create API  https://review.opendev.org/c/openstack/cinder/+/80278120:48
opendevreviewSeena Fallah proposed openstack/cinder master: snapshot: allow revert to any point for rbd driver  https://review.opendev.org/c/openstack/cinder/+/80680720:54
opendevreviewSeena Fallah proposed openstack/cinder master: snapshot: allow revert to any point for rbd driver  https://review.opendev.org/c/openstack/cinder/+/80680721:03
opendevreviewAlan Bishop proposed openstack/cinder master: Add infrastructure for testing new RBAC policies  https://review.opendev.org/c/openstack/cinder/+/80531622:41
opendevreviewAlan Bishop proposed openstack/cinder master: Implement project personas for messages  https://review.opendev.org/c/openstack/cinder/+/80663122:43
opendevreviewAlan Bishop proposed openstack/cinder master: Implement project personas for backups API  https://review.opendev.org/c/openstack/cinder/+/80445922:49
opendevreviewAlan Bishop proposed openstack/cinder master: Implement project personas for attachments  https://review.opendev.org/c/openstack/cinder/+/75994322:50
opendevreviewBrian Rosmaita proposed openstack/python-cinderclient master: WIP: updates for mv 3.66  https://review.opendev.org/c/openstack/python-cinderclient/+/80681723:10

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!