Thursday, 2019-07-25

*** whoami-rajat has quit IRC00:01
*** Kuirong has quit IRC00:04
*** tejdeep has quit IRC00:19
*** tejdeep has joined #openstack-cinder00:58
*** davee_ has quit IRC01:03
*** davee_ has joined #openstack-cinder01:04
*** KeithMnemonic has quit IRC01:16
*** Kuirong has joined #openstack-cinder01:20
*** deiter has quit IRC01:26
*** brinzhang has joined #openstack-cinder01:28
*** zhengMa has joined #openstack-cinder01:36
*** enriquetaso has quit IRC01:40
*** mvkr_ has quit IRC01:41
openstackgerritwanghao proposed openstack/cinder-specs master: Update the spec of filtering by time comparison operators for Train
*** mriedem has quit IRC01:49
*** mvkr_ has joined #openstack-cinder01:54
*** tejdeep has quit IRC02:22
*** tejdeep has joined #openstack-cinder02:38
*** tejdeep has quit IRC02:48
*** Liang__ has joined #openstack-cinder03:00
*** Liang__ is now known as LiangFang03:01
*** whoami-rajat has joined #openstack-cinder03:06
*** zhengMa has quit IRC03:33
*** dviroel has quit IRC03:37
*** psachin has joined #openstack-cinder03:38
*** rcernin has quit IRC03:55
*** udesale has joined #openstack-cinder04:06
*** gkadam has joined #openstack-cinder04:08
*** gkadam has quit IRC04:09
*** pcaruana has joined #openstack-cinder04:44
*** vishalmanchanda has joined #openstack-cinder04:46
*** boxiang has quit IRC04:48
*** boxiang has joined #openstack-cinder04:48
*** pcaruana has quit IRC04:56
*** Luzi has joined #openstack-cinder05:03
*** zhengMa has joined #openstack-cinder05:27
*** rcernin has joined #openstack-cinder05:33
*** ruffian_sheep has joined #openstack-cinder05:40
*** ruffian_sheep has quit IRC05:48
*** m75abrams has joined #openstack-cinder05:50
openstackgerritBhaa Shakur proposed openstack/cinder master: Zadara VPSA: Move to API access key authentication
*** rcernin has quit IRC06:03
*** brinzhang has quit IRC06:10
*** brinzhang has joined #openstack-cinder06:11
*** rcernin has joined #openstack-cinder06:18
*** sahid has joined #openstack-cinder06:20
*** pcaruana has joined #openstack-cinder06:21
*** rcernin has quit IRC06:21
*** rcernin has joined #openstack-cinder06:21
*** dpawlik has joined #openstack-cinder06:31
*** udesale has quit IRC06:33
*** udesale has joined #openstack-cinder06:34
-openstackstatus- NOTICE: The git service on is currently down.06:47
*** ChanServ changes topic to "The git service on is currently down."06:48
*** dpawlik has quit IRC06:50
*** yaawang has quit IRC06:55
*** yaawang has joined #openstack-cinder06:57
*** rcernin has quit IRC07:06
*** tesseract has joined #openstack-cinder07:15
*** udesale has quit IRC07:16
*** dpawlik has joined #openstack-cinder07:17
*** udesale has joined #openstack-cinder07:18
*** zhengMa has left #openstack-cinder07:24
*** boxiang has quit IRC07:30
*** boxiang_ has joined #openstack-cinder07:30
*** bhagyashris has joined #openstack-cinder07:39
geguileohemna: hope it also resolves your customer issue  :-)07:43
*** ruffian_sheep has joined #openstack-cinder07:52
*** tkajinam has quit IRC07:53
ruffian_sheepwhoami-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-cinder07:53
whoami-rajatruffian_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_sheepOK,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_sheepwhoami-rajat:Does this mean that there is nowhere else to modify? Just wait for the driver to join the main line?08:00
whoami-rajatruffian_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-rajatruffian_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_sheepwhoami-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-cinder08:14
whoami-rajatruffian_sheep: oh08:23
*** tosky has joined #openstack-cinder08:30
-openstackstatus- NOTICE: Services at like our git server and at are currently down, looks like an outage in one of our cloud providers.08:31
*** ChanServ changes topic to "Services at like our git server and at are currently down, looks like an outage in one of our cloud providers."08:31
*** tkajinam has quit IRC08:36
*** ChanServ changes topic to "The Block Storage Project | |"08:38
-openstackstatus- NOTICE: The problem in our cloud provider has been fixed, services should be working again08:38
*** ruffian_sheep has quit IRC08:59
*** ruffian_sheep has joined #openstack-cinder09:02
whoami-rajatruffian_sheep: i think gerrit server is up now09:03
ruffian_sheepNot yet,still cannot upload or update the cinder09:04
*** sahid has quit IRC09:07
ruffian_sheepwhoami-rajat :Now,I can download it by still can update it ,maybe still need some times.09:10
*** lemko has joined #openstack-cinder09:11
*** lpetrut has joined #openstack-cinder09:15
*** lpetrut has quit IRC09:16
*** lennyb has joined #openstack-cinder09:16
*** sahid has joined #openstack-cinder09:16
*** lpetrut has joined #openstack-cinder09:16
*** sahid has quit IRC09:29
*** sahid has joined #openstack-cinder09:29
openstackgerrithjy proposed openstack/cinder master: Add MacroSAN cinder driver
*** e0ne has joined #openstack-cinder09:32
*** LiangFang has quit IRC09:33
*** ag-47 has quit IRC09:35
*** raghavendrat has joined #openstack-cinder09:44
raghavendrathi reviewers09:51
raghavendratI have a query regarding my patch [1] and another patch [2]
raghavendratpatch [2] has got merged.09:51
raghavendratmy patch [1] doesn't have changes mentioned in patch [2]09:51
raghavendrathowever i do not see any conflicts.09:51
raghavendratI wish to avoid any merge failures [if it may arise].09:52
raghavendratShould i continue working on my patch [1] OR should I incorporate changes of patch [2] and then submit new code review.09:52
raghavendratany pointer would be appreciated.09:52
whoami-rajatraghavendrat: merge conflicts occur when the same code is changed by both the patches.09:59
whoami-rajatraghavendrat: but still you should rebase your patch to have patch2 changes in your patch to be on safe side.10:00
whoami-rajats/code/code block10:01
raghavendratthanks whoami-rajat:10:03
raghavendratdid not understand -> s/code/code block10:03
whoami-rajat... when the same code block is changed ...10:03
*** carloss has joined #openstack-cinder10:23
*** sahid has quit IRC10:29
*** ruffian_sheep has quit IRC10:29
*** bhagyashris has quit IRC10:50
*** sahid has joined #openstack-cinder10:56
*** udesale has quit IRC11:13
*** baojg has quit IRC11:17
*** stakeda has quit IRC11:39
*** pcaruana has quit IRC11:42
*** dviroel has joined #openstack-cinder11:51
*** mriedem has joined #openstack-cinder11:51
*** sapd1_ has joined #openstack-cinder11:59
*** sapd1 has quit IRC11:59
*** dpawlik has quit IRC12:02
*** martinkennelly has joined #openstack-cinder12:21
*** dpawlik has joined #openstack-cinder12:21
*** pcaruana has joined #openstack-cinder12:22
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR: Provide new option to specify NSP for single path attachments
whoami-rajatjungleboyj: Hey, i talked to the macrosan driver maintainer, their CI is reporting now
whoami-rajatalso Eric's comment was regarding the use of XML files for loading config options which i think is also fixed.13:03
openstackgerritMerged openstack/cinder-specs master: Leverage compression hardware accelerator
*** udesale has joined #openstack-cinder13:16
*** mvkr_ has quit IRC13:17
*** _hemna has joined #openstack-cinder13:30
jungleboyjhemna:  Morning.13:32
*** Luzi has quit IRC13:33
*** _hemna has quit IRC13:35
jungleboyjwhoami-rajat:  Ok.  eharney still needs to remove his -2.13:35
eharneyi'm changing it to a -113:35
jungleboyjI do not see the tempest results in their logs.13:36
openstackgerritBhaa Shakur proposed openstack/cinder master: Zadara VPSA: Move to API access key authentication
jungleboyjOh, I found them.13:40
rosmaitajungleboyj: need you to look at when you have time (stable/rocky release patch)13:41
jungleboyjrosmaita:  Ok.  Will do.13:41
whoami-rajatjungleboyj: the test results13:43
jungleboyjYeah.  I found those.13:44
jungleboyjNot 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
jungleboyjThey 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
hemnawhich review?13:47
jungleboyjTrying to get in at the last minute here.13:48
eharneythey are as long as someone is sure that the secret=True not being there doesn't result in passwords being logged13:48
hemnaok new driver13:48
* hemna looks13:48
eharneyi think they're maybe caught by a filter that looks for the word "password", but not 100% sure13:48
jungleboyjeharney:  Ok.  And it could be handled in a follow up patch.13:49
eharneyi mean, if it's ok to ship a driver with a security problem, i guess?13:49
jungleboyjAh, good point.13:49
jungleboyjWe could just make that edit.13:50
*** sahid has quit IRC13:50
eharneyyeah probably the best way to go13:50
jungleboyjwhoami-rajat:  Is the developer still on IRC now?13:50
whoami-rajatjungleboyj: he's on my timezone and is offline currently, he's with the IRC name ruffian_sheep13:51
hemnadamn, I think that driver needs more options....13:51
jungleboyjOh.  ruffian_sheep .  Ok.  He has been active for a while.13:51
*** enriquetaso has joined #openstack-cinder13:52
eharneythere'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 option13:53
hemnawe allow delete on attached volumes now?13:53
eharneyi'm just reading what the code does in the driver13:53
hemnaI guess today is the new driver deadline?13:55
jungleboyjhemna:  Yeah ...13:56
jungleboyjWas 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
jungleboyjI would shrug and say too bad but I kind-of feel like we failed here too.13:57
*** mvkr_ has joined #openstack-cinder13:58
jungleboyjSo, trying to figure out the fair thing to do here.13:58
eharneyi'd say fix the secret=True options, merge it, and write a bug for cleaning up the rest of it, which are small things13:58
hemnamissing VERSION13:58
hemnaother issues too13:59
jungleboyjeharney:  We can always revert before the end of the cycle if they don't fix things.13:59
jungleboyjwhoami-rajat: Would you be willing to shepherd getting the fixes done with ruffian_sheep?13:59
*** sahid has joined #openstack-cinder14:00
whoami-rajatjungleboyj: yeah, he's available almost everyday, i could ask him to address all the other comments in a followup14:00
*** lemko has quit IRC14:00
jungleboyjwhoami-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-rajatjungleboyj: great!14:02
whoami-rajatthanks jungleboyj eharney hemna14:02
jungleboyjhemna:  Do you have any concerns with that based on what you are finding?14:03
hemnasmall ones that are easy to fix14:06
hemnatheir docs say they require adding a blacklist to multipath.conf14:06
hemnaas well as require friendly names being off14:06
hemnathis is odd14:06
hemnathey are getting the FC wwns from the cinder volume host14:09
hemnathey are creating os-brick connectofrs14:10
hemnaline 119814:11
*** baojg has joined #openstack-cinder14:12
hemnathey are doing it multiple places14:14
jungleboyjWell, make your comments and they will have to follow up.14:15
hemnaok I added my comments14:15
hemnathat's a big one14:15
hemnathere is no reason to be doing that in a driver14:15
hemnaI -2'd the HPE driver patch for the same reason14:15
jungleboyjhemna:  Thanks.14:21
openstackgerritJay Bryant proposed openstack/cinder master: Add MacroSAN cinder driver
jungleboyjOk.  The security issue is resolved.14:24
jungleboyjI will review and make comments.14:24
hemnalots of other comments from patchset 19 that need addressing14:27
potsHi 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 IRC14:28
potstox 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
toskypots: you can configure tox to pass environment variables14:29
toskythrough passenv:
potsi saw that but it seems like the developer of the multiattach tests would've done something more straightforward?14:32
toskypots: you can set it in tempest.conf, I think; let me check14:34
potsI see there's a TOX_TESTENV_PASSENV variable, maybe that's the best route?14:36
toskyif it's a tempest test and you need to set that for all your tests, why not setting it through tempest.conf?14:37
jungleboyjhemna:  Right.  I will call that patchset out in the bug.14:37
jungleboyjpots: 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
jungleboyjThought you were off today.14:38
toskypots: just in case, please see:
jungleboyjOh, yeah, tosky  has been looking at that stuff lately too.  :-)14:40
potsI 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 IRC14:42
potsi'm off today, i have to leave now.  but thanks, tosky.  i'll ping gegeuileo tomorrow.14:42
toskyenabling a feature in devstack does not mean that it's automatically exposed by the API, if the client needs to set a specific API microversion14:42
toskyoki :)14:42
jungleboyjwhoami-rajat:  Please make sure to make your vote on the Macrosan driver.14:51
whoami-rajatjungleboyj: sure, just waiting for CI resu14:52
whoami-rajatjungleboyj: also is it good to merge it with a negative comment?14:52
jungleboyjwhoami-rajat:  No, but we are making out comments and merging it today so that we can meet the deadline.14:53
jungleboyjSo, if you feel we should merge it and fix the issues later than give your +2.  :-)14:53
whoami-rajatjungleboyj:  oh ok. Thanks for all the effort :)14:54
jungleboyjYep.  Thank you for advocating for the developer.15:00
*** deiter has joined #openstack-cinder15:04
deiterHello! Please review Thank you!15:04
hemnathe macrosan driver needs some work15:05
hemnathey have some dead functions15:05
hemnaand calling os-brick connectors directly15:05
*** jmlowe has joined #openstack-cinder15:07
*** m75abrams has quit IRC15:12
*** dklyle has quit IRC15:17
*** _erlon_ has joined #openstack-cinder15:18
*** dklyle has joined #openstack-cinder15:18
*** e0ne has quit IRC15:23
walshh_is anyone available to review and  Both have clean CI.  Thank you15:37
*** sahid has quit IRC15:38
*** sahid has joined #openstack-cinder15:43
jungleboyjOMG, it is impossible to review large patches with the way gerrit is jumping around the file.15:43
*** gnufied has joined #openstack-cinder15:48
*** altlogbot_1 has quit IRC15:48
*** altlogbot_3 has joined #openstack-cinder15:49
*** tesseract has quit IRC15:50
*** henriqueof has joined #openstack-cinder15:55
*** jmlowe has quit IRC16:04
whoami-rajathemna: yeah i understand, those will be the first thing in the morning for them to fix. :)16:05
whoami-rajatjungleboyj: yeah, i've been struggling with that too recently.16:05
*** enriquetaso has quit IRC16:09
openstackgerritAndreas Jaeger proposed openstack/python-cinderclient master: Update api-ref location
jungleboyjwhoami-rajat: I have left my comments.  Will leave it to you to merge after you have made your comments.16:15
*** brinzhang_ has quit IRC16:18
*** brinzhang has quit IRC16:18
whoami-rajatjungleboyj: sure, my comments were addressed previously but will go through it again. Thanks again for last moment work :)16:21
*** henriqueof has quit IRC16:21
jungleboyjYep.  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 IRC16:29
*** sahid has quit IRC16:34
*** jrubenst has joined #openstack-cinder16:36
openstackgerritLuigi Toscano proposed openstack/python-cinderclient master: Migrate the functional job to Zuul v3
hemnaanyone tried ipv6 with cinder/drivers ?17:01
hemnajungleboyj: ok it's not just me then.  I thought I was having problem with my browser, but it's gerrit ui then?17:02
jungleboyjhemna:  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
hemnaah ok, I guess that's why I hadn't run into it yet17:07
hemnaI only saw it on the big file17:07
*** jmlowe has joined #openstack-cinder17:13
*** udesale has quit IRC17:34
*** mvkr_ has quit IRC17:35
*** henriqueof has joined #openstack-cinder17:38
*** tejdeep has joined #openstack-cinder17:39
*** deiter has quit IRC17:42
*** martinkennelly has quit IRC17:45
*** psachin has quit IRC17:50
*** deiter has joined #openstack-cinder17:51
openstackgerritLuigi Toscano proposed openstack/cinderlib master: zuul: change the cinderlib test runner into a role
*** dklyle has quit IRC18:11
*** deiter has quit IRC18:11
*** tejdeep has quit IRC18:12
*** dklyle has joined #openstack-cinder18:12
*** tejdeep has joined #openstack-cinder18:12
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Update cinder-tempest-plugin-lvm-lio (revamp)
openstackgerritMerged openstack/cinderlib master: Fix cleanup of persistence tests
openstackgerritLuigi Toscano proposed openstack/cinder master: Port several legacy tempest jobs to Zuul v3
openstackgerritLuigi Toscano proposed openstack/python-cinderclient master: Migrate the functional job to Zuul v3
*** pck has quit IRC18:23
*** pck has joined #openstack-cinder18:30
*** brinzhang has joined #openstack-cinder18:37
*** brinzhang_ has joined #openstack-cinder18:37
*** eharney has quit IRC18:39
*** brinzhang_ has quit IRC18:42
*** brinzhang has quit IRC18:42
*** brinzhang has joined #openstack-cinder18:43
*** brinzhang_ has joined #openstack-cinder18:43
*** mriedem has quit IRC18:54
*** mriedem has joined #openstack-cinder19:03
*** enriquetaso has joined #openstack-cinder19:09
*** tosky has quit IRC19:10
*** senrique_ has joined #openstack-cinder19:10
*** enriquetaso has quit IRC19:13
*** e0ne has joined #openstack-cinder19:39
*** dasp has quit IRC19:51
*** senrique_ is now known as enriquetaso19:52
*** dasp has joined #openstack-cinder19:54
jungleboyjhemna:  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
hemnalooks like they addressed the doc issue I had with the jar19:57
jungleboyjYeah, and I don't think Eric's remaining comment is a blocker.19:57
hemnaerr done even19:58
jungleboyjhemna:  This needs a +W today as well.20:01
jungleboyjand this:
jungleboyje0ne:  Is this one not going to make it now?20:02
jungleboyjhemna:  And this one:
e0nejungleboyj: sorry, didn't get time today to address Gorka's comments :(20:03
jungleboyjApparently my pleas in the meeting yesterday went unheard.20:03
jungleboyje0ne: Ok.  How do you want to proceed?20:03
e0newill do it tomorrow morning20:03
e0neeven if it's too late for this cycle20:04
jungleboyje0ne: 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
hemnaimage encryption20:05
e0nejungleboyj: need to go and see what I already have20:06
e0neif there is a more or less working prototype - I'll ask for spec freeze exception20:07
jungleboyje0ne:  ++20:09
e0neI'll put it as a top priority for me for tomorrow20:10
jungleboyje0ne:  Ok.  Thank you.20:11
e0nejungleboyj: thanks for the help and reminder20:11
jungleboyjYep.  You are welcome.20:11
openstackgerritMerged openstack/cinder-specs master: Volume Rekey spec
jungleboyjhemna: Thanks for the help.  That just leaves:
openstackgerritMerged openstack/cinder-specs master: Spec for the Cinder part of Image Encryption
rosmaitajungleboyj: also ?20:14
rosmaitajungleboyj: i think we definitely want to get in, maybe we can approve and revisit the question of allowing __DEFAULT__ volume type to be updated during implementation?20:19
jungleboyjrosmaita: Ok, the filtering spec is now approved.20:20
jungleboyjrosmaita:  I am ok with that.20:20
rosmaitaok, i will put a note on the review and approve it20:20
jungleboyjNeed to stop bikeshedding.20:20
rosmaitaok, done20:23
hemnabikeshedding is so productive though...20:23
*** raghavendrat has quit IRC20:23
openstackgerritMerged openstack/cinder-specs master: Update the spec of filtering by time comparison operators for Train
openstackgerritMerged openstack/cinder-specs master: Untyped vol to default vol type
* jungleboyj rolls my eyes20:38
jungleboyjrosmaita:  Do you know where the info is for setting up the right package Repo to install OpenStack on a Red Hat box?20:38
rosmaitajungleboyj: maybe this?
*** boxiang_ has quit IRC20:48
*** boxiang_ has joined #openstack-cinder20:48
jungleboyjYeah, 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
jungleboyjTrying to add Cinder to an RHOSP install where they didn't have it without redeploying.20:50
*** mriedem has quit IRC20:52
*** mriedem has joined #openstack-cinder20:53
openstackgerritMerged openstack/cinder master: Add MacroSAN cinder driver
*** whoami-rajat has quit IRC21:28
*** pcaruana has quit IRC21:28
*** enriquetaso has quit IRC21:31
*** _hemna has joined #openstack-cinder21:31
*** enriquetaso has joined #openstack-cinder21:32
*** _hemna has quit IRC21:36
*** Kuirong has quit IRC21:40
*** e0ne has quit IRC21:56
*** henriqueof has quit IRC22:01
openstackgerritMerged openstack/cinder master: Re-add Infortrend Cinder volume driver
*** Liang__ has joined #openstack-cinder22:15
*** rcernin has joined #openstack-cinder22:16
*** carloss has quit IRC22:38
*** Liang__ has quit IRC22:43
*** _erlon_ has quit IRC23:07
*** mriedem has quit IRC23:18
*** brinzhang has quit IRC23:24
*** brinzhang has joined #openstack-cinder23:25
*** deiter has joined #openstack-cinder23:40
*** brinzhang_ has quit IRC23:52
*** brinzhang_ has joined #openstack-cinder23:53
*** enriquetaso has quit IRC23:55
*** smcginnis has quit IRC23:56

Generated by 2.15.3 by Marius Gedminas - find it at!