Wednesday, 2019-05-08

*** mmethot has quit IRC00:06
*** _hemna has joined #openstack-cinder00:12
*** _hemna has quit IRC00:17
*** lixiaoy1 has joined #openstack-cinder00:35
*** KeithMnemonic has quit IRC00:56
*** _alastor_ has quit IRC00:57
*** boxiang has quit IRC01:00
*** boxiang has joined #openstack-cinder01:00
openstackgerritMerged openstack/os-brick master: Provide setting to ignore lvm descriptor leak warnings  https://review.opendev.org/65279701:34
*** baojg has joined #openstack-cinder01:36
openstackgerrittonybrad proposed openstack/cinder master: Drop use git.openstack.org for tox deps  https://review.opendev.org/65772302:19
*** whoami-rajat has joined #openstack-cinder03:01
*** tejdeep has quit IRC03:06
*** psachin has joined #openstack-cinder03:06
*** gnufied has quit IRC03:07
*** itlinux has joined #openstack-cinder03:20
*** itlinux has quit IRC03:22
*** enriquetaso has quit IRC03:44
*** udesale has joined #openstack-cinder03:55
openstackgerritMerged openstack/os-brick master: Replace git.openstack.org URLs with opendev.org URLs  https://review.opendev.org/65505304:10
*** boxiang has quit IRC05:20
*** boxiang has joined #openstack-cinder05:21
*** dims has quit IRC05:32
*** dims has joined #openstack-cinder05:33
*** boxiang has quit IRC05:36
*** boxiang has joined #openstack-cinder05:36
openstackgerritMerged openstack/cinder stable/rocky: Remove LOCI publishing from the post pipeline  https://review.opendev.org/65763505:48
*** boxiang has quit IRC05:50
*** boxiang has joined #openstack-cinder05:51
*** anks2k has joined #openstack-cinder05:56
*** anks2k has quit IRC05:56
*** boxiang has quit IRC06:03
*** boxiang has joined #openstack-cinder06:04
*** udesale has quit IRC06:05
*** udesale has joined #openstack-cinder06:05
*** dims has quit IRC06:16
*** dims has joined #openstack-cinder06:42
*** dims has quit IRC06:49
*** dims has joined #openstack-cinder06:54
*** boxiang has quit IRC06:57
*** boxiang has joined #openstack-cinder06:58
*** rcernin has quit IRC07:01
*** udesale has quit IRC07:03
*** udesale has joined #openstack-cinder07:03
*** boxiang has quit IRC07:18
*** boxiang has joined #openstack-cinder07:20
*** brinzhang has joined #openstack-cinder07:21
*** boxiang has quit IRC07:36
*** boxiang has joined #openstack-cinder07:37
*** Luzi has joined #openstack-cinder07:47
*** rtjure has quit IRC07:48
*** rtjure has joined #openstack-cinder07:54
*** dpawlik has joined #openstack-cinder07:56
*** rtjure has quit IRC08:16
*** lemko has joined #openstack-cinder08:24
*** lpetrut has joined #openstack-cinder08:38
*** tkajinam has quit IRC08:46
*** lixiaoy1 has quit IRC08:54
openstackgerritzhufl proposed openstack/cinder master: Use assertNotIn to check dict item  https://review.opendev.org/65775908:55
*** _hemna has joined #openstack-cinder09:12
*** _hemna has quit IRC09:16
*** helenafm has joined #openstack-cinder09:23
*** carloss has quit IRC10:56
openstackgerritRajat Dhasmana proposed openstack/cinder master: Remove Violin driver requirements  https://review.opendev.org/65777711:06
*** sapd1_x has joined #openstack-cinder11:11
*** udesale has quit IRC11:14
*** phasespace has joined #openstack-cinder11:16
hemnamornin11:21
*** dviroel has joined #openstack-cinder11:26
*** jsquare has quit IRC11:30
*** jsquare has joined #openstack-cinder11:31
*** baojg has quit IRC11:40
*** helenafm has quit IRC12:05
*** _alastor_ has joined #openstack-cinder12:07
*** abishop has joined #openstack-cinder12:11
*** _alastor_ has quit IRC12:12
*** henriqueof has quit IRC12:12
*** henriqueof has joined #openstack-cinder12:17
*** mmethot has joined #openstack-cinder12:32
*** rosmaita has quit IRC12:34
*** rosmaita has joined #openstack-cinder12:35
hemnawhat version of python3 are we going to support when we drop py27 ?12:38
openstackgerritRaghavendra Tilay proposed openstack/cinder master: [WIP] 3PAR: Provide option to specify zone information for bootable volume  https://review.opendev.org/65758512:42
*** henriqueof has quit IRC12:43
*** henriqueof has joined #openstack-cinder12:43
hemnais the ci watch down?12:45
*** hemna has quit IRC12:54
*** hemna has joined #openstack-cinder12:55
hemnaSo it looks like we have problems with some driver's 3rd party libraries13:02
hemnathose are going to need to get fixed13:02
hemnaI think it could be time to deprecate the hpelefthand driver13:04
hemnalast I had heard at hpe, they were discontinuing the lefthand devices13:04
*** kaisers has quit IRC13:05
*** kaisers has joined #openstack-cinder13:06
*** davee__ has joined #openstack-cinder13:10
*** davee_ has quit IRC13:11
*** _alastor_ has joined #openstack-cinder13:17
*** brinzhang has quit IRC13:23
*** nikeshm has joined #openstack-cinder13:23
nikeshmHi everyone13:24
nikeshmWe want to certify the Kaminario cinder volume driver with RedHat RHOSP13. But test_force_detach_volume test is failing, so if we fix it in master, can i backport it in stable/queens used in RED HAT OPENSTACK PLATFORM 1313:24
smcginnisnikeshm: Bug fixes can be backported, yes.13:28
smcginnishemna: Was thinking that too. If the drivers or their libraries don't support py3, then we should mark them unsupported now so we can remove them in U when we go to py3-only.13:29
smcginnishemna: https://governance.openstack.org/tc/reference/runtimes/train.html13:30
nikeshm@smcginnis : is there any documentation or guidelines to check  py3 support13:31
nikeshmI mean openstack documentation13:32
nikeshmor guidelines13:32
*** baojg has joined #openstack-cinder13:34
hemnaso python3.6 is the minimum13:34
smcginnisYeah, but things also are expected to run on py3.7.13:35
*** helenafm has joined #openstack-cinder13:38
hemnaso that makes 3.7 the minimum by default then13:41
hemnaif it's required to run13:41
hemnaI think this is a big problem for us13:42
smcginnisYeah, that's why I keep bringing up that third party CI needs to be switched over to using py3 at a minimum. I think we're going to hit a lot of issues, so I'm worried it's taking everyone so long to actually start testing this.13:45
smcginnisWe already know VNX has problems (or taskflow) with py3.7.13:45
*** baojg has quit IRC13:45
smcginnisNote on those LOCI job backports, those don't need to follow "normal" backport policy since they are CI job fixes. The sooner we just get those through the better.13:50
hemnathe purestorage client doesn't even adhere to normal pypi projects13:51
hemnano tox13:51
hemnaand no declared support for py versions13:51
*** mriedem has joined #openstack-cinder13:52
hemnaI think we need a mailing list post13:52
hemnaand a timeframe for getting CI switched to py313:52
smcginnissimondodsley: ^^13:52
smcginnishemna: Want to send something out? I do think the more we start shouting about this the better.13:52
hemnasince we are going to require py37, then I presume the CI tests need to run under py37 by default13:52
smcginnisYep13:53
hemnasure13:53
smcginnisWe need the py36 and py37 unit tests running at an absolute minimum.13:53
*** Luzi has quit IRC13:57
*** lpetrut has quit IRC13:59
jungleboyjsmcginnis:  Ok.  Good to know.  Will work on just getting those in.14:01
*** gnufied has joined #openstack-cinder14:04
*** phasespace has quit IRC14:06
*** psachin has quit IRC14:08
*** _alastor_ has quit IRC14:19
smcginnisjungleboyj: Did we agree at the PTG that milestone 2 is the deadline for CIs to be updated to running py3?14:22
smcginnisMight be good to send something to the ML to start pushing that widely.14:22
jungleboyjsmcginnis: That sounds right.  I am going through all those notes today and can verify.14:22
jungleboyjsmcginnis:  Can do.14:23
smcginnis++14:23
*** lpetrut has joined #openstack-cinder14:24
hemnaI sent an ML post14:35
hemnahaven't seen it come up yet14:35
openstackgerritChristopher Brown proposed openstack/cinder stable/queens: Dell EMC: SC FC Driver wwns api response in lower  https://review.opendev.org/65782614:36
*** lpetrut has quit IRC14:37
*** tesseract has joined #openstack-cinder14:38
*** lpetrut has joined #openstack-cinder14:39
smcginnishemna: I haven't seen it yet either. Typo in address maybe?14:41
hemnahrmm14:49
hemnaguess it's been a while since I've made a post14:52
smcginnisStill sending to -dev? :)14:56
hemnayah14:56
jungleboyjOoops.14:56
hemnahad to resubscribe14:57
hemnatrying again14:57
*** whoami-rajat has quit IRC15:02
*** itlinux has joined #openstack-cinder15:08
*** sapd1_x has quit IRC15:09
*** sapd1_x has joined #openstack-cinder15:09
*** tesseract has quit IRC15:09
*** imacdonn has quit IRC15:11
jungleboyjhemna:  Good note.  Thank you.15:11
*** imacdonn has joined #openstack-cinder15:12
hemnait showed up?15:12
jungleboyjsmcginnis:  hemna  I was going to respond with a follow-up that drivers that fail to meet the requirement will have a Unsupported patch pushed up.15:13
hemnaah ok, crap I forgot that part15:13
jungleboyjNo problem.  I will be bad cop.15:13
jgriffithhemna: there's a new list as of like December15:13
hemnajgriffith: thanks.  I had just resubscribed, but aren't seeing any yet15:13
jgriffithhemna: oh... waiting for the bits to catch up probably :)15:14
hemnayah15:15
jungleboyjOk.  I have added that note.15:15
smcginnisjgriffith: o/15:17
*** altlogbot_1 has quit IRC15:21
*** altlogbot_3 has joined #openstack-cinder15:21
*** irclogbot_2 has quit IRC15:22
*** enriquetaso has joined #openstack-cinder15:22
jungleboyjjgriffith:  What happened at the Summit?  I saw you and then you were gone.15:23
*** irclogbot_1 has joined #openstack-cinder15:23
jgriffithjungleboyj: I was only there that evening to catch up and have dinner... turns out walking the streets of Denver during an OpenStack summit can result in a person like me being very easily distracted15:23
jungleboyjjgriffith: :-)  Didn't interrupt when I saw you as I thought we would see you at dinner.  Sorry we didn't get to catch up.15:24
jgriffithPopped into a few stops along the way during the walk, next thing I knew it was around 9:30 or 10:00 and had to make the drive home15:24
jgriffithYeah, me too... I didn't get to catch up with many people unfortunatly15:25
jungleboyj:-)15:25
jungleboyjWell, it has been nice to see you in the channel again.   :-)15:25
jgriffithLiar :)15:26
jungleboyj:-p15:27
*** altlogbot_3 has quit IRC15:27
jungleboyjThe PTGs are so dull without you and Duncan fighting.15:27
*** altlogbot_2 has joined #openstack-cinder15:27
jgriffithhardy har har15:27
*** _alastor_ has joined #openstack-cinder15:27
*** irclogbot_1 has quit IRC15:28
*** irclogbot_0 has joined #openstack-cinder15:29
woojayjungleboyj: smcginnis: Y'all good w/ this one, to remove drbdmanage from the requirement?  https://review.opendev.org/#/c/657630/15:31
*** henriqueof has quit IRC15:32
smcginnisYep!15:32
woojayThank you!  I will follow up w/ another one for python-linstor shortly then.15:33
openstackgerritMerged openstack/cinder stable/pike: Remove LOCI publishing from the post pipeline  https://review.opendev.org/65763715:41
*** itlinux has quit IRC15:42
woojaysmcginnis: my drbdmanage patch failed py27 test at the gate just now.  http://logs.openstack.org/30/657630/1/gate/openstack-tox-py27/1540c55/ara-report/ with silly 'ERROR: networkx requires Python '>=3.5' but the running Python is 2.7.15'.  Do I have to 'recheck' later then?15:47
smcginnisThat's odd, I've seen a few of those now.15:48
smcginniswoojay: Recheck will probably take care of it, but maybe we should track down what's going on there a little better.15:48
smcginnisThought it was just a weird fluke before, but maybe not.15:48
woojayI wasn't sure if it was testbed specific or something else...15:49
woojayI will get the python-linstor one started in the mean time, and try a recheck on the failing one later today.15:50
*** e0ne has joined #openstack-cinder15:50
*** e0ne has quit IRC15:53
openstackgerritMerged openstack/cinder stable/queens: Remove LOCI publishing from the post pipeline  https://review.opendev.org/65763615:57
smcginnisHmm, according to upper-constraints, py27 jobs should not have ended up trying to install networkx 2.3 - https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L8615:57
smcginnisSomething weird is going on with those failures.15:57
openstackgerritJay Rubenstein proposed openstack/cinder master: SF: Handle qos values on extend volume  https://review.opendev.org/60695916:00
woojayFor sure.  It definitely tried to pull a wrong version.16:01
jungleboyjsmcginnis:  rosmaita  Thoughts on this now that it passed the check?16:02
jungleboyjhttps://review.opendev.org/64919316:02
smcginnisMan, all that red from third party.16:02
hemnayah, I think we have a mess on our hands (ci)16:04
jungleboyj:-(16:04
jungleboyjThere is no reason that CIs should fail on that.16:05
smcginnisI'm sure most are probably due to the openstack-dev to openstack namespace change for devstack.16:05
jungleboyjsmcginnis:  Ugh, Ok.  Let me see if I can fix those cases.16:06
smcginnisjungleboyj: Probably just a quick if check first.16:06
jungleboyjRight.  :-)16:07
smcginnisLooks good with a real cinder.conf file.16:09
geguileojungleboyj: Is there something wrong with my calendar or should we be already having the cinder meeting?  r:-???16:11
jungleboyjgeguileo:  We cancelled it this week to give people a break after the PTG.16:11
jungleboyjMany people were traveling, etc.16:11
*** lpetrut has quit IRC16:11
jungleboyjI thought I sent an e-mail about it.16:11
geguileojungleboyj: oh, I read the email and somehow assumed it was last weeks email...16:11
*** e0ne has joined #openstack-cinder16:11
jungleboyj:-)16:11
geguileomy bad  XD16:12
geguileojungleboyj: thanks :-)16:12
jungleboyjgeguileo:  How was the wedding?16:12
mszwed_same story for me - I thought it was regarding last week :)16:12
geguileojungleboyj: great, they even got married, though the priest forgot the book thingy, so we couldn't sign it...16:12
smcginnisWOw16:13
jungleboyjOoops.16:13
*** mszwed_ is now known as mszwed16:13
jungleboyjgeguileo:  Glad it went well otherwise.16:16
geguileothanks16:16
jungleboyjWe missed you but enjoyed delegating all the work to you.  ;-)16:16
*** whoami-rajat has joined #openstack-cinder16:16
*** david-lyle has quit IRC16:19
*** dklyle has joined #openstack-cinder16:20
hemna2019-05-07 01:39:18.255 | fatal: repository 'https://review.opendev.org/p/openstack-dev/devstack/' not found16:23
hemna2019-05-07 01:39:18.257 | error: Could not fetch originy16:23
hemnapurestorage ci is failing due to that16:23
*** helenafm has quit IRC16:24
*** e0ne has quit IRC16:26
smcginnisThe few I looked at were all like that.16:26
openstackgerritJay Bryant proposed openstack/cinder master: Remove the Tintri Driver  https://review.opendev.org/64793416:41
*** henriqueof has joined #openstack-cinder16:42
*** lseki has joined #openstack-cinder16:55
tonybhemna: Let me know if my comment on 657395 needs clarification17:05
*** e0ne has joined #openstack-cinder17:10
*** e0ne has quit IRC17:12
*** e0ne has joined #openstack-cinder17:18
*** e0ne has quit IRC17:21
*** ianychoi_ has joined #openstack-cinder17:29
*** ianychoi has quit IRC17:31
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move Nexenta exception  https://review.opendev.org/65785517:33
openstackgerritWoojay Poynter proposed openstack/cinder master: Removes python-linstor as a requirement for LINSTOR driver  https://review.opendev.org/65785617:41
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move sheepdog exception  https://review.opendev.org/65785917:53
*** e0ne has joined #openstack-cinder17:59
woojaysmcginnis: Looks like that tox-py27 test doesn't want to play nice today...  Quite a few cinder patches are failing at the 'check' so far...18:05
*** e0ne has quit IRC18:07
smcginnisOlder releases of taskflow were uncapped, but it should be picking up the new one.18:23
smcginnisMust be getting an older release for some reason. I would expect it to happen on stable branches, but not master.18:23
woojaythank you so much for looking.  I will give it a recheck later in the evening.18:25
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move dothill exceptions  https://review.opendev.org/65786618:43
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move storpool exception  https://review.opendev.org/65786818:58
*** lpetrut has joined #openstack-cinder19:08
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move xtremio exceptions  https://review.opendev.org/65787119:10
*** lpetrut has quit IRC19:12
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move zfssa exception  https://review.opendev.org/65787219:21
*** lemko has quit IRC19:22
*** carloss has joined #openstack-cinder19:24
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move zadara exceptions  https://review.opendev.org/65787419:34
hemnaweee!19:34
smcginnisNice19:34
* jungleboyj is in expense report hell!19:34
* smcginnis already had his approved19:35
jungleboyjThe best part is now it is complaining that I have too many items on the report.  Seriously?19:37
smcginnisBig spender.19:39
jungleboyjI guess so.  But now I have to split it into two reports.  *Sigh*19:40
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move datera exception  https://review.opendev.org/65787619:45
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: remove unused exceptions  https://review.opendev.org/65787819:51
hemnaman this stuff has needed a good cleaning for a while evidently19:53
jungleboyjhemna:  Thanks for doing it.19:57
hemnanp.  I think cinder will be slightly cleaner once it's done19:59
smcginnisDirty, dirty cinder.20:00
openstackgerritSean McGinnis proposed openstack/cinder master: Raise lower constraint for taskflow  https://review.opendev.org/65788020:03
openstackgerritTony Breeds proposed openstack/cinder master: Make sure we install cinder requirements during the correct tox phase  https://review.opendev.org/65788120:12
*** david-lyle has joined #openstack-cinder20:12
tonyb^^^ probably should be backported as far as rocky20:13
*** itlinux has joined #openstack-cinder20:13
*** itlinux has quit IRC20:13
*** niceplace has quit IRC20:14
*** obre has quit IRC20:14
*** dklyle has quit IRC20:15
*** hyang has quit IRC20:15
*** dansmith has quit IRC20:15
*** pots has quit IRC20:15
*** hyang has joined #openstack-cinder20:16
*** obre has joined #openstack-cinder20:16
*** pots has joined #openstack-cinder20:17
smcginnisHappened in Mitaka - https://github.com/openstack/cinder/commit/17cc592c68cb94461c6377dd5049fc300227844a20:19
*** dansmith has joined #openstack-cinder20:20
*** niceplace has joined #openstack-cinder20:22
*** itlinux has joined #openstack-cinder20:43
*** itlinux has quit IRC20:45
hemnaERROR: networkx requires Python '>=3.5' but the running Python is 2.7.1520:48
hemnaseeing that in my exceptions py27 jobs20:48
smcginnishemna: That's what we were talking about earlier ^20:54
*** abishop has quit IRC20:56
smcginnishemna: This will hopefully fix it and passes check: https://review.opendev.org/#/c/657881/21:02
openstackgerritMerged openstack/cinder stable/ocata: Remove LOCI publishing from the post pipeline  https://review.opendev.org/65763921:03
rosmaitasmcginnis: if you cherry-pick https://review.opendev.org/#/c/654609/ to stable/rocky, i'll write the release note ... deal?21:17
smcginnisHah!21:18
smcginnisrosmaita: Want to just do both so you can add the release note to the patch right away?21:18
rosmaitasmcginnis: i was going to do it on a different patch so the cherry-pick stays clean21:19
rosmaitabut i guess it's not going into queens, so i could do that21:19
rosmaitaall right, i'll do that and then come back and bug you and jay to review21:20
smcginnisSounds good. Just thinking we had wanted a release note to backport it, so it would be good to include it in one.21:20
rosmaitamakes sense21:20
openstackgerritWoojay Poynter proposed openstack/cinder master: Removes python-linstor as a requirement for LINSTOR driver  https://review.opendev.org/65785621:20
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: move netapp exception  https://review.opendev.org/65788921:22
hemnasmcginnis: oh ok coolio.  I missed that.  been kneck deep in netapp code...21:23
rosmaitasmcginnis: while i've got you, can you go to https://review.opendev.org/#/c/653473/ and hit the cherry-pick button to stable/stein21:24
smcginnisrosmaita: Doesn't look like it, but I can get it going on the command line.21:24
smcginnisNo idea why git thought that was a merge conflict and couldn't figure it out.21:26
rosmaitaweird21:26
rosmaitathanks21:26
smcginnisDang, can't even git review because the rebase that happens thinks it's a conflict too.21:27
openstackgerritSean McGinnis proposed openstack/python-cinderclient stable/stein: Raise API max version for Stein updates  https://review.opendev.org/65789121:28
openstackgerritSean McGinnis proposed openstack/python-cinderclient stable/stein: Raise API max version for Stein updates  https://review.opendev.org/65789121:31
*** kaisers has quit IRC21:31
rosmaitasmcginnis: i wonder if the problem is that the parent of that is 8f7cc5b, and stable/stein head is currently at ee37f26, which is your other patch that changed the API max21:32
smcginnisI didn't think I had a local stable/stein branch, but I guess I did/21:32
smcginnisThat second one should be OK.21:32
rosmaitaand it looks like you already figured that out21:32
rosmaita:)21:32
*** _hemna has joined #openstack-cinder21:32
smcginnis;)21:32
smcginnisThat was confusing21:32
smcginnisBut yeah, looks like it's in the check queue now and (knock on wood) should be OK.21:32
rosmaitacool, thanks21:33
smcginnisThanks for the reminder.21:33
*** _hemna has quit IRC21:36
rosmaitasmcginnis: i think we also need https://review.opendev.org/#/c/653472/ to be backported to stein?  i am losing track21:37
*** henriqueof has quit IRC21:44
smcginnisrosmaita: It was functionality added in Stein, but that now gets a little iffey with stable backport policy.22:03
*** _hemna has joined #openstack-cinder22:03
smcginnisMight be better to just recommend anyone that needs that use the first train client version to pick it up.22:04
smcginnisBah, but we are raising the microversion in the other ones, so it will be weird if that's missing.22:04
openstackgerritSean McGinnis proposed openstack/python-cinderclient stable/stein: Add support for backup user ID  https://review.opendev.org/65789522:05
*** carloss has quit IRC22:24
*** whoami-rajat has quit IRC22:26
*** rcernin has joined #openstack-cinder22:31
*** _hemna has quit IRC22:37
*** boxiang has quit IRC22:45
*** boxiang has joined #openstack-cinder22:45
openstackgerritMerged openstack/cinder master: Make sure we install cinder requirements during the correct tox phase  https://review.opendev.org/65788122:49
*** tkajinam has joined #openstack-cinder23:03
*** dviroel has quit IRC23:14
*** e0ne has joined #openstack-cinder23:35
openstackgerritBrian Rosmaita proposed openstack/python-cinderclient stable/rocky: Raise API max version for Rocky updates  https://review.opendev.org/65790523:48
*** e0ne has quit IRC23:52

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!