*** k0da has quit IRC | 00:01 | |
*** kylek3h has joined #openstack-powervm | 00:13 | |
*** kriskend has quit IRC | 01:03 | |
*** thorst has joined #openstack-powervm | 01:17 | |
*** seroyer has joined #openstack-powervm | 01:18 | |
*** thorst has quit IRC | 01:18 | |
*** thorst has joined #openstack-powervm | 01:27 | |
*** jpasqualetto has quit IRC | 01:31 | |
*** edmondsw has joined #openstack-powervm | 02:00 | |
*** edmondsw has quit IRC | 02:00 | |
*** edmondsw has joined #openstack-powervm | 02:00 | |
*** jpasqualetto has joined #openstack-powervm | 02:01 | |
*** jpasqualetto has quit IRC | 02:02 | |
*** thorst has quit IRC | 02:33 | |
*** thorst has joined #openstack-powervm | 02:33 | |
*** thorst has quit IRC | 02:38 | |
*** edmondsw has quit IRC | 02:55 | |
*** edmondsw has joined #openstack-powervm | 02:56 | |
*** edmondsw has quit IRC | 03:01 | |
*** tlian has quit IRC | 03:14 | |
*** tlian has joined #openstack-powervm | 03:33 | |
*** thorst has joined #openstack-powervm | 03:34 | |
*** seroyer has quit IRC | 03:35 | |
*** thorst has quit IRC | 03:38 | |
*** mdrabe has quit IRC | 03:54 | |
*** thorst has joined #openstack-powervm | 03:56 | |
*** thorst has quit IRC | 03:56 | |
*** thorst has joined #openstack-powervm | 04:57 | |
*** thorst has quit IRC | 05:02 | |
*** tlian has quit IRC | 05:47 | |
*** thorst has joined #openstack-powervm | 05:58 | |
*** thorst has quit IRC | 06:03 | |
*** thorst has joined #openstack-powervm | 06:59 | |
*** thorst has quit IRC | 07:04 | |
*** thorst has joined #openstack-powervm | 08:00 | |
*** thorst has quit IRC | 08:04 | |
*** k0da has joined #openstack-powervm | 08:34 | |
*** thorst has joined #openstack-powervm | 09:01 | |
*** thorst has quit IRC | 09:05 | |
*** thorst has joined #openstack-powervm | 10:02 | |
*** thorst has quit IRC | 10:06 | |
*** openstackgerrit has quit IRC | 10:33 | |
*** thorst has joined #openstack-powervm | 11:02 | |
*** thorst has quit IRC | 11:07 | |
*** edmondsw has joined #openstack-powervm | 11:19 | |
*** edmondsw has quit IRC | 11:23 | |
*** thorst has joined #openstack-powervm | 12:03 | |
*** thorst has quit IRC | 12:08 | |
*** smatzek has joined #openstack-powervm | 12:31 | |
*** smatzek_ has joined #openstack-powervm | 12:33 | |
*** thorst has joined #openstack-powervm | 12:34 | |
*** smatzek has quit IRC | 12:37 | |
*** seroyer has joined #openstack-powervm | 12:55 | |
*** edmondsw has joined #openstack-powervm | 13:00 | |
*** tblakes has joined #openstack-powervm | 13:01 | |
*** edmondsw has quit IRC | 13:06 | |
*** edmondsw has joined #openstack-powervm | 13:08 | |
*** edmondsw has quit IRC | 13:12 | |
*** edmondsw has joined #openstack-powervm | 13:14 | |
*** edmondsw has quit IRC | 13:14 | |
*** edmondsw has joined #openstack-powervm | 13:15 | |
*** seroyer has quit IRC | 13:18 | |
*** k0da has quit IRC | 13:42 | |
*** jpasqualetto has joined #openstack-powervm | 13:46 | |
*** k0da has joined #openstack-powervm | 13:46 | |
*** nbante has joined #openstack-powervm | 13:48 | |
*** esberglu has joined #openstack-powervm | 13:51 | |
esberglu | #startmeeting powervm_driver_meeting | 14:00 |
---|---|---|
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:00 |
thorst | o/ | 14:02 |
adreznec | o/ | 14:03 |
*** apearson has joined #openstack-powervm | 14:04 | |
esberglu | #topic In-tree driver | 14:04 |
esberglu | I don't think much has changed for in-tree | 14:05 |
esberglu | Still waiting for wider reviews on the earlier patches, still WIP on the later ones | 14:06 |
esberglu | Anyone have anything they want to talk about in tree? | 14:07 |
adreznec | Nothing here | 14:08 |
*** mdrabe has joined #openstack-powervm | 14:10 | |
esberglu | #topic OOT driver | 14:10 |
esberglu | Any items to discuss here? | 14:10 |
*** kriskend has joined #openstack-powervm | 14:11 | |
esberglu | Sounds like a no. I didn't have anything | 14:14 |
esberglu | #topic CI | 14:14 |
esberglu | A change went in yesterday evening that broke OOT CI on the master branch | 14:14 |
esberglu | Looks like it's just an import error though, not expecting it to be a tough fix | 14:15 |
esberglu | #action esberglu: Find solution for failing OOT CI on master | 14:16 |
esberglu | newton and ocata are also failing | 14:16 |
esberglu | but for a different reason | 14:16 |
esberglu | They are failing to stack and there is a devstack bug open | 14:17 |
*** seroyer has joined #openstack-powervm | 14:17 | |
esberglu | #action esberglu: Get fix for newton/ocata stacking bug | 14:18 |
*** seroyer has quit IRC | 14:19 | |
esberglu | Any final topics before I close the meeting? | 14:21 |
adreznec | Nothing from me | 14:22 |
esberglu | #endmeeting | 14:27 |
openstack | Meeting ended Tue Mar 7 14:27:54 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:27 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-03-07-14.00.html | 14:27 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-03-07-14.00.txt | 14:27 |
openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2017/powervm_driver_meeting.2017-03-07-14.00.log.html | 14:27 |
*** smatzek_ has quit IRC | 14:32 | |
*** seroyer has joined #openstack-powervm | 14:40 | |
*** k0da has quit IRC | 14:41 | |
*** dwayne has joined #openstack-powervm | 14:45 | |
*** kriskend has quit IRC | 14:49 | |
*** k0da has joined #openstack-powervm | 14:53 | |
*** smatzek_ has joined #openstack-powervm | 14:55 | |
*** openstackgerrit has joined #openstack-powervm | 14:58 | |
openstackgerrit | Eric Berglund proposed openstack/networking-powervm master: Import context from neutron_lib https://review.openstack.org/442540 | 14:58 |
*** kjw3 has joined #openstack-powervm | 15:03 | |
*** alainfisher has joined #openstack-powervm | 15:48 | |
*** alainfisher has quit IRC | 15:49 | |
*** kjw3 has quit IRC | 15:49 | |
*** kjw3 has joined #openstack-powervm | 16:01 | |
*** tjakobs has joined #openstack-powervm | 16:01 | |
esberglu | https://github.com/openstack/neutron/commit/ca751a1486812e498d9539980d8cd7bf6d7ebab7 | 16:27 |
esberglu | This change removed context from neutron which is breaking our OOT master branch CI. Now it's in neutron_lib. So I changed the import | 16:27 |
esberglu | from neutron import context | 16:27 |
esberglu | to | 16:27 |
esberglu | from neutron_lib import context | 16:27 |
esberglu | I can't get it to pass tox though. | 16:27 |
esberglu | When running tox with the old import, I get an import error and no tests are run | 16:27 |
esberglu | When running tox with new version, the tests run but I get import errors from the tests | 16:27 |
esberglu | http://logs.openstack.org/40/442540/1/check/gate-networking-powervm-python27-ubuntu-xenial/a78cb3f/testr_results.html.gz | 16:27 |
esberglu | Not sure what I'm missing here | 16:27 |
openstackgerrit | Merged openstack/nova-powervm master: Move _get_vg_uuid to pypowervm's find_vg https://review.openstack.org/440784 | 16:34 |
esberglu | The change passes our CI, just not tox | 16:34 |
thorst | esberglu: do we have neutron_lib as one of the projects we update as part of the ready script? | 16:35 |
thorst | each night we clone down nova, neutron, cinder, etc... | 16:35 |
thorst | then the ready script does a git pull | 16:35 |
thorst | is neutron_lib part of that? I wonder if we have an old neutron_lib laying around basically | 16:35 |
esberglu | It passes CI fine | 16:36 |
thorst | hmm | 16:36 |
thorst | let me try on my box | 16:36 |
esberglu | I've tried locally as well and it failed | 16:36 |
esberglu | https://review.openstack.org/#/c/442540/ | 16:36 |
esberglu | Change in question | 16:36 |
thorst | ahh, svenkat was just talking to me about proposing that change | 16:37 |
openstackgerrit | Merged openstack/nova-powervm master: In-tree backports: SSP UT https://review.openstack.org/437992 | 16:42 |
thorst | esberglu: I think it has to do with the requirements.txt | 16:43 |
thorst | https://github.com/openstack/neutron/blob/master/requirements.txt#L18 | 16:43 |
thorst | update us to 1.2.0 and I bet we're gold | 16:43 |
thorst | trying locally myself. | 16:44 |
thorst | another bonus that we'll have with the requirements bot... | 16:44 |
adreznec | thorst: I actually already have that in https://review.openstack.org/#/c/439805/2 | 16:46 |
thorst | right, but because they're not interconnected... | 16:46 |
adreznec | Have just been waiting for another +2 on it, but I can just merge it we want | 16:46 |
thorst | I'm not sure we can merge yours now | 16:46 |
thorst | because its broken without the change from esberglu | 16:46 |
thorst | won't it fail in the gate? | 16:46 |
adreznec | Well... it passed both jenkins and our CI | 16:47 |
esberglu | Yeah. Because the CI run went through before the neutron change broke us | 16:47 |
esberglu | Otherwise thorst would be correct | 16:47 |
thorst | so we can w+1 it and merge? | 16:48 |
esberglu | I'm still getting tox errors with 1.2.0 | 16:50 |
*** kjw3 has quit IRC | 16:51 | |
*** mdrabe has quit IRC | 16:55 | |
thorst | yeah...same here. | 16:56 |
adreznec | esberglu: thorst what kind of tox errors? | 16:59 |
thorst | its just not part of 1.2.0 | 17:00 |
thorst | looks like 6 days ago they released a 1.2.1 | 17:00 |
thorst | which I bet we need | 17:00 |
thorst | but it begs the question, how is neutron itself passing | 17:00 |
esberglu | Where are you seeing 1.2.1? | 17:00 |
thorst | random google. | 17:00 |
thorst | I'm trying it now | 17:00 |
*** mdrabe has joined #openstack-powervm | 17:02 | |
esberglu | context exists in 1.2.0 though | 17:02 |
esberglu | https://github.com/openstack/neutron-lib/blob/1.2.0/neutron_lib/context.py | 17:02 |
thorst | esberglu: actually, it looks like other things just need to be flipped over | 17:03 |
thorst | found it... | 17:05 |
thorst | https://github.com/openstack/networking-powervm/blob/master/networking_powervm/tests/unit/plugins/ibm/powervm/base.py#L49 | 17:05 |
thorst | so leave neutron-lib at 1.2.0, update that fixture to use the new path | 17:06 |
esberglu | Ah good find thanks | 17:07 |
openstackgerrit | Eric Berglund proposed openstack/networking-powervm master: Import context from neutron_lib https://review.openstack.org/442540 | 17:07 |
*** kjw3 has joined #openstack-powervm | 17:11 | |
*** jpasqualetto has quit IRC | 17:48 | |
*** jpasqualetto has joined #openstack-powervm | 17:55 | |
*** nbante has quit IRC | 18:24 | |
*** nbante has joined #openstack-powervm | 18:31 | |
thorst | esberglu: did that work? | 18:37 |
*** kjw3 has quit IRC | 18:40 | |
*** kjw3 has joined #openstack-powervm | 18:53 | |
esberglu | thorst: Yep. https://review.openstack.org/#/c/442540/ | 18:54 |
*** nbante has quit IRC | 18:58 | |
thorst | esberglu: you rock dude | 19:21 |
*** kjw3 has quit IRC | 19:22 | |
openstackgerrit | Merged openstack/networking-powervm master: Import context from neutron_lib https://review.openstack.org/442540 | 19:26 |
*** kjw3 has joined #openstack-powervm | 19:41 | |
*** kjw3 has quit IRC | 20:37 | |
*** kjw3 has joined #openstack-powervm | 21:22 | |
*** smatzek_ has quit IRC | 21:28 | |
efried | esberglu 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 |
thorst | have a link handy? I deleted them | 21:31 |
efried | thorst http://logs.openstack.org/88/391288/38/check/gate-nova-pep8-ubuntu-xenial/118aab6/ | 21:31 |
efried | deleted what? | 21:32 |
thorst | the e-mails saying we had a failure | 21:32 |
efried | It's jenkins, not our CI. | 21:32 |
efried | pep8 | 21:32 |
thorst | so I'm betting they updated pep8 version on the CI servers | 21:32 |
thorst | and we're just getting hit by it now | 21:32 |
thorst | I think we just need to update the code to get through the flake8, but I bet ocata and mitaka are now in trouble | 21:35 |
efried | thorst How do I get that new pep8 locally on my dev box? | 21:36 |
thorst | sudo pip install --upgrade pep8 | 21:36 |
thorst | I think | 21:36 |
efried | Well, that did something. Lemme see if it now hits the problem. | 21:36 |
efried | nope | 21:38 |
thorst | hmmm | 21:38 |
thorst | adreznec: any ideas? | 21:38 |
efried | thorst Hold the phone, it's using pep8 from the virtualenv. | 21:39 |
efried | Trying with that guy upgraded... | 21:39 |
adreznec | Was there actually a new pep8 version released that it's using? | 21:41 |
adreznec | The job logs say pep8==1.5.7 | 21:41 |
adreznec | But that was released in like 2014 | 21:41 |
thorst | hmm...I was just going off of a theory | 21:42 |
efried | adreznec 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 |
thorst | I didn't investigate said theory | 21:42 |
efried | Cause something is requiring 1.5.7 | 21:43 |
adreznec | efried: Right, I'm just saying that it's not using 1.7.0 in the gate | 21:43 |
efried | Like the global requirements project. | 21:43 |
adreznec | Yeah | 21:43 |
adreznec | requirements has pep8==1.5.7 | 21:43 |
efried | Yeah, and that failure log says it's using 1.5.7 too. | 21:44 |
efried | so where is the failure coming from? | 21:44 |
adreznec | and that hasn't changed in over a year | 21:44 |
thorst | do we some how inherit rules from nova...and did they remove rules? | 21:45 |
*** kjw3 has quit IRC | 21:46 | |
adreznec | was this an in-tree patch? or out of tree? | 21:46 |
thorst | oot | 21:46 |
thorst | but I thought our hacking rules imported from nova | 21:46 |
thorst | but this isn't a hacking issue...is it? | 21:46 |
adreznec | This is an in-tree patch | 21:47 |
adreznec | https://review.openstack.org/#/c/391288/ | 21:47 |
thorst | o derp | 21:47 |
thorst | then its just a fix needed to the patch :-D | 21:47 |
adreznec | Nova must have updated rules or something | 21:47 |
adreznec | efried: ^ | 21:47 |
efried | adreznec If it's the rules in tox.ini we're talking about, those haven't changed in two years. | 21:49 |
efried | oh. Except for "enable extensions". | 21:50 |
efried | Even there... | 21:51 |
efried | I don't see what's causing the problem. | 21:51 |
efried | gr | 21:51 |
efried | Aha. It's this guy: flake8-import-order==0.11 | 21:59 |
efried | It'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 IRC | 22:09 | |
*** thorst has quit IRC | 22:12 | |
*** smatzek_ has joined #openstack-powervm | 22:25 | |
*** tblakes has quit IRC | 22:33 | |
*** smatzek_ has quit IRC | 22:36 | |
*** esberglu has quit IRC | 22:49 | |
*** esberglu has joined #openstack-powervm | 22:49 | |
*** mdrabe has quit IRC | 22:50 | |
*** dwayne has quit IRC | 22:52 | |
*** seroyer has quit IRC | 22:52 | |
*** esberglu_ has joined #openstack-powervm | 22:53 | |
*** esberglu has quit IRC | 22:54 | |
*** esberglu_ has quit IRC | 22:57 | |
*** jpasqualetto has quit IRC | 23:08 | |
*** edmondsw has quit IRC | 23:16 | |
*** k0da has quit IRC | 23:26 | |
*** tjakobs has quit IRC | 23:33 | |
*** thorst has joined #openstack-powervm | 23:45 | |
*** thorst has quit IRC | 23:45 | |
*** edmondsw has joined #openstack-powervm | 23:52 | |
*** dwayne has joined #openstack-powervm | 23:55 | |
*** thorst has joined #openstack-powervm | 23:55 | |
*** edmondsw has quit IRC | 23:56 | |
*** esberglu has joined #openstack-powervm | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!