| *** edmondsw has joined #openstack-powervm | 00:17 | |
| *** edmondsw has quit IRC | 00:25 | |
| *** adreznec has joined #openstack-powervm | 05:21 | |
| *** edmondsw has joined #openstack-powervm | 05:41 | |
| *** edmondsw has quit IRC | 05:45 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 07:45 | |
| *** chas_ has joined #openstack-powervm | 07:56 | |
| *** AlexeyAbashkin has quit IRC | 10:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/519296 | 10:17 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/519121 | 10:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/519129 | 10:24 |
| *** AlexeyAbashkin has joined #openstack-powervm | 11:23 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: Get host-level cpu metrics from pypowervm metric cache https://review.openstack.org/512864 | 12:13 |
| openstackgerrit | Merged openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/519296 | 12:22 |
| openstackgerrit | Merged openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/519129 | 12:27 |
| openstackgerrit | Merged openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/519121 | 12:58 |
| *** edmondsw has joined #openstack-powervm | 13:19 | |
| *** edmondsw has quit IRC | 13:20 | |
| *** edmondsw has joined #openstack-powervm | 13:20 | |
| openstackgerrit | Sridhar Venkat proposed openstack/nova-powervm master: Tap devices are removed when host is rebooted https://review.openstack.org/519030 | 13:36 |
| efried | edmondsw you there? | 13:43 |
| edmondsw | efried yes... looking at ^ now | 13:43 |
| efried | Thanks, read my mind :) | 13:43 |
| edmondsw | is there a fix proposed for pypowervm? | 13:43 |
| edmondsw | efried ^ | 13:45 |
| edmondsw | comment says "This will be removed in future when pypowervm is fixed" | 13:46 |
| efried | edmondsw Yes, it's merged. Lemme get you the commit... | 13:46 |
| efried | edmondsw 6131 | 13:46 |
| edmondsw | efried so shouldn't we use that in nova-powervm master, and only use this hack in stable branches? | 13:47 |
| efried | edmondsw Long term, yes (maybe). But we needed this fix ASAP. | 13:47 |
| efried | We have to get the pypowervm fix into a release and through g-r, which seems to be taking a while lately. | 13:47 |
| *** edmondsw_ has joined #openstack-powervm | 13:53 | |
| efried | edmondsw We're also in negotiations to get 1.1.10 into stable u-c (but not g-r). Weighing the relative merits of that. | 13:53 |
| *** edmondsw has quit IRC | 13:53 | |
| edmondsw_ | efried negotiations with whom? | 13:53 |
| *** edmondsw_ is now known as edmondsw | 13:54 | |
| efried | edmondsw #openstack-requirements folks. | 13:54 |
| edmondsw | what is driving the request? | 13:54 |
| edmondsw | we haven't even released 1.1.10 yet have we? | 13:54 |
| efried | No. This bug is driving the request. | 13:55 |
| edmondsw | or did we just do that to pick this up? | 13:55 |
| efried | We haven't done anything yet. | 13:56 |
| efried | Just svenkat's emergency-fix-with-horrid-monkeypatching over the weekend. | 13:56 |
| efried | edmondsw Oh, yeah, cause there's also 6138 that's needed for this. | 13:57 |
| *** svenkat has joined #openstack-powervm | 14:02 | |
| * efried waves to svenkat | 14:05 | |
| svenkat | Hello! | 14:06 |
| efried | edmondsw Are you still reviewing the patch? | 14:07 |
| edmondsw | efried yes... and making a ton of comments | 14:07 |
| svenkat | which issue is being discussed here? | 14:07 |
| edmondsw | 519030 | 14:07 |
| svenkat | ohok. | 14:08 |
| efried | edmondsw Okay; but be aware that this is a deliberate hack for the sake of expediency. | 14:08 |
| edmondsw | efried yeah, I get that... the comments are wonky, though. I assume we can afford an hour to let svenkat get off the phone and fix them | 14:08 |
| edmondsw | efried as is I'm struggling to figure this out | 14:09 |
| efried | edmondsw When you're done redlining that guy, wanna push https://review.openstack.org/#/c/512864/ ? | 14:12 |
| svenkat | I am off.. let me look at your comments | 14:15 |
| svenkat | i do not see any new comments in https://review.openstack.org/#/c/519030/4 | 14:16 |
| edmondsw | efried done | 14:16 |
| edmondsw | svenkat haven't saved yet | 14:16 |
| svenkat | ok.. will wait | 14:17 |
| edmondsw | I'll walk over, easier to discuss that way | 14:17 |
| svenkat | sure | 14:17 |
| efried | edmondsw svenkat But I'd still like to see the executive summary in the review when you're done. | 14:18 |
| *** edmondsw has quit IRC | 14:20 | |
| *** edmondsw has joined #openstack-powervm | 14:20 | |
| *** edmondsw has quit IRC | 14:21 | |
| *** edmondsw has joined #openstack-powervm | 14:21 | |
| openstackgerrit | Merged openstack/nova-powervm master: Get host-level cpu metrics from pypowervm metric cache https://review.openstack.org/512864 | 14:28 |
| edmondsw | efried yep, one sec and I'll save these comments, which will give you that. Then I'll throw up a new ps that addresses them | 14:44 |
| efried | edmondsw Roger that. | 14:44 |
| edmondsw | efried one of the comments was a TODO to add caching... but this code is only hit when the agent has just restarted, so any cache would always be empty, no? | 14:55 |
| efried | edmondsw It's hit once per vif, though. We could certainly cache the VIOS wrapper(s), and perhaps the CNA lists too. | 14:56 |
| edmondsw | k that makes more sense | 14:56 |
| efried | The latter would help if >1 vif per VM. | 14:56 |
| *** miltonm has joined #openstack-powervm | 15:00 | |
| *** tjakobs has joined #openstack-powervm | 15:01 | |
| *** esberglu has joined #openstack-powervm | 15:07 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova-powervm master: Tap devices are removed when host is rebooted https://review.openstack.org/519030 | 15:44 |
| edmondsw | efried ^ | 15:45 |
| efried | edmondsw +2. Assuming we don't get thorst to look, I'll +W it when svenkat +1s. | 15:48 |
| edmondsw | efried well let's at least sort out this slack conversation first | 15:50 |
| svenkat | I added my +1 | 15:51 |
| edmondsw | efried svenkat ok, I think we're good here... I'll +2/+W | 15:56 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: Replace TestCase with NoDBTestCase https://review.openstack.org/518378 | 16:08 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova-powervm master: Tap devices are removed when host is rebooted https://review.openstack.org/519030 | 16:16 |
| openstackgerrit | Merged openstack/nova-powervm master: Tap devices are removed when host is rebooted https://review.openstack.org/519030 | 16:35 |
| *** AlexeyAbashkin has quit IRC | 18:18 | |
| *** chas_ has quit IRC | 18:28 | |
| *** chas_ has joined #openstack-powervm | 18:28 | |
| *** chas_ has quit IRC | 18:32 | |
| edmondsw | efried what do you think of my comment on 6107? | 18:56 |
| *** chas_ has joined #openstack-powervm | 19:30 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 19:37 | |
| *** AlexeyAbashkin has quit IRC | 19:41 | |
| *** chas_ has quit IRC | 19:47 | |
| *** chas_ has joined #openstack-powervm | 19:47 | |
| *** esberglu has quit IRC | 19:50 | |
| *** esberglu has joined #openstack-powervm | 19:50 | |
| *** chas_ has quit IRC | 19:51 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 19:59 | |
| *** AlexeyAbashkin has quit IRC | 20:06 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 20:06 | |
| efried | edmondsw I suggested the same at some point. Must not have been in a code comment, but - great minds, and all that. | 20:18 |
| edmondsw | efried alright, I'm fine with making it a future change if you are | 20:20 |
| efried | Yuh. Want to do a full sweep of pypowervm and get rid of all references to sys as parent. Then we can do the same in all community code. | 20:21 |
| efried | esberglu ^ | 20:21 |
| esberglu | efried: ack. Is there something similar to the etherpads for pypowervm that I'm not aware of? | 20:23 |
| efried | We had a temporary internal etherpad at some point, but we had knocked it all out. | 20:24 |
| efried | At this point we can track it via the LP bug. Sec... | 20:24 |
| efried | esberglu https://bugs.launchpad.net/pypowervm/+bug/1637668 | 20:24 |
| openstack | Launchpad bug 1637668 in pypowervm "Get rid of need for host uuid" [Wishlist,Confirmed] - Assigned to Eric Fried (efried) | 20:24 |
| esberglu | Cool tx | 20:25 |
| *** Alexey_Abashkin has joined #openstack-powervm | 20:27 | |
| efried | esberglu I made notes and assigned to you. | 20:27 |
| esberglu | ok | 20:27 |
| *** Alexey_Abashkin1 has joined #openstack-powervm | 20:28 | |
| *** AlexeyAbashkin has quit IRC | 20:30 | |
| *** Alexey_Abashkin1 is now known as AlexeyAbashkin | 20:30 | |
| *** Alexey_Abashkin has quit IRC | 20:31 | |
| efried | esberglu Has 6107 been through a community CI run yet? | 20:50 |
| efried | Just with base code to ensure it's backward compatible for now. Then we can rip the host_uuid out of the community code subsequently. | 20:51 |
| esberglu | efried: Nope will do | 20:51 |
| efried | thx | 20:51 |
| *** AlexeyAbashkin has quit IRC | 20:53 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 21:32 | |
| *** AlexeyAbashkin has quit IRC | 21:36 | |
| *** chas_ has joined #openstack-powervm | 21:37 | |
| *** svenkat has quit IRC | 21:56 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 22:26 | |
| *** AlexeyAbashkin has quit IRC | 22:31 | |
| edmondsw | esberglu are you going to propose something similar to https://review.openstack.org/#/c/518378 for the IT driver? | 22:33 |
| edmondsw | there are 3 places we are using TestCase instead of NoDBTestCase in the IT driver | 22:34 |
| esberglu | edmondsw: Yeah I have it on my personal TODO list, I'll add to the etherpad too | 22:34 |
| edmondsw | tx | 22:34 |
| *** esberglu_ has joined #openstack-powervm | 22:35 | |
| *** esberglu has quit IRC | 22:39 | |
| openstackgerrit | Merged openstack/nova-powervm master: Replace TestCase with NoDBTestCase https://review.openstack.org/518378 | 22:42 |
| *** esberglu_ has quit IRC | 22:43 | |
| *** esberglu has joined #openstack-powervm | 22:44 | |
| *** esberglu has quit IRC | 22:48 | |
| *** esberglu has joined #openstack-powervm | 22:57 | |
| *** tjakobs has quit IRC | 23:10 | |
| *** esberglu has quit IRC | 23:15 | |
| *** chas_ has quit IRC | 23:20 | |
| *** chas_ has joined #openstack-powervm | 23:21 | |
| *** chas_ has quit IRC | 23:25 | |
| *** chas_ has joined #openstack-powervm | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!