*** xuanyd has joined #openstack-cinder | 01:06 | |
xuanyd | whoami-rajat: Hello, the patch has passed zuul check, please review if you have time, thanks,https://review.opendev.org/#/c/718121/ | 01:10 |
---|---|---|
*** zhanglong has joined #openstack-cinder | 01:11 | |
*** yedongcan has joined #openstack-cinder | 01:13 | |
*** Liang__ has joined #openstack-cinder | 01:13 | |
*** xuanyd has quit IRC | 01:30 | |
*** lvbin02 has joined #openstack-cinder | 02:12 | |
*** lvbin01 has quit IRC | 02:15 | |
*** lvbin02 is now known as lvbin01 | 02:15 | |
*** zhanglong has quit IRC | 02:35 | |
*** zhanglong has joined #openstack-cinder | 02:40 | |
*** lvbin02 has joined #openstack-cinder | 02:44 | |
*** lvbin01 has quit IRC | 02:47 | |
*** lvbin02 is now known as lvbin01 | 02:47 | |
*** brinzhang_ has quit IRC | 02:59 | |
*** brinzhang_ has joined #openstack-cinder | 02:59 | |
*** psachin has joined #openstack-cinder | 03:28 | |
*** zhanglong has quit IRC | 03:38 | |
*** zhanglong has joined #openstack-cinder | 03:43 | |
*** yaawang__ has quit IRC | 03:53 | |
*** vishalmanchanda has joined #openstack-cinder | 03:53 | |
*** ykarel|away is now known as ykarel | 04:24 | |
*** andrebeltrami has quit IRC | 04:26 | |
*** ociuhandu has joined #openstack-cinder | 04:47 | |
*** ociuhandu has quit IRC | 04:51 | |
*** brinzhang has joined #openstack-cinder | 05:08 | |
*** brinzhang_ has quit IRC | 05:11 | |
*** udesale has joined #openstack-cinder | 05:22 | |
*** udesale has quit IRC | 05:23 | |
*** udesale has joined #openstack-cinder | 05:23 | |
*** yaawang has joined #openstack-cinder | 05:58 | |
*** xuanyd has joined #openstack-cinder | 05:59 | |
*** raghavendrat has joined #openstack-cinder | 06:09 | |
*** lvbin02 has joined #openstack-cinder | 06:16 | |
*** lvbin01 has quit IRC | 06:18 | |
*** lvbin02 is now known as lvbin01 | 06:18 | |
*** xuanyd has quit IRC | 06:18 | |
*** lvbin02 has joined #openstack-cinder | 06:34 | |
*** lvbin01 has quit IRC | 06:35 | |
*** lvbin02 is now known as lvbin01 | 06:35 | |
*** xinliang has joined #openstack-cinder | 06:58 | |
*** yaawang has quit IRC | 07:04 | |
*** yaawang has joined #openstack-cinder | 07:05 | |
*** lpetrut has joined #openstack-cinder | 07:09 | |
*** raghavendrat has quit IRC | 07:18 | |
*** Xuchu has quit IRC | 07:24 | |
*** Xuchu has joined #openstack-cinder | 07:24 | |
*** lvbin02 has joined #openstack-cinder | 07:30 | |
*** lvbin01 has quit IRC | 07:33 | |
*** lvbin02 is now known as lvbin01 | 07:33 | |
*** e0ne has joined #openstack-cinder | 07:38 | |
*** ociuhandu has joined #openstack-cinder | 07:44 | |
*** brinzhang_ has joined #openstack-cinder | 07:48 | |
*** brinzhang has quit IRC | 07:51 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/718333 | 07:55 |
*** zhanglong has quit IRC | 07:59 | |
*** tosky has joined #openstack-cinder | 08:00 | |
*** ociuhandu has quit IRC | 08:02 | |
*** ociuhandu has joined #openstack-cinder | 08:03 | |
*** dpawlik has joined #openstack-cinder | 08:08 | |
*** ociuhandu has quit IRC | 08:08 | |
*** brinzhang has joined #openstack-cinder | 08:10 | |
*** xuanyd has joined #openstack-cinder | 08:11 | |
*** zhanglong has joined #openstack-cinder | 08:11 | |
*** brinzhang_ has quit IRC | 08:13 | |
xuanyd | whoami-rajat: thanks for review, about your comments, Do i need to update? https://review.opendev.org/#/c/718121/ | 08:13 |
*** brinzhang_ has joined #openstack-cinder | 08:14 | |
whoami-rajat | xuanyd, if you could quick update, i can review again | 08:14 |
*** brinzhang has quit IRC | 08:15 | |
xuanyd | Ok. | 08:18 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: Followup to address open review comments https://review.opendev.org/718121 | 08:24 |
xuanyd | whoami-rajat: I updated it, please review again, thanks very much. | 08:25 |
whoami-rajat | xuanyd, done | 08:26 |
*** rcernin has quit IRC | 08:27 | |
*** martinkennelly has joined #openstack-cinder | 08:28 | |
openstackgerrit | Merged openstack/cinder master: PowerMax Driver - SRDF Replication Fixes https://review.opendev.org/714723 | 08:31 |
openstackgerrit | Merged openstack/cinder master: Revert "Remove the Veritas Access Driver" https://review.opendev.org/717378 | 08:31 |
openstackgerrit | Merged openstack/cinder master: Revert "Remove the Virtuozzo Storage Driver" https://review.opendev.org/717383 | 08:31 |
*** zhanglong has quit IRC | 08:32 | |
*** xinliang has quit IRC | 08:32 | |
*** zhanglong has joined #openstack-cinder | 08:36 | |
*** ociuhandu has joined #openstack-cinder | 08:45 | |
*** xuanyd has quit IRC | 08:47 | |
*** yaawang has quit IRC | 08:48 | |
*** yaawang has joined #openstack-cinder | 08:50 | |
*** ociuhandu has quit IRC | 08:50 | |
*** tkajinam has quit IRC | 08:51 | |
*** ociuhandu has joined #openstack-cinder | 08:52 | |
*** ykarel is now known as ykarel|lunch | 08:53 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Refreshing stale metadata https://review.opendev.org/712977 | 08:59 |
*** ociuhandu has quit IRC | 09:04 | |
*** ociuhandu has joined #openstack-cinder | 09:04 | |
*** brinzhang has joined #openstack-cinder | 09:06 | |
*** brinzhang_ has quit IRC | 09:09 | |
*** ociuhandu has quit IRC | 09:09 | |
*** raghavendrat has joined #openstack-cinder | 09:12 | |
raghavendrat | hi e0ne: geguileo: whoami-rajat: | 09:13 |
whoami-rajat | raghavendrat, hey | 09:13 |
*** yaawang has quit IRC | 09:14 | |
raghavendrat | this is regarding https://review.opendev.org/#/c/713803/ got +1 from zuul and HPE Storage CI has passed | 09:14 |
raghavendrat | whenever you get time, it would be great if you can have a look. Appreciate your feedback. | 09:14 |
*** n-saito has quit IRC | 09:15 | |
*** yaawang has joined #openstack-cinder | 09:16 | |
openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: Extending testing scope of Incremental Backup https://review.opendev.org/652817 | 09:20 |
tosky | (just a rebase) | 09:23 |
tosky | whoami-rajat: I added a point about cinder-tempest-plugin to today's meeting agenda; I'm sure you may want to add something and for sure chime in | 09:24 |
whoami-rajat | raghavendrat, okay, will take a look | 09:25 |
whoami-rajat | tosky, looking | 09:25 |
raghavendrat | thanks | 09:26 |
whoami-rajat | tosky, can we also add the new tgt job patch as well in the review list? | 09:27 |
tosky | whoami-rajat: sure - have you seen my comment there? | 09:28 |
whoami-rajat | tosky, i see i still have to address your review comments on my patches, i will try to do it before the meeting | 09:28 |
whoami-rajat | yep | 09:28 |
whoami-rajat | will try to modify it asap i'm done with the current work | 09:28 |
tosky | oki; no rush; let's see how much we can have it merged, we definitely need to cover more | 09:30 |
*** yaawang has quit IRC | 09:35 | |
*** yaawang has joined #openstack-cinder | 09:36 | |
whoami-rajat | tosky, yeah a lot. thanks for looking into the plugin patches. | 09:37 |
*** Liang__ has quit IRC | 09:38 | |
tosky | thanks to working on them in the first place! | 09:38 |
tosky | I just nag people | 09:38 |
openstackgerrit | Michael McAleer proposed openstack/cinder master: PowerMax Driver - Replication Metadata Fix https://review.opendev.org/712977 | 09:39 |
*** ociuhandu has joined #openstack-cinder | 09:40 | |
geguileo | raghavendrat: thanks for making the changes. The code looks good, but there are 2 things that need to be changed | 09:41 |
geguileo | raghavendrat: adding a release note | 09:42 |
geguileo | raghavendrat: and you need to return the configuration as part of the conf options reported by the driver as used | 09:42 |
geguileo | raghavendrat: sorry I didn't mention the release notes in my previous review | 09:43 |
*** zhanglong has quit IRC | 09:44 | |
raghavendrat | geguileo: i will add release notes | 09:44 |
raghavendrat | didn't understand 2nd point: need to return the configuration as part of the conf options reported by the driver as used | 09:45 |
raghavendrat | can you please elaborate | 09:45 |
raghavendrat | sorry, didn't notice your comments in code review link. now taking look | 09:46 |
raghavendrat | thanks geguileo: for quick review. i will work on review comments. | 09:50 |
geguileo | raghavendrat: they should be easy to fix, please ping me once they are done and I'll quickly review it again | 09:50 |
*** brinzhang_ has joined #openstack-cinder | 09:52 | |
raghavendrat | ok. sure. | 09:52 |
*** brinzhang has quit IRC | 09:55 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Deletion of group with volumes https://review.opendev.org/707828 | 09:58 |
*** ykarel|lunch is now known as ykarel | 10:12 | |
whoami-rajat | tosky, around? | 10:31 |
whoami-rajat | i've a question regarding https://review.opendev.org/#/c/537658/6/.zuul.yaml@56 | 10:32 |
*** brinzhang has joined #openstack-cinder | 10:37 | |
*** raghavendrat has quit IRC | 10:39 | |
*** brinzhang_ has quit IRC | 10:40 | |
openstackgerrit | Michael McAleer proposed openstack/cinder master: PowerMax Driver - Compression Change Bug Fix https://review.opendev.org/715965 | 10:44 |
*** brinzhang_ has joined #openstack-cinder | 10:57 | |
*** e0ne has quit IRC | 10:58 | |
*** e0ne has joined #openstack-cinder | 10:58 | |
*** brinzhang has quit IRC | 11:00 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add LVM+tgt tempest job https://review.opendev.org/537658 | 11:24 |
openstackgerrit | Lucian Petrut proposed openstack/os-brick stable/train: WIP: rbd Windows support https://review.opendev.org/718396 | 11:24 |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add LVM+tgt tempest job https://review.opendev.org/537658 | 11:32 |
*** ociuhandu has quit IRC | 11:34 | |
*** ociuhandu has joined #openstack-cinder | 11:35 | |
openstackgerrit | Lucian Petrut proposed openstack/os-brick master: WIP: rbd Windows support https://review.opendev.org/718403 | 11:39 |
*** ociuhandu has quit IRC | 11:39 | |
*** zhanglong has joined #openstack-cinder | 11:40 | |
*** zhanglong has quit IRC | 11:45 | |
*** brinzhang has joined #openstack-cinder | 11:47 | |
*** zhanglong has joined #openstack-cinder | 11:47 | |
*** brinzhang_ has quit IRC | 11:50 | |
*** ociuhandu has joined #openstack-cinder | 11:51 | |
tosky | whoami-rajat: sure, please ask (but my answer will take a while) | 11:58 |
*** zhanglong has quit IRC | 12:01 | |
whoami-rajat | tosky, hah np. i was thinking of keeping tgt as base job and deriving the lio job from it but it isn't too good for extensibility so i did your changes | 12:01 |
whoami-rajat | tosky, you can review again if i missed something whenever you get time https://review.opendev.org/#/c/537658/ | 12:02 |
tosky | sure I will, thanks! | 12:06 |
*** ociuhandu has quit IRC | 12:10 | |
*** ociuhandu has joined #openstack-cinder | 12:11 | |
whoami-rajat | tosky, thanks! | 12:12 |
*** tkajinam has joined #openstack-cinder | 12:14 | |
*** ociuhandu has quit IRC | 12:16 | |
*** brinzhang has quit IRC | 12:19 | |
*** brinzhang has joined #openstack-cinder | 12:23 | |
*** brinzhang has quit IRC | 12:23 | |
*** andrebeltrami has joined #openstack-cinder | 12:40 | |
hemna_ | morning | 12:44 |
hemna_ | smcginnis rosmaita you guys heard of/used openstack watcher? | 12:44 |
rosmaita | not me | 12:46 |
smcginnis | Definitely heard of, but never used. | 12:49 |
rosmaita | smcginnis: could you take a quick look at https://review.opendev.org/#/c/717385/ (restore prophetstor driver) | 12:51 |
smcginnis | rosmaita: We should add the UNSUPPORTED=True to the driver classes. And probably mention something at the top of https://review.opendev.org/#/c/717385/3/doc/source/configuration/block-storage/drivers/prophetstor-dpl-driver.rst | 12:53 |
smcginnis | I'm fine with that being a follow up, but we should make sure that is done. Looks like this was dropped before we added that flag or something. Or I'm just not seeing it. | 12:54 |
rosmaita | let me check, they should have been unsupported before they were removed | 12:54 |
smcginnis | I would have thought so. | 12:54 |
smcginnis | But I don't even see the VERSION value either. | 12:54 |
rosmaita | it's in dplcommon.py | 12:55 |
rosmaita | line 672 | 12:55 |
smcginnis | Ah, there it is. OK, I'm just blind. :) | 12:56 |
rosmaita | good idea about a ReST warning in the docs, though | 12:56 |
rosmaita | i'll add that to the bp | 12:56 |
*** ykarel is now known as ykarel|afk | 12:56 | |
rosmaita | planning to do an overall release note about the status of these unsupported but not removed drivers | 12:56 |
smcginnis | ++ | 12:57 |
smcginnis | I haven't had any time to look at what we need to change with the upgrade checker either. | 12:57 |
rosmaita | i think just remove them from the removed driver list | 12:57 |
rosmaita | there are also a few drivers we need to add | 12:57 |
smcginnis | Yeah. | 12:58 |
*** spatel has joined #openstack-cinder | 12:58 | |
*** spatel has quit IRC | 12:59 | |
Xuchu | smcginnis: I updated this patch according to your comments, Do you have time to review it agian? Thanks. https://review.opendev.org/715766 | 12:59 |
zigo | Hi here ! Is there a way to do some QoS on the volume migration traffic ? | 13:00 |
zigo | It's going too fast, and disturbing production VMs... | 13:00 |
zigo | Is it volume_copy_bps_limit ?!? | 13:02 |
smcginnis | Xuchu: Done | 13:04 |
smcginnis | zigo: I believe so. This is the description for that option: "The upper limit of bandwidth of volume copy. 0 => unlimited (integer value)" | 13:05 |
Xuchu | smcginnis: Thank you very much | 13:05 |
Xuchu | zigo: https://docs.openstack.org/cinder/latest/admin/blockstorage-basic-volume-qos.html, Did you try this ? | 13:05 |
rosmaita | hemna: do you have a minute to look over the glanceclient release note? once that's approved, i can put up the release patch: https://review.opendev.org/#/c/718234/1 | 13:08 |
zigo | Xuchu: The problem for me is not the QoS in general, but just during the "openstack volume migrate" operation, so I'm searching for a way to slow it down. | 13:10 |
zigo | Something like 1Gbits/s for our RAID10 SSDs would do ... | 13:10 |
ioni | hey guys | 13:16 |
ioni | i have an issue with creating volume snapshots having ceph backend with two volume types | 13:16 |
ioni | the scheduler is crashing | 13:17 |
ioni | https://paste.xinu.at/V5vW9/ | 13:17 |
ioni | i have the following filters for scheduling: https://paste.xinu.at/H9BW/ | 13:17 |
*** udesale_ has joined #openstack-cinder | 13:18 | |
whoami-rajat | ioni, which release ? | 13:18 |
ioni | it seems that it doesn't get the volume type in the driver_filter.py | 13:18 |
ioni | stein | 13:18 |
ioni | https://opendev.org/openstack/cinder/src/branch/stable/stein/cinder/scheduler/filters/driver_filter.py#L118 | 13:20 |
*** udesale has quit IRC | 13:20 | |
whoami-rajat | ioni, i think it's related to this https://review.opendev.org/#/c/698921/ | 13:20 |
ioni | whoami-rajat, thanks | 13:21 |
whoami-rajat | ioni, now i see hemna_ mentioned we should backport that fix and now we got a failure | 13:21 |
*** zhanglong has joined #openstack-cinder | 13:21 | |
Xuchu | zigo: volume_copy_bps_limit may not work in your scenario. https://docs.openstack.org/cinder/latest/admin/blockstorage-ratelimit-volume-copy-bandwidth.html. I think you can set a write QoS for target volume to mitigate the performance impact. | 13:22 |
whoami-rajat | ioni, could you make that change and confirm if the issue is resolved? | 13:22 |
ioni | whoami-rajat, sure, let me apply it | 13:22 |
*** jv has quit IRC | 13:23 | |
zigo | Xuchu: Thanks ! | 13:26 |
zigo | I'll try. | 13:26 |
*** jv has joined #openstack-cinder | 13:27 | |
ioni | whoami-rajat, thanks for the hint. it works now | 13:27 |
smcginnis | whoami-rajat: Want to propose that for backport? | 13:27 |
whoami-rajat | ioni, great. | 13:28 |
whoami-rajat | smcginnis, yep. will do. | 13:28 |
smcginnis | Awesome, thanks! | 13:28 |
*** eharney has joined #openstack-cinder | 13:28 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder stable/train: Fix: failed to create snapshot with DriverFilter https://review.opendev.org/718447 | 13:29 |
*** zhanglong has quit IRC | 13:30 | |
*** zhanglong has joined #openstack-cinder | 13:31 | |
*** ociuhandu has joined #openstack-cinder | 13:33 | |
*** ykarel|afk is now known as ykarel | 13:37 | |
*** enriquetaso has joined #openstack-cinder | 13:39 | |
*** Liang__ has joined #openstack-cinder | 13:50 | |
rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir ^^ | 13:59 |
*** throne82 has joined #openstack-cinder | 14:00 | |
*** Liang__ is now known as LiangFang | 14:00 | |
*** kaisers_ has joined #openstack-cinder | 14:11 | |
ganso | enriquetaso: Hi! I just added a topic about that bug to the agenda. I see your latest update in the launchpad but I don't understand it 100%, did you mean that we don't really need to do any major changes as I had proposed in that other etherpad? | 14:18 |
openstackgerrit | Merged openstack/python-cinderclient master: Add release note for Ussuri cinderclient release. https://review.opendev.org/718234 | 14:22 |
*** martinkennelly has quit IRC | 14:23 | |
*** zhanglong has quit IRC | 14:24 | |
*** martinkennelly has joined #openstack-cinder | 14:25 | |
*** zhanglong has joined #openstack-cinder | 14:26 | |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Manage volume emulation check https://review.opendev.org/718475 | 14:33 |
*** udesale_ has quit IRC | 14:46 | |
*** lpetrut has quit IRC | 14:46 | |
*** hoonetorg has quit IRC | 14:56 | |
smcginnis | rosmaita, eharney, jungleboyj: I think it would be good to get e0ne's patch in so we have a little runtime on it: https://review.opendev.org/#/c/656888/ | 15:00 |
rosmaita | enriquetaso: sorry about that ... looks like you got the discussion started, but no answers yet | 15:00 |
e0ne | smcginnis: thanks, Sean! | 15:01 |
rosmaita | yes, that has been sitting for quite a while | 15:01 |
rajinir | https://etherpad.openstack.org/p/Dell_EMC_Driver_Feature_Patches_Ussuri - Thank you all for the reviews | 15:01 |
*** LiangFang has quit IRC | 15:01 | |
smcginnis | e0ne: Thanks for doing that. That's one of those things that I've always wondered why we didn't make more common like that. | 15:01 |
jungleboyj | rajinir: Thanks. Will take a look. | 15:02 |
e0ne | smcginnis: :) | 15:02 |
enriquetaso | rosmaita, don't worry... maybe we could add the topic again next week because not sure what to do | 15:02 |
rosmaita | enriquetaso: yes, looks like we have a bunch of those! | 15:02 |
enriquetaso | hahaha | 15:03 |
jungleboyj | smcginnis: Good call. Will take a look. | 15:03 |
*** ykarel is now known as ykarel|away | 15:04 | |
rosmaita | rajinir: ty for the list, guess my afternoon is all planned out now | 15:10 |
rajinir | rosmaita :) ty | 15:10 |
*** zhanglong has quit IRC | 15:14 | |
*** zhanglong has joined #openstack-cinder | 15:16 | |
enriquetaso | ganso, hi! Sorry we can't discuss it today. | 15:16 |
enriquetaso | ganso, please take a look at the API using context.elevated() https://github.com/openstack/cinder/blob/master/cinder/backup/api.py#L266 when the manager doesn't https://github.com/openstack/cinder/blob/master/cinder/backup/manager.py#L438-L439 | 15:21 |
*** zhanglong has quit IRC | 15:27 | |
jungleboyj | rosmaita: smcginnis I just forwarded you guys an e-mail from digvijay who is working on the GPFS driver. | 15:31 |
jungleboyj | Did you guys see that? | 15:31 |
smcginnis | Yep. | 15:32 |
jungleboyj | Any concerns with a little grace period there as they have been keeping us updated on things? | 15:32 |
ganso | enriquetaso: sorry for the delay, I'm in another meeting. I am going to test take a look at this. thank you! | 15:34 |
enriquetaso | ganso++ | 15:34 |
rosmaita | jungleboyj: probably not, looking for the email now | 15:36 |
jungleboyj | Since really it is RC-1 when we really need to have that all ironed out. Right. | 15:37 |
jungleboyj | Getting him registered on IRC so he can join us here. | 15:37 |
rosmaita | ok, cool | 15:38 |
*** digvijay has joined #openstack-cinder | 15:48 | |
digvijay | @jungleboyj: thanks for help! | 15:49 |
jungleboyj | Yay! Welcome! | 15:49 |
digvijay | We've some request for 3rd party CI checkpoint | 15:49 |
digvijay | IBM Spectrum Scale (GPFS) is planing to add cinder driver support back to community form the Ussuri release. | 15:50 |
digvijay | We're aware of the Cinder 3rd party CI Compliance Checkpoint deadline is 10th of April, but we're facing few technical issues with running the CI. | 15:50 |
digvijay | Is there any way you can extend the timeline for CI checkpoint? | 15:50 |
digvijay | We can even share you the results of tempest test suit ran with devstack & GPFS, if that can extend the window. | 15:50 |
jungleboyj | digvijay: I think if you can share the results showing that you can run it and that it passes that will help. | 15:51 |
digvijay | jungleboyj: sure.. let me know where I can share the results.. | 15:52 |
jungleboyj | Can you just put them in paste.openstack.org and share the link? | 15:52 |
rosmaita | digvijay: good to see you in irc | 15:52 |
jungleboyj | Do you think you can having things working by 4/20 ? | 15:52 |
digvijay | rosmaita: its good to see you too :) | 15:55 |
rosmaita | digvijay: if you can get the CI running reliably by 4/20, you can still get the driver marked 'supported' in the ussuri release | 16:01 |
*** hoonetorg has joined #openstack-cinder | 16:02 | |
jungleboyj | rosmaita: ++ | 16:03 |
rosmaita | digvijay: do you have a patch up for that yet? | 16:03 |
digvijay | jungleboyj: thanks for relaxing timeline.. we'll fix the issue and start reposrting asap | 16:03 |
jungleboyj | e0ne: Please see my comment on your patch: https://review.opendev.org/#/c/656888/7 | 16:03 |
jungleboyj | digvijay: Thank rosmaita ;-) | 16:03 |
rosmaita | you can thank me, jungleboyj has been doing all the work though | 16:04 |
jungleboyj | digvijay: I am happy to see GPFS coming back. Just keep us updated on here as much as possible. We appreciate that. | 16:04 |
digvijay | rosmaita: we are working on patch | 16:04 |
jungleboyj | rosmaita: You are too modest. | 16:04 |
e0ne | jungleboyj: oh.. one mode rebase issue. will fix it a bit later tonigh | 16:04 |
e0ne | tonight | 16:04 |
rosmaita | digvijay: ok, that's good ... i will add a blueprint so we can keep track that you are working on this | 16:04 |
jungleboyj | e0ne: Ok, I think there was one other rebase issue. Let me find it quick. | 16:04 |
digvijay | rosmaita: thanks for pitching in :) | 16:05 |
digvijay | jungleboyj, rosmaita: will update you soon | 16:07 |
rosmaita | digvijay: https://blueprints.launchpad.net/cinder/+spec/ibm-gpfs-supported | 16:09 |
rosmaita | digvijay: put "Partially-implements: bp ibm-gpfs-supported" in the commit message when you post the patch | 16:10 |
digvijay | rosmaita: thanks for quick blueprint.. will add the line in commit message | 16:14 |
rosmaita | ok, cool | 16:14 |
*** ociuhandu has quit IRC | 16:16 | |
*** ociuhandu has joined #openstack-cinder | 16:17 | |
*** ociuhandu has quit IRC | 16:22 | |
*** psachin has quit IRC | 16:24 | |
*** dpawlik has quit IRC | 16:29 | |
*** digvijay has quit IRC | 16:32 | |
*** yedongcan has left #openstack-cinder | 17:03 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Move get_volume_stats impl to the base volume driver https://review.opendev.org/656888 | 17:07 |
*** ociuhandu has joined #openstack-cinder | 17:10 | |
*** ociuhandu has quit IRC | 17:16 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Move get_volume_stats impl to the base volume driver https://review.opendev.org/656888 | 17:25 |
smcginnis | A/23 | 17:39 |
*** e0ne has quit IRC | 18:21 | |
*** ociuhandu has joined #openstack-cinder | 18:26 | |
*** kaisers_ has quit IRC | 18:31 | |
smcginnis | rosmaita: It doesn't look like we have any pending cinderclient patches. Are we good to go ahead with the final release for that? | 18:45 |
rosmaita | was just putting it together | 18:45 |
rosmaita | wanted to look at brick-cinderclient-ext also | 18:45 |
smcginnis | rosmaita: OK, I updated the one with the branching, but I didn't look to see if the hash should have changed. | 18:47 |
rosmaita | ok, i will check | 18:47 |
smcginnis | We have a new job that will help propose patches to update the cycle-specific job templates. Looking for a good first patch to watch to make sure it works right. ;) | 18:48 |
rosmaita | :) | 18:48 |
rosmaita | smcginnis: i updated the cinderclient release patch | 18:51 |
*** AJaeger has joined #openstack-cinder | 18:53 | |
smcginnis | Excellent. I will shepard that one through. | 18:53 |
AJaeger | could I get another review on this os-brick change, please? https://review.opendev.org/717610 it finishes py27 dropping | 18:54 |
smcginnis | On it | 18:54 |
AJaeger | thanks, smcginnis | 18:55 |
openstackgerrit | Sofia Enriquez proposed openstack/cinder master: NFS encrypted volume support https://review.opendev.org/597148 | 19:03 |
*** AJaeger has left #openstack-cinder | 19:15 | |
*** throne82 has quit IRC | 19:16 | |
openstackgerrit | Merged openstack/cinder stable/stein: PowerMax Driver - Legacy volume not found https://review.opendev.org/715004 | 19:32 |
*** vishalmanchanda has quit IRC | 19:32 | |
openstackgerrit | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/718333 | 19:32 |
openstackgerrit | Merged openstack/cinder master: Revert "Remove ProphetStor Flexvisor Driver" https://review.opendev.org/717385 | 19:33 |
openstackgerrit | Merged openstack/cinder master: Followup to address open review comments https://review.opendev.org/718121 | 19:33 |
openstackgerrit | Merged openstack/cinder master: RBD: add support for revert-to-snapshot https://review.opendev.org/710334 | 19:33 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-cinderclient stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/718540 | 19:40 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-cinderclient stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/718541 | 19:40 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-cinderclient master: Update master for stable/ussuri https://review.opendev.org/718542 | 19:40 |
*** Lucas_Gray has joined #openstack-cinder | 19:50 | |
*** Lucas_Gray has quit IRC | 20:18 | |
*** Lucas_Gray has joined #openstack-cinder | 20:19 | |
*** Lucas_Gray has quit IRC | 20:33 | |
openstackgerrit | Merged openstack/python-cinderclient stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/718540 | 20:33 |
openstackgerrit | Merged openstack/python-cinderclient master: Update master for stable/ussuri https://review.opendev.org/718542 | 20:51 |
*** rosmaita has quit IRC | 20:58 | |
*** Lucas_Gray has joined #openstack-cinder | 21:01 | |
openstackgerrit | Merged openstack/python-cinderclient stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/718541 | 21:14 |
*** raunak12 has joined #openstack-cinder | 21:18 | |
*** rosmaita has joined #openstack-cinder | 21:23 | |
*** enriquetaso has quit IRC | 21:48 | |
openstackgerrit | OpenStack Release Bot proposed openstack/python-brick-cinderclient-ext stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/718560 | 21:50 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-brick-cinderclient-ext stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/718561 | 21:50 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-brick-cinderclient-ext master: Update master for stable/ussuri https://review.opendev.org/718562 | 21:50 |
rosmaita | jungleboyj: smcginnis: could use feedback on https://review.opendev.org/718563 (cycle highlights) | 21:53 |
jungleboyj | Highlight ... We are all still alive. ;-) | 21:54 |
jungleboyj | Looks good to me. | 21:55 |
rosmaita | that is a highlight ... i just got back online after power outage and tornado warning (though no actual tornado) | 21:56 |
openstackgerrit | Raunak Kumar proposed openstack/cinder master: Mark Nimble Storage Driver supported https://review.opendev.org/716495 | 21:57 |
rosmaita | jungleboyj: just noticed that i need you to look over https://review.opendev.org/#/c/713710/ so i'm not a liar | 21:58 |
smcginnis | rosmaita: It would probably help them to be more specific. Like which new drivers were added. What does image data colocation mean for end users, etc. | 21:59 |
*** Lucas_Gray has quit IRC | 22:01 | |
rosmaita | smcginnis: well, the problem is that i don't think any of them have actually been added yet! | 22:04 |
smcginnis | Well that's a problem. :D | 22:05 |
rosmaita | well, zuul didn't like that patch anyway | 22:06 |
zigo | I don't understand what python-brick-cinderclient-ext is for. Can someone enlight a poor Debian package maintainer ? :) | 22:16 |
smcginnis | zigo: It is an extension of python-cinderclient for performing local storage management operations. It is used for the cases where the ultimate consumer of Cinder created volumes is not something like nova. | 22:17 |
smcginnis | So a user can create a volume and expose it to their own host, then use this extension to finish setting it up for the parts that are normally taken care of by nova. | 22:18 |
zigo | smcginnis: Ok, so it's useful only outside of OpenStack ? | 22:18 |
smcginnis | Yeah, really the target would be someone using stand alone cinder. | 22:18 |
zigo | Ok, thanks. | 22:18 |
zigo | smcginnis: Do you want it in Debian ? :) | 22:18 |
zigo | I don't think I'd care, but if you do, I can do the work. | 22:19 |
smcginnis | I'm really not sure if it would be useful to apt install it or not. Maybe for someone. Personally, I'm fine if it is not packaged. It can always be pip installed after the fact for those that actually need it. | 22:20 |
zigo | Ok, fair enough. | 22:20 |
smcginnis | zigo: Thanks for checking. | 22:20 |
zigo | Thanks to you for the details! | 22:20 |
*** martinkennelly has quit IRC | 22:25 | |
rosmaita | hemna_: around? | 22:26 |
*** rcernin has joined #openstack-cinder | 22:32 | |
openstackgerrit | Merged openstack/cinder master: Add support for VxFlex OS 3.5 to VxFlex OS driver https://review.opendev.org/705176 | 23:01 |
openstackgerrit | Merged openstack/cinder master: Add OpenStack volume replication v2.1 support in VxFlex OS driver https://review.opendev.org/707390 | 23:01 |
openstackgerrit | Merged openstack/cinder stable/rocky: RBD: fix volume reference handling in clone logic https://review.opendev.org/716081 | 23:01 |
*** threestrands has joined #openstack-cinder | 23:10 | |
*** tosky has quit IRC | 23:20 | |
*** gary_perkins has quit IRC | 23:26 | |
*** gary_perkins has joined #openstack-cinder | 23:32 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!