| *** esberglu has quit IRC | 01:13 | |
| *** esberglu has joined #openstack-powervm | 01:14 | |
| *** thorst has quit IRC | 01:17 | |
| *** esberglu has quit IRC | 01:19 | |
| *** esberglu has joined #openstack-powervm | 01:25 | |
| *** esberglu has quit IRC | 01:26 | |
| *** thorst has joined #openstack-powervm | 01:33 | |
| *** thorst has quit IRC | 01:33 | |
| *** chas has joined #openstack-powervm | 01:40 | |
| *** jwcroppe has joined #openstack-powervm | 01:41 | |
| *** chas has quit IRC | 01:45 | |
| *** lan has joined #openstack-powervm | 01:52 | |
| *** thorst has joined #openstack-powervm | 01:53 | |
| *** thorst has quit IRC | 01:53 | |
| *** dwayne has quit IRC | 01:55 | |
| *** lan1 has joined #openstack-powervm | 02:00 | |
| *** lan has quit IRC | 02:01 | |
| *** lan2 has joined #openstack-powervm | 02:03 | |
| *** lan1 has quit IRC | 02:06 | |
| *** thorst has joined #openstack-powervm | 02:09 | |
| *** thorst has quit IRC | 02:10 | |
| *** edmondsw has joined #openstack-powervm | 02:11 | |
| *** edmondsw has quit IRC | 02:16 | |
| *** thorst has joined #openstack-powervm | 02:34 | |
| *** thorst has quit IRC | 02:38 | |
| *** thorst has joined #openstack-powervm | 03:38 | |
| *** chas has joined #openstack-powervm | 03:41 | |
| *** chas has quit IRC | 03:46 | |
| *** lan2 has quit IRC | 03:49 | |
| *** thorst has quit IRC | 03:57 | |
| *** chhavi has joined #openstack-powervm | 04:07 | |
| *** thorst has joined #openstack-powervm | 04:55 | |
| *** thorst has quit IRC | 04:59 | |
| *** chas has joined #openstack-powervm | 05:42 | |
| *** arunman has joined #openstack-powervm | 05:43 | |
| *** edmondsw has joined #openstack-powervm | 05:47 | |
| *** chas has quit IRC | 05:49 | |
| *** edmondsw has quit IRC | 05:52 | |
| *** thorst has joined #openstack-powervm | 05:56 | |
| *** thorst has quit IRC | 06:00 | |
| *** chas has joined #openstack-powervm | 06:07 | |
| *** tjakobs has joined #openstack-powervm | 06:38 | |
| *** chas has quit IRC | 06:55 | |
| *** thorst has joined #openstack-powervm | 06:56 | |
| *** tjakobs has quit IRC | 06:59 | |
| *** thorst has quit IRC | 07:01 | |
| *** k0da has joined #openstack-powervm | 07:45 | |
| *** thorst has joined #openstack-powervm | 07:58 | |
| *** k0da has quit IRC | 08:09 | |
| *** thorst has quit IRC | 08:16 | |
| *** chas has joined #openstack-powervm | 08:46 | |
| *** chas has quit IRC | 08:51 | |
| *** chas has joined #openstack-powervm | 09:07 | |
| *** chas has quit IRC | 09:12 | |
| *** thorst has joined #openstack-powervm | 09:13 | |
| *** thorst has quit IRC | 09:17 | |
| *** edmondsw has joined #openstack-powervm | 09:23 | |
| *** edmondsw has quit IRC | 09:27 | |
| *** chas has joined #openstack-powervm | 09:28 | |
| *** chas has quit IRC | 09:32 | |
| *** openstackstatus has quit IRC | 09:36 | |
| *** openstackstatus has joined #openstack-powervm | 09:38 | |
| *** ChanServ sets mode: +v openstackstatus | 09:38 | |
| -openstackstatus- NOTICE: There is a known issue with setuptools 36.0.0 and errors about the "six" package. For current details see https://github.com/pypa/setuptools/issues/1042 and monitor #openstack-infra | 09:45 | |
| *** chas has joined #openstack-powervm | 09:49 | |
| *** chas has quit IRC | 09:53 | |
| *** chas has joined #openstack-powervm | 10:10 | |
| *** chas has quit IRC | 10:14 | |
| *** chas has joined #openstack-powervm | 10:21 | |
| *** chas has quit IRC | 10:26 | |
| *** smatzek has joined #openstack-powervm | 10:58 | |
| *** smatzek_ has joined #openstack-powervm | 10:59 | |
| *** smatzek has quit IRC | 11:03 | |
| *** k0da has joined #openstack-powervm | 11:10 | |
| *** thorst has joined #openstack-powervm | 11:15 | |
| *** thorst has quit IRC | 11:19 | |
| *** thorst has joined #openstack-powervm | 11:41 | |
| chhavi | efried: sonar is complaining for the max argument issue | 11:46 |
|---|---|---|
| *** arunman has quit IRC | 11:51 | |
| *** svenkat has joined #openstack-powervm | 12:00 | |
| *** chas has joined #openstack-powervm | 12:22 | |
| *** edmondsw has joined #openstack-powervm | 12:26 | |
| *** chas has quit IRC | 12:26 | |
| *** kylek3h has quit IRC | 12:32 | |
| *** kylek3h has joined #openstack-powervm | 12:58 | |
| *** arunman has joined #openstack-powervm | 12:59 | |
| *** AndyWojo has joined #openstack-powervm | 12:59 | |
| *** mdrabe has joined #openstack-powervm | 13:08 | |
| edmondsw | efried thinking about how we organize this etherpad as it continues to grow over time... how do you feel about adding "Completed in Pike", etc. sections at the bottom and moving things there when we complete them | 13:14 |
| edmondsw | so they don't continue to clutter up the higher sections | 13:14 |
| edmondsw | and could serve as a good reminder of what we've done as well, I guess | 13:14 |
| *** jpasqualetto has joined #openstack-powervm | 13:20 | |
| *** smatzek_ has quit IRC | 13:31 | |
| chhavi | edmondsw: how about separate etherpad per release and just maintaining the link in the parent one, the way its been used in PTG per discussion | 13:32 |
| *** esberglu has joined #openstack-powervm | 13:33 | |
| edmondsw | chhavi that could work, but it may get tricky if we're not sure which release something is going to be done in... wouldn't know which etherpad to put it in | 13:36 |
| chhavi | Yeah, i was just visualizing as a single etherpad, which has links: "Things to Do", "Items completed", "Issues" | 13:42 |
| chhavi | something like this, its just a thought. | 13:42 |
| *** jwcroppe has quit IRC | 13:48 | |
| *** smatzek_ has joined #openstack-powervm | 13:58 | |
| *** kriskend has joined #openstack-powervm | 13:58 | |
| *** jwcroppe has joined #openstack-powervm | 14:01 | |
| efried | edmondsw Yeah, I'm good with moving completed items to the bottom; or just dropping them off after some time. | 14:01 |
| efried | Would rather avoid having multiple etherpads, I think. We should have a small enough set of to-dos that we can keep it to one. | 14:03 |
| esberglu | edmondsw: efried: thorst: Still working on CI stability. Updated the novalink packages on all of the hosts and was seeing some green last night before something broke the world | 14:05 |
| esberglu | http://ci-watch.tintri.com/project?project=nova | 14:05 |
| efried | yikes. | 14:07 |
| efried | Interestingly, UT for py27 is passing, but py35 isn't. | 14:07 |
| esberglu | efried: It was causing devstack to fail like 4 minutes in all of last night. Looks like whatever the issue was may have been resolved already. I'm seeing runs complete the stack now | 14:10 |
| efried | okay. | 14:10 |
| efried | Discussing it in #openstack-meeting right now. | 14:11 |
| efried | "the setuptools issue" - getting link | 14:11 |
| efried | esberglu https://github.com/pypa/setuptools/issues/1042 | 14:12 |
| esberglu | Stupid setuptool | 14:13 |
| esberglu | Seems like there have been a lot of issues with it that break devstack | 14:14 |
| edmondsw | esberglu maybe they're learning... https://github.com/pypa/setuptools/issues/1045 | 14:32 |
| edmondsw | or at least people are complaining :) | 14:33 |
| -openstackstatus- NOTICE: python-setuptools 36.0.1 has been released and now making its way into jobs. Feel free to 'recheck' your failures. If you have any problems, please join #openstack-infra | 14:34 | |
| *** openstackgerrit has quit IRC | 14:34 | |
| *** tjakobs has joined #openstack-powervm | 14:37 | |
| *** tjakobs has quit IRC | 14:43 | |
| *** chas has joined #openstack-powervm | 14:53 | |
| *** k0da has quit IRC | 14:56 | |
| *** chas has quit IRC | 14:58 | |
| esberglu | thorst: Dang we had a successful run go through on https://review.openstack.org/#/c/469291/ | 15:06 |
| esberglu | But another recheck got posted | 15:06 |
| esberglu | Hopefully that one is successful as well | 15:06 |
| thorst | heh | 15:06 |
| thorst | well, we'll let Esha's go through I guess... | 15:06 |
| thorst | odd that Esha's check didn't cancel out yours? | 15:06 |
| esberglu | Actually that's right | 15:07 |
| esberglu | recheck's don't cancel out other rechecks | 15:07 |
| esberglu | But a new patch coming in will | 15:07 |
| thorst | o...I thought thye did | 15:07 |
| thorst | ok | 15:07 |
| thorst | TIL :-) | 15:07 |
| esberglu | So her's didn't actually do anything | 15:07 |
| thorst | so it no-ops? | 15:07 |
| thorst | or it adds to the queue? | 15:07 |
| esberglu | no-ops | 15:07 |
| thorst | neat | 15:08 |
| esberglu | Yep just checked the queue to confirm. Not there | 15:08 |
| thorst | efried: you good if I w+1 it? | 15:08 |
| efried | thorst Yes. esberglu does our CI trigger without the colon? | 15:09 |
| esberglu | Yeah it actually triggers on a bunch of stuff | 15:09 |
| esberglu | powervm:recheck (with or without spaces in betwee) | 15:09 |
| esberglu | powervm recheck (with any whitespace) | 15:09 |
| esberglu | powervm-recheck | 15:10 |
| efried | k, wasn't sure. | 15:10 |
| efried | thorst's powervmci:recheck was ignored ;-) | 15:10 |
| esberglu | On which patch? That shouldn't be the case | 15:11 |
| *** tjakobs has joined #openstack-powervm | 15:12 | |
| efried | esberglu On https://review.openstack.org/#/c/469291/ (PS2, yesterday morning) | 15:14 |
| efried | esberglu I posted my recheck that afternoon because nothing had happened and somebody asked me why. | 15:15 |
| efried | esberglu So next question: I seem to have posted my recheck on PS1 - does that actually run the CI on PS1, or does it always choose latest? | 15:15 |
| esberglu | Nah the results are posted at 8:28 from his check | 15:16 |
| esberglu | Oh wait I was looking at the wrong recheck. Idk why it didn't get picked up | 15:16 |
| *** openstackgerrit has joined #openstack-powervm | 15:16 | |
| openstackgerrit | Merged openstack/networking-powervm master: VIOS availability check during agent startup https://review.openstack.org/469291 | 15:16 |
| efried | esberglu Because he spelled it powervm*ci*:recheck | 15:18 |
| esberglu | lol yeah that would do it | 15:18 |
| efried | edmondsw thorst FYI, esberglu just patched drmgr on the CI nvls in a bid to fix the vterm issues we've been having. So keep an eye out for CI failures caused by slow/timing-out vterm close. | 15:19 |
| thorst | was that apearson's update? | 15:20 |
| esberglu | yeah | 15:20 |
| efried | The drmgr patch was provided by apearson; word is that the fix is proposed to be backported, but not yet there. | 15:20 |
| edmondsw | thorst esberglu can one of you add me to the list of those notified when the CI fails? | 15:23 |
| thorst | esberglu: edmondsw is asking to be added to the powervmci failure e-mail list :-) | 15:24 |
| thorst | edmondsw: brace yourself.... | 15:24 |
| * edmondsw ducks | 15:24 | |
| *** dwayne has joined #openstack-powervm | 15:27 | |
| esberglu | edmondsw: I would highly recommend having a filter set up | 15:30 |
| esberglu | efried: Regarding your earlier question. Looks like the recheck on PS1 runs using PS1 code | 15:31 |
| esberglu | Not the latest | 15:32 |
| efried | esberglu While that seems logical on some level, I wonder if it's really what we want. | 15:32 |
| esberglu | I don't think we have any control over that unless we want to patch zuul | 15:33 |
| *** openstackgerrit has quit IRC | 15:33 | |
| efried | I guess it's useful if we want to see if an old patch set passes where a new one failed. That could also be done by reproposing the old patch - but that's messier. | 15:33 |
| efried | Okay, if it's hard to change, so be it. | 15:33 |
| efried | thorst edmondsw Can you review my review on 5267? | 15:35 |
| efried | thorst edmondsw Mainly looking for consensus on whether we should split it up. More I think about it, more I think that's appropriate. | 15:36 |
| edmondsw | efried will pull it up | 15:36 |
| thorst | efried: on it. | 15:38 |
| efried | edmondsw thorst But I'm way happier wrt the footprint of the change, especially the fact that the method sig change is backward compatible so we don't need to dork with pypowervm reqs. | 15:39 |
| thorst | efried: I dunno...feels like little value in pulling that out | 15:39 |
| thorst | as long as chhavi updates the commit message, I'm fine leaving it in personally | 15:40 |
| *** mdrabe has quit IRC | 15:41 | |
| efried | thorst Okay. My thought was to isolate the part that changes a method signature. An argument can be made here that both things are changing the API. But the two changes really are unrelated. | 15:41 |
| efried | It's a red flag when the commit message has a conjunction ("ISCSI: Support LUN ID *and* handle error conditions") | 15:42 |
| *** chas has joined #openstack-powervm | 15:43 | |
| thorst | yeah, that to me could be 'iSCSI robustness updates' | 15:43 |
| edmondsw | efried the lun portion of this is just about 5 lines, right? | 15:46 |
| efried | edmondsw Yeah. | 15:46 |
| efried | Well | 15:46 |
| efried | With the suggested additional tests, it would be a bit more. | 15:46 |
| efried | Just thinking for future debuggability/maintainability, it would be nice to be able to point to a single change that alters the method sig. | 15:47 |
| efried | Like, if that breaks something, we would be able to revert just that bit with one click, rather than having to unwind it manually. | 15:47 |
| efried | kind of thing. | 15:47 |
| edmondsw | efried sure, tests would be more lines | 15:47 |
| edmondsw | I think I'd prefer to split that out into a separate change unless chhavi has a good reason it needs to be in this one that I'm missing | 15:48 |
| edmondsw | seems like it should be easy to split that out when she's doing this other cleanup | 15:48 |
| efried | agree the mechanics aren't tough | 15:49 |
| efried | We've probably spent more time discussing it than it would have taken to do it ;-) | 15:49 |
| esberglu | edmondsw: Just added you to the mailing list | 15:51 |
| edmondsw | esberglu thank you? ;) | 15:51 |
| efried | I haven't been seeing a whole lot of traffic on that list | 15:52 |
| efried | esberglu Am I on it? | 15:52 |
| efried | I would have thought so. Didn't I create it? | 15:52 |
| esberglu | Yeah you did and you are on it | 15:53 |
| esberglu | Are you not getting them? There has been TONS of traffic the last week | 15:54 |
| efried | esberglu Example subject line? | 15:54 |
| esberglu | Check of openstack/nova 403660,42 failed | 15:54 |
| chhavi | efried: just saw the review comments, I can update the commit message and Doc Strings. | 16:03 |
| chhavi | There is one test case for ISCSIDiscoveryFailed, and handling with lunid as well | 16:04 |
| efried | esberglu Aha, got those going into a folder I never look at. 7,048 unread messages in there. | 16:05 |
| efried | 0 messages in there. | 16:05 |
| esberglu | Yeah you definitely would know if they were going to your main inbox ;) | 16:05 |
| efried | chhavi Thanks - but also see discussion on splitting this into two changes. | 16:06 |
| edmondsw | chhavi I think efried was trying to say that you test for ISCSIDiscoveryFailed on discover_iscsi, but not for discover_initiator | 16:06 |
| efried | I don't know if it's a democracy, but if it is, thorst is outvoted ;-) | 16:07 |
| edmondsw | and you test the case that lunid is specified, but not the case that it isn't | 16:07 |
| efried | chhavi Yes, what edmondsw said is correct. | 16:07 |
| chhavi | efried: I think we can have a single change set, and go with updating the change set. | 16:09 |
| edmondsw | chhavi why don't you want to split it? shouldn't be difficult | 16:10 |
| chhavi | not with respect to diffiulty, but all this got discovered and build as a part of single issue. | 16:13 |
| *** chas has quit IRC | 16:18 | |
| chhavi | We should not be seing any backward compatibility with method signature, considering its constant, and also its not a mandatory job paramter. | 16:18 |
| *** mdrabe has joined #openstack-powervm | 16:18 | |
| chhavi | if we all still feel that having separate will be a good option, i am ok to spend time in doing so. | 16:19 |
| esberglu | thorst: There aren't any more ocata changes that CI is blocking correct? | 16:19 |
| esberglu | https://review.openstack.org/#/c/468515/ | 16:19 |
| thorst | don't think so | 16:20 |
| esberglu | ^ that's already included in the build? | 16:20 |
| thorst | just svenkat's and erlarese's | 16:20 |
| thorst | and they've gotten their fix into PVC through other means, but a backport removes the tech debt | 16:20 |
| thorst | so we still need to get it back there. | 16:20 |
| esberglu | Yep. The stable runs have been looking really good today so that last one should go through with the latest recheck | 16:21 |
| thorst | rockin | 16:23 |
| thorst | :-) | 16:23 |
| efried | thorst Note that there's some question whether pvc will be respinning or absorbing the tech debt regardless. Talk to svenkat if there's more stuff in the pipe that they care about. | 16:23 |
| thorst | I can. The neat thing about PVC though is as soon as we merge it, it just disappears from their tech debt | 16:23 |
| thorst | I think due to how they cherry pick | 16:23 |
| svenkat | efried : there are no other changes needed at this time… | 16:23 |
| chhavi | efried, thorst, edmondsw: So whats the vote, shall we split or update commit msg. based on that i can update the review. | 16:25 |
| edmondsw | efried and I would prefer it to be split unless thorst wants to overrule that | 16:26 |
| chhavi | lets split it then | 16:27 |
| chhavi | it will take less time then taking everyone votes :) | 16:27 |
| efried | +1 | 16:32 |
| thorst | sorry - sounds good. | 16:44 |
| chhavi | efried: got the first change set for the Return code | 17:07 |
| efried | chhavi Reviewed. | 17:11 |
| efried | thorst edmondsw 5267 is now just the return code handling. | 17:12 |
| *** chas has joined #openstack-powervm | 17:18 | |
| chhavi | efried: You have to review once again until i get +2 :) | 17:19 |
| efried | chhavi Want to make sure thorst and edmondsw are happy first. | 17:19 |
| thorst | looking | 17:19 |
| thorst | we plan to merge these basically together...right? | 17:20 |
| *** AndyWojo has quit IRC | 17:20 | |
| chhavi | once this is merged i will post the lun id changes | 17:21 |
| thorst | ok | 17:21 |
| chhavi | so that there will be no conflicts | 17:21 |
| thorst | I'm good with PS30 | 17:21 |
| thorst | waiting for Jenkins to come back happy | 17:21 |
| efried | thorst https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/event.py#L63 | 17:30 |
| thorst | yes... | 17:30 |
| efried | thorst It appears as though we would handle an NVRAM event even if the path was SomethingOtherThanLogicalPartiton/{pvm_uuid] | 17:31 |
| thorst | Is there anything else that ends in LogicalPartition? | 17:31 |
| thorst | are you saying it should be /LogicalPartition/ ? | 17:31 |
| efried | Hold on | 17:31 |
| efried | No, that's not what I was saying. But I think I've gorped up my sandbox and confused myself. | 17:32 |
| efried | Ignore me for now, will come back in a bit if I'm still confused. | 17:32 |
| thorst | cool | 17:32 |
| chhavi | build done | 17:38 |
| edmondsw | chhavi couple spacing issues, but otherwise LGTM | 17:39 |
| chhavi | can't take anymore | 17:40 |
| edmondsw | chhavi sorry, this should be it :( | 17:40 |
| chhavi | :) | 17:41 |
| edmondsw | is trove not like this? | 17:41 |
| chhavi | never | 17:41 |
| edmondsw | efried esberglu is there a reason we don't have a .gitreview file in pypowervm or powervm-ci, or is that just an oversight? | 17:42 |
| efried | edmondsw Because we don't want to publish the hostname of our internal gerrit repo. | 17:42 |
| chhavi | we focus more on code then on syntax, basic coding guidelines should be followed. | 17:42 |
| chhavi | better to have these spacing rules included in pep8 | 17:44 |
| edmondsw | efried ok, duh... I was thinking those were public but I guess not totally | 17:44 |
| edmondsw | chhavi yeah, I agree that they should have been caught in pep8 | 17:44 |
| efried | edmondsw Not at all public. Agree it's a minor pain, but c'est la vie. | 17:44 |
| edmondsw | efried source is public on github... but of course that doesn't mean dev has to be public | 17:45 |
| efried | edmondsw Right, the gerrit repo is private. | 17:46 |
| *** openstackgerrit has joined #openstack-powervm | 17:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/465572 | 17:54 |
| *** arunmani_ has joined #openstack-powervm | 17:56 | |
| edmondsw | chhavi... you put those first 2 blank lines in the wrong place | 17:57 |
| *** arunman has quit IRC | 17:58 | |
| *** arunmani_ has quit IRC | 18:05 | |
| edmondsw | efried I just added my +2 on 5267, please look before chhavi decides to strangle one of us | 18:08 |
| efried | edmondsw chhavi +2 | 18:08 |
| *** arunmani_ has joined #openstack-powervm | 18:11 | |
| chhavi | Thanks all of you for doing so many levels of reviews, its tough time :) | 18:11 |
| edmondsw | chhavi thanks for not strangling us ;) | 18:20 |
| chhavi | but i think if we can add some more rules in pep8 checks for blanklines and spacing it will help | 18:21 |
| edmondsw | chhavi agreed... will add that to the TODO list :) | 18:22 |
| esberglu | thorst: You have a chance to give the remote console for CI any more thought? | 18:35 |
| edmondsw | efried should I open a LP bug for the "deactivate compute service if pvm-rest is dead", or would we just fix that with no bug number? | 18:36 |
| efried | edmondsw I don't think we need a bug. How do you feel, thorst ? | 18:36 |
| efried | Not something we're going to backport, so we don't *need* it. Beyond that, is it actually a bug? I say nah. | 18:37 |
| *** arunmani_ has quit IRC | 18:42 | |
| *** chas has quit IRC | 18:55 | |
| *** chas has joined #openstack-powervm | 18:55 | |
| *** chhavi has quit IRC | 18:56 | |
| *** k0da has joined #openstack-powervm | 19:01 | |
| *** chas has quit IRC | 19:03 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: WIP: Delay get_instance in event processing https://review.openstack.org/469982 | 19:20 |
| edmondsw | efried should we also deactivate the compute service when we can't find an active VIOS? | 19:33 |
| edmondsw | currently raises an exception from init_host in that case | 19:33 |
| efried | edmondsw Yeah, I think we should dig into (and possibly change) that validate_vios_ready business. | 19:34 |
| edmondsw | yep | 19:34 |
| efried | Right now I think it spins for quite a while waiting for the VIOS to be ready. | 19:34 |
| edmondsw | yes | 19:34 |
| efried | We should instead include vios readiness in the criteria for marking the compute host up/down. | 19:34 |
| efried | So whenever our periodic task hits, compute_host_state = up if (pvm-rest && vios ready) | 19:35 |
| efried | But hold on. | 19:35 |
| efried | thorst should weigh in here. | 19:35 |
| efried | Cause there's *some* stuff we can still do without VIOSes. | 19:35 |
| efried | Like report LPAR states. | 19:35 |
| efried | certain resizes, probably... | 19:36 |
| efried | power on/off/reboot | 19:36 |
| efried | What we can't do is schedule new instances, delete existing, map volumes, or (if SEA/SR-IOV) change networking. | 19:37 |
| edmondsw | efried even if we want it to be up when there's no VIOS, we need changes | 19:37 |
| efried | Yeah | 19:37 |
| efried | I don't fully know what it means for a compute host to be down. | 19:37 |
| edmondsw | thorst, when you have a minute to discuss this... | 19:37 |
| efried | Does the conductor just not even try to ask the compute anything? | 19:38 |
| edmondsw | idk, but it at least won't schedule new instances | 19:39 |
| edmondsw | efried and we can get rid of conn_tries=60 entirely and let it use the default of 1, right? | 19:43 |
| edmondsw | on the Session create | 19:44 |
| edmondsw | or is there a reason to still try a handful of times? | 19:44 |
| efried | edmondsw That would be nice, but I think a little more investigation is needed as to how often we'll get to check. | 19:44 |
| efried | If we only get hit once every five minutes, that means it'll almost always be five minutes after compute startup before we're alive. | 19:45 |
| efried | (on a reboot) | 19:45 |
| efried | If we get hit once every 30s or something, not so bad. | 19:45 |
| edmondsw | efried can you walk me through how you setup an env to try things | 19:53 |
| efried | edmondsw Okay. You got a victim? | 19:53 |
| edmondsw | victim? | 19:53 |
| efried | a neo you can use | 19:54 |
| edmondsw | no. I've got one I can use with a proxy, but not directly, and if we're going to be restarting things... | 19:54 |
| edmondsw | that's part of my question :) | 19:54 |
| efried | thorst How are we on hardware? | 19:55 |
| efried | We need to get edmondsw a neo one way or another. | 19:56 |
| *** smatzek_ has quit IRC | 20:06 | |
| edmondsw | efried if we use get_available_resource as you suggested, that gets called every 60s by default | 20:16 |
| edmondsw | it's configurable with CONF.update_resources_interval | 20:17 |
| *** jwcroppe has quit IRC | 20:24 | |
| thorst | edmondsw: I can get you a system tomorrow? | 20:31 |
| edmondsw | thorst that's fine | 20:31 |
| edmondsw | thorst can you get me wiki access today? | 20:31 |
| thorst | PM me the wiki link? | 20:31 |
| edmondsw | it's in your slack | 20:31 |
| thorst | and for vios readiness...I don't think the host is ready unless you have VIOS ready | 20:32 |
| thorst | so I like that idea. | 20:32 |
| *** svenkat has quit IRC | 20:33 | |
| *** edmondsw has quit IRC | 20:52 | |
| *** thorst has quit IRC | 20:57 | |
| *** kriskend has quit IRC | 20:59 | |
| *** thorst has joined #openstack-powervm | 21:02 | |
| *** chas has joined #openstack-powervm | 21:03 | |
| *** jwcroppe has joined #openstack-powervm | 21:04 | |
| *** jwcroppe has quit IRC | 21:04 | |
| *** thorst has quit IRC | 21:07 | |
| *** chas has quit IRC | 21:09 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: WIP: Delay get_instance in event processing https://review.openstack.org/469982 | 21:17 |
| *** thorst has joined #openstack-powervm | 21:20 | |
| *** thorst has quit IRC | 21:24 | |
| *** jpasqualetto has quit IRC | 21:24 | |
| *** edmondsw has joined #openstack-powervm | 21:32 | |
| *** edmondsw has quit IRC | 21:37 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: WIP: Delay get_instance in event processing https://review.openstack.org/469982 | 21:42 |
| *** k0da has quit IRC | 22:00 | |
| *** kylek3h has quit IRC | 22:06 | |
| *** mdrabe has quit IRC | 22:16 | |
| *** tjakobs has quit IRC | 22:38 | |
| *** svenkat has joined #openstack-powervm | 23:00 | |
| *** jwcroppe has joined #openstack-powervm | 23:48 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!