opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 01:57 |
---|---|---|
opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Support revert any snapshot to the volume https://review.opendev.org/c/openstack/cinder-specs/+/736111 | 01:58 |
opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Update original volume az https://review.opendev.org/c/openstack/cinder-specs/+/778437 | 02:00 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 02:06 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 02:49 |
opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Update original volume az https://review.opendev.org/c/openstack/cinder-specs/+/778437 | 03:00 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 05:24 |
opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Update original resource's az https://review.opendev.org/c/openstack/cinder-specs/+/778437 | 06:04 |
opendevreview | Tushar Trambak Gite proposed openstack/cinderlib master: Update TOX setenv https://review.opendev.org/c/openstack/cinderlib/+/796819 | 08:01 |
opendevreview | Tushar 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/+/798618 | 09:13 |
*** bhagyashris_ is now known as bhagyashris | 09:15 | |
opendevreview | Tushar 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/+/798630 | 09:55 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations https://review.opendev.org/c/openstack/cinder/+/786627 | 09:57 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user message for revert to snapshot https://review.opendev.org/c/openstack/cinder/+/792209 | 09:57 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations https://review.opendev.org/c/openstack/cinder/+/793515 | 09:57 |
opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix datetime serialization https://review.opendev.org/c/openstack/cinderlib/+/798633 | 10:00 |
geguileo | whoami-rajat: rosmaita that was the cinderlib patch I would like to include before we release | 10:01 |
whoami-rajat | geguileo: ack | 10:28 |
*** amar7ibm is now known as amar7_ | 10:29 | |
whoami-rajat | geguileo: I've left some comments on your patch, mostly nits repeated in tests | 10:53 |
opendevreview | Ivan Pchelintsev proposed openstack/cinder master: Add Cinder NFS driver for Dell EMC PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 10:53 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations https://review.opendev.org/c/openstack/cinder/+/786627 | 11:43 |
opendevreview | Merged openstack/os-brick stable/train: Avoid unhandled exceptions during connecting to iSCSI portals https://review.opendev.org/c/openstack/os-brick/+/796671 | 11:48 |
opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix datetime serialization https://review.opendev.org/c/openstack/cinderlib/+/798633 | 12:07 |
geguileo | whoami-rajat: fixed, thanks | 12:07 |
rosmaita | geguileo: ack, will review | 12:10 |
rosmaita | geguileo: have a question and a nit on https://review.opendev.org/c/openstack/cinderlib/+/798633 | 12:47 |
geguileo | rosmaita: replied, though I'm not sure if my answer is clear enough | 12:52 |
opendevreview | Ivan Pchelintsev proposed openstack/cinder master: Add Cinder NFS driver for Dell EMC PowerStore https://review.opendev.org/c/openstack/cinder/+/797608 | 12:53 |
xuany | rosmaita: hello | 12:54 |
rosmaita | xuany: hello | 13:00 |
xuany | Are you free now | 13:02 |
rosmaita | geguileo: ty, your answer is perfectly clear | 13:02 |
rosmaita | xuany: yes, i am ... what's on your mind? | 13:03 |
geguileo | :-) | 13:03 |
xuany | Is there any question about the bp spec: Support revert any snapshot to the volume? | 13:03 |
xuany | and this bp: update_original_volume_az? | 13:04 |
xuany | The spec merge deadline is comming | 13:05 |
rosmaita | xuany: i have not read your updated specs yet | 13:05 |
rosmaita | and the spec merge deadline was 25 June | 13:05 |
xuany | On, Is the deadline up yet ? | 13:07 |
rosmaita | yes, but you can continue working on this for the Y release | 13:08 |
rosmaita | xuany: did you see my email last week: http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023272.html | 13:09 |
xuany | Sorry, i missed it. | 13:12 |
rosmaita | xuany: that's OK, hopefully it gives you enough info on what to do next | 13:19 |
xuany | Ok, thanks very much. | 13:20 |
xuany | Have a good day! | 13:20 |
rosmaita | xuany: you too! | 13:21 |
MrClayPole | Hi, 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 |
opendevreview | Sofia Enriquez proposed openstack/cinder stable/wallaby: LVM: Retry lvdisplay and lvcreate calls on segfault https://review.opendev.org/c/openstack/cinder/+/798671 | 14:29 |
opendevreview | Sofia Enriquez proposed openstack/cinder stable/wallaby: LVM: Retry lvdisplay and lvcreate calls on segfault https://review.opendev.org/c/openstack/cinder/+/798671 | 14:39 |
opendevreview | Eric Harney proposed openstack/cinder stable/wallaby: LVM: Use --readonly for lvdisplay in lv_has_snapshot https://review.opendev.org/c/openstack/cinder/+/798696 | 14:40 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations https://review.opendev.org/c/openstack/cinder/+/793515 | 14:57 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations https://review.opendev.org/c/openstack/cinder/+/786627 | 15:09 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user message for revert to snapshot https://review.opendev.org/c/openstack/cinder/+/792209 | 15:09 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations https://review.opendev.org/c/openstack/cinder/+/793515 | 15:11 |
opendevreview | Merged openstack/cinder master: LVM: Retry lvdisplay and lvcreate calls on segfault https://review.opendev.org/c/openstack/cinder/+/796950 | 15:25 |
opendevreview | Sofia Enriquez proposed openstack/cinder stable/wallaby: LVM: Retry lvdisplay and lvcreate calls on segfault https://review.opendev.org/c/openstack/cinder/+/798671 | 15:30 |
opendevreview | Simon Dodsley proposed openstack/cinder stable/victoria: [Pure Storage] Ensure multiattach volumes are not disconnected early https://review.opendev.org/c/openstack/cinder/+/798672 | 15:31 |
opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix datetime serialization https://review.opendev.org/c/openstack/cinderlib/+/798633 | 15:50 |
opendevreview | Gorka Eguileor proposed openstack/cinderlib master: Fix test_global_setup unit test https://review.opendev.org/c/openstack/cinderlib/+/798706 | 15:50 |
geguileo | whoami-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 |
rosmaita | geguileo: ack | 15:50 |
*** dviroel is now known as dviroel|lunch | 16:03 | |
geguileo | rosmaita: you are right about the auditor/system reader, I had somehow forgotten about that | 16:10 |
geguileo | rosmaita: no need to check with Lance, that's what was agreed | 16:10 |
rosmaita | geguileo: ok, cool | 16:10 |
rosmaita | i will probably still check to make sure nothing changed in the past few months | 16:11 |
opendevreview | Brian Rosmaita proposed openstack/cinder master: Add cinder permissions matrix https://review.opendev.org/c/openstack/cinder/+/763306 | 16:41 |
*** dviroel|lunch is now known as dviroel | 17:08 | |
opendevreview | Merged openstack/cinderlib master: Fix test_global_setup unit test https://review.opendev.org/c/openstack/cinderlib/+/798706 | 19:27 |
opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes https://review.opendev.org/c/openstack/cinder/+/796286 | 19:54 |
opendevreview | Sofia Enriquez proposed openstack/os-brick master: LVM: Retry lv calls on segfault due to lvs failure https://review.opendev.org/c/openstack/os-brick/+/798743 | 20:06 |
simondodsley | Hi - 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 |
opendevreview | Merged openstack/cinderlib master: Fix datetime serialization https://review.opendev.org/c/openstack/cinderlib/+/798633 | 20:57 |
opendevreview | Simon Dodsley proposed openstack/cinder stable/victoria: [Pure Storage] Ensure multiattach volumes are not disconnected early https://review.opendev.org/c/openstack/cinder/+/798672 | 20:57 |
rosmaita | geguileo: review for future gorka: https://review.opendev.org/c/openstack/releases/+/797422 | 21:07 |
geguileo | rosmaita: future gorka disapproves };-~ | 21:08 |
geguileo | rosmaita: done, thanks | 21:09 |
rosmaita | cool, ty | 21:09 |
geguileo | ty | 21:10 |
opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Open cinderlib for xena development https://review.opendev.org/c/openstack/cinderlib/+/798749 | 21:21 |
*** dviroel is now known as dviroel|out | 21:22 | |
geguileo | rosmaita: thanks for that patch ^ :-) | 21:55 |
opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 8 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/729766 | 22:08 |
opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 8 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/729766 | 22:11 |
opendevreview | Eric Harney proposed openstack/cinder master: Tests: Simplify LVM fake_execute https://review.opendev.org/c/openstack/cinder/+/769602 | 22:28 |
opendevreview | Eric Harney proposed openstack/cinder master: LVM: Simplify version fetching https://review.opendev.org/c/openstack/cinder/+/796946 | 22:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!