Tuesday, 2017-03-07

*** k0da has quit IRC00:01
*** kylek3h has joined #openstack-powervm00:13
*** kriskend has quit IRC01:03
*** thorst has joined #openstack-powervm01:17
*** seroyer has joined #openstack-powervm01:18
*** thorst has quit IRC01:18
*** thorst has joined #openstack-powervm01:27
*** jpasqualetto has quit IRC01:31
*** edmondsw has joined #openstack-powervm02:00
*** edmondsw has quit IRC02:00
*** edmondsw has joined #openstack-powervm02:00
*** jpasqualetto has joined #openstack-powervm02:01
*** jpasqualetto has quit IRC02:02
*** thorst has quit IRC02:33
*** thorst has joined #openstack-powervm02:33
*** thorst has quit IRC02:38
*** edmondsw has quit IRC02:55
*** edmondsw has joined #openstack-powervm02:56
*** edmondsw has quit IRC03:01
*** tlian has quit IRC03:14
*** tlian has joined #openstack-powervm03:33
*** thorst has joined #openstack-powervm03:34
*** seroyer has quit IRC03:35
*** thorst has quit IRC03:38
*** mdrabe has quit IRC03:54
*** thorst has joined #openstack-powervm03:56
*** thorst has quit IRC03:56
*** thorst has joined #openstack-powervm04:57
*** thorst has quit IRC05:02
*** tlian has quit IRC05:47
*** thorst has joined #openstack-powervm05:58
*** thorst has quit IRC06:03
*** thorst has joined #openstack-powervm06:59
*** thorst has quit IRC07:04
*** thorst has joined #openstack-powervm08:00
*** thorst has quit IRC08:04
*** k0da has joined #openstack-powervm08:34
*** thorst has joined #openstack-powervm09:01
*** thorst has quit IRC09:05
*** thorst has joined #openstack-powervm10:02
*** thorst has quit IRC10:06
*** openstackgerrit has quit IRC10:33
*** thorst has joined #openstack-powervm11:02
*** thorst has quit IRC11:07
*** edmondsw has joined #openstack-powervm11:19
*** edmondsw has quit IRC11:23
*** thorst has joined #openstack-powervm12:03
*** thorst has quit IRC12:08
*** smatzek has joined #openstack-powervm12:31
*** smatzek_ has joined #openstack-powervm12:33
*** thorst has joined #openstack-powervm12:34
*** smatzek has quit IRC12:37
*** seroyer has joined #openstack-powervm12:55
*** edmondsw has joined #openstack-powervm13:00
*** tblakes has joined #openstack-powervm13:01
*** edmondsw has quit IRC13:06
*** edmondsw has joined #openstack-powervm13:08
*** edmondsw has quit IRC13:12
*** edmondsw has joined #openstack-powervm13:14
*** edmondsw has quit IRC13:14
*** edmondsw has joined #openstack-powervm13:15
*** seroyer has quit IRC13:18
*** k0da has quit IRC13:42
*** jpasqualetto has joined #openstack-powervm13:46
*** k0da has joined #openstack-powervm13:46
*** nbante has joined #openstack-powervm13:48
*** esberglu has joined #openstack-powervm13:51
esberglu#startmeeting powervm_driver_meeting14:00
openstackMeeting started Tue Mar  7 14:00:19 2017 UTC and is due to finish in 60 minutes.  The chair is esberglu. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
openstackThe meeting name has been set to 'powervm_driver_meeting'14:00
thorsto/14:02
adrezneco/14:03
*** apearson has joined #openstack-powervm14:04
esberglu#topic In-tree driver14:04
esbergluI don't think much has changed for in-tree14:05
esbergluStill waiting for wider reviews on the earlier patches, still WIP on the later ones14:06
esbergluAnyone have anything they want to talk about in tree?14:07
adreznecNothing here14:08
*** mdrabe has joined #openstack-powervm14:10
esberglu#topic OOT driver14:10
esbergluAny items to discuss here?14:10
*** kriskend has joined #openstack-powervm14:11
esbergluSounds like a no. I didn't have anything14:14
esberglu#topic CI14:14
esbergluA change went in yesterday evening that broke OOT CI on the master branch14:14
esbergluLooks like it's just an import error though, not expecting it to be a tough fix14:15
esberglu#action esberglu: Find solution for failing OOT CI on master14:16
esberglunewton and ocata are also failing14:16
esberglubut for a different reason14:16
esbergluThey are failing to stack and there is a devstack bug open14:17
*** seroyer has joined #openstack-powervm14:17
esberglu#action esberglu: Get fix for newton/ocata stacking bug14:18
*** seroyer has quit IRC14:19
esbergluAny final topics before I close the meeting?14:21
adreznecNothing from me14:22
esberglu#endmeeting14:27
openstackMeeting ended Tue Mar  7 14:27:54 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:27
openstackMinutes:        http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-03-07-14.00.html14:27
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-03-07-14.00.txt14:27
openstackLog:            http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-03-07-14.00.log.html14:27
*** smatzek_ has quit IRC14:32
*** seroyer has joined #openstack-powervm14:40
*** k0da has quit IRC14:41
*** dwayne has joined #openstack-powervm14:45
*** kriskend has quit IRC14:49
*** k0da has joined #openstack-powervm14:53
*** smatzek_ has joined #openstack-powervm14:55
*** openstackgerrit has joined #openstack-powervm14:58
openstackgerritEric Berglund proposed openstack/networking-powervm master: Import context from neutron_lib  https://review.openstack.org/44254014:58
*** kjw3 has joined #openstack-powervm15:03
*** alainfisher has joined #openstack-powervm15:48
*** alainfisher has quit IRC15:49
*** kjw3 has quit IRC15:49
*** kjw3 has joined #openstack-powervm16:01
*** tjakobs has joined #openstack-powervm16:01
esbergluhttps://github.com/openstack/neutron/commit/ca751a1486812e498d9539980d8cd7bf6d7ebab716:27
esbergluThis change removed context from neutron which is breaking our OOT master branch CI. Now it's in neutron_lib. So I changed the import16:27
esberglufrom neutron import context16:27
esbergluto16:27
esberglufrom neutron_lib import context16:27
esbergluI can't get it to pass tox though.16:27
esbergluWhen running tox with the old import, I get an import error and no tests are run16:27
esbergluWhen running tox with new version, the tests run but I get import errors from the tests16:27
esbergluhttp://logs.openstack.org/40/442540/1/check/gate-networking-powervm-python27-ubuntu-xenial/a78cb3f/testr_results.html.gz16:27
esbergluNot sure what I'm missing here16:27
openstackgerritMerged openstack/nova-powervm master: Move _get_vg_uuid to pypowervm's find_vg  https://review.openstack.org/44078416:34
esbergluThe change passes our CI, just not tox16:34
thorstesberglu: do we have neutron_lib as one of the projects we update as part of the ready script?16:35
thorsteach night we clone down nova, neutron, cinder, etc...16:35
thorstthen the ready script does a git pull16:35
thorstis neutron_lib part of that?  I wonder if we have an old neutron_lib laying around basically16:35
esbergluIt passes CI fine16:36
thorsthmm16:36
thorstlet me try on my box16:36
esbergluI've tried locally as well and it failed16:36
esbergluhttps://review.openstack.org/#/c/442540/16:36
esbergluChange in question16:36
thorstahh, svenkat was just talking to me about proposing that change16:37
openstackgerritMerged openstack/nova-powervm master: In-tree backports: SSP UT  https://review.openstack.org/43799216:42
thorstesberglu: I think it has to do with the requirements.txt16:43
thorsthttps://github.com/openstack/neutron/blob/master/requirements.txt#L1816:43
thorstupdate us to 1.2.0 and I bet we're gold16:43
thorsttrying locally myself.16:44
thorstanother bonus that we'll have with the requirements bot...16:44
adreznecthorst: I actually already have that in https://review.openstack.org/#/c/439805/216:46
thorstright, but because they're not interconnected...16:46
adreznecHave just been waiting for another +2 on it, but I can just merge it we want16:46
thorstI'm not sure we can merge yours now16:46
thorstbecause its broken without the change from esberglu16:46
thorstwon't it fail in the gate?16:46
adreznecWell... it passed both jenkins and our CI16:47
esbergluYeah. Because the CI run went through before the neutron change broke us16:47
esbergluOtherwise thorst would be correct16:47
thorstso we can w+1 it and merge?16:48
esbergluI'm still getting tox errors with 1.2.016:50
*** kjw3 has quit IRC16:51
*** mdrabe has quit IRC16:55
thorstyeah...same here.16:56
adreznecesberglu: thorst what kind of tox errors?16:59
thorstits just not part of 1.2.017:00
thorstlooks like 6 days ago they released a 1.2.117:00
thorstwhich I bet we need17:00
thorstbut it begs the question, how is neutron itself passing17:00
esbergluWhere are you seeing 1.2.1?17:00
thorstrandom google.17:00
thorstI'm trying it now17:00
*** mdrabe has joined #openstack-powervm17:02
esberglucontext exists in 1.2.0 though17:02
esbergluhttps://github.com/openstack/neutron-lib/blob/1.2.0/neutron_lib/context.py17:02
thorstesberglu: actually, it looks like other things just need to be flipped over17:03
thorstfound it...17:05
thorsthttps://github.com/openstack/networking-powervm/blob/master/networking_powervm/tests/unit/plugins/ibm/powervm/base.py#L4917:05
thorstso leave neutron-lib at 1.2.0, update that fixture to use the new path17:06
esbergluAh good find thanks17:07
openstackgerritEric Berglund proposed openstack/networking-powervm master: Import context from neutron_lib  https://review.openstack.org/44254017:07
*** kjw3 has joined #openstack-powervm17:11
*** jpasqualetto has quit IRC17:48
*** jpasqualetto has joined #openstack-powervm17:55
*** nbante has quit IRC18:24
*** nbante has joined #openstack-powervm18:31
thorstesberglu: did that work?18:37
*** kjw3 has quit IRC18:40
*** kjw3 has joined #openstack-powervm18:53
esbergluthorst: Yep. https://review.openstack.org/#/c/442540/18:54
*** nbante has quit IRC18:58
thorstesberglu: you rock dude19:21
*** kjw3 has quit IRC19:22
openstackgerritMerged openstack/networking-powervm master: Import context from neutron_lib  https://review.openstack.org/44254019:26
*** kjw3 has joined #openstack-powervm19:41
*** kjw3 has quit IRC20:37
*** kjw3 has joined #openstack-powervm21:22
*** smatzek_ has quit IRC21:28
efriedesberglu thorst adreznec Check out recent jenkins failures.  Looks like we may have gotten some new flake8 rules in place.  Any ideas where that came from, or how I can get those same rules locally?21:31
thorsthave a link handy?  I deleted them21:31
efriedthorst http://logs.openstack.org/88/391288/38/check/gate-nova-pep8-ubuntu-xenial/118aab6/21:31
efrieddeleted what?21:32
thorstthe e-mails saying we had a failure21:32
efriedIt's jenkins, not our CI.21:32
efriedpep821:32
thorstso I'm betting they updated pep8 version on the CI servers21:32
thorstand we're just getting hit by it now21:32
thorstI think we just need to update the code to get through the flake8, but I bet ocata and mitaka are now in trouble21:35
efriedthorst How do I get that new pep8 locally on my dev box?21:36
thorstsudo pip install --upgrade pep821:36
thorstI think21:36
efriedWell, that did something.  Lemme see if it now hits the problem.21:36
efriednope21:38
thorsthmmm21:38
thorstadreznec: any ideas?21:38
efriedthorst Hold the phone, it's using pep8 from the virtualenv.21:39
efriedTrying with that guy upgraded...21:39
adreznecWas there actually a new pep8 version released that it's using?21:41
adreznecThe job logs say pep8==1.5.721:41
adreznecBut that was released in like 201421:41
thorsthmm...I was just going off of a theory21:42
efriedadreznec The latest seems to be 1.7.0, but I'm having a devil of a time getting it to run in my env.21:42
thorstI didn't investigate said theory21:42
efriedCause something is requiring 1.5.721:43
adreznecefried: Right, I'm just saying that it's not using 1.7.0 in the gate21:43
efriedLike the global requirements project.21:43
adreznecYeah21:43
adreznecrequirements has pep8==1.5.721:43
efriedYeah, and that failure log says it's using 1.5.7 too.21:44
efriedso where is the failure coming from?21:44
adreznecand that hasn't changed in over a year21:44
thorstdo we some how inherit rules from nova...and did they remove rules?21:45
*** kjw3 has quit IRC21:46
adreznecwas this an in-tree patch? or out of tree?21:46
thorstoot21:46
thorstbut I thought our hacking rules imported from nova21:46
thorstbut this isn't a hacking issue...is it?21:46
adreznecThis is an in-tree patch21:47
adreznechttps://review.openstack.org/#/c/391288/21:47
thorsto derp21:47
thorstthen its just a fix needed to the patch  :-D21:47
adreznecNova must have updated rules or something21:47
adreznecefried: ^21:47
efriedadreznec If it's the rules in tox.ini we're talking about, those haven't changed in two years.21:49
efriedoh.  Except for "enable extensions".21:50
efriedEven there...21:51
efriedI don't see what's causing the problem.21:51
efriedgr21:51
efriedAha.  It's this guy: flake8-import-order==0.1121:59
efriedIt's possible if I rebuild the virtualenv it'll get pulled in.  Course, it's also possible it won't.22:01
*** apearson has quit IRC22:09
*** thorst has quit IRC22:12
*** smatzek_ has joined #openstack-powervm22:25
*** tblakes has quit IRC22:33
*** smatzek_ has quit IRC22:36
*** esberglu has quit IRC22:49
*** esberglu has joined #openstack-powervm22:49
*** mdrabe has quit IRC22:50
*** dwayne has quit IRC22:52
*** seroyer has quit IRC22:52
*** esberglu_ has joined #openstack-powervm22:53
*** esberglu has quit IRC22:54
*** esberglu_ has quit IRC22:57
*** jpasqualetto has quit IRC23:08
*** edmondsw has quit IRC23:16
*** k0da has quit IRC23:26
*** tjakobs has quit IRC23:33
*** thorst has joined #openstack-powervm23:45
*** thorst has quit IRC23:45
*** edmondsw has joined #openstack-powervm23:52
*** dwayne has joined #openstack-powervm23:55
*** thorst has joined #openstack-powervm23:55
*** edmondsw has quit IRC23:56
*** esberglu has joined #openstack-powervm23:58

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