| *** pcaruana has quit IRC | 00:04 | |
| *** pcaruana has joined #openstack-cinder | 00:16 | |
| *** Xuchu has joined #openstack-cinder | 00:46 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder master: Revert "Remove Nimble Storage Driver" https://review.opendev.org/711362 | 00:52 |
|---|---|---|
| *** sfernand has quit IRC | 00:58 | |
| *** Liang__ has joined #openstack-cinder | 01:12 | |
| openstackgerrit | Xuchu Jiang proposed openstack/cinder master: Add an option to specify iSCSI IP version https://review.opendev.org/698630 | 01:17 |
| *** thgcorrea has quit IRC | 01:18 | |
| Xuchu | rosmaita: I updated this patch based on your comments, can you help to review it agian while you are avaiable ? | 01:20 |
| *** zhanglong has joined #openstack-cinder | 01:28 | |
| *** lvbin02 has joined #openstack-cinder | 01:46 | |
| *** lvbin01 has quit IRC | 01:49 | |
| *** lvbin02 is now known as lvbin01 | 01:49 | |
| *** spatel has joined #openstack-cinder | 02:03 | |
| *** zhenguo_ has joined #openstack-cinder | 02:33 | |
| openstackgerrit | DingDong proposed openstack/cinder master: [Unity] Support create volume with tiering policy. https://review.opendev.org/668621 | 02:33 |
| openstackgerrit | Xuchu Jiang proposed openstack/cinder master: Fix revert snapshot issue https://review.opendev.org/707873 | 02:45 |
| *** psachin has joined #openstack-cinder | 03:29 | |
| *** threestrands has joined #openstack-cinder | 03:39 | |
| *** ociuhandu has joined #openstack-cinder | 03:43 | |
| *** ociuhandu has quit IRC | 03:48 | |
| *** spatel has quit IRC | 04:11 | |
| *** eharney has quit IRC | 04:19 | |
| *** eharney has joined #openstack-cinder | 04:31 | |
| *** udesale has joined #openstack-cinder | 04:42 | |
| *** zhenguo_ has quit IRC | 04:42 | |
| *** ykarel|away is now known as ykarel | 04:45 | |
| *** dave-mccowan has quit IRC | 05:03 | |
| *** Xuchu has quit IRC | 05:05 | |
| *** Xuchu has joined #openstack-cinder | 05:10 | |
| *** dave-mccowan has joined #openstack-cinder | 05:11 | |
| *** Liang__ is now known as LiangFang | 05:18 | |
| *** ruffian_sheep has joined #openstack-cinder | 05:38 | |
| *** rosmaita has quit IRC | 05:39 | |
| *** rosmaita has joined #openstack-cinder | 05:40 | |
| ruffian_sheep | whoami-rajat: Hello i am back again | 05:42 |
| openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add support for VxFlex OS 3.5 to VxFlex OS driver https://review.opendev.org/705176 | 05:42 |
| openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add OpenStack volume replication v2.1 support in VxFlex OS driver https://review.opendev.org/707390 | 05:42 |
| openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add support for volume migration in VxFlex OS driver https://review.opendev.org/711624 | 05:42 |
| openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add revert to snapshot support in VxFlex OS driver https://review.opendev.org/712622 | 05:42 |
| ruffian_sheep | lol | 05:42 |
| ruffian_sheep | rosmaita : Do you have time to help locate the problem? About temepst test case | 05:43 |
| openstackgerrit | Yong Huang proposed openstack/cinder stable/rocky: [Unity] Fix TypeError for test case test_delete_host_wo_lock https://review.opendev.org/716156 | 06:01 |
| *** dpawlik has joined #openstack-cinder | 06:20 | |
| openstackgerrit | wuqiongdan proposed openstack/cinder master: Add sandstone iscsi driver. https://review.opendev.org/708673 | 06:25 |
| *** dpawlik has quit IRC | 06:28 | |
| *** vishalmanchanda has joined #openstack-cinder | 06:38 | |
| *** dpawlik has joined #openstack-cinder | 06:47 | |
| *** ociuhandu has joined #openstack-cinder | 07:24 | |
| *** ociuhandu has quit IRC | 07:29 | |
| *** tosky has joined #openstack-cinder | 07:29 | |
| *** lvbin02 has joined #openstack-cinder | 07:36 | |
| *** lvbin01 has quit IRC | 07:38 | |
| *** lvbin02 is now known as lvbin01 | 07:38 | |
| *** ruffian_sheep has quit IRC | 07:39 | |
| *** e0ne has joined #openstack-cinder | 07:44 | |
| *** ociuhandu has joined #openstack-cinder | 07:55 | |
| openstackgerrit | DingDong proposed openstack/cinder master: [Unity] Support consistency group replication https://review.opendev.org/691557 | 07:58 |
| *** yaawang has quit IRC | 08:23 | |
| *** yaawang has joined #openstack-cinder | 08:25 | |
| *** threestrands has quit IRC | 08:26 | |
| *** zhanglong has quit IRC | 08:33 | |
| *** martinkennelly has joined #openstack-cinder | 08:37 | |
| *** Luzi has joined #openstack-cinder | 08:44 | |
| *** zhanglong has joined #openstack-cinder | 08:48 | |
| *** ociuhandu has quit IRC | 08:49 | |
| *** ociuhandu has joined #openstack-cinder | 08:50 | |
| *** lpetrut has joined #openstack-cinder | 08:53 | |
| *** rcernin has quit IRC | 08:54 | |
| *** ruffian_sheep has joined #openstack-cinder | 09:08 | |
| ruffian_sheep | whoami-rajat: hi,are u online? | 09:08 |
| *** ruffian_sheep has quit IRC | 09:16 | |
| *** LiangFang has quit IRC | 09:16 | |
| *** n-saito has quit IRC | 09:17 | |
| *** zhanglong has quit IRC | 09:21 | |
| *** zhanglong has joined #openstack-cinder | 09:22 | |
| *** zhanglong has quit IRC | 09:45 | |
| *** ociuhandu has quit IRC | 09:46 | |
| *** lvbin02 has joined #openstack-cinder | 09:47 | |
| *** lvbin01 has quit IRC | 09:49 | |
| *** lvbin02 is now known as lvbin01 | 09:49 | |
| *** ociuhandu has joined #openstack-cinder | 09:58 | |
| *** rcernin has joined #openstack-cinder | 09:59 | |
| *** yaawang has quit IRC | 10:08 | |
| *** ociuhandu has quit IRC | 10:09 | |
| *** yaawang has joined #openstack-cinder | 10:09 | |
| *** ociuhandu has joined #openstack-cinder | 10:10 | |
| *** ykarel is now known as ykarel|lunch | 10:11 | |
| *** rcernin has quit IRC | 10:14 | |
| *** ociuhandu has quit IRC | 10:14 | |
| *** ociuhandu has joined #openstack-cinder | 10:14 | |
| *** zzzeek has quit IRC | 10:44 | |
| *** zzzeek has joined #openstack-cinder | 10:45 | |
| *** ykarel|lunch is now known as ykarel | 10:52 | |
| *** felipe_rodrigues has quit IRC | 11:18 | |
| *** psachin has quit IRC | 11:35 | |
| *** psachin has joined #openstack-cinder | 11:36 | |
| *** psachin has quit IRC | 11:53 | |
| *** psachin has joined #openstack-cinder | 11:54 | |
| *** ociuhandu has quit IRC | 12:04 | |
| *** ociuhandu has joined #openstack-cinder | 12:04 | |
| *** zzzeek has quit IRC | 12:05 | |
| *** zzzeek has joined #openstack-cinder | 12:06 | |
| *** ociuhandu has quit IRC | 12:10 | |
| hemna | mep | 12:18 |
| openstackgerrit | DingDong proposed openstack/cinder master: [Unity] Support consistency group replication https://review.opendev.org/691557 | 12:20 |
| openstackgerrit | wuqiongdan proposed openstack/cinder master: Add sandstone iscsi driver. https://review.opendev.org/708673 | 12:34 |
| *** udesale_ has joined #openstack-cinder | 12:38 | |
| *** udesale has quit IRC | 12:41 | |
| *** zzzeek has quit IRC | 12:44 | |
| *** ociuhandu has joined #openstack-cinder | 12:45 | |
| *** zzzeek has joined #openstack-cinder | 12:45 | |
| *** ociuhandu has quit IRC | 12:58 | |
| *** ociuhandu has joined #openstack-cinder | 12:58 | |
| *** ociuhandu has quit IRC | 13:04 | |
| *** ociuhandu has joined #openstack-cinder | 13:04 | |
| *** swegener has quit IRC | 13:05 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Version comparison correction https://review.opendev.org/713329 | 13:09 |
| *** ociuhandu has quit IRC | 13:15 | |
| *** ociuhandu has joined #openstack-cinder | 13:16 | |
| *** ociuhandu has quit IRC | 13:21 | |
| *** ykarel is now known as ykarel|mtg | 13:22 | |
| openstackgerrit | Merged openstack/cinder master: PowerMax Driver - Support of Multiple Replication https://review.opendev.org/709669 | 13:29 |
| *** ociuhandu has joined #openstack-cinder | 13:32 | |
| *** ykarel|mtg is now known as ykarel | 13:35 | |
| jungleboyj | rosmaita: Do we want to add a note here that the driver was removed and resotred? https://review.opendev.org/#/c/711362/7/releasenotes/notes/nimble-mark-unsupported-0c8e5e21c8d2179d.yaml | 13:36 |
| rosmaita | i don't think so | 13:36 |
| rosmaita | if we remove and put it back in the same release, it's like it was never removed | 13:36 |
| jungleboyj | Ok. I had thought maybe but am ok not doing that. | 13:37 |
| jungleboyj | Oh, yeah, all in the same release. Ok. | 13:37 |
| rosmaita | yeah, i think it's complicated enough as it is! | 13:37 |
| *** zzzeek has quit IRC | 13:38 | |
| *** zzzeek has joined #openstack-cinder | 13:39 | |
| smcginnis | rosmaita, jungleboyj: We announced in Train that the driver was unsupported and needed the flag set. Not sure why they are adding another note stating that when the driver is actually now being supported. | 13:43 |
| rosmaita | smcginnis: the situation is that this patch isn't marking it supported because the CI is still unstable | 13:44 |
| rosmaita | it's just restoring the driver | 13:44 |
| rosmaita | because our new policy wouldn't have removed it | 13:44 |
| smcginnis | So why are we adding another deprecation notice in Ussuri? Should just drop any release notes from there. | 13:45 |
| rosmaita | well, the train note said unsupported in T and dropped in U | 13:45 |
| smcginnis | Otherwise that's just confusing to have another mention is ussuri that the driver is deprecated in train. | 13:45 |
| smcginnis | Plans change. :) | 13:45 |
| rosmaita | so we want to say that it's still there in U | 13:46 |
| smcginnis | Then we should say that. | 13:46 |
| smcginnis | The text right now is misleading and confusing. | 13:46 |
| jungleboyj | smcginnis: ++ | 13:46 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Version comparison correction https://review.opendev.org/713329 | 13:46 |
| smcginnis | I think the better approach would just be add a prelude section overall for this release stating that some previously deprecated drivers are still present and can be used at your own risk, yada yada yada. | 13:46 |
| rosmaita | i was going to add a list to the upgrade section, but same basic idea | 13:47 |
| smcginnis | Good, then I would say definitely remove this note. | 13:47 |
| rosmaita | smcginnis: jungleboyj: so is the consensus that they do *not* need a release note on the nimble restore patch? | 13:48 |
| rosmaita | in other words, PS6 was fine? | 13:49 |
| smcginnis | That would be my preference, and what I think would be the least confusing. Especially if we follow through on a general statement about unsupported drivers. | 13:49 |
| jungleboyj | Ooops. | 13:49 |
| rosmaita | :) | 13:49 |
| rosmaita | ok, that's what I'll tell them then | 13:49 |
| jungleboyj | Sorry, I didn't know about the plan for the more general note. | 13:50 |
| rosmaita | yeah, i should have mentioned it, i was thinking we need to do that because of the other drivers that will be un-removed over the next week | 13:51 |
| *** Luzi has quit IRC | 13:54 | |
| jungleboyj | Ok. That makes sense. Sorry for causing confusion. | 13:56 |
| smcginnis | rosmaita, jungleboyj: Would one of you have a minute to take a look at https://review.opendev.org/#/c/715683/ | 13:58 |
| rosmaita | sure | 13:59 |
| smcginnis | Reenables our hacking checks, so I think it would be good to get that in soon to make sure nothing sneaks in. | 13:59 |
| rosmaita | good point | 13:59 |
| jungleboyj | Looking. | 13:59 |
| *** lpetrut has quit IRC | 14:01 | |
| jungleboyj | Yeah, that would be good to get back. | 14:01 |
| jungleboyj | Done. | 14:02 |
| smcginnis | Thanks! | 14:02 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Compression Change Bug Fix https://review.opendev.org/715965 | 14:02 |
| jungleboyj | rosmaita: Do you want to look or do you want me to merge? | 14:02 |
| rosmaita | just finished, i'll merge | 14:03 |
| jungleboyj | Cool. Thanks. | 14:04 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Drop unicode() hacking check https://review.opendev.org/715742 | 14:06 |
| *** lpetrut has joined #openstack-cinder | 14:08 | |
| jungleboyj | rosmaita: Happy to see you running for PTL again. :-) | 14:18 |
| rosmaita | :D | 14:19 |
| *** lvbin01 has quit IRC | 14:20 | |
| *** lvbin01 has joined #openstack-cinder | 14:20 | |
| *** ajitha has joined #openstack-cinder | 14:21 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Refreshing stale metadata https://review.opendev.org/712977 | 14:22 |
| smcginnis | Yeah, thanks rosmaita! | 14:22 |
| *** dave-mccowan has quit IRC | 14:23 | |
| *** ociuhandu has quit IRC | 14:24 | |
| openstackgerrit | Xuan Yandong proposed openstack/cinder master: Add features for add backup id to volume metadata https://review.opendev.org/705876 | 14:25 |
| *** ociuhandu has joined #openstack-cinder | 14:26 | |
| *** zzzeek has quit IRC | 14:28 | |
| *** ociuhandu has quit IRC | 14:31 | |
| *** zzzeek has joined #openstack-cinder | 14:32 | |
| *** ociuhandu has joined #openstack-cinder | 14:35 | |
| openstackgerrit | DingDong proposed openstack/cinder master: [Unity] Support consistency group replication https://review.opendev.org/691557 | 14:40 |
| *** ykarel is now known as ykarel|away | 14:41 | |
| *** tkajinam has quit IRC | 14:45 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Deletion of group with volumes https://review.opendev.org/707828 | 14:49 |
| *** dave-mccowan has joined #openstack-cinder | 15:06 | |
| openstackgerrit | Merged openstack/cinder master: Re-enable local hacking checks https://review.opendev.org/715683 | 15:18 |
| *** vishalmanchanda has quit IRC | 15:18 | |
| *** raunak12 has joined #openstack-cinder | 15:32 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder master: Revert "Remove Nimble Storage Driver" https://review.opendev.org/711362 | 15:34 |
| *** Xuchu has quit IRC | 15:39 | |
| *** vishalmanchanda has joined #openstack-cinder | 15:41 | |
| *** Xuchu has joined #openstack-cinder | 15:50 | |
| *** thgcorrea has joined #openstack-cinder | 15:59 | |
| *** udesale_ has quit IRC | 16:01 | |
| *** eharney has quit IRC | 16:15 | |
| *** ajitha has quit IRC | 16:16 | |
| *** eharney has joined #openstack-cinder | 16:19 | |
| *** lpetrut has quit IRC | 16:24 | |
| *** raunak12 has quit IRC | 16:37 | |
| *** psachin has quit IRC | 16:50 | |
| *** pcaruana has quit IRC | 16:56 | |
| *** pcaruana has joined #openstack-cinder | 17:10 | |
| openstackgerrit | Brian Rosmaita proposed openstack/cinder master: RBD: add support for revert-to-snapshot https://review.opendev.org/710334 | 17:26 |
| *** ociuhandu has quit IRC | 17:33 | |
| *** ociuhandu has joined #openstack-cinder | 17:34 | |
| *** ociuhandu has quit IRC | 17:39 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add hacking check for 3rd party mock https://review.opendev.org/716380 | 18:16 |
| *** martinkennelly has quit IRC | 18:35 | |
| thgcorrea | Hi all, I've added the unit test and got a +1 on this patch, I'd appreciate if anyone could review it: https://review.opendev.org/#/c/710336 thanks | 18:48 |
| *** e0ne has quit IRC | 18:54 | |
| *** e0ne has joined #openstack-cinder | 18:55 | |
| rosmaita | smcginnis: i came across another one of those orphaned release notes: cinder/volume/drivers/hpe/releasenotes/notes/hpe_3par_multiattach_detach_fix-d2d3785d656fba90.yaml -- it's from Train -- what should we do about it? | 19:14 |
| smcginnis | Bahhhh! | 19:17 |
| rosmaita | yeah | 19:17 |
| rosmaita | it's a real short note, maybe no one will miss it | 19:17 |
| smcginnis | rosmaita: Unless something has changed, only way to get it to work right now is to propose moving it directly to stable/train. | 19:17 |
| rosmaita | (especially since no one's ever seen it!) | 19:17 |
| smcginnis | Haha, yeah. | 19:17 |
| rosmaita | ok, i was wondering | 19:17 |
| rosmaita | i'll put that on my list | 19:18 |
| rosmaita | thanks | 19:18 |
| smcginnis | Certainly an option to just drop the file if it's not too critical. | 19:18 |
| rosmaita | if we move it to stable/train, will it be included in the master releasenotes? | 19:18 |
| rosmaita | i think not because of the branch point? | 19:19 |
| smcginnis | As long as the move is only done in stable/train, then reno should only pick it up there. | 19:19 |
| smcginnis | And we can just drop it in master. | 19:19 |
| rosmaita | ok, sounds good | 19:20 |
| *** vishalmanchanda has quit IRC | 19:28 | |
| *** Lucas_Gray has joined #openstack-cinder | 19:41 | |
| *** Lucas_Gray has quit IRC | 19:48 | |
| *** Lucas_Gray has joined #openstack-cinder | 19:51 | |
| openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Remove HPE Lefthand Driver https://review.opendev.org/716413 | 19:59 |
| rosmaita | my first driver removal! | 19:59 |
| openstackgerrit | Merged openstack/cinder stable/train: PowerMax Driver - Legacy volume not found https://review.opendev.org/715003 | 20:03 |
| *** ociuhandu has joined #openstack-cinder | 20:12 | |
| *** e0ne has quit IRC | 20:14 | |
| *** ociuhandu has quit IRC | 20:18 | |
| *** raunak12 has joined #openstack-cinder | 20:28 | |
| *** Lucas_Gray has quit IRC | 20:31 | |
| openstackgerrit | Merged openstack/cinder master: SAN: Add missing driver options https://review.opendev.org/713683 | 20:33 |
| openstackgerrit | Merged openstack/cinder master: RBD: Add missing driver options https://review.opendev.org/713684 | 20:48 |
| *** Lucas_Gray has joined #openstack-cinder | 20:59 | |
| *** rcernin has joined #openstack-cinder | 21:02 | |
| *** Lucas_Gray has quit IRC | 21:03 | |
| *** Lucas_Gray has joined #openstack-cinder | 21:04 | |
| openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Move release note to correct directory https://review.opendev.org/716436 | 21:10 |
| smcginnis | rosmaita: Was that the train release note you mentioned? ^ | 21:14 |
| rosmaita | smcginnis: yes, i changed the text a bit to submit it to master, then we can backport to stable/train | 21:14 |
| rosmaita | see what you think | 21:14 |
| smcginnis | A little odd to announce it in ussuri release notes. But it also won't show up in train until 15.0.x releases, so either way I guess it's not going to be where it's supposed to be. | 21:16 |
| smcginnis | Thinking out loud - yeah, that works for me. | 21:16 |
| rosmaita | let's see what it looks like when it builds | 21:16 |
| rosmaita | somehow i hosed by git repo and releasenotes won't build correctly | 21:16 |
| smcginnis | Did you rebuild your venv? tox -re releasenotes? | 21:17 |
| rosmaita | yeah, i have no idea what happened | 21:17 |
| *** e0ne has joined #openstack-cinder | 21:46 | |
| *** e0ne has quit IRC | 21:47 | |
| *** Wryhder has joined #openstack-cinder | 21:55 | |
| *** Lucas_Gray has quit IRC | 21:55 | |
| *** Wryhder is now known as Lucas_Gray | 21:56 | |
| *** Lucas_Gray has quit IRC | 22:45 | |
| *** tkajinam has joined #openstack-cinder | 22:49 | |
| *** tosky has quit IRC | 22:54 | |
| *** ianychoi has quit IRC | 23:15 | |
| *** ianychoi has joined #openstack-cinder | 23:16 | |
| openstackgerrit | Merged openstack/cinder master: Drop unicode() hacking check https://review.opendev.org/715742 | 23:43 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!