Monday, 2018-01-15

*** k0da has quit IRC00:03
*** edmondsw has joined #openstack-powervm00:24
*** edmondsw has quit IRC00:28
*** edmondsw has joined #openstack-powervm04:00
*** edmondsw has quit IRC04:05
-openstackstatus- NOTICE: The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck.04:49
*** ChanServ changes topic to "The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck."04:49
*** ChanServ changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack."04:53
*** edmondsw has joined #openstack-powervm05:48
*** edmondsw has quit IRC05:53
*** edmondsw has joined #openstack-powervm07:36
*** edmondsw has quit IRC07:41
*** AlexeyAbashkin has joined #openstack-powervm07:54
*** k0da has joined #openstack-powervm08:48
*** k0da has quit IRC09:06
*** edmondsw has joined #openstack-powervm09:25
*** k0da has joined #openstack-powervm09:25
*** edmondsw has quit IRC09:29
*** k0da has quit IRC09:32
*** k0da has joined #openstack-powervm10:54
*** edmondsw has joined #openstack-powervm11:13
*** edmondsw has quit IRC11:17
*** edmondsw has joined #openstack-powervm12:55
*** openstackgerrit has quit IRC13:18
*** esberglu has joined #openstack-powervm13:40
*** esberglu has quit IRC13:48
*** apearson__ has joined #openstack-powervm14:02
*** esberglu has joined #openstack-powervm14:07
*** csky has joined #openstack-powervm14:54
*** hkran has joined #openstack-powervm14:54
*** hkran has quit IRC14:56
*** hkran has joined #openstack-powervm14:57
*** csky_ has joined #openstack-powervm14:57
*** hkran has left #openstack-powervm14:57
*** csky has quit IRC15:00
*** hkran has joined #openstack-powervm15:10
*** chhavi__ has joined #openstack-powervm15:13
*** chhagarw has joined #openstack-powervm15:13
hkranhi, I got the image  Ubuntu_16-04-01_2g.img.gz  from the box for nova-powervm, and setup the testing environment for nova-powervm, but i do not know how to  use the image... how to upload the image into the glance service? thanks15:15
efriedesberglu ^ you got that command handy?15:18
esbergluhkran: Try this15:19
efriedhkran It might look something like: glance --os-image-api-version 2 image-create --file /path/to/Ubuntu_16-04-01_2g.img.gz --progress --disk-format=raw --container-format=bare --property name=ubuntu16 --property visibility=public15:19
efriedoh, but whatever esberglu says is gonna be better.15:19
esbergluopenstack image create --disk-format raw --container-format bare 2gUbuntu1604 --file <img_loc>15:19
efried++15:20
hkran i am going to try it , thank you very much15:21
hkrananother question, I am using kolla to deploy openstack controll part, but have to configure nova-compute manually on novalink part(it seems neutron-openvswitch-agent in docker working fine),is it possible to put the nova-powervm in a docker container?  is it possible to make kolla support nova-powervm?15:25
efriedapearson__ you around?15:27
efriedhkran I think this is something we're looking into, but don't support right now.  If you can get it working, we'd be pretty interested to know about it :)15:28
efriedesberglu FYI, gibi left a review on the ovs patch.  In my experience, gibi is about as sharp as they come, so his comments are likely on point.15:30
esbergluefried: ack15:31
edmondswhkran I don't know that we've done anything with kolla to date, but as efried said we would definitely be interested in anything you figure out there15:39
hkranthank for all your info, anyway, kolla is  something cool... just wish kolla can support power better, i suffer so much when trying to build docker image on power....15:44
edmondswhkran yep, totally understand that. Hope we can improve that over time15:44
edmondswesberglu re: gibi's comments, look mostly like questions that may or may not really require changes. E.g. in norm_mac I had assumed that line was there in case norm_mac was called with something that had already been normalized. We would need to make sure that can't happen before removing that line.15:48
edmondswthough if the line is needed, maybe add a comment explaining why15:48
esbergluefried: edmondsw: https://review.openstack.org/#/c/422512/36/nova/virt/powervm/media.py@18116:12
esbergluI'm just gonna rip that comment out unless you have objections16:13
efriedshrug16:13
*** k0da has quit IRC16:18
*** AlexeyAbashkin has quit IRC16:39
*** AlexeyAbashkin has joined #openstack-powervm16:39
edmondswesberglu I would move the comment to line 199, where it's actually used16:41
edmondswe.g. # Insert FF:FE per Appendix A of RFC 429116:44
edmondswactually, the bit flip after that also comes from that appendix...16:46
*** AlexeyAbashkin has quit IRC16:47
esbergluefried: edmondsw: https://review.openstack.org/#/c/422512/36/nova/virt/powervm/tasks/network.py@14516:54
esbergluFrom what I can tell is_neutron() would be true in both the OVS and SEA cases unless I'm missing something16:54
edmondswI think that's answered on line 13916:54
edmondswI think is_neutron would be true if you're using neutron, and false if you're using nova-network16:55
esbergluedmondsw: I guess this goes back to whether or not this works with nova networking, which I didn't test since I thought it was deprecated16:55
esbergluSince it is deprecated16:55
edmondswthe code appears to be structured in a way that nova-network could work, but we'd have to ask thorst whether that was ever tested16:56
edmondswI don't know that I would spend time testing it now unless pressed16:57
edmondswI would just answer that comment with something like "See comments line 137-141"16:57
*** chhavi__ has quit IRC17:03
*** chhagarw has quit IRC17:03
*** hkran has quit IRC17:10
edmondswefried esberglu it looks like networkx was added to pypowervm's requirements.txt just recently with 1.1.9... any idea why? I don't see anything using it17:13
esbergluedmondsw: https://github.com/powervm/pypowervm/commit/a7ccb6eb01217:14
efriedLooks like it was added explicitly to cap the version because 2.0 broke us.17:14
efriedSo basically somebody needs to go through and make the fixes necessary to make 2.0 work, and uncap it.17:15
edmondswtaskflow caps it: https://github.com/openstack/taskflow/blob/master/requirements.txt#L2317:15
edmondswso we shouldn't need to do it for them17:15
efriedPresumably they're uncapping as part of this effort.17:16
edmondswoh, they didn't always cap it in taskflow like they should have... e.g. https://github.com/openstack/taskflow/blob/2.7.0/requirements.txt#L2317:17
efriedright, that was probably why we had to do it.17:17
efriedAnd presumably taskflow is going to have to uncap now as part of this effort as well.17:17
efriedBut I'm not sure if other projects are using taskflow.  If they're not, they might not uncap it.  Which would be a problem for us.17:17
efriedCan we ask harlowja in openstack-oslo?17:18
edmondswso I think the answer here is that we can remove this if/when openstack bumps the req for taskflow to >=2.16.0 (where taskflow added the cap)17:18
efriedneat17:19
*** efried is now known as efried_rollin17:19
edmondswefried I'll add a comment in the pypowervm bug17:21
edmondswand I'll propose the taskflow bump17:37
edmondswhttps://review.openstack.org/53375118:02
-openstackstatus- NOTICE: Zuul has been restarted and has lost queue contents; changes in progress will need to be rechecked.18:23
*** csky_ has quit IRC18:38
*** csky has joined #openstack-powervm18:38
*** csky has left #openstack-powervm18:39
esbergluefried_rollin: edmondsw: Respun OVS based on the comments from gibi. Also rebased SEA onto it and hit some nits from the previous version18:49
esbergluReady for re-review18:49
edmondswesberglu ack18:49
*** AlexeyAbashkin has joined #openstack-powervm18:50
*** AlexeyAbashkin has quit IRC18:55
edmondswesberglu I posted 3 comments on ovs19:00
esbergluedmondsw: Fixed and patch is up19:13
edmondswesberglu ack19:17
edmondswesberglu one comment on SEA19:29
esbergluedmondsw: Hmm that looks familiar lol. Done19:31
edmondsw:)19:31
edmondswefried we've got the OVS and SEA patches fixed up and I've +1d both now... once you approve, ping gibi and let him know?19:32
*** k0da has joined #openstack-powervm19:44
*** AlexeyAbashkin has joined #openstack-powervm19:51
*** efried_rollin is now known as efried19:55
*** AlexeyAbashkin has quit IRC19:55
edmondswefried esberglu I just noticed that nova have pypowervm listed in its requirements.txt but not taskflow, which the IT driver also requires19:58
edmondswshould I propose that fix to requirements.txt, or should we pull both that and pypowervm out into setup.cfg extras as optional (since you don't have to use the powervm driver)?19:58
edmondswI'm guessing the former... suits us better to know it's always there, and nova is presumably ok with that since they let pypowervm in19:59
efriedis it in test-reqs?20:05
edmondswefried nope, not there either20:10
edmondswI'll propose adding it to requirements.txt unless you have any objections20:11
edmondswgot the patch ready to submit20:11
efriedhmph, that is odd.  Guess we've just been getting lucky20:11
*** openstackgerrit has joined #openstack-powervm20:13
openstackgerritEric Berglund proposed openstack/nova-powervm master: Cleanup unit conversions  https://review.openstack.org/53377320:13
edmondswre: taskflow  ->  https://review.openstack.org/#/c/533779/20:29
*** AlexeyAbashkin has joined #openstack-powervm20:50
*** AlexeyAbashkin has quit IRC20:54
*** apearson__ has quit IRC20:59
*** apearson__ has joined #openstack-powervm21:01
openstackgerritMerged openstack/nova-powervm master: Cleanup unit conversions  https://review.openstack.org/53377321:14
*** apearson__ has quit IRC21:18
*** apearson has joined #openstack-powervm21:29
*** AlexeyAbashkin has joined #openstack-powervm21:50
*** AlexeyAbashkin has quit IRC21:54
*** csky has joined #openstack-powervm22:45
*** csky has quit IRC22:49
*** csky has joined #openstack-powervm22:58
*** chhavi__ has joined #openstack-powervm23:00
*** chhagarw has joined #openstack-powervm23:00
*** chhavi__ has quit IRC23:04
*** chhagarw has quit IRC23:04
openstackgerritEric Berglund proposed openstack/nova-powervm master: Replace double spaces with single space  https://review.openstack.org/53381623:05
*** esberglu has quit IRC23:10
*** esberglu has joined #openstack-powervm23:11
efriedreally23:13
*** esberglu has quit IRC23:16
*** apearson has quit IRC23:20
*** esberglu has joined #openstack-powervm23:27
*** esberglu has quit IRC23:28
*** apearson has joined #openstack-powervm23:29
*** apearson has quit IRC23:38
*** k0da has quit IRC23:44

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