*** whoami-rajat has quit IRC | 00:01 | |
*** Kuirong has quit IRC | 00:04 | |
*** tejdeep has quit IRC | 00:19 | |
*** tejdeep has joined #openstack-cinder | 00:58 | |
*** davee_ has quit IRC | 01:03 | |
*** davee_ has joined #openstack-cinder | 01:04 | |
*** KeithMnemonic has quit IRC | 01:16 | |
*** Kuirong has joined #openstack-cinder | 01:20 | |
*** deiter has quit IRC | 01:26 | |
*** brinzhang has joined #openstack-cinder | 01:28 | |
*** zhengMa has joined #openstack-cinder | 01:36 | |
*** enriquetaso has quit IRC | 01:40 | |
*** mvkr_ has quit IRC | 01:41 | |
openstackgerrit | wanghao proposed openstack/cinder-specs master: Update the spec of filtering by time comparison operators for Train https://review.opendev.org/658954 | 01:43 |
---|---|---|
*** mriedem has quit IRC | 01:49 | |
*** mvkr_ has joined #openstack-cinder | 01:54 | |
*** tejdeep has quit IRC | 02:22 | |
*** tejdeep has joined #openstack-cinder | 02:38 | |
*** tejdeep has quit IRC | 02:48 | |
*** Liang__ has joined #openstack-cinder | 03:00 | |
*** Liang__ is now known as LiangFang | 03:01 | |
*** whoami-rajat has joined #openstack-cinder | 03:06 | |
*** zhengMa has quit IRC | 03:33 | |
*** dviroel has quit IRC | 03:37 | |
*** psachin has joined #openstack-cinder | 03:38 | |
*** rcernin has quit IRC | 03:55 | |
*** udesale has joined #openstack-cinder | 04:06 | |
*** gkadam has joined #openstack-cinder | 04:08 | |
*** gkadam has quit IRC | 04:09 | |
*** pcaruana has joined #openstack-cinder | 04:44 | |
*** vishalmanchanda has joined #openstack-cinder | 04:46 | |
*** boxiang has quit IRC | 04:48 | |
*** boxiang has joined #openstack-cinder | 04:48 | |
*** pcaruana has quit IRC | 04:56 | |
*** Luzi has joined #openstack-cinder | 05:03 | |
*** zhengMa has joined #openstack-cinder | 05:27 | |
*** rcernin has joined #openstack-cinder | 05:33 | |
*** ruffian_sheep has joined #openstack-cinder | 05:40 | |
*** ruffian_sheep has quit IRC | 05:48 | |
*** m75abrams has joined #openstack-cinder | 05:50 | |
openstackgerrit | Bhaa Shakur proposed openstack/cinder master: Zadara VPSA: Move to API access key authentication https://review.opendev.org/670715 | 05:54 |
*** rcernin has quit IRC | 06:03 | |
*** brinzhang has quit IRC | 06:10 | |
*** brinzhang has joined #openstack-cinder | 06:11 | |
*** rcernin has joined #openstack-cinder | 06:18 | |
*** sahid has joined #openstack-cinder | 06:20 | |
*** pcaruana has joined #openstack-cinder | 06:21 | |
*** rcernin has quit IRC | 06:21 | |
*** rcernin has joined #openstack-cinder | 06:21 | |
*** dpawlik has joined #openstack-cinder | 06:31 | |
*** udesale has quit IRC | 06:33 | |
*** udesale has joined #openstack-cinder | 06:34 | |
-openstackstatus- NOTICE: The git service on opendev.org is currently down. | 06:47 | |
*** ChanServ changes topic to "The git service on opendev.org is currently down." | 06:48 | |
*** dpawlik has quit IRC | 06:50 | |
*** yaawang has quit IRC | 06:55 | |
*** yaawang has joined #openstack-cinder | 06:57 | |
*** rcernin has quit IRC | 07:06 | |
*** tesseract has joined #openstack-cinder | 07:15 | |
*** udesale has quit IRC | 07:16 | |
*** dpawlik has joined #openstack-cinder | 07:17 | |
*** udesale has joined #openstack-cinder | 07:18 | |
*** zhengMa has left #openstack-cinder | 07:24 | |
*** boxiang has quit IRC | 07:30 | |
*** boxiang_ has joined #openstack-cinder | 07:30 | |
*** bhagyashris has joined #openstack-cinder | 07:39 | |
geguileo | hemna: hope it also resolves your customer issue :-) | 07:43 |
*** ruffian_sheep has joined #openstack-cinder | 07:52 | |
*** tkajinam has quit IRC | 07:53 | |
ruffian_sheep | whoami-rajat:Hi,The latest comments have been met and submitted, and the results have been feedback. It will be appriciated if you can check it out as early as possible. | 07:53 |
*** tkajinam has joined #openstack-cinder | 07:53 | |
whoami-rajat | ruffian_sheep: I discussed the change at cinder meeting yesterday but there wasn't any CI result on it so other members were concerned. I can see CI running there, will notify other members regarding the same. | 07:56 |
ruffian_sheep | OK,thanks.There are two problems in the driver I uploaded yesterday. But does not affect the driver function, one is spelling, one is sleep. I just found out that git had a problem when I was uploading. I can't fix these two problems for the time being. | 07:59 |
ruffian_sheep | whoami-rajat:Does this mean that there is nowhere else to modify? Just wait for the driver to join the main line? | 08:00 |
whoami-rajat | ruffian_sheep: we discussed to review the driver yesterday but I haven't seen anyone did that. There are other drivers and specs on priority too. We can only wait now. | 08:09 |
whoami-rajat | ruffian_sheep: if you can fix the sleep issue and get your CI run in next few hours then it will be great. | 08:10 |
ruffian_sheep | whoami-rajat:I have fixed it, but there is no way to upload the driver to gerrit right now. . . Look at the situation is that the git service has stopped working. | 08:12 |
*** ag-47 has joined #openstack-cinder | 08:14 | |
whoami-rajat | ruffian_sheep: oh | 08:23 |
*** tosky has joined #openstack-cinder | 08:30 | |
-openstackstatus- NOTICE: Services at opendev.org like our git server and at openstack.org are currently down, looks like an outage in one of our cloud providers. | 08:31 | |
*** ChanServ changes topic to "Services at opendev.org like our git server and at openstack.org are currently down, looks like an outage in one of our cloud providers." | 08:31 | |
*** tkajinam has quit IRC | 08:36 | |
*** ChanServ changes topic to "The Block Storage Project | https://wiki.openstack.org/wiki/Cinder | https://tiny.cc/CinderPriorities" | 08:38 | |
-openstackstatus- NOTICE: The problem in our cloud provider has been fixed, services should be working again | 08:38 | |
*** ruffian_sheep has quit IRC | 08:59 | |
*** ruffian_sheep has joined #openstack-cinder | 09:02 | |
whoami-rajat | ruffian_sheep: i think gerrit server is up now | 09:03 |
ruffian_sheep | Not yet,still cannot upload or update the cinder | 09:04 |
*** sahid has quit IRC | 09:07 | |
ruffian_sheep | whoami-rajat :Now,I can download it by still can update it ,maybe still need some times. | 09:10 |
ruffian_sheep | but | 09:10 |
*** lemko has joined #openstack-cinder | 09:11 | |
*** lpetrut has joined #openstack-cinder | 09:15 | |
*** lpetrut has quit IRC | 09:16 | |
*** lennyb has joined #openstack-cinder | 09:16 | |
*** sahid has joined #openstack-cinder | 09:16 | |
*** lpetrut has joined #openstack-cinder | 09:16 | |
*** sahid has quit IRC | 09:29 | |
*** sahid has joined #openstack-cinder | 09:29 | |
openstackgerrit | hjy proposed openstack/cinder master: Add MacroSAN cinder driver https://review.opendev.org/612311 | 09:30 |
*** e0ne has joined #openstack-cinder | 09:32 | |
*** LiangFang has quit IRC | 09:33 | |
*** ag-47 has quit IRC | 09:35 | |
*** raghavendrat has joined #openstack-cinder | 09:44 | |
raghavendrat | hi reviewers | 09:51 |
raghavendrat | I have a query regarding my patch [1] https://review.opendev.org/#/c/657585/ and another patch [2] https://review.opendev.org/#/c/668793 | 09:51 |
raghavendrat | patch [2] has got merged. | 09:51 |
raghavendrat | my patch [1] doesn't have changes mentioned in patch [2] | 09:51 |
raghavendrat | however i do not see any conflicts. | 09:51 |
raghavendrat | I wish to avoid any merge failures [if it may arise]. | 09:52 |
raghavendrat | Should i continue working on my patch [1] OR should I incorporate changes of patch [2] and then submit new code review. | 09:52 |
raghavendrat | any pointer would be appreciated. | 09:52 |
whoami-rajat | raghavendrat: merge conflicts occur when the same code is changed by both the patches. | 09:59 |
whoami-rajat | raghavendrat: but still you should rebase your patch to have patch2 changes in your patch to be on safe side. | 10:00 |
whoami-rajat | s/code/code block | 10:01 |
raghavendrat | thanks whoami-rajat: | 10:03 |
raghavendrat | did not understand -> s/code/code block | 10:03 |
whoami-rajat | ... when the same code block is changed ... | 10:03 |
raghavendrat | ok | 10:04 |
*** carloss has joined #openstack-cinder | 10:23 | |
*** sahid has quit IRC | 10:29 | |
*** ruffian_sheep has quit IRC | 10:29 | |
*** bhagyashris has quit IRC | 10:50 | |
*** sahid has joined #openstack-cinder | 10:56 | |
*** udesale has quit IRC | 11:13 | |
*** baojg has quit IRC | 11:17 | |
*** stakeda has quit IRC | 11:39 | |
*** pcaruana has quit IRC | 11:42 | |
*** dviroel has joined #openstack-cinder | 11:51 | |
*** mriedem has joined #openstack-cinder | 11:51 | |
*** sapd1_ has joined #openstack-cinder | 11:59 | |
*** sapd1 has quit IRC | 11:59 | |
*** dpawlik has quit IRC | 12:02 | |
*** martinkennelly has joined #openstack-cinder | 12:21 | |
*** dpawlik has joined #openstack-cinder | 12:21 | |
*** pcaruana has joined #openstack-cinder | 12:22 | |
hemna | mornin | 12:48 |
openstackgerrit | Raghavendra Tilay proposed openstack/cinder master: 3PAR: Provide new option to specify NSP for single path attachments https://review.opendev.org/657585 | 12:54 |
whoami-rajat | jungleboyj: Hey, i talked to the macrosan driver maintainer, their CI is reporting now https://review.opendev.org/#/c/612311 | 13:02 |
whoami-rajat | also Eric's comment was regarding the use of XML files for loading config options which i think is also fixed. | 13:03 |
openstackgerrit | Merged openstack/cinder-specs master: Leverage compression hardware accelerator https://review.opendev.org/652275 | 13:10 |
*** udesale has joined #openstack-cinder | 13:16 | |
*** mvkr_ has quit IRC | 13:17 | |
*** _hemna has joined #openstack-cinder | 13:30 | |
jungleboyj | hemna: Morning. | 13:32 |
*** Luzi has quit IRC | 13:33 | |
*** _hemna has quit IRC | 13:35 | |
jungleboyj | whoami-rajat: Ok. eharney still needs to remove his -2. | 13:35 |
eharney | i'm changing it to a -1 | 13:35 |
jungleboyj | I do not see the tempest results in their logs. | 13:36 |
openstackgerrit | Bhaa Shakur proposed openstack/cinder master: Zadara VPSA: Move to API access key authentication https://review.opendev.org/670715 | 13:39 |
jungleboyj | Oh, I found them. | 13:40 |
rosmaita | jungleboyj: need you to look at https://review.opendev.org/#/c/669770/ when you have time (stable/rocky release patch) | 13:41 |
jungleboyj | rosmaita: Ok. Will do. | 13:41 |
whoami-rajat | jungleboyj: http://120.77.149.115/11/612311/18/check/cinder-isicsi-driver/96b7742/tempest_log/tox.log the test results | 13:43 |
jungleboyj | Yeah. I found those. | 13:44 |
jungleboyj | Not sure how to proceed on this one. eharney looked like your comments are ones that could be addressed in a follow-up patch> | 13:46 |
jungleboyj | They have been working on this for the last month. I think it didn't get more reviews because it was blocked by a -2. | 13:47 |
hemna | which review? | 13:47 |
jungleboyj | https://review.opendev.org/#/c/612311/ | 13:48 |
jungleboyj | Trying to get in at the last minute here. | 13:48 |
eharney | they are as long as someone is sure that the secret=True not being there doesn't result in passwords being logged | 13:48 |
hemna | ok new driver | 13:48 |
* hemna looks | 13:48 | |
eharney | i think they're maybe caught by a filter that looks for the word "password", but not 100% sure | 13:48 |
jungleboyj | eharney: Ok. And it could be handled in a follow up patch. | 13:49 |
eharney | i mean, if it's ok to ship a driver with a security problem, i guess? | 13:49 |
jungleboyj | Ah, good point. | 13:49 |
jungleboyj | We could just make that edit. | 13:50 |
*** sahid has quit IRC | 13:50 | |
eharney | yeah probably the best way to go | 13:50 |
jungleboyj | whoami-rajat: Is the developer still on IRC now? | 13:50 |
whoami-rajat | jungleboyj: he's on my timezone and is offline currently, he's with the IRC name ruffian_sheep | 13:51 |
hemna | damn, I think that driver needs more options.... | 13:51 |
jungleboyj | Oh. ruffian_sheep . Ok. He has been active for a while. | 13:51 |
*** enriquetaso has joined #openstack-cinder | 13:52 | |
eharney | there's also an option in there for whether the driver should force terminate a connection on volume delete. that's not a decision that should be pushed off to admins in a config option | 13:53 |
hemna | we allow delete on attached volumes now? | 13:53 |
eharney | i'm just reading what the code does in the driver | 13:53 |
hemna | I guess today is the new driver deadline? | 13:55 |
jungleboyj | hemna: Yeah ... | 13:56 |
jungleboyj | Was just having a side conversationg with whoami-rajat I guess he has been working very actively with ruffian_sheep on this and the driver didn't get wider core review due to TZ differences and possibly also the fact that we didn't get the -2 removed. | 13:57 |
jungleboyj | I would shrug and say too bad but I kind-of feel like we failed here too. | 13:57 |
*** mvkr_ has joined #openstack-cinder | 13:58 | |
jungleboyj | So, trying to figure out the fair thing to do here. | 13:58 |
eharney | i'd say fix the secret=True options, merge it, and write a bug for cleaning up the rest of it, which are small things | 13:58 |
hemna | missing VERSION | 13:58 |
hemna | other issues too | 13:59 |
jungleboyj | eharney: We can always revert before the end of the cycle if they don't fix things. | 13:59 |
jungleboyj | whoami-rajat: Would you be willing to shepherd getting the fixes done with ruffian_sheep? | 13:59 |
*** sahid has joined #openstack-cinder | 14:00 | |
whoami-rajat | jungleboyj: yeah, he's available almost everyday, i could ask him to address all the other comments in a followup | 14:00 |
*** lemko has quit IRC | 14:00 | |
jungleboyj | whoami-rajat: Ok. Let me push up a patch that fixes the security issue. Then lets make all of our comments on it today/merge it and I will get a bug opened against it for follow-up. | 14:02 |
whoami-rajat | jungleboyj: great! | 14:02 |
whoami-rajat | thanks jungleboyj eharney hemna | 14:02 |
jungleboyj | hemna: Do you have any concerns with that based on what you are finding? | 14:03 |
hemna | small ones that are easy to fix | 14:06 |
hemna | their docs say they require adding a blacklist to multipath.conf | 14:06 |
hemna | as well as require friendly names being off | 14:06 |
hemna | this is odd | 14:06 |
hemna | wth | 14:09 |
hemna | they are getting the FC wwns from the cinder volume host | 14:09 |
hemna | ?! | 14:09 |
hemna | they are creating os-brick connectofrs | 14:10 |
hemna | with | 14:10 |
hemna | https://review.opendev.org/#/c/612311/19/cinder/volume/drivers/macrosan/driver.py | 14:10 |
hemna | line 1198 | 14:11 |
hemna | wtf | 14:11 |
*** baojg has joined #openstack-cinder | 14:12 | |
hemna | they are doing it multiple places | 14:14 |
hemna | :( | 14:14 |
jungleboyj | Well, make your comments and they will have to follow up. | 14:15 |
hemna | ok I added my comments | 14:15 |
hemna | that's a big one | 14:15 |
hemna | there is no reason to be doing that in a driver | 14:15 |
hemna | I -2'd the HPE driver patch for the same reason | 14:15 |
jungleboyj | hemna: Thanks. | 14:21 |
openstackgerrit | Jay Bryant proposed openstack/cinder master: Add MacroSAN cinder driver https://review.opendev.org/612311 | 14:24 |
jungleboyj | Ok. The security issue is resolved. | 14:24 |
jungleboyj | I will review and make comments. | 14:24 |
hemna | weird | 14:26 |
hemna | lots of other comments from patchset 19 that need addressing | 14:27 |
pots | Hi folks, is anyone familiar with running the CI multiattach tests? When I run through all the tests with "tox -eall -- volume' I get about a dozen failures due to the client not using the right microversion? | 14:27 |
*** dpawlik has quit IRC | 14:28 | |
pots | tox doesn't pass environment variables, so setting OS_COMPUTE_API_VERSION=2.60 isn't effective. is there some alternate way to do that, or should I just be skipping those tests? | 14:29 |
tosky | pots: you can configure tox to pass environment variables | 14:29 |
tosky | through passenv: https://tox.readthedocs.io/en/latest/config.html#conf-passenv | 14:30 |
pots | i saw that but it seems like the developer of the multiattach tests would've done something more straightforward? | 14:32 |
tosky | pots: you can set it in tempest.conf, I think; let me check | 14:34 |
pots | I see there's a TOX_TESTENV_PASSENV variable, maybe that's the best route? | 14:36 |
tosky | if it's a tempest test and you need to set that for all your tests, why not setting it through tempest.conf? | 14:37 |
jungleboyj | hemna: Right. I will call that patchset out in the bug. | 14:37 |
jungleboyj | pots: Hey, I saw your e-mail. I think geguileo can help with the multi-attach questions and eharney knows tempest as well as anyone. :-) | 14:38 |
jungleboyj | Thought you were off today. | 14:38 |
tosky | pots: just in case, please see: https://docs.openstack.org/tempest/latest/microversion_testing.html#configuration-options-for-microversion | 14:39 |
jungleboyj | Oh, yeah, tosky has been looking at that stuff lately too. :-) | 14:40 |
pots | I would think there would be a standard way to set up for multiattach in local.conf, e.g. that's where devstack configures compute-feature-enabled/volume_multiattach=True. | 14:40 |
*** jmlowe has quit IRC | 14:42 | |
pots | i'm off today, i have to leave now. but thanks, tosky. i'll ping gegeuileo tomorrow. | 14:42 |
tosky | enabling a feature in devstack does not mean that it's automatically exposed by the API, if the client needs to set a specific API microversion | 14:42 |
tosky | oki :) | 14:42 |
jungleboyj | whoami-rajat: Please make sure to make your vote on the Macrosan driver. | 14:51 |
whoami-rajat | jungleboyj: sure, just waiting for CI resu | 14:52 |
whoami-rajat | results* | 14:52 |
jungleboyj | Ok. | 14:52 |
whoami-rajat | jungleboyj: also is it good to merge it with a negative comment? | 14:52 |
whoami-rajat | s/comment/vote | 14:53 |
jungleboyj | whoami-rajat: No, but we are making out comments and merging it today so that we can meet the deadline. | 14:53 |
jungleboyj | So, if you feel we should merge it and fix the issues later than give your +2. :-) | 14:53 |
whoami-rajat | jungleboyj: oh ok. Thanks for all the effort :) | 14:54 |
jungleboyj | Yep. Thank you for advocating for the developer. | 15:00 |
*** deiter has joined #openstack-cinder | 15:04 | |
deiter | Hello! Please review https://review.opendev.org/#/c/669736/ Thank you! | 15:04 |
hemna | the macrosan driver needs some work | 15:05 |
hemna | they have some dead functions | 15:05 |
hemna | and calling os-brick connectors directly | 15:05 |
*** jmlowe has joined #openstack-cinder | 15:07 | |
*** m75abrams has quit IRC | 15:12 | |
*** dklyle has quit IRC | 15:17 | |
*** _erlon_ has joined #openstack-cinder | 15:18 | |
*** dklyle has joined #openstack-cinder | 15:18 | |
*** e0ne has quit IRC | 15:23 | |
walshh_ | is anyone available to review https://review.opendev.org/#/c/664860 and https://review.opendev.org/#/c/669835? Both have clean CI. Thank you | 15:37 |
*** sahid has quit IRC | 15:38 | |
*** sahid has joined #openstack-cinder | 15:43 | |
jungleboyj | OMG, it is impossible to review large patches with the way gerrit is jumping around the file. | 15:43 |
*** gnufied has joined #openstack-cinder | 15:48 | |
*** altlogbot_1 has quit IRC | 15:48 | |
*** altlogbot_3 has joined #openstack-cinder | 15:49 | |
*** tesseract has quit IRC | 15:50 | |
*** henriqueof has joined #openstack-cinder | 15:55 | |
*** jmlowe has quit IRC | 16:04 | |
whoami-rajat | hemna: yeah i understand, those will be the first thing in the morning for them to fix. :) | 16:05 |
whoami-rajat | jungleboyj: yeah, i've been struggling with that too recently. | 16:05 |
*** enriquetaso has quit IRC | 16:09 | |
openstackgerrit | Andreas Jaeger proposed openstack/python-cinderclient master: Update api-ref location https://review.opendev.org/672770 | 16:10 |
jungleboyj | whoami-rajat: I have left my comments. Will leave it to you to merge after you have made your comments. | 16:15 |
*** brinzhang_ has quit IRC | 16:18 | |
*** brinzhang has quit IRC | 16:18 | |
whoami-rajat | jungleboyj: sure, my comments were addressed previously but will go through it again. Thanks again for last moment work :) | 16:21 |
*** henriqueof has quit IRC | 16:21 | |
jungleboyj | Yep. You are welcome. They are lucky. Back in the day this would have caused quite the row over the whole thing. | 16:22 |
*** lpetrut has quit IRC | 16:29 | |
*** sahid has quit IRC | 16:34 | |
*** jrubenst has joined #openstack-cinder | 16:36 | |
openstackgerrit | Luigi Toscano proposed openstack/python-cinderclient master: Migrate the functional job to Zuul v3 https://review.opendev.org/672784 | 16:40 |
hemna | anyone tried ipv6 with cinder/drivers ? | 17:01 |
hemna | jungleboyj: ok it's not just me then. I thought I was having problem with my browser, but it's gerrit ui then? | 17:02 |
jungleboyj | hemna: Not just you. In fact I have heard numerous people with the same complaint. It has been an issue with larger reviews for quite some time. | 17:06 |
hemna | ah ok, I guess that's why I hadn't run into it yet | 17:07 |
hemna | I only saw it on the big driver.py file | 17:07 |
*** jmlowe has joined #openstack-cinder | 17:13 | |
*** udesale has quit IRC | 17:34 | |
*** mvkr_ has quit IRC | 17:35 | |
*** henriqueof has joined #openstack-cinder | 17:38 | |
*** tejdeep has joined #openstack-cinder | 17:39 | |
*** deiter has quit IRC | 17:42 | |
*** martinkennelly has quit IRC | 17:45 | |
*** psachin has quit IRC | 17:50 | |
*** deiter has joined #openstack-cinder | 17:51 | |
openstackgerrit | Luigi Toscano proposed openstack/cinderlib master: zuul: change the cinderlib test runner into a role https://review.opendev.org/671699 | 17:58 |
*** dklyle has quit IRC | 18:11 | |
*** deiter has quit IRC | 18:11 | |
*** tejdeep has quit IRC | 18:12 | |
*** dklyle has joined #openstack-cinder | 18:12 | |
*** tejdeep has joined #openstack-cinder | 18:12 | |
openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: Update cinder-tempest-plugin-lvm-lio (revamp) https://review.opendev.org/672804 | 18:16 |
openstackgerrit | Merged openstack/cinderlib master: Fix cleanup of persistence tests https://review.opendev.org/672508 | 18:16 |
openstackgerrit | Luigi Toscano proposed openstack/cinder master: Port several legacy tempest jobs to Zuul v3 https://review.opendev.org/671945 | 18:18 |
openstackgerrit | Luigi Toscano proposed openstack/python-cinderclient master: Migrate the functional job to Zuul v3 https://review.opendev.org/672784 | 18:21 |
*** pck has quit IRC | 18:23 | |
*** pck has joined #openstack-cinder | 18:30 | |
*** brinzhang has joined #openstack-cinder | 18:37 | |
*** brinzhang_ has joined #openstack-cinder | 18:37 | |
*** eharney has quit IRC | 18:39 | |
*** brinzhang_ has quit IRC | 18:42 | |
*** brinzhang has quit IRC | 18:42 | |
*** brinzhang has joined #openstack-cinder | 18:43 | |
*** brinzhang_ has joined #openstack-cinder | 18:43 | |
*** mriedem has quit IRC | 18:54 | |
*** mriedem has joined #openstack-cinder | 19:03 | |
*** enriquetaso has joined #openstack-cinder | 19:09 | |
*** tosky has quit IRC | 19:10 | |
*** senrique_ has joined #openstack-cinder | 19:10 | |
*** enriquetaso has quit IRC | 19:13 | |
*** e0ne has joined #openstack-cinder | 19:39 | |
*** dasp has quit IRC | 19:51 | |
*** senrique_ is now known as enriquetaso | 19:52 | |
*** dasp has joined #openstack-cinder | 19:54 | |
jungleboyj | hemna: whoami-rajat or smcginnis Can someone give the infortrend driver a +W . I think all the issues ahve been addressed and we are down to things that can be fixed in patches if there are more things. | 19:55 |
hemna | url | 19:55 |
jungleboyj | https://review.opendev.org/#/c/523659 | 19:56 |
hemna | looks like they addressed the doc issue I had with the jar | 19:57 |
jungleboyj | Yeah, and I don't think Eric's remaining comment is a blocker. | 19:57 |
hemna | udone | 19:58 |
hemna | err done even | 19:58 |
jungleboyj | Thanks. | 19:58 |
jungleboyj | hemna: This needs a +W today as well. | 20:01 |
jungleboyj | and this: https://review.opendev.org/#/c/664973/ | 20:01 |
jungleboyj | e0ne: Is this one not going to make it now? | 20:02 |
jungleboyj | https://review.opendev.org/#/c/556529/ | 20:02 |
jungleboyj | hemna: And this one: https://review.opendev.org/#/c/608663/ | 20:03 |
e0ne | jungleboyj: sorry, didn't get time today to address Gorka's comments :( | 20:03 |
jungleboyj | Apparently my pleas in the meeting yesterday went unheard. | 20:03 |
jungleboyj | e0ne: Ok. How do you want to proceed? | 20:03 |
hemna | looking | 20:03 |
e0ne | will do it tomorrow morning | 20:03 |
e0ne | even if it's too late for this cycle | 20:04 |
jungleboyj | e0ne: That or you can submit a Spec Freeze exception request. ;-) Do you think you will be able to get the associated code done? | 20:05 |
hemna | image encryption | 20:05 |
hemna | done | 20:06 |
e0ne | jungleboyj: need to go and see what I already have | 20:06 |
e0ne | if there is a more or less working prototype - I'll ask for spec freeze exception | 20:07 |
jungleboyj | e0ne: ++ | 20:09 |
e0ne | I'll put it as a top priority for me for tomorrow | 20:10 |
jungleboyj | e0ne: Ok. Thank you. | 20:11 |
e0ne | jungleboyj: thanks for the help and reminder | 20:11 |
jungleboyj | Yep. You are welcome. | 20:11 |
openstackgerrit | Merged openstack/cinder-specs master: Volume Rekey spec https://review.opendev.org/664973 | 20:11 |
jungleboyj | hemna: Thanks for the help. That just leaves: https://review.opendev.org/#/c/651480/ | 20:13 |
openstackgerrit | Merged openstack/cinder-specs master: Spec for the Cinder part of Image Encryption https://review.opendev.org/608663 | 20:13 |
rosmaita | jungleboyj: also https://review.opendev.org/#/c/658954/7 ? | 20:14 |
rosmaita | jungleboyj: i think we definitely want to get https://review.opendev.org/#/c/651480/ in, maybe we can approve and revisit the question of allowing __DEFAULT__ volume type to be updated during implementation? | 20:19 |
jungleboyj | rosmaita: Ok, the filtering spec is now approved. | 20:20 |
jungleboyj | rosmaita: I am ok with that. | 20:20 |
rosmaita | ok, i will put a note on the review and approve it | 20:20 |
jungleboyj | Need to stop bikeshedding. | 20:20 |
rosmaita | ok, done | 20:23 |
hemna | bikeshedding is so productive though... | 20:23 |
*** raghavendrat has quit IRC | 20:23 | |
openstackgerrit | Merged openstack/cinder-specs master: Update the spec of filtering by time comparison operators for Train https://review.opendev.org/658954 | 20:30 |
openstackgerrit | Merged openstack/cinder-specs master: Untyped vol to default vol type https://review.opendev.org/651480 | 20:33 |
* jungleboyj rolls my eyes | 20:38 | |
jungleboyj | rosmaita: Do you know where the info is for setting up the right package Repo to install OpenStack on a Red Hat box? | 20:38 |
rosmaita | jungleboyj: maybe this? https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/14/html/release_notes/chap-introduction#Content_Delivery_Network_CDN_Channels | 20:47 |
*** boxiang_ has quit IRC | 20:48 | |
*** boxiang_ has joined #openstack-cinder | 20:48 | |
jungleboyj | Yeah, that might help if I can get it registered to a Satellite or something. Going to have our guy in SH see if we can get it registered so I don't spend a whole day just trying to pull together the right packages. | 20:49 |
jungleboyj | Trying to add Cinder to an RHOSP install where they didn't have it without redeploying. | 20:50 |
*** mriedem has quit IRC | 20:52 | |
*** mriedem has joined #openstack-cinder | 20:53 | |
openstackgerrit | Merged openstack/cinder master: Add MacroSAN cinder driver https://review.opendev.org/612311 | 20:59 |
*** whoami-rajat has quit IRC | 21:28 | |
*** pcaruana has quit IRC | 21:28 | |
*** enriquetaso has quit IRC | 21:31 | |
*** _hemna has joined #openstack-cinder | 21:31 | |
*** enriquetaso has joined #openstack-cinder | 21:32 | |
*** _hemna has quit IRC | 21:36 | |
*** Kuirong has quit IRC | 21:40 | |
*** e0ne has quit IRC | 21:56 | |
*** henriqueof has quit IRC | 22:01 | |
openstackgerrit | Merged openstack/cinder master: Re-add Infortrend Cinder volume driver https://review.opendev.org/523659 | 22:01 |
*** Liang__ has joined #openstack-cinder | 22:15 | |
*** rcernin has joined #openstack-cinder | 22:16 | |
*** carloss has quit IRC | 22:38 | |
*** Liang__ has quit IRC | 22:43 | |
*** _erlon_ has quit IRC | 23:07 | |
*** mriedem has quit IRC | 23:18 | |
*** brinzhang has quit IRC | 23:24 | |
*** brinzhang has joined #openstack-cinder | 23:25 | |
*** deiter has joined #openstack-cinder | 23:40 | |
*** brinzhang_ has quit IRC | 23:52 | |
*** brinzhang_ has joined #openstack-cinder | 23:53 | |
*** enriquetaso has quit IRC | 23:55 | |
*** smcginnis has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!