Wednesday, 2016-11-16

« Tuesday, 2016-11-15 Index Thursday, 2016-11-17 »
*** chas_ has quit IRC00:03
*** jwcroppe has quit IRC00:13
*** kriskend has joined #openstack-powervm00:46
*** YuYangWang has joined #openstack-powervm00:46
*** seroyer has joined #openstack-powervm00:55
*** jwcroppe has joined #openstack-powervm00:59
*** thorst has joined #openstack-powervm01:41
*** thorst has quit IRC01:45
*** thorst has joined #openstack-powervm01:49
*** thorst has quit IRC01:51
*** thorst has joined #openstack-powervm02:20
*** kriskend has quit IRC02:36
*** thorst has quit IRC02:40
*** chas has joined #openstack-powervm02:40
*** chas has quit IRC02:45
*** chas has joined #openstack-powervm03:08
*** seroyer has quit IRC03:11
*** chas has quit IRC03:12
*** chas has joined #openstack-powervm03:29
*** chas has quit IRC03:33
*** chas has joined #openstack-powervm04:22
*** chas has quit IRC04:28
*** chas has joined #openstack-powervm06:17
*** chas has quit IRC06:21
*** kotra03 has joined #openstack-powervm06:33
*** Cartoon has joined #openstack-powervm06:42
*** chas has joined #openstack-powervm06:45
*** chas has quit IRC06:49
*** chas has joined #openstack-powervm07:12
*** chas has quit IRC07:16
*** Cartoon has quit IRC07:46
*** k0da has joined #openstack-powervm08:35
*** dwayne has quit IRC08:45
*** Cartoon has joined #openstack-powervm08:46
*** Cartoon has quit IRC09:10
*** dwayne has joined #openstack-powervm09:16
*** chas has joined #openstack-powervm09:31
*** chas has quit IRC09:52
*** chas has joined #openstack-powervm09:52
*** chas has quit IRC09:57
*** chas has joined #openstack-powervm09:58
*** chas has quit IRC10:02
*** chas has joined #openstack-powervm10:04
*** chas has quit IRC10:07
*** chas has joined #openstack-powervm10:07
*** chas has quit IRC10:18
*** chas has joined #openstack-powervm10:24
*** chas has quit IRC10:29
*** chas has joined #openstack-powervm10:49
*** chas has quit IRC10:55
*** Cartoon has joined #openstack-powervm11:08
*** chas has joined #openstack-powervm11:09
*** chas has quit IRC11:13
*** chas has joined #openstack-powervm11:15
*** chas has quit IRC11:19
*** chas has joined #openstack-powervm11:21
*** chas has quit IRC11:25
*** kotra03 has quit IRC11:29
*** chas has joined #openstack-powervm11:51
*** chas has quit IRC11:55
*** kotra03 has joined #openstack-powervm11:58
*** chas has joined #openstack-powervm12:22
*** chas has quit IRC12:26
*** seroyer has joined #openstack-powervm12:32
*** seroyer has quit IRC12:48
*** chas has joined #openstack-powervm12:52
*** thorst_ has joined #openstack-powervm12:53
*** thorst_ has quit IRC12:54
*** thorst_ has joined #openstack-powervm12:54
*** chas has quit IRC12:57
*** chas has joined #openstack-powervm13:00
*** tblakes has joined #openstack-powervm13:06
*** kylek3h has quit IRC13:09
*** kylek3h has joined #openstack-powervm13:09
*** kylek3h has quit IRC13:13
*** svenkat has joined #openstack-powervm13:14
*** kotra03 has quit IRC13:27
openstackgerritSridhar Venkat proposed openstack/nova-powervm: ProvisionRequest does not distinguish event source  https://review.openstack.org/39673913:40
*** kylek3h has joined #openstack-powervm13:42
*** esberglu has quit IRC13:42
*** edmondsw has joined #openstack-powervm13:43
*** seroyer has joined #openstack-powervm13:43
*** esberglu has joined #openstack-powervm13:45
*** esberglu has quit IRC13:53
*** apearson has joined #openstack-powervm14:04
*** esberglu has joined #openstack-powervm14:12
*** kriskend has joined #openstack-powervm14:15
*** mdrabe has joined #openstack-powervm14:22
*** chas has quit IRC14:47
*** chas has joined #openstack-powervm14:47
efried1thorst_, adreznec (and svenkat) - Requirements change merged.... with an unintended consequence.15:10
efried1http://logs.openstack.org/39/396739/3/check/gate-nova-powervm-python34/df4ba48/tox/py34-1.log.txt15:10
efried1All of our out-of-tree drivers are now broken.15:10
efried1I think we'll have to remove the explicit pypowervm requirements from them.15:11
*** mdrabe has quit IRC15:11
*** tjakobs has joined #openstack-powervm15:11
*** Cartoon has quit IRC15:13
thorst_efried1: whey did we need the upper-constraints?15:15
thorst_and that's unfortunate, because we'll need to be able to have the out of tree be higher...15:15
efried1thorst_, we got a verification failure when upper-constraints didn't include it.  So I went and read the README in the requirements project, which explained what upper-constraints really is and outlined what you're supposed to do to populate it.15:16
thorst_efried1: hmm...  adreznec any ideas on how we get our out of tree to support that?15:17
thorst_cause that needs to be on develop branch  :-)15:17
thorst_(at least master does)15:17
thorst_efried1: I think he's presenting charts at the moment15:18
esbergluthorst_: Yeah he is15:19
adreznecSorry, was presenting15:22
adreznecreading backscroll15:22
efried1thorst_, it may mean maintaining our own upper-constraints file.15:22
*** kriskend has quit IRC15:22
thorst_efried1: ew.15:23
*** dwayne has quit IRC15:23
adreznecthorst_: efried1 ew, this is gross15:27
adreznecHmm15:27
thorst_well, we know how we need it to work...question is how to get it to do that.15:27
thorst_if we need gork in the out of tree driver...that's probably OK15:28
adreznecRight15:28
*** kriskend has joined #openstack-powervm15:29
adreznecSo with devstack we can use the edit-constraints stuff to fix this...15:32
adreznecDo we care about this anywhere outside of devstack deploys?15:33
adreznecefried1: thorst_ esberglu ^15:33
thorst_adreznec: yeah...15:33
thorst_for sure.15:33
thorst_out-of-tree...we're running develop15:33
thorst_at least for master branch15:33
thorst_we obviously do that now so that we can get the changes in early in pypowervm...they bake ... then we bring in-tree in very stable release levels15:34
thorst_and of course we stabilize on stable/xxx releases on the out-of-tree as well15:34
adreznecSure15:34
adreznecLet me play with the tox stuff then15:35
adreznecI really don't want to have our own upper-constraints file15:35
*** jwcroppe has quit IRC15:37
thorst_this is another fun wrinkle for the CI too15:37
thorst_bleh15:37
adreznecWell we could work around it for the CI for right now by doing something like https://github.com/openstack/requirements#edit-constraints15:38
adreznecWhile we figure out the right solution15:38
thorst_but we're also stuck on getting new changes in while we're figuring it out.15:39
thorst_so I am hoping we just solve this today.15:39
adreznecWorking on it...15:39
thorst_adreznec: thank you sir!  :-)15:39
adreznecI mean it's a terrible option, but if we really need to merge things right now15:40
adreznecWe could just remove our upper-constraints enforcement...15:40
thorst_adreznec: that breaks things in other ways15:49
thorst_the reason we put that upper constraints thing in was due to another bug.15:49
adreznecI think I have a way around it15:49
adreznecI know15:49
adreznecI think that bug has been maybe fixed upstream15:49
adreznecBut I did say it was a terrible option :P15:49
thorst_k15:50
esbergluadreznec: So do I need to do something for the CI? Not really sure what that link above is telling me to do15:50
adreznecesberglu: I think I have a solution, give me ~30 minutes15:51
esbergluOkay cool15:51
esbergluthorst_: That image build made it all the way through15:52
esbergluFinally15:52
thorst_esberglu: heh, but now we know we need retry logic15:52
thorst_cause devstack does it15:52
esbergluYep15:53
*** jwcroppe has joined #openstack-powervm15:55
*** tlian has joined #openstack-powervm15:58
*** dwayne has joined #openstack-powervm16:00
*** k0da has quit IRC16:05
thorst_esberglu: when you get a chance...can you verify that on our CI systems we're deleting vOpt?16:12
thorst_it would be a pvmctl vom list16:12
esbergluFailed to find any VirtualOpticalMedia entries.16:12
thorst_yeah, but we just cleaned, right/16:12
esbergluYeah. When do they get created?16:13
thorst_oops, nevermind, I see where we do it in the code16:13
thorst_it'd be something to check after a few deploys16:13
thorst_after a couple tempest runs16:13
*** kriskend has quit IRC16:17
thorst_adreznec: FYI that tlian pinged me on the issue too with the out of tree driver.  Didn't take long for others to notice  :-)16:17
thorst_I'm going to grab burrito and discuss SDN for an hour...16:17
adreznecWith the burrito?16:17
* adreznec imagines thorst_ having an in-depth technical discussion on SDN with his burrito16:18
tlianthat would utterly be one sided beating :- )16:35
efried1adreznec, how's it going?  Need any help?16:53
adreznecefried1: Still working through an issue, but I think I have the general solution figured out. Doing some testing now17:00
efried1Okay.17:01
efried1adreznec  Do you (or thorst_) know what tdurakov means here? https://review.openstack.org/#/c/391284/3/nova/objects/migrate_data.py@30117:01
efried1I assume we're talking about from/to_legacy_dict, but I'd like to know when/why/how those are used.17:01
adreznecHmm, not 100% sure. Is he referring to implementing at 'obj_make_compatible' method like libvirt and hyperv have?17:05
adreznecWhere we would delete unsupported primitives from the object if we were migrating to a system running with an older migration object?17:06
seroyerThe obj_make_compatible is what I was assuming...17:06
efried1adreznec/seroyer - what I'm wondering is under what conditions those are actually used.  Like, are we supporting old enough envs that they're even relevant?17:13
*** tjakobs has quit IRC17:15
adreznecefried1: probably not17:20
*** tjakobs has joined #openstack-powervm17:21
*** chas has quit IRC17:24
*** chas has joined #openstack-powervm17:24
*** chas has quit IRC17:29
adreznecFYI efried1 thorst_ esberglu I have a solution that seems to work. tox is running now locally17:37
adreznecNot sure how much I like it ,but...17:37
adreznecIt's based on what some other libs are doing17:37
*** esberglu has quit IRC17:39
*** esberglu has joined #openstack-powervm17:39
*** esberglu has quit IRC17:44
efried1adreznec, got a review up yet?17:44
adreznecWriting commit message17:45
efried1coo17:45
openstackgerritAdam Reznechek proposed openstack/nova-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39851317:48
adreznecefried1: thorst_ esberglu svenkat ^17:49
adreznec#workedinmyenvironment17:49
*** esberglu has joined #openstack-powervm17:51
*** esberglu_ has joined #openstack-powervm17:52
*** esberglu has quit IRC17:55
adreznecWell, it passed the gate17:58
esberglu_adreznec: Should I turn on nova-powervm and recheck to see what happens in the CI?18:01
adreznecAs long as you run with that patch, sure :)18:02
adreznecAh, wait18:02
adreznecThe CI uses networking-powervm too, right?18:02
esberglu_Yeah18:02
thorst_it installs all three18:02
adreznecnetworking-powervm and ceilometer-powervm will have this same problem18:02
thorst_adreznec: did you get that code from somewhere?18:02
thorst_or all net new/18:02
adreznecNo, a good chunk of that script is from the *-lib projects18:03
thorst_copyright add needed?18:03
thorst_also, does this get us develop branch?18:03
adreznecYes18:03
thorst_o bleh, I see it18:03
adreznecand none of the lib projects had copyright on it18:03
adreznecSo... idk?18:03
thorst_k, maybe just a note at top where it was pulled from?18:04
thorst_we did that in past when we had a similar issue18:04
adreznecSure18:04
thorst_but I think the approach is good...and we should move forward with it18:04
thorst_I'll be back in an hour...fun day.18:04
adreznecOk18:04
efried1Seems like we could use this to help with some of our other versioning issues.18:05
*** dwayne has quit IRC18:05
efried1Like the fact that there's currently like six places we have to change whenever a new openstack release happens.18:05
openstackgerritAdam Reznechek proposed openstack/nova-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39851318:06
adreznecefried1: Yeah, potentially18:08
*** dwayne has joined #openstack-powervm18:16
*** efried1 has quit IRC18:50
*** mdrabe has joined #openstack-powervm18:54
*** k0da has joined #openstack-powervm18:54
*** efried has joined #openstack-powervm18:58
*** apearson has quit IRC18:58
*** jwcroppe has quit IRC19:00
*** apearson has joined #openstack-powervm19:01
*** apearson has quit IRC19:10
*** apearson has joined #openstack-powervm19:14
*** jwcroppe has joined #openstack-powervm19:15
thorst_adreznec: I'm +2 on that19:19
thorst_do we have a similar one for networking-powervm and ceilometer-powervm19:19
*** dwayne has quit IRC19:34
openstackgerritAdam Reznechek proposed openstack/networking-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39855419:35
openstackgerritMerged openstack/nova-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39851319:36
openstackgerritAdam Reznechek proposed openstack/ceilometer-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39855519:38
adreznecthorst_: efried esberglu_ I'm going to be AFK for a while here, so merge those as needed ^19:43
esberglu_adreznec: Cool thanks19:44
*** dwayne has joined #openstack-powervm19:48
openstackgerritEric Berglund proposed openstack/ceilometer-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39855519:57
openstackgerritEric Berglund proposed openstack/ceilometer-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39855520:00
openstackgerritEric Berglund proposed openstack/networking-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39855420:00
thorst_efried: want to take a look as well?  Since you had the -1?20:01
efried...20:02
efriedthorst_, esberglu_, done20:03
thorst_everything is W+120:04
thorst_esberglu_: once through, update and run a job in CI?20:04
thorst_(if its ready)20:04
esberglu_thorst_: Don't think I will need to update anything. It pulls new each tempest run20:08
esberglu_So I will kick one off as soon as they merge20:08
openstackgerritMerged openstack/ceilometer-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39855520:10
thorst_rockin20:11
openstackgerritMerged openstack/networking-powervm: Add tox_install.sh for pypowervm upper-constraints workaround  https://review.openstack.org/39855420:12
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova-powervm: ProvisionRequest does not distinguish event source  https://review.openstack.org/39673920:17
openstackgerritEric Berglund proposed openstack/nova-powervm: DNM: ci check  https://review.openstack.org/32831520:19
*** jwcroppe has quit IRC20:30
*** jwcroppe has joined #openstack-powervm20:32
thorst_efried: you got time to drive a pypowervm one to closure?20:41
efriedthorst_, maybe.  What's up?21:00
thorst_we've got an ask to try to get 4504 done and 453321:00
thorst_if I take 4533, could you finish out 4504?21:01
*** k0da has quit IRC21:05
efriedthorst_, okay.  Did you want to discuss 4533 at all?21:08
thorst_is there a discussion left on 4533?21:08
thorst_I'm chicken heading again.21:08
efriedthorst_, I left you comments.21:09
efriedIn my usual paragraph format.21:09
thorst_efried: bleh21:10
thorst_ok21:10
thorst_will look at that21:10
efriedthorst_, btw, 4504 doesn't go without 450721:11
efried...which still needs reviews.21:11
efriedthorst_/esberglu_ - has 4504 been verified live?21:11
thorst_efried: we're hitting issues in CI on the AIO21:12
thorst_so I dont' think so21:12
thorst_basically, now that upper requirements has gone in21:12
thorst_devstack is wiping out our develop branch pypowervm21:12
thorst_and installing 1.0.0.3.921:12
efriedmm21:12
thorst_yeah21:13
thorst_that's why we're failing atm21:13
thorst_I don't see a paragraph on 450421:13
thorst_just one small q21:13
thorst_am I missing something?21:13
efried453321:14
efriedwas the one with the paragraphs.21:14
thorst_o, 4533 you had really legit things.21:14
thorst_and I'm changing how I approach it21:14
efriedyes21:14
efriedokay.21:14
thorst_which is why I didn't dump that on you21:14
thorst_4507 looks good to me21:16
efriedthorst_, see new comment on 450421:20
thorst_shouldn't need to21:21
thorst_at least I dont' think so yet21:21
thorst_(re do we need to wrap it)21:21
thorst_because the first stream will send the close which will pipe into the second stream21:22
efriedokay.21:22
thorst_though...I suppose it wouldn't hurt21:22
efriedthorst_, yea or nay?21:24
thorst_lets separate it out into another change set21:24
thorst_we've got people waiting21:24
efriedk21:26
efriedthorst_, should be good to go now.21:30
thorst_efried: thx.21:35
thorst_I have no clue how to move esberglu_ forward now in the CI though21:35
esberglu_^^ same21:35
thorst_cause you can't uninstall it after the fact...it needs to be done, perhaps within the devstack plugins themselves.21:35
*** apearson has quit IRC21:35
thorst_maybe that's the fix...21:36
thorst_esberglu_ efried adreznec: maybe something in here that installs the develop branch: https://github.com/openstack/nova-powervm/blob/master/devstack/plugin.sh#L7321:36
efriedthorst_, we already have that.21:37
efriedI think we would have to specify the flag that says "don't use constraints".21:37
thorst_efried: huh?  We do for tox but not for devstack?21:38
thorst_and the tox thing we just added?21:38
efriedthorst_ adreznec esberglu_ https://github.com/openstack/requirements/blob/master/README.rst#devstack21:39
thorst_so set in the local.conf USE_CONSTRAINTS = False21:40
thorst_and give it a test.21:40
efriedYeah, not sure that would actually work.  But if it's relatively quick to try...21:40
esberglu_There's also a way to edit single constraints in devstack21:40
efriedYeah, or that.21:40
efriedOr have our CI scripts manually remove pypowervm from global-requirements and upper-constraints - i.e. put our CI env back to what it was before some schlub added pypowervm to requirements.21:41
thorst_the thing is, I don't want devstack broken outside of CI either...21:41
thorst_we have to be cognizant of that too.21:41
*** apearson has joined #openstack-powervm21:42
esberglu_Go ahead and try the USE_CONSTRAINTS thing for now and see what happens?21:46
thorst_if its easy to test, yes21:46
esberglu_Easy yes. But it would take some time. It would have to rebuild the nodepool image again21:47
*** jwcroppe has quit IRC21:50
efriedthorst_, then we can have our plugin.sh/powervm-settings.sh do the same.21:59
*** apearson has quit IRC21:59
thorst_esberglu_: lets see what happens22:00
thorst_then figure out the long term fix22:00
thorst_but yeah, efried I like that approach22:00
*** svenkat has quit IRC22:00
efriedthorst_, the requirements readme basically says USE_CONSTRAINTS=False is not a good idea.22:00
efriedwhether it works or not.22:00
thorst_agree.22:01
thorst_I'd almost rather have it be in the powervm-settings.sh22:01
thorst_thoughts?22:01
*** apearson has joined #openstack-powervm22:02
efriedthorst_, I'd be down with that.22:03
efriedsed -i /pypowervm/d22:04
thorst_efried: I'm going to try to finish that pypowervm delete one quick22:05
thorst_if you get a chance, work on that?  Or we can chase tomorrow?22:05
efriedI'm working on https://bugs.launchpad.net/pypowervm/+bug/1642397 at the moment, but I should be able to nail it quick.22:05
openstackLaunchpad bug 1642397 in pypowervm "LPARBuilder._shared_proc_keys_specified has a logic flaw" [Undecided,In progress] - Assigned to Eric Fried (efried)22:05
thorst_o, I saw that come in22:06
thorst_nice external one!22:06
*** jwcroppe has joined #openstack-powervm22:07
*** kylek3h has quit IRC22:09
thorst_efried: did a pass at 4533.  I think we'll need to revisit the how do we fix every path use case22:18
thorst_but wanted to get something out to help them out now.22:18
efriedaaight.22:18
thorst_gotta run, looks like Jenkins is upset anyway.22:18
thorst_will check back tonight22:18
*** thorst_ has quit IRC22:18
*** thorst_ has joined #openstack-powervm22:19
*** apearson has quit IRC22:21
*** chas has joined #openstack-powervm22:23
*** thorst_ has quit IRC22:24
*** chas has quit IRC22:27
*** chas has joined #openstack-powervm22:31
*** chas has quit IRC22:33
*** chas has joined #openstack-powervm22:34
*** tblakes has quit IRC22:39
*** kylek3h has joined #openstack-powervm22:43
*** mdrabe has quit IRC23:08
*** jwcroppe has quit IRC23:09
*** jwcroppe has joined #openstack-powervm23:10
*** jwcroppe has quit IRC23:14
*** edmondsw has quit IRC23:17
*** seroyer has quit IRC23:19
*** jwcroppe has joined #openstack-powervm23:26
*** jwcroppe has quit IRC23:31
esberglu_efried: thorst: I can't apply the pypowervm patches during the deployment now either. Looking into it.23:36
esberglu_Nvm user error23:39
esberglu_Actually they still don't work23:40
« Tuesday, 2016-11-15 Index Thursday, 2016-11-17 »

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