Tuesday, 2021-06-29

opendevreviewTakashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone  https://review.opendev.org/c/openstack/cinder/+/79724501:57
opendevreviewXuan Yandong proposed openstack/cinder-specs master: Support revert any snapshot to the volume  https://review.opendev.org/c/openstack/cinder-specs/+/73611101:58
opendevreviewXuan Yandong proposed openstack/cinder-specs master: Update original volume az  https://review.opendev.org/c/openstack/cinder-specs/+/77843702:00
opendevreviewTakashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone  https://review.opendev.org/c/openstack/cinder/+/79724502:06
opendevreviewTakashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone  https://review.opendev.org/c/openstack/cinder/+/79724502:49
opendevreviewXuan Yandong proposed openstack/cinder-specs master: Update original volume az  https://review.opendev.org/c/openstack/cinder-specs/+/77843703:00
opendevreviewTakashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone  https://review.opendev.org/c/openstack/cinder/+/79724505:24
opendevreviewXuan Yandong proposed openstack/cinder-specs master: Update original resource's az  https://review.opendev.org/c/openstack/cinder-specs/+/77843706:04
opendevreviewTushar Trambak Gite proposed openstack/cinderlib master: Update TOX setenv  https://review.opendev.org/c/openstack/cinderlib/+/79681908:01
opendevreviewTushar Trambak Gite proposed openstack/python-brick-cinderclient-ext master: Move python-brick-cinderclient-ext to new hacking 4.0.0  https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/79861809:13
*** bhagyashris_ is now known as bhagyashris09:15
opendevreviewTushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Change minversion to 3.18.0 in tox  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/79863009:55
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations  https://review.opendev.org/c/openstack/cinder/+/78662709:57
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user message for revert to snapshot  https://review.opendev.org/c/openstack/cinder/+/79220909:57
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations  https://review.opendev.org/c/openstack/cinder/+/79351509:57
opendevreviewGorka Eguileor proposed openstack/cinderlib master: Fix datetime serialization  https://review.opendev.org/c/openstack/cinderlib/+/79863310:00
geguileowhoami-rajat: rosmaita  that was the cinderlib patch I would like to include before we release10:01
whoami-rajatgeguileo: ack10:28
*** amar7ibm is now known as amar7_10:29
whoami-rajatgeguileo: I've left some comments on your patch, mostly nits repeated in tests10:53
opendevreviewIvan Pchelintsev proposed openstack/cinder master: Add Cinder NFS driver for Dell EMC PowerStore  https://review.opendev.org/c/openstack/cinder/+/79760810:53
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations  https://review.opendev.org/c/openstack/cinder/+/78662711:43
opendevreviewMerged openstack/os-brick stable/train: Avoid unhandled exceptions during connecting to iSCSI portals  https://review.opendev.org/c/openstack/os-brick/+/79667111:48
opendevreviewGorka Eguileor proposed openstack/cinderlib master: Fix datetime serialization  https://review.opendev.org/c/openstack/cinderlib/+/79863312:07
geguileowhoami-rajat: fixed, thanks12:07
rosmaitageguileo: ack, will review12:10
rosmaitageguileo: have a question and a nit on https://review.opendev.org/c/openstack/cinderlib/+/79863312:47
geguileorosmaita: replied, though I'm not sure if my answer is clear enough12:52
opendevreviewIvan Pchelintsev proposed openstack/cinder master: Add Cinder NFS driver for Dell EMC PowerStore  https://review.opendev.org/c/openstack/cinder/+/79760812:53
xuanyrosmaita: hello12:54
rosmaitaxuany: hello13:00
xuanyAre you free now13:02
rosmaitageguileo: ty, your answer is perfectly clear13:02
rosmaitaxuany: yes, i am ... what's on your mind?13:03
geguileo:-)13:03
xuanyIs there any question about the bp spec: Support revert any snapshot to the volume?13:03
xuanyand this bp: update_original_volume_az?13:04
xuanyThe spec merge deadline is comming13:05
rosmaitaxuany: i have not read your updated specs yet13:05
rosmaitaand the spec merge deadline was 25 June13:05
xuanyOn, Is the deadline up yet ?13:07
rosmaitayes, but you can continue working on this for the Y release13:08
rosmaitaxuany: did you see my email last week: http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023272.html13:09
xuanySorry, i missed it.13:12
rosmaitaxuany: that's OK, hopefully it gives you enough info on what to do next13:19
xuanyOk, thanks very much.13:20
xuanyHave a good day!13:20
rosmaitaxuany: you too!13:21
MrClayPoleHi, I've been reviewing the "cinder attachment-show" command. I have a powered on instance with 2 volumes but only 1 of the volumes shows the iscsi LUN number. Is this normal? The commands and its output are shown here http://paste.openstack.org/raw/807026/14:25
opendevreviewSofia Enriquez proposed openstack/cinder stable/wallaby: LVM: Retry lvdisplay and lvcreate calls on segfault  https://review.opendev.org/c/openstack/cinder/+/79867114:29
opendevreviewSofia Enriquez proposed openstack/cinder stable/wallaby: LVM: Retry lvdisplay and lvcreate calls on segfault  https://review.opendev.org/c/openstack/cinder/+/79867114:39
opendevreviewEric Harney proposed openstack/cinder stable/wallaby: LVM: Use --readonly for lvdisplay in lv_has_snapshot  https://review.opendev.org/c/openstack/cinder/+/79869614:40
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations  https://review.opendev.org/c/openstack/cinder/+/79351514:57
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations  https://review.opendev.org/c/openstack/cinder/+/78662715:09
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user message for revert to snapshot  https://review.opendev.org/c/openstack/cinder/+/79220915:09
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations  https://review.opendev.org/c/openstack/cinder/+/79351515:11
opendevreviewMerged openstack/cinder master: LVM: Retry lvdisplay and lvcreate calls on segfault  https://review.opendev.org/c/openstack/cinder/+/79695015:25
opendevreviewSofia Enriquez proposed openstack/cinder stable/wallaby: LVM: Retry lvdisplay and lvcreate calls on segfault  https://review.opendev.org/c/openstack/cinder/+/79867115:30
opendevreviewSimon Dodsley proposed openstack/cinder stable/victoria: [Pure Storage] Ensure multiattach volumes are not disconnected early  https://review.opendev.org/c/openstack/cinder/+/79867215:31
opendevreviewGorka Eguileor proposed openstack/cinderlib master: Fix datetime serialization  https://review.opendev.org/c/openstack/cinderlib/+/79863315:50
opendevreviewGorka Eguileor proposed openstack/cinderlib master: Fix test_global_setup unit test  https://review.opendev.org/c/openstack/cinderlib/+/79870615:50
geguileowhoami-rajat: rosmaita the cinderlib patch failed due to an unrelated unit test issue.  I'm proposed a patch to fix that issue and rebased that patch on top of it  ^15:50
rosmaitageguileo: ack15:50
*** dviroel is now known as dviroel|lunch16:03
geguileorosmaita: you are right about the auditor/system reader, I had somehow forgotten about that16:10
geguileorosmaita: no need to check with Lance, that's what was agreed16:10
rosmaitageguileo: ok, cool16:10
rosmaitai will probably still check to make sure nothing changed in the past few months16:11
opendevreviewBrian Rosmaita proposed openstack/cinder master: Add cinder permissions matrix  https://review.opendev.org/c/openstack/cinder/+/76330616:41
*** dviroel|lunch is now known as dviroel17:08
opendevreviewMerged openstack/cinderlib master: Fix test_global_setup unit test  https://review.opendev.org/c/openstack/cinderlib/+/79870619:27
opendevreviewHelen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes  https://review.opendev.org/c/openstack/cinder/+/79628619:54
opendevreviewSofia Enriquez proposed openstack/os-brick master: LVM: Retry lv calls on segfault due to lvs failure  https://review.opendev.org/c/openstack/os-brick/+/79874320:06
simondodsleyHi - I have restored https://review.opendev.org/c/openstack/cinder/+/779781 as this is required to allow any backports to Victoria to work for our driver. Given this is a simple flag to allow devstack to correctly work I don't see that there is too much of an issue here.20:08
opendevreviewMerged openstack/cinderlib master: Fix datetime serialization  https://review.opendev.org/c/openstack/cinderlib/+/79863320:57
opendevreviewSimon Dodsley proposed openstack/cinder stable/victoria: [Pure Storage] Ensure multiattach volumes are not disconnected early  https://review.opendev.org/c/openstack/cinder/+/79867220:57
rosmaitageguileo: review for future gorka: https://review.opendev.org/c/openstack/releases/+/79742221:07
geguileorosmaita: future gorka disapproves  };-~21:08
geguileorosmaita: done, thanks21:09
rosmaitacool, ty21:09
geguileoty21:10
opendevreviewBrian Rosmaita proposed openstack/cinderlib master: Open cinderlib for xena development  https://review.opendev.org/c/openstack/cinderlib/+/79874921:21
*** dviroel is now known as dviroel|out21:22
geguileorosmaita: thanks for that patch ^  :-)21:55
opendevreviewLuigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 8  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/72976622:08
opendevreviewLuigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 8  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/72976622:11
opendevreviewEric Harney proposed openstack/cinder master: Tests: Simplify LVM fake_execute  https://review.opendev.org/c/openstack/cinder/+/76960222:28
opendevreviewEric Harney proposed openstack/cinder master: LVM: Simplify version fetching  https://review.opendev.org/c/openstack/cinder/+/79694622:28

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