| opendevreview | katari manoj kumar proposed openstack/cinder stable/victoria: [SVF]:Storwize hyperswap volume is not deleting. https://review.opendev.org/c/openstack/cinder/+/798104 | 05:13 |
|---|---|---|
| opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Support revert any snapshot to the volume https://review.opendev.org/c/openstack/cinder-specs/+/736111 | 06:37 |
| opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Update original volume az https://review.opendev.org/c/openstack/cinder-specs/+/778437 | 06:52 |
| opendevreview | Xuan Yandong proposed openstack/os-brick master: Fix instance hard reboot error https://review.opendev.org/c/openstack/os-brick/+/780511 | 07:10 |
| opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Update original volume az https://review.opendev.org/c/openstack/cinder-specs/+/778437 | 07:27 |
| opendevreview | Tobias Urdin proposed openstack/cinder master: Add backup_swift_create_storage_policy config opt https://review.opendev.org/c/openstack/cinder/+/761113 | 07:37 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations https://review.opendev.org/c/openstack/cinder/+/786627 | 08:58 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user message for revert to snapshot https://review.opendev.org/c/openstack/cinder/+/792209 | 08:58 |
| *** bhagyashris_ is now known as bhagyashris | 09:40 | |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: NetApp: Fix terminate_connection on unmapped vol https://review.opendev.org/c/openstack/cinder/+/731167 | 10:28 |
| geguileo | tosky: you around? | 11:44 |
| tosky | geguileo: yes | 11:52 |
| geguileo | tosky: when you wrote https://review.opendev.org/c/openstack/cinder/+/671945 do you remember if you removed cinderlib tests from LVM job on purpose or was it accidentally? | 11:52 |
| tosky | geguileo: which job exactly? cinder-tempest-plugin-lvm-lio-barbican ? | 11:56 |
| geguileo | tosky: cinder-tempest-dsvm-lvm-lio-barbican | 11:57 |
| tosky | geguileo: cinder-tempest-dsvm-lvm-lio-barbican is replaced by cinder-tempest-plugin-lvm-lio-barbican | 11:59 |
| tosky | which is defined elsewhere and runs cinderlib tests | 11:59 |
| tosky | tl;dr last time I've checked cinderlib tests are running as expected on all branches >= train | 12:00 |
| geguileo | tosky: thanks | 12:00 |
| geguileo | tosky: I see it now: https://opendev.org/openstack/cinder-tempest-plugin/src/branch/master/.zuul.yaml#L102 | 12:01 |
| opendevreview | Merged openstack/cinder stable/wallaby: Retry "lvs" call on segfault for _get_thin_pool_free_space https://review.opendev.org/c/openstack/cinder/+/797161 | 13:10 |
| *** abhishekk is now known as akekane|home | 13:39 | |
| *** akekane|home is now known as abhishekk | 13:39 | |
| opendevreview | Merged openstack/cinderlib master: Continue wallaby development https://review.opendev.org/c/openstack/cinderlib/+/785677 | 13:51 |
| whoami-rajat | rosmaita: geguileo hey, I've updated the hash for cinderlib wallaby release https://review.opendev.org/c/openstack/releases/+/797422 | 14:00 |
| rosmaita | thanks! | 14:00 |
| whoami-rajat | np | 14:00 |
| geguileo | whoami-rajat: rosmaita could you give me one day? I may need to squeeze one patch in | 14:01 |
| rosmaita | geguileo: sure | 14:01 |
| geguileo | it's about the json serialization, where cinderlib is loosing precission on datetimes | 14:01 |
| whoami-rajat | the deadline is 2nd july so fine by me | 14:01 |
| geguileo | thanks! | 14:01 |
| whoami-rajat | geguileo: ack, can you -1 the release patch so release team doesn't merge it? | 14:02 |
| geguileo | whoami-rajat: yup | 14:02 |
| whoami-rajat | thanks | 14:02 |
| whoami-rajat | rosmaita: can you take a look at https://review.opendev.org/c/openstack/python-cinderclient/+/783628 , last patch blocking glance side | 14:04 |
| rosmaita | ok | 14:04 |
| rosmaita | we're not going to release cinderclient before v2 is removed, though | 14:04 |
| whoami-rajat | np, the glance patch depends on this patch so it should merge | 14:05 |
| rosmaita | ok | 14:05 |
| rosmaita | whoami-rajat: how do you feel about a release note? | 14:08 |
| rosmaita | whoami-rajat: left a comment for you on https://review.opendev.org/c/openstack/python-cinderclient/+/783628 | 14:18 |
| whoami-rajat | rosmaita: I'm not really sure if end users use the attachment create API directly for attachment (since it only reserves attachment and you need os-brick connector to update the attachment which end users don't have), the usecase I'm aware of is via nova and glance only. what do you think? | 14:25 |
| whoami-rajat | s/API/command | 14:25 |
| rosmaita | doesn't it impact use of python-brick-cinderclient-ext ? | 14:26 |
| rosmaita | whoami-rajat: why don't you say on your patch that you'll do it as a followup, and it may not be necessary, and we can figure it out later | 14:29 |
| whoami-rajat | rosmaita: I'm not very aware of brick client but i went through the code to see this strange thing https://github.com/openstack/python-brick-cinderclient-ext/blob/master/brick_cinderclient_ext/client.py#L142-L145 | 14:30 |
| whoami-rajat | rosmaita: sure, should i reply to your comment or mention in commit message? | 14:30 |
| rosmaita | in the comment | 14:31 |
| whoami-rajat | rosmaita: done, thanks | 14:32 |
| rosmaita | ty | 14:33 |
| whoami-rajat | I'm still not sure why brick is generating the random uuid for instance, maybe it just wants to do a host attach and using that uuid because it's mandatory to pass, in that case we need to update brick client code as well | 14:35 |
| rosmaita | whoami-rajat: that's exactly what is going on there | 14:37 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 14:49 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Query internal project/user id from Keystone https://review.opendev.org/c/openstack/cinder/+/797245 | 14:53 |
| *** dviroel is now known as dviroel|lunch | 15:07 | |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: zuul: pin cinder-tempest-plugin/stein-last on stein https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/798344 | 15:34 |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/stein: DNM Dummy change to test a cinder-tempest-plugin/stein fix https://review.opendev.org/c/openstack/cinder/+/798347 | 15:47 |
| *** abhishekk is now known as abhishekk|dinner | 15:55 | |
| tobias-urdin | rosmaita: took a while for https://review.opendev.org/c/openstack/cinder/+/761113 to pass due to barbican blocking, but it already has one +2 if you wouldn't mind having a look | 15:58 |
| rosmaita | tobias-urdin: ack (and i mean it this time :)) | 15:59 |
| tobias-urdin | thanks! | 15:59 |
| *** dviroel|lunch is now known as dviroel | 16:00 | |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/stein: DNM Dummy change to test a cinder-tempest-plugin/stein fix https://review.opendev.org/c/openstack/cinder/+/798347 | 16:24 |
| *** abhishekk|dinner is now known as abhishekk|home | 16:28 | |
| *** abhishekk|home is now known as abhishekk | 16:28 | |
| fungi | does anyone know if these two wiki pages are intentionally duplicates of one another, or whether one supersedes the other and should be redirected to? https://wiki.openstack.org/wiki/Reset_state https://wiki.openstack.org/wiki/Reset_State_Robustification | 16:44 |
| fungi | also adding some context in them would be nice, a page which is just a naked google docs url is fairly confusing | 16:44 |
| opendevreview | Eric Harney proposed openstack/cinder stable/victoria: Retry "lvs" call on segfault for _get_thin_pool_free_space https://review.opendev.org/c/openstack/cinder/+/798356 | 16:52 |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Update contributor docs https://review.opendev.org/c/openstack/cinderlib/+/772558 | 17:03 |
| simondodsley | Hi Team - can I get some reviews on this backport https://review.opendev.org/c/openstack/cinder/+/797107 and this patch https://review.opendev.org/c/openstack/cinder/+/792462. Both already have a +2 | 17:33 |
| opendevreview | Merged openstack/cinder master: Add backup_swift_create_storage_policy config opt https://review.opendev.org/c/openstack/cinder/+/761113 | 18:56 |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/stein: DNM Dummy change to test a cinder-tempest-plugin/stein fix https://review.opendev.org/c/openstack/cinder/+/798347 | 20:33 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp ONTAP: Fix create FlexVol pool replica https://review.opendev.org/c/openstack/cinder/+/798384 | 21:11 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required https://review.opendev.org/c/openstack/cinder/+/792462 | 21:42 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required https://review.opendev.org/c/openstack/cinder/+/792462 | 21:45 |
| opendevreview | Merged openstack/cinder stable/wallaby: [Pure Storage] Ensure multiattach volumes are not disconnected early https://review.opendev.org/c/openstack/cinder/+/797107 | 22:13 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Fix minimum SDK version required https://review.opendev.org/c/openstack/cinder/+/792462 | 22:41 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Add cinder permissions matrix https://review.opendev.org/c/openstack/cinder/+/763306 | 23:17 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!