| opendevreview | Girish Chilukuri proposed openstack/cinder master: [SVF]:Retype in-use hyperswap volume https://review.opendev.org/c/openstack/cinder/+/782390 | 02:24 |
|---|---|---|
| amar_ | jungleboyj: Thank you : | 03:00 |
| jungleboyj | amar_: Welcome. :-) | 03:00 |
| opendevreview | Zohar Mamedov proposed openstack/os-brick master: NVMeOF Connection Agent https://review.opendev.org/c/openstack/os-brick/+/802691 | 03:24 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 04:12 |
| opendevreview | katari manoj kumar proposed openstack/cinder master: Doc: IBM Storages user documentation update https://review.opendev.org/c/openstack/cinder/+/803901 | 04:25 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 07:11 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 09:46 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Migrate all quota parameters to [quota] section. https://review.opendev.org/c/openstack/cinder/+/772181 | 10:00 |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [svf] Fix rccg_name based on group display_name https://review.opendev.org/c/openstack/cinder/+/792800 | 10:47 |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [svf] Fix rccg_name based on group display_name https://review.opendev.org/c/openstack/cinder/+/792800 | 10:51 |
| opendevreview | yangheng proposed openstack/cinder master: Support Cinder FC driver for TOYOU NetStor https://review.opendev.org/c/openstack/cinder/+/803000 | 11:25 |
| *** dviroel|out is now known as dviroel | 11:26 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 12:02 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix cinder-manage clean_locks command https://review.opendev.org/c/openstack/cinder/+/804071 | 12:10 |
| opendevreview | Merged openstack/cinder stable/victoria: NFS: Update connection info on online snap create https://review.opendev.org/c/openstack/cinder/+/802039 | 12:21 |
| *** geguileo is now known as Guest3812 | 12:30 | |
| *** geguileor is now known as geguileo | 12:48 | |
| geguileo | rosmaita: guess who messed up the cinder-manage command to clean file locks... https://review.opendev.org/c/openstack/cinder/+/804071 | 12:50 |
| rosmaita | geguileo: guess you really needed that vacation! | 12:51 |
| geguileo | lol | 13:02 |
| geguileo | rosmaita: I'm taking next week as well lol | 13:02 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: image cache https://review.opendev.org/c/openstack/cinder/+/788363 | 13:24 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: continued manager, scheduler, rpcapi https://review.opendev.org/c/openstack/cinder/+/784453 | 13:25 |
| *** lbragstad__ is now known as lbragstad | 13:35 | |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: [WIP] NetApp ONTAP: Add storage assisted migration support https://review.opendev.org/c/openstack/cinder/+/799668 | 13:42 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add option to report storage provisioned capacity https://review.opendev.org/c/openstack/cinder/+/798198 | 13:50 |
| venkatakrishnat | Could someone please review https://review.opendev.org/c/openstack/cinder/+/774594 which is to support extend feature to HyperSwap volume on IBM cinder driver. Waiting for second +2 | 14:03 |
| *** whoami-rajat__ is now known as whoami-rajat | 14:06 | |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: create_volume flows https://review.opendev.org/c/openstack/cinder/+/780933 | 14:27 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: image cache https://review.opendev.org/c/openstack/cinder/+/788363 | 14:27 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: continued manager, scheduler, rpcapi https://review.opendev.org/c/openstack/cinder/+/784453 | 14:27 |
| gmann | rosmaita: can you check this, if sdk job is helpful for cinder gate https://review.opendev.org/c/openstack/cinder/+/801875 | 14:37 |
| opendevreview | Fábio Oliveira proposed openstack/cinder master: Netapp ONTAP: Add support to revert to snapshot https://review.opendev.org/c/openstack/cinder/+/804093 | 14:44 |
| rosmaita | gmann: looking | 14:51 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: continued manager, scheduler, rpcapi https://review.opendev.org/c/openstack/cinder/+/784453 | 14:52 |
| rosmaita | gmann: would there be any point in running the job in check only? (though i notice that nova runs it in both check and gate) | 15:02 |
| amar__ | Hi jungleboyj, There is small conflict at Zuul gating after +2 & W . So resolved it in the new patch and CI is also green now. Could you give your +2 for this patch again : https://review.opendev.org/c/openstack/cinder/+/782390 | 15:41 |
| zenkuro | hi rosmaita! can you please check https://review.opendev.org/c/openstack/cinder/+/794962 | 15:44 |
| rosmaita | zenkuro: do you have a blueprint filed in launchpad for that? | 15:45 |
| gmann | rosmaita: actually, for all voting jobs has to run in check and gate pipeline. | 15:45 |
| rosmaita | gmann: didn't realize that ... ik | 15:45 |
| rosmaita | i mean, "ok" | 15:45 |
| gmann | that is guidelines actually. not like zuul will fail. | 15:46 |
| rosmaita | gmann: so the jobs here: https://opendev.org/openstack/python-cinderclient/src/branch/master/.zuul.yaml#L41-L42 should also be run in the gate, I guess | 15:47 |
| gmann | rosmaita: yeah that will help to avoid integrated issue can occur for related repo change merging at same time | 15:49 |
| rosmaita | gmann: thanks, glad I asked ... i will put up a patch. I thought we were being clever and conserving resources, I didn't realize we were ignoring a guideline! | 15:50 |
| gmann | +1, thanks | 15:50 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: DNM: prototype base policy file https://review.opendev.org/c/openstack/cinder/+/803748 | 16:07 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Implement project personas for attachments https://review.opendev.org/c/openstack/cinder/+/759943 | 16:11 |
| manoj_katari | @core team: could someone review this change https://review.opendev.org/c/openstack/cinder/+/784092 pending for a long time. | 16:39 |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: Add functional jobs to the gate https://review.opendev.org/c/openstack/python-cinderclient/+/804115 | 16:42 |
| opendevreview | Eric Harney proposed openstack/cinder master: conditional_update(): disable false pylint error message https://review.opendev.org/c/openstack/cinder/+/804118 | 17:00 |
| gksk | To the Core reviewers pls review the below patch related to update volume rep properties fix for empty values | 17:04 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/801617/2 | 17:04 |
| simondodsley | Can cores have a look at this backport - needs final +2 and +W: https://review.opendev.org/c/openstack/cinder/+/798879 | 17:29 |
| opendevreview | Merged openstack/cinder master: Add openstacksdk-functional-devstack job in cinder gate https://review.opendev.org/c/openstack/cinder/+/801875 | 17:48 |
| rosmaita | simondodsley: got it | 18:36 |
| *** dviroel is now known as dviroel|brb | 18:41 | |
| *** dviroel|brb is now known as dviroel | 19:00 | |
| *** aprice_ is now known as aprice | 19:02 | |
| *** lbragstad_ is now known as lbragstad | 19:45 | |
| opendevreview | Merged openstack/cinder stable/wallaby: [Pure Storage] Fix minimum SDK version required https://review.opendev.org/c/openstack/cinder/+/798879 | 19:51 |
| simondodsley | Can I get some reviews from core for this XS patch: https://review.opendev.org/c/openstack/cinder/+/803046 | 19:55 |
| *** dviroel is now known as dviroel|out | 20:52 | |
| Roamer` | so... if anyone has two minutes to look at two trivial bugfixes, one in the StorPool driver (https://review.opendev.org/c/openstack/cinder/+/787296), the other one in the general handling of Cinder-backed Glance images (https://review.opendev.org/c/openstack/cinder/+/803862), it'd be great :) BTW both of the bugs are present in Wallaby, maybe way further back. | 21:41 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!