| *** esberglu has quit IRC | 00:06 | |
| *** esberglu has joined #openstack-powervm | 00:07 | |
| *** esberglu has quit IRC | 00:12 | |
| *** svenkat has quit IRC | 00:17 | |
| *** toan has quit IRC | 00:24 | |
| *** efried has quit IRC | 00:26 | |
| *** esberglu has joined #openstack-powervm | 00:29 | |
| *** toan has joined #openstack-powervm | 00:29 | |
| *** esberglu has quit IRC | 00:31 | |
| *** esberglu has joined #openstack-powervm | 00:31 | |
| *** efried has joined #openstack-powervm | 00:32 | |
| *** esberglu has quit IRC | 00:36 | |
| *** esberglu has joined #openstack-powervm | 01:02 | |
| *** thorst has joined #openstack-powervm | 01:04 | |
| *** thorst has quit IRC | 01:22 | |
| *** esberglu has quit IRC | 01:24 | |
| *** esberglu has joined #openstack-powervm | 01:30 | |
| *** esberglu has quit IRC | 01:31 | |
| *** esberglu has joined #openstack-powervm | 01:32 | |
| *** esberglu has quit IRC | 01:33 | |
| *** esberglu has joined #openstack-powervm | 01:36 | |
| *** esberglu has quit IRC | 01:39 | |
| *** tjakobs has joined #openstack-powervm | 01:53 | |
| *** thorst has joined #openstack-powervm | 01:58 | |
| *** thorst has quit IRC | 02:03 | |
| *** apearson has joined #openstack-powervm | 02:15 | |
| *** tjakobs has quit IRC | 02:20 | |
| *** esberglu has joined #openstack-powervm | 02:27 | |
| *** apearson has joined #openstack-powervm | 02:28 | |
| *** esberglu has quit IRC | 02:32 | |
| *** esberglu has joined #openstack-powervm | 03:21 | |
| *** esberglu has quit IRC | 03:32 | |
| *** esberglu has joined #openstack-powervm | 03:39 | |
| *** esberglu has quit IRC | 03:40 | |
| *** seroyer has joined #openstack-powervm | 04:10 | |
| *** tjakobs has joined #openstack-powervm | 04:21 | |
| *** seroyer has quit IRC | 04:21 | |
| *** mdrabe has quit IRC | 04:24 | |
| *** mdrabe has joined #openstack-powervm | 04:25 | |
| *** mdrabe has quit IRC | 04:33 | |
| *** tjakobs has quit IRC | 05:33 | |
| *** kylek3h_ has joined #openstack-powervm | 05:36 | |
| *** kylek3h has quit IRC | 05:38 | |
| *** k0da has joined #openstack-powervm | 06:17 | |
| *** k0da has quit IRC | 06:49 | |
| *** k0da has joined #openstack-powervm | 08:12 | |
| *** k0da has quit IRC | 09:05 | |
| *** esberglu has joined #openstack-powervm | 10:19 | |
| *** esberglu has quit IRC | 10:24 | |
| *** k0da has joined #openstack-powervm | 11:13 | |
| *** seroyer has joined #openstack-powervm | 11:27 | |
| *** thorst has joined #openstack-powervm | 11:47 | |
| *** seroyer has quit IRC | 11:56 | |
| *** seroyer has joined #openstack-powervm | 11:56 | |
| *** tblakes has joined #openstack-powervm | 12:00 | |
| *** edmondsw has joined #openstack-powervm | 12:07 | |
| *** esberglu has joined #openstack-powervm | 12:08 | |
| *** esberglu has quit IRC | 12:13 | |
| *** apearson has quit IRC | 12:15 | |
| *** k0da has quit IRC | 12:18 | |
| *** tblakes_ has joined #openstack-powervm | 12:30 | |
| *** tblakes_ has quit IRC | 12:30 | |
| *** tblakes has quit IRC | 12:30 | |
| *** tblakes has joined #openstack-powervm | 12:30 | |
| openstackgerrit | Zhuangqing Xiao proposed openstack/ceilometer-powervm: clean up oslo-incubator related stuff https://review.openstack.org/362705 | 12:34 |
|---|---|---|
| *** svenkat has joined #openstack-powervm | 12:37 | |
| *** apearson has joined #openstack-powervm | 12:51 | |
| *** esberglu_ has joined #openstack-powervm | 13:08 | |
| *** mdrabe has joined #openstack-powervm | 13:14 | |
| *** mdrabe_ has joined #openstack-powervm | 13:17 | |
| *** mdrabe_ has quit IRC | 13:18 | |
| *** mdrabe has quit IRC | 13:18 | |
| thorst | efried: FYI that ceilometer-powervm appears to be broken due to an upstream change...I'm proposing a fix | 13:30 |
| thorst | need to validate if it hits newton... | 13:30 |
| *** seroyer has quit IRC | 13:31 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/ceilometer-powervm: Support new InterfaceStats https://review.openstack.org/385480 | 13:43 |
| *** seroyer has joined #openstack-powervm | 13:47 | |
| *** esberglu_ has quit IRC | 13:51 | |
| *** k0da has joined #openstack-powervm | 13:51 | |
| thorst | efried: several bug reviews for you now :-) | 13:53 |
| thorst | adreznec: you too | 13:53 |
| thorst | they're small though | 13:53 |
| adreznec | ack | 13:53 |
| efried | thorst, I +2/W+1ed the tox.ini changes. The inspector I gave +2 - wouldn't mind another reviewer, but merge if you wish. | 14:00 |
| thorst | efried: thx. We apparently also have some hanging reviews from others that we're unaware of because they didn't add us as reviewers. | 14:01 |
| efried | Oh? | 14:02 |
| thorst | so we should check to see if we have any *-powervm reviews hanging out there. | 14:02 |
| thorst | at least I was made aware of one today. | 14:02 |
| thorst | which triggered this rabbit hole... | 14:02 |
| adreznec | Hmm | 14:03 |
| adreznec | Yeah, we should probably all start checking something like this | 14:03 |
| adreznec | https://review.openstack.org/#/q/project:%255Eopenstack/.*powervm+is:open | 14:03 |
| efried | thorst, I checked all three projects. | 14:04 |
| thorst | adreznec: https://review.openstack.org/#/c/267676/ | 14:04 |
| thorst | +2? | 14:04 |
| thorst | or.... | 14:04 |
| efried | Other than the one from Xiao, nothing recent. | 14:04 |
| thorst | efried: we may need to check more than recent. | 14:04 |
| efried | okay. I'll add *-powervm-core to whatever I see. | 14:04 |
| thorst | that'd be good. | 14:05 |
| thorst | bleh...but good | 14:05 |
| adreznec | Humm | 14:05 |
| adreznec | thorst: that's a weird patch, let me look at it a bit more... and make sure it still actually applies | 14:06 |
| efried | thorst, adreznec: Should kylek3h_ still be a nova-powervm-core? | 14:06 |
| efried | And Sam Matzek? | 14:07 |
| adreznec | Maybe if we leave them, they'll show up and take care of our code review backlog? :P | 14:09 |
| adreznec | But no, probably not I suppose | 14:09 |
| *** k0da has quit IRC | 14:18 | |
| thorst | adreznec: yeah, I guess I'm OK removing as cores for now. I don't think they'd mind | 14:19 |
| *** k0da has joined #openstack-powervm | 14:19 | |
| adreznec | thorst: efried Ok, cleaned up | 14:21 |
| thorst | adreznec: you do across all projects? | 14:21 |
| openstackgerrit | Merged openstack/networking-powervm: Merge tag '1.0.0' https://review.openstack.org/267676 | 14:21 |
| adreznec | They were only cores for nova-powervm | 14:21 |
| adreznec | networking/ceilometer-powervm lists look good | 14:21 |
| thorst | adreznec: rockin | 14:21 |
| *** burgerk has joined #openstack-powervm | 14:22 | |
| *** esberglu has joined #openstack-powervm | 14:30 | |
| *** k0da has quit IRC | 14:31 | |
| *** esberglu has quit IRC | 14:34 | |
| thorst | seroyer: would love a review from you on https://review.openstack.org/#/c/385480/ | 14:34 |
| *** esberglu has joined #openstack-powervm | 14:34 | |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Add instance to many log events https://review.openstack.org/385144 | 14:35 |
| *** tjakobs has joined #openstack-powervm | 14:44 | |
| *** k0da has joined #openstack-powervm | 15:11 | |
| *** mdrabe has joined #openstack-powervm | 15:20 | |
| *** esberglu has quit IRC | 15:30 | |
| *** esberglu has joined #openstack-powervm | 15:31 | |
| *** esberglu_ has joined #openstack-powervm | 15:32 | |
| *** kylek3h_ is now known as kylek3h | 15:33 | |
| *** esberglu has quit IRC | 15:35 | |
| esberglu_ | ls | 15:36 |
| *** esberglu_ has left #openstack-powervm | 15:36 | |
| *** esberglu_ has joined #openstack-powervm | 15:36 | |
| *** esberglu_ has quit IRC | 15:36 | |
| *** k0da has quit IRC | 15:38 | |
| *** mdrabe has quit IRC | 15:49 | |
| *** mdrabe has joined #openstack-powervm | 15:49 | |
| seroyer | thorst, -1. I’d like to talk about the assumption that the drops are likely tx. I don’t think that’s likely. | 16:03 |
| thorst | seroyer: that's why I wanted to ask you :-D | 16:04 |
| thorst | grabbing a burrito quick | 16:04 |
| *** thorst is now known as thorst_afk | 16:04 | |
| seroyer | I’m not sure where you get your data, but the hypervisor reports drops on RX only. Drops on TX tend to be way less interesting because of broadcast/multicast. | 16:05 |
| seroyer | And, drops on RX are only reported due to lack of a receive buffer to put it in. | 16:06 |
| thorst_afk | well, the pcm data only gives me drops. I can't separate them out :-( | 16:09 |
| thorst_afk | ok, really afk for now | 16:09 |
| thorst_afk | https://github.com/powervm/pypowervm/blob/master/pypowervm/wrappers/pcm/phyp.py#L228 | 16:10 |
| seroyer | Assuming that data comes from where I think it comes from, drops reported through there are always for RX, not TX. | 16:11 |
| *** esberglu has joined #openstack-powervm | 16:24 | |
| *** esberglu has quit IRC | 16:42 | |
| thorst_afk | seroyer: cool...did you add a comment? I'm trying to finish the LPM thing then I'll fix that | 17:14 |
| *** thorst_afk is now known as thorst | 17:14 | |
| seroyer | thorst: I did. | 17:17 |
| thorst | seroyer: as always, you rock | 17:36 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/nova-powervm: Pre-allocate trunk adapter on target host https://review.openstack.org/350231 | 17:57 |
| openstackgerrit | Drew Thorstensen (thorst) proposed openstack/ceilometer-powervm: Support new InterfaceStats https://review.openstack.org/385480 | 18:00 |
| openstackgerrit | Eric Fried proposed openstack/networking-powervm: Refactor, consolidate, and clean up agents https://review.openstack.org/378000 | 18:24 |
| efried | thorst, I added a Closes-Bug to ^^. We about ready to get that sucker merged? | 18:24 |
| thorst | efried: let me re-review...while I'm at it, can you re review this sucker? https://review.openstack.org/#/c/350231/ | 18:34 |
| efried | Yes, on it now. | 18:34 |
| thorst | your changes (small) looked good. I gave you a +2 | 18:35 |
| thorst | will let you W+1 it | 18:35 |
| thorst | I think it'll also increase the reliability of it a lot. I've seen the old model go down too often...saw that on two clouds now | 18:35 |
| openstackgerrit | Merged openstack/ceilometer-powervm: Support new InterfaceStats https://review.openstack.org/385480 | 18:38 |
| openstackgerrit | Merged openstack/ceilometer-powervm: Remove discover from test-requirements https://review.openstack.org/345208 | 18:40 |
| efried | seroyer, thorst, this may be a stupid question, but... for SR-IOV cards to be usable for vNIC-ness, do they need to be assigned to the NovaLink partition, or totally unassigned? | 18:45 |
| seroyer | efried: SR-IOV card needs to be unassigned to use it for SR-IOV with or without vNIC. | 18:46 |
| efried | seroyer, thanks. thorst, wonder if that should be included in our docs somewhere - or if it's "obvious" ;-) | 18:47 |
| efried | thorst, gave https://review.openstack.org/#/c/350231/ a +1. Will promote to +2 once someone can validate the actual function instead of just nitpicking your spelling. | 18:49 |
| *** k0da has joined #openstack-powervm | 18:52 | |
| *** adi___ has quit IRC | 18:58 | |
| openstackgerrit | Merged openstack/networking-powervm: Refactor, consolidate, and clean up agents https://review.openstack.org/378000 | 18:59 |
| *** adi___ has joined #openstack-powervm | 18:59 | |
| thorst | efried: when you say validate, you mean live testing (which I've done) or validation of concepts? If second, it should be seroyer | 19:26 |
| seroyer | FYI: I do not have +2 authority. | 19:27 |
| thorst | seroyer: I think he says he'll +2 if you +1 | 19:28 |
| efried | thorst, seroyer, yeah. | 19:30 |
| thorst | efried: was this cherry picked, but hand fixed? | 19:31 |
| thorst | https://review.openstack.org/#/c/385592/ | 19:31 |
| thorst | hand fixed ==> hand merged? | 19:31 |
| efried | I didn't do anything by hand. | 19:31 |
| efried | should I need to? | 19:32 |
| efried | I think the branches were the same up to that point. | 19:32 |
| efried | Other than maybe the logging business. | 19:32 |
| thorst | o, I thought if you clicked the cherry pick button it would lead to a different patch set or something | 19:32 |
| efried | The Change-Id is the same, and its brother shows up under Cherry-Picks in gerrit. | 19:33 |
| thorst | k | 19:33 |
| efried | I also thought the commit message got an extra "cherry picked from" bit | 19:33 |
| efried | But maybe that only happens in nova-powervm - see https://review.openstack.org/#/c/385593/ | 19:34 |
| thorst | efried: i didn't see that bit, which is why I asked | 19:34 |
| thorst | no, it should always happen if you hit that cherry-pick button | 19:34 |
| efried | I did those both the same way, so it must be project-specific? | 19:34 |
| efried | Yeah, mysterious. | 19:34 |
| thorst | efried: It looks good, but lets make sure the nova change to stable/newton is merged first | 19:40 |
| efried | yes, definitely | 19:40 |
| efried | looks like it just went. | 19:41 |
| *** k0da has quit IRC | 20:40 | |
| *** thorst has quit IRC | 21:05 | |
| *** svenkat has quit IRC | 21:07 | |
| *** burgerk_ has joined #openstack-powervm | 21:08 | |
| *** esberglu has joined #openstack-powervm | 21:09 | |
| *** thorst has joined #openstack-powervm | 21:11 | |
| *** apearson has quit IRC | 21:16 | |
| *** thorst has quit IRC | 21:16 | |
| *** burgerk__ has joined #openstack-powervm | 21:18 | |
| *** burgerk has quit IRC | 21:19 | |
| *** burgerk_ has quit IRC | 21:20 | |
| *** adreznec has quit IRC | 21:23 | |
| *** adreznec has joined #openstack-powervm | 21:28 | |
| *** kriskend has joined #openstack-powervm | 21:30 | |
| *** tblakes_ has joined #openstack-powervm | 21:32 | |
| *** tblakes has quit IRC | 21:35 | |
| *** tblakes_ is now known as tblakes | 21:35 | |
| *** edmondsw has quit IRC | 21:41 | |
| *** tblakes has quit IRC | 21:42 | |
| *** thorst has joined #openstack-powervm | 21:45 | |
| *** mdrabe has quit IRC | 21:49 | |
| *** thorst has quit IRC | 21:49 | |
| *** mdrabe has joined #openstack-powervm | 21:49 | |
| *** kriskend has quit IRC | 21:49 | |
| *** burgerk__ has quit IRC | 21:52 | |
| *** thorst has joined #openstack-powervm | 21:55 | |
| *** thorst has quit IRC | 22:00 | |
| *** mdrabe has quit IRC | 22:01 | |
| *** edmondsw has joined #openstack-powervm | 22:16 | |
| *** smatzek has joined #openstack-powervm | 22:27 | |
| *** esberglu has quit IRC | 22:31 | |
| *** seroyer has quit IRC | 22:31 | |
| *** esberglu has joined #openstack-powervm | 22:35 | |
| *** esberglu has quit IRC | 22:35 | |
| *** esberglu has joined #openstack-powervm | 22:43 | |
| *** esberglu has quit IRC | 22:45 | |
| *** smatzek has quit IRC | 22:49 | |
| *** tjakobs has quit IRC | 22:52 | |
| openstackgerrit | Merged openstack/nova-powervm: Pre-allocate trunk adapter on target host https://review.openstack.org/350231 | 22:54 |
| *** edmondsw has quit IRC | 23:08 | |
| *** esberglu has joined #openstack-powervm | 23:36 | |
| *** seroyer has joined #openstack-powervm | 23:54 | |
| *** esberglu has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!