Friday, 2016-11-11

*** chas_ has joined #openstack-powervm00:05
*** chas_ has quit IRC00:09
*** chas_ has joined #openstack-powervm00:24
*** chas_ has quit IRC00:28
*** jwcroppe has quit IRC00:44
*** jwcroppe has joined #openstack-powervm00:44
*** svenkat has joined #openstack-powervm00:59
*** jwcroppe has quit IRC01:13
*** jwcroppe has joined #openstack-powervm01:15
*** edmondsw has quit IRC01:21
*** smatzek has joined #openstack-powervm01:22
*** chas_ has joined #openstack-powervm01:24
*** chas_ has quit IRC01:29
openstackgerritSridhar Venkat proposed openstack/networking-powervm: ProvisionRequest does not distinguish event source  https://review.openstack.org/39646702:11
*** svenkat has quit IRC02:17
*** chas_ has joined #openstack-powervm02:25
*** chas_ has quit IRC02:30
*** tjakobs has joined #openstack-powervm02:39
*** tjakobs has quit IRC02:43
*** esberglu has joined #openstack-powervm03:07
*** smatzek has quit IRC03:09
*** esberglu has quit IRC03:11
*** tjakobs has joined #openstack-powervm04:03
*** jwcroppe has quit IRC04:13
*** jwcroppe has joined #openstack-powervm04:14
*** apearson has joined #openstack-powervm04:15
*** jwcroppe has quit IRC04:18
*** chas_ has joined #openstack-powervm04:26
*** chas_ has quit IRC04:31
*** jwcroppe has joined #openstack-powervm04:36
*** esberglu has joined #openstack-powervm04:56
*** esberglu has quit IRC05:00
*** tjakobs has quit IRC05:24
*** chas_ has joined #openstack-powervm05:28
*** kotra03 has joined #openstack-powervm05:28
*** chas_ has quit IRC05:32
*** tjakobs has joined #openstack-powervm05:39
*** tjakobs has quit IRC05:43
*** thorst_ has quit IRC05:49
*** thorst_ has joined #openstack-powervm05:50
*** thorst_ has quit IRC05:59
*** apearson has quit IRC06:04
*** apearson has joined #openstack-powervm06:04
*** apearson has quit IRC06:05
*** apearson has joined #openstack-powervm06:05
*** apearson has joined #openstack-powervm06:06
*** apearson has quit IRC06:07
*** apearson has joined #openstack-powervm06:07
*** apearson has quit IRC06:07
*** apearson has joined #openstack-powervm06:08
*** apearson has quit IRC06:08
*** apearson has joined #openstack-powervm06:08
*** apearson has quit IRC06:09
*** apearson has joined #openstack-powervm06:09
*** apearson has quit IRC06:10
*** apearson has joined #openstack-powervm06:10
*** apearson has quit IRC06:11
*** apearson has joined #openstack-powervm06:11
*** apearson has quit IRC06:11
*** apearson has joined #openstack-powervm06:12
*** apearson has quit IRC06:12
*** apearson has joined #openstack-powervm06:12
*** apearson has quit IRC06:13
*** esberglu has joined #openstack-powervm06:45
*** esberglu has quit IRC06:49
*** jwcroppe has quit IRC07:11
*** jwcroppe has joined #openstack-powervm07:12
*** jwcroppe has quit IRC07:16
*** thorst_ has joined #openstack-powervm08:01
*** thorst_ has quit IRC08:08
*** esberglu has joined #openstack-powervm08:34
*** esberglu has quit IRC08:38
*** thorst_ has joined #openstack-powervm09:07
*** adreznec has quit IRC09:09
*** adreznec has joined #openstack-powervm09:11
*** thorst_ has quit IRC09:14
*** kylek3h has quit IRC09:16
*** kylek3h has joined #openstack-powervm09:23
*** esberglu has joined #openstack-powervm09:34
*** esberglu has quit IRC09:38
*** openstackgerrit has quit IRC09:48
*** openstackgerrit has joined #openstack-powervm09:49
*** thorst_ has joined #openstack-powervm10:12
*** thorst_ has quit IRC10:18
*** esberglu has joined #openstack-powervm10:28
*** esberglu has quit IRC10:33
*** thorst_ has joined #openstack-powervm11:19
*** thorst_ has quit IRC11:24
*** esberglu has joined #openstack-powervm11:31
*** esberglu has quit IRC11:36
*** jwcroppe has joined #openstack-powervm11:42
*** jwcroppe has quit IRC11:47
*** smatzek has joined #openstack-powervm11:52
-openstackstatus- NOTICE: Our OpenStack CI systems are stuck and no new jobs are submitted. Please do not recheck - and do not approve changes until this is fixed.12:01
*** thorst has joined #openstack-powervm12:20
*** thorst has quit IRC12:29
*** chas_ has joined #openstack-powervm12:38
*** chas_ has quit IRC12:43
*** chas_ has joined #openstack-powervm12:49
-openstackstatus- NOTICE: Our OpenStack CI system is coming back online again. Thanks for your patience.12:59
*** thorst has joined #openstack-powervm13:01
*** thorst_ has joined #openstack-powervm13:03
*** thorst has quit IRC13:05
*** thorst_ is now known as thorst13:12
*** svenkat has joined #openstack-powervm13:16
*** edmondsw has joined #openstack-powervm13:16
*** jwcroppe has joined #openstack-powervm13:29
*** jwcroppe has quit IRC13:29
*** jwcroppe has joined #openstack-powervm13:30
*** seroyer has joined #openstack-powervm13:34
*** jwcroppe has quit IRC13:34
*** tblakes has joined #openstack-powervm14:11
*** esberglu has joined #openstack-powervm14:14
*** smatzek has quit IRC14:20
*** jwcroppe has joined #openstack-powervm14:28
*** seroyer has quit IRC14:30
*** seroyer has joined #openstack-powervm14:40
openstackgerritSridhar Venkat proposed openstack/networking-powervm: ProvisionRequest does not distinguish event source  https://review.openstack.org/39646714:41
thorstsvenkat: did you get a chance to connect with efried on the ones above?14:43
svenkatyes… i juste uploaded second patch set based on Eric’s comments14:44
thorstawesome14:44
*** apearson has joined #openstack-powervm14:45
openstackgerritMerged openstack/ceilometer-powervm: Update the init of the collector to take in a conf  https://review.openstack.org/39591814:47
openstackgerritMerged openstack/nova-powervm: Fix rollback op's search call  https://review.openstack.org/39380114:47
*** kriskend has joined #openstack-powervm14:54
*** AlexeyAbashkin has quit IRC15:11
*** smatzek has joined #openstack-powervm15:14
*** apearson has quit IRC15:31
thorstesberglu: I think we should probably rev the version of nova-powervm to 3.0.1.  Thoughts?15:40
thorstadreznec efried: ^^15:40
efriedThe master branch?15:41
adreznecProbably stable/newton15:41
esbergluNo the stable/newton15:41
thorststable/newton.15:41
adreznecThe master branch would be 4.0.0-dev now15:41
adreznecI guess that seems fine to me15:41
*** seroyer has quit IRC15:42
*** tjakobs has joined #openstack-powervm15:44
esbergluthorst: adreznec: Is that a go?15:46
thorstk.  I'm seeing some patch set changes from svenkat that I think will warrant nova-powervm to later jump to 3.0.2 and networking-powervm to go to 3.0.115:46
thorstyeah, I think that's a go for the 3.0.1 nova-powervm stable/mitaka version update15:46
adreznecnewton15:47
adreznecBut yes15:47
adreznecFire away15:47
thorstman...I screw that up ALL THE TIME15:48
adreznecthorst: just really, really liked mitaka15:48
thorstguess so15:49
esbergluthorst: Donw15:50
esbergluDone15:50
thorstu rock15:50
thorstefried: If possible, i think you'd take interest in my comments here: https://review.openstack.org/#/c/396467/215:51
thorstI talked to svenkat about it too15:51
svenkatthorst: I am working to get event source from nova…15:52
thorstsvenkat: I put the exact LoC where it happens15:52
svenkatsure. thanks.15:52
thorstthe detail is where you'd add something.  maybe a 'type='15:53
thorstand good find/debug on that issue15:53
efriedthorst, that rebase is done - take a look.15:53
thorstlooking now15:53
efriedthorst, your comment on svenkat's change set is the same one I made ;-)15:54
efriedgreat minds, and all that.15:54
*** seroyer has joined #openstack-powervm15:55
*** smatzek has quit IRC15:57
*** smatzek has joined #openstack-powervm15:58
thorstefried: excellent15:58
thorstgetting used to these pok addresses is fun.15:59
thorst...15:59
*** apearson has joined #openstack-powervm16:02
thorstefried: I'm good with that change16:08
efriedthorst, so do we get esberglu to yank it into the CI env at this point?16:08
thorstI think he already has.16:09
thorstand he's fighting devstack issues now16:09
thorstesberglu: can you confirm?16:09
*** efried1 has joined #openstack-powervm16:20
*** efried has quit IRC16:22
*** kotra03 has quit IRC16:25
*** k0da has joined #openstack-powervm16:30
esbergluthorst: Are we talking about the remote patch?16:36
efried1thorst, you good with 4487 yet?16:38
efried1If PM comes after me, I'm deflecting 'em to you.16:39
esbergluthorst: And yeah I'm fighting devstack16:40
esbergluDevstack is basically my nemesis at this point16:41
adreznecUndercloud or overcloud?16:44
esbergluAIO devstack master16:45
*** k0da has quit IRC16:51
*** seroyer has quit IRC17:02
efried1adreznec, any idea why this keeps failing to verify?  https://review.openstack.org/#/c/391288/17:10
efried1I looked into the errors and they're wildly unrelated even to compute.  Like, in the cinder driver and stuff.17:10
*** mdrabe has joined #openstack-powervm17:11
adreznecefried1: Let me take a look here17:13
thorstefried1: yeah, good with 448717:17
thorstesberglu: Anything we need to help with for devstack/17:17
efried1thorst, am I ready to merge stuff at this point?  Infra all set up for it?17:18
thorstefried1: unclear17:18
thorstthat's a dom q17:18
efried1k, asking dclain.17:18
efried1thorst, is there any need for APINotLocal exception anymore?  Any code paths we want to block thusly?17:20
efried1thorst, also, 4475, I put up a patch showing how it would work with hand-parsing to avoid circular imports.  Not urgent, but on your list somewhere.17:22
thorstefried1: lets not pull APINotLocal yet...17:23
thorstthat would require a full on scour17:23
thorst(cause we may need it in places we missed)17:23
efried1thorst, huh?  It was *just* introduced17:23
thorstwill look at 447517:24
thorstefried1: o, well, cut it17:24
efried1The only place it lives right now is in the UT.  If we need to reintroduce it, we can.  But I'd rather do that than leave it hanging around unused and forget to purge it.17:24
thorstmeh, we can cut it17:25
thorstI like 4475 more17:25
efried1REALLY?17:27
efried1okay.17:27
thorstlol - personal preference17:28
*** seroyer has joined #openstack-powervm17:46
*** smatzek has quit IRC17:51
*** kriskend has quit IRC17:51
*** esberglu has quit IRC17:51
*** tjakobs has quit IRC17:51
*** smatzek has joined #openstack-powervm17:52
*** esberglu has joined #openstack-powervm17:52
*** kriskend has joined #openstack-powervm17:52
*** tjakobs has joined #openstack-powervm17:52
adreznecSo efried1 I looked at that patch. The intel NFV CI is just hitting a timeout at 3 hours... not sure if that's a known problem or not18:06
adreznecThe xenserver logs were gone18:07
adreznecI retriggered that to look18:07
efried1adreznec, been retrying it for a week-plus18:07
adreznecIt wasn't getting retriggered in any of your other retries18:07
efried1Yeah, the xenserver logs are never there by the time I go look.18:07
efried1Oh.18:07
adreznecIt was looking specifically for 'xenserver: recheck' I guess18:07
efried1'recheck' doesn't hit everything?18:07
efried1Booooooh gus.18:07
adreznecGuess not for that one18:07
*** chas_ has quit IRC18:30
*** chas_ has joined #openstack-powervm18:30
esbergluthorst: adreznec: I found the source of the devstack issue. Some deprecated exceptions were retired. I think we just need to change the import from18:32
esberglufrom neutron.common import exceptions18:32
esbergluto18:32
esberglufrom neutron_lib import exceptions18:32
thorstesberglu: yeah, I noticed that in a change from svenkat18:33
*** smatzek_ has joined #openstack-powervm18:33
thorstI in fact noted we should do that in a separate change set18:33
thorstdid you want to fire that off and I can +2 it?18:33
esbergluYeah. I know which file its coming from but I will check and see if we are using the incorrect import anywhere else18:33
svenkatthorst: i will take it off of my change set.18:34
*** chas_ has quit IRC18:34
*** smatzek has quit IRC18:36
openstackgerritEric Berglund proposed openstack/networking-powervm: Import exceptions from neutron_lib  https://review.openstack.org/39672818:38
esbergluthorst: ^^18:39
thorstesberglu: thx18:43
* thorst that feeling when the nova cores tell others to 'do what nova-powervm does'18:43
esbergluefried1: That remote change is good to go right?18:43
efried1esberglu: Well, no.18:44
efried1Hold on, let me get a new patch set up.18:44
thorstefried1 esberglu: good for CI, not yet ready to be in pypowervm18:44
efried1It'll have UT incomplete, but it'll be missing a couple of bugs you would introduce otherwise...18:44
thorstfinishing UT I think18:44
esbergluYeah thats what I meant18:44
efried1thorst, no, there's a bug in the vopt path.18:44
thorstahh, then devstack would fail there18:45
esbergluSo I need to hold off for now?18:45
efried1esberglu, no, I just posted the update.  thorst, you want to pull in the 'ensure close' bit too, right?18:46
thorstefried1: yep.18:46
efried1So esberglu, pull 4504 and you'll get both.18:46
esbergluCool. I will redeploy with that and hopefully by then the networking-powervm fix will be in and devstack will work18:46
efried1This is in release branch, btw esberglu.18:47
efried1not develop18:47
efried1in pypowervm18:47
openstackgerritMerged openstack/networking-powervm: Import exceptions from neutron_lib  https://review.openstack.org/39672818:55
thorstesberglu: you're merged  :-)18:55
openstackgerritSridhar Venkat proposed openstack/nova-powervm: ProvisionRequest does not distinguish event source  https://review.openstack.org/39673919:10
esbergluthorst: adreznec: efried: Right now the CI is using the develop branch for pypowervm. Will there be impacts if I change it to release?19:16
esbergluSince the local2remote patch is on develop and that new patch from 4504 is release.19:16
esbergluOr does it even matter? Maybe I can apply anyways?19:18
esbergluLooks like it will apply anyways19:25
thorstesberglu: yep...should be able to just apply since branches are near identical since they branched recently19:26
esbergluthorst: Verified locally, management playbook is running right now19:31
efried1thorst, did you get a chance to grok this subprocess._eintr_retry_call thing?19:46
efried1kinda threw that over the fence in the middle of this kafuffle, but we didn't really talk about it.  We should.19:46
thorstefried1: haven't looked yet.19:47
efried1thorst, it won't be obvious what's going on by looking at it.19:47
efried1We had one failure yesterday where basically open() got a signal, EINTR, and that blew up the process.19:47
efried1I did some research: apparently those signals "just happen" from time to time, and we're supposed to ignore them and retry.19:49
efried1They fixed this at the lowest level in py3 by doing exactly what _eintr_retry_call does19:49
efried1But in py2, we're still subject to this behavior, so we're supposed to do that retry ourselves.19:49
efried1It's a bit ugly, cause instead of just being able to make syscalls like open() and close() and read() and write(), we have to wrap 'em in this retry nonsense.19:50
thorstwhoa...ok19:50
efried1...or we could just take the risk and accept these failures when they happen.  But that doesn't seem very robust.19:50
efried1Now, of course, sonar hates the fact that I'm using a private member of a foreign thingy (subprocess._eintr_retry_call)19:51
efried1The alternative is to dup the code inline as a util or something.19:51
efried1It's only like 7LOC or so.19:51
efried1How do you feel about that?19:51
thorstI'm ok with that for now19:54
thorstassuming I don't change my mind after i see it19:55
efried1rgr19:56
efried1thorst, FYI: https://www.python.org/dev/peps/pep-0475/19:57
thorstmy word...so that could be our whole 'this is why glance is failing' thing?20:12
kriskendso these signals just randomly happen?  Is this a version of chaos monkey?20:18
*** tblakes has quit IRC20:21
esbergluthorst: efried: Might be a while before runs go through with that patch20:25
esberglu100% (115572/115572), 45.06 MiB | 96.00 KiB/s20:25
thorstyikes...20:30
*** thorst has quit IRC20:31
*** thorst has joined #openstack-powervm20:32
*** thorst has quit IRC20:36
*** tblakes has joined #openstack-powervm20:37
*** smatzek_ has quit IRC20:41
*** jwcroppe has quit IRC20:44
*** jwcroppe has joined #openstack-powervm20:45
*** jwcroppe has quit IRC20:49
efried1thorst, kriskend, seroyer - yes, and may in fact make the following comment obsolete:20:50
efried1# Very rarely (think one in a thousand) there appears to be a20:50
efried1# hiccup in the upload processing.  A simple retry mechanism20:50
efried1# should be enough to push it through20:50
efried1May even make _wrap_user_stream unnecessary.20:51
*** thorst has joined #openstack-powervm20:54
*** thorst has quit IRC20:58
*** svenkat has quit IRC21:03
*** thorst has joined #openstack-powervm21:04
*** jwcroppe has joined #openstack-powervm21:07
*** jwcroppe has joined #openstack-powervm21:07
*** k0da has joined #openstack-powervm21:57
efried1adreznec, how do I find out what magic incantation I need to recheck these other failing CIs?22:00
adreznecIt should say in the comments from the CI systems22:00
adreznecYou might have to hit the toggle CI button22:00
adreznecTo show the comments22:00
efried1ah22:00
*** seroyer_ has joined #openstack-powervm22:01
*** seroyer has quit IRC22:02
*** seroyer_ is now known as seroyer22:02
*** kriskend has quit IRC22:07
*** edmondsw has quit IRC22:12
*** tblakes has quit IRC22:34
*** tjakobs has quit IRC22:35
*** esberglu has quit IRC22:41
*** esberglu has joined #openstack-powervm22:41
*** seroyer has quit IRC22:44
*** esberglu has quit IRC22:46
*** mdrabe has quit IRC22:58
*** chas_ has joined #openstack-powervm23:09
*** chas_ has quit IRC23:13
*** apearson has quit IRC23:15
*** tjakobs has joined #openstack-powervm23:15
*** tjakobs has quit IRC23:24

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