| *** tetsuro has joined #openstack-placement | 03:02 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 03:36 |
|---|---|---|
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 03:36 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 03:36 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 03:36 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 03:36 |
| *** tetsuro has quit IRC | 03:37 | |
| *** tetsuro has joined #openstack-placement | 03:38 | |
| *** tetsuro has quit IRC | 03:41 | |
| *** tetsuro has joined #openstack-placement | 03:42 | |
| *** tetsuro has quit IRC | 03:55 | |
| *** tetsuro has joined #openstack-placement | 04:12 | |
| *** tetsuro has quit IRC | 04:26 | |
| *** tetsuro has joined #openstack-placement | 04:53 | |
| *** tetsuro has quit IRC | 04:54 | |
| *** e0ne has joined #openstack-placement | 04:59 | |
| *** e0ne has quit IRC | 04:59 | |
| *** tetsuro has joined #openstack-placement | 05:11 | |
| *** tetsuro has quit IRC | 05:14 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Adjust log style and remove ocata support https://review.openstack.org/576462 | 06:55 |
| openstackgerrit | jichenjc proposed openstack/nova master: Not reraise DiskNotFound if instance is resized https://review.openstack.org/571410 | 07:00 |
| openstackgerrit | jichenjc proposed openstack/nova master: mv generate_glance_url to get_image_endpoint_url https://review.openstack.org/511400 | 07:03 |
| *** peereb has joined #openstack-placement | 07:17 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove ec2 object definitions https://review.openstack.org/557150 | 07:19 |
| openstackgerrit | jichenjc proposed openstack/nova master: remove ec2 db functions https://review.openstack.org/557572 | 07:19 |
| openstackgerrit | sahid proposed openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168 | 07:34 |
| *** giblet_off is now known as gibi | 07:38 | |
| openstackgerrit | Tuan Do Anh proposed openstack/nova-specs master: fix tox python3 overrides https://review.openstack.org/579793 | 07:47 |
| *** ttsiouts has joined #openstack-placement | 07:53 | |
| *** cdent has joined #openstack-placement | 08:10 | |
| *** tssurya has joined #openstack-placement | 08:16 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Avoid raise InstanceNotRunning exception https://review.openstack.org/541152 | 08:24 |
| *** e0ne has joined #openstack-placement | 08:56 | |
| *** e0ne has quit IRC | 09:05 | |
| *** edmondsw has joined #openstack-placement | 09:10 | |
| *** e0ne has joined #openstack-placement | 09:13 | |
| *** edmondsw has quit IRC | 09:15 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899 | 09:15 |
| *** peereb has quit IRC | 09:19 | |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317 | 09:26 |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Docs for reporting CPU traits feature https://review.openstack.org/582902 | 09:26 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Address nits in server group policy series https://review.openstack.org/581616 | 09:31 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove support for /os-floating-ips-bulk REST API https://review.openstack.org/582912 | 10:06 |
| *** e0ne has quit IRC | 10:28 | |
| *** ttsiouts has quit IRC | 10:32 | |
| *** e0ne has joined #openstack-placement | 10:34 | |
| *** edmondsw has joined #openstack-placement | 10:58 | |
| *** e0ne has quit IRC | 11:02 | |
| *** edmondsw has quit IRC | 11:03 | |
| *** e0ne has joined #openstack-placement | 11:15 | |
| *** ttsiouts has joined #openstack-placement | 11:18 | |
| cdent | efried, jaypipes: I think I've got /reshaper past where eric rang out of time last week. Could of the existing gabbi tests are failing where expected. I'll go ahead and push it back up so we can discuss it more | 11:18 |
| cdent | The failing tests are failing in ways that seemed expected based on eric's review of the parent | 11:19 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | 11:24 |
| cdent | jaypipes: efried : ^ | 11:25 |
| cdent | will look into additional tests after a couple of other things | 11:25 |
| deepak_mourya | cdent: Hi | 11:25 |
| cdent | hi deepak_mourya | 11:26 |
| deepak_mourya | Is there anything left for this patch https://review.openstack.org/#/c/579423/ ? | 11:26 |
| deepak_mourya | error_code one | 11:26 |
| * cdent looks | 11:26 | |
| cdent | deepak_mourya: looks good. when jaypipes comes around today he ought to be able to merge that | 11:28 |
| cdent | biab | 11:29 |
| deepak_mourya | cdent: okay no problem :) | 11:30 |
| *** edmondsw has joined #openstack-placement | 11:53 | |
| *** edmondsw has quit IRC | 11:56 | |
| jaypipes | cdent: col, ty sir | 12:07 |
| cdent | jaypipes: diddling with the tests some more now | 12:08 |
| jaypipes | deepak_mourya: +W'd! | 12:09 |
| jaypipes | cdent: ack, they're tricky. tetsuro had some good input on them on one of the revisions. | 12:09 |
| * cdent nods | 12:09 | |
| openstackgerrit | Merged openstack/nova master: network: Always retrieve network information if available https://review.openstack.org/564444 | 12:18 |
| openstackgerrit | sean mooney proposed openstack/nova master: DNM test of intel nfv ci test changes https://review.openstack.org/582941 | 12:30 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove support for /os-floating-ip-dns REST API https://review.openstack.org/582943 | 12:33 |
| *** edmondsw has joined #openstack-placement | 12:34 | |
| efried | ō/ | 12:35 |
| efried | cdent, jaypipes: What's the latest? Anything I should look at right away? | 12:36 |
| cdent | efried: I'm just gonna push up some additional tests on https://review.openstack.org/#/c/576927/ | 12:36 |
| cdent | I fixed the rebase block you timed out on | 12:36 |
| efried | cdent: rgr. And fwiw, when I started playing, I was going down the exact same path. | 12:37 |
| efried | pulling that "ensure consumers for allocations" method out of set_allocations | 12:37 |
| efried | and reusing it. | 12:37 |
| *** edmondsw has quit IRC | 12:39 | |
| jaypipes | efried, cdent: I'm trying to wrap up the pupdate for last week... sorry, was super sick all weekend (and still sick) and didn't get it done | 12:39 |
| efried | Thanks jaypipes. Hope you feel better quickly. | 12:40 |
| cdent | sick on weekends is the worst | 12:40 |
| efried | cdent: The 'capacity exceeded' errors on the gabbi tests, those are ones we expect, right? | 12:42 |
| cdent | yes | 12:42 |
| cdent | there's more about that in my forthcoming commit (which clarifies the tests and adds a bit more error handling) | 12:43 |
| efried | beaut | 12:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove support for /os-floating-ips-bulk REST API https://review.openstack.org/582912 | 12:51 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove support for /os-floating-ip-dns REST API https://review.openstack.org/582943 | 12:51 |
| cdent | sweet, hit an unexpected but real bug | 12:54 |
| efried | yay testing | 12:54 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | 12:58 |
| cdent | efried, jaypipes : please chew on that for while. I'm going to have some lunch. It's still not complete but is closer in ways that ought to be informative | 12:58 |
| cdent | the last test in the gabbit is the 'unexpected but real bug' | 12:59 |
| cdent | biab | 12:59 |
| *** edmondsw has joined #openstack-placement | 13:04 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add UUID validation for consumer_uuid https://review.openstack.org/580373 | 13:23 |
| *** e0ne has quit IRC | 13:25 | |
| *** alex_xu has quit IRC | 13:29 | |
| *** e0ne has joined #openstack-placement | 13:32 | |
| *** alex_xu has joined #openstack-placement | 13:32 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Escalate UUID validation warning to error in test https://review.openstack.org/540386 | 13:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.openstack.org/512552 | 13:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.openstack.org/512553 | 13:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.openstack.org/512623 | 13:42 |
| openstack | bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer) | 13:42 |
| *** mriedem has joined #openstack-placement | 13:48 | |
| efried | Scheduler/placement meeting in #openstack-meeting-alt in 7 minutes | 13:53 |
| *** takashin has joined #openstack-placement | 13:55 | |
| jaypipes | efried: were you wanting to add in tetsuro's additional test assertions from his comment on https://review.openstack.org/#/c/582383/2/nova/tests/functional/api/openstack/placement/db/test_reshape.py? | 14:01 |
| efried | jaypipes: Did you see my response there? I thought it would be as/more effective to add them to the gabbits in the subsequent patch. | 14:02 |
| jaypipes | efried: if not, I can. | 14:02 |
| efried | jaypipes: you can see some of that from cdent in his latest updates. | 14:02 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Replace support matrix ext with common library https://review.openstack.org/481304 | 14:04 |
| *** openstackgerrit has quit IRC | 14:04 | |
| *** openstackgerrit has joined #openstack-placement | 14:10 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Call generate_image_url only for legacy notification https://review.openstack.org/564528 | 14:10 |
| openstackgerrit | Merged openstack/nova stable/queens: hardware: fix hugepages memory usage per intances https://review.openstack.org/581736 | 14:27 |
| *** ttsiouts has quit IRC | 14:44 | |
| *** ttsiouts has joined #openstack-placement | 14:47 | |
| *** takashin has left #openstack-placement | 14:47 | |
| openstackgerrit | Merged openstack/nova stable/pike: Backport tox.ini to switch to stestr https://review.openstack.org/580492 | 14:52 |
| openstackgerrit | Merged openstack/nova master: Resource_provider API handler does not return specific error codes https://review.openstack.org/579423 | 14:52 |
| jaypipes | cdent: yes, I am not going to summits any more. or at least, as long as they are all marketing BS. | 15:07 |
| jaypipes | cdent: I'll probably end up having to attend the "T" or "U" summits because they are being merged back together with the PTGs, unfortunately. | 15:07 |
| jaypipes | cdent: but I'll end up going to as little of the summit side as possible and sticking to the PTG parts. | 15:07 |
| cdent | since boston, all I've done is forum stuff | 15:08 |
| cdent | except to watch efried and edleafe | 15:09 |
| cdent | jaypipes: yeah, killing the ptg is very :( | 15:10 |
| gibi | do I see correctly that DELETE /allocations/{consumer_uuid} does not have a way to provide / assert consumer generation? I don't say I desperatly need to have that I'm just curious | 15:10 |
| jaypipes | cdent: I watched efried and edleafe afterwards on video. good presentation. | 15:11 |
| jaypipes | cdent: same with belmiro's presentation. | 15:11 |
| cdent | I always say to myself that I will watch the video and never do. | 15:11 |
| jaypipes | cdent: but given that I can watch those (few) interesting presentations after the fact, I don't see a need to attend the summit as a whole. certainly not for the keynotes. | 15:11 |
| * cdent nods | 15:11 | |
| jaypipes | cdent: the one nice thing about watching the vids online is trying to identify contributors from the back of their heads. luckily, your hair is easily identifiable ;) | 15:12 |
| jaypipes | deepak_mourya: your patch is now merged (rp error codes) | 15:13 |
| jaypipes | deepak_mourya: ty for your patience in reviews and for these patches. they are very much welcome! | 15:13 |
| efried | gibi: That's one of the main reasons when we did the consumer gen work that we made it so you can PUT with an empty 'allocations' dict to delete. | 15:14 |
| openstackgerrit | Merged openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | 15:15 |
| gibi | efried: ahh I see. So when we bump the report client to 1.28 then we need to replace DELETE /allocations with PUT /allocations as well to keep it safe | 15:16 |
| gibi | efried: like in https://github.com/openstack/nova/blob/6ca30fd8323029604e60b611fc63a80c87dee84a/nova/scheduler/client/report.py#L1837 | 15:16 |
| efried | gibi: Yes, exactly so. | 15:16 |
| gibi | efried: thanks | 15:16 |
| efried | gibi: I did something similar for inventory deletions a while back. | 15:17 |
| openstackgerrit | sean mooney proposed openstack/nova master: DNM test of intel-nfv-ci-test changes https://review.openstack.org/582941 | 15:19 |
| cdent | jaypipes, efried, gibi : I've got to take sarah to the doc (sprained her ankle this is a followup) but I'll be back around a bit later to followup on reshaper stuff | 15:20 |
| gibi | efried: btw, we have hard dependency between consumer gen (1.28) and nrp a_c (1.29) due to the fact that claim uses the api version that is used by the a_c query https://github.com/openstack/nova/blob/6ca30fd8323029604e60b611fc63a80c87dee84a/nova/scheduler/client/report.py#L1553 | 15:20 |
| jaypipes | cdent: no worries, take care | 15:21 |
| gibi | cdent: take care | 15:21 |
| efried | gibi: If we're going through this stuff, it wouldn't hurt my feelings if we unwound that so it didn't have to be one version. | 15:22 |
| efried | gibi: Better yet, I would prefer if we didn't take that parameter at all, and just used the right versions internally. | 15:22 |
| efried | gibi: Since we now require placement to be updated first, we ought to be able to do that, and remove the business where we try to detect the host's max version. | 15:23 |
| gibi | efried: I have to read the history of this param first to know why we introduced it in the first place | 15:24 |
| efried | gibi: I suspect it's because we didn't used to enforce "placement upgrades first". So we couldn't be sure we had the latest, and had to figure out from the host. See https://github.com/openstack/nova/blob/6ca30fd8323029604e60b611fc63a80c87dee84a/nova/conductor/manager.py#L630 | 15:24 |
| *** tssurya has quit IRC | 15:30 | |
| gibi | efried: when you say 'host' which host do you mean? compute host? For me it seems that the version is determined only be code in the get_allocation_candidates call | 15:33 |
| gibi | https://github.com/openstack/nova/blob/16a2db6f23d150330dc82a56d0cf10366b004210/nova/scheduler/manager.py#L148 | 15:33 |
| efried | gibi: I was just trying to track that down. I don't know what that 'host' variable actually *is* at that point. | 15:33 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update HostState.instances during _consume_selected_host https://review.openstack.org/582976 | 15:34 |
| gibi | efried: I think that is a Selection object | 15:34 |
| efried | gibi: It looks like edleafe did a lot of the code around this. Perhaps he can shed some light on allocation_request_version - and tell us if we don't need it anymore, which would be neat. | 15:35 |
| gibi | efried: good point | 15:37 |
| efried | gibi: Here's where that param was added: https://review.openstack.org/#/c/495854/ | 15:39 |
| openstackgerrit | Merged openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534 | 15:41 |
| openstackgerrit | Merged openstack/nova master: do not assume 1 consumer in AllocList.delete_all() https://review.openstack.org/582382 | 15:43 |
| jaypipes | efried: fix for LP #1781430 now merged ^ | 15:44 |
| openstack | Launchpad bug 1781430 in OpenStack Compute (nova) "AllocationList.delete_all() incorrectly assumes a single consumer" [High,Fix released] https://launchpad.net/bugs/1781430 - Assigned to Jay Pipes (jaypipes) | 15:44 |
| efried | jaypipes: woot | 15:55 |
| edleafe | efried: I argued against the need for allocation_request_version, but others insisted it was necessary. It would be an extremely contrived situation for an AR to be received at one version, and then used as a claim for a different microversion. I had convinced jaypipes at one point, but dansmith overruled. | 16:19 |
| edleafe | So we're stuck with it. If you want to remove it, you need to talk to dansmith when he returns | 16:20 |
| *** e0ne has quit IRC | 16:21 | |
| efried | edleafe: Do you recall if it had anything to do with the possibility (now extinct) of placement being at different versions on different hosts? | 16:24 |
| efried | edleafe: or on the host vs conductor? That's not possible anymore, right, since you have to upgrade placement first? | 16:24 |
| jaypipes | efried: that was exactly the reason dansmith wanted to have the AR version in the payload. | 16:25 |
| edleafe | efried: to be honest, I really don't remember. Like I said, take it up with Dan | 16:25 |
| efried | jaypipes: Okay, and at this point is that still a thing? | 16:26 |
| jaypipes | efried: no idea. is there a reason for the question? have you run into something about that? | 16:26 |
| efried | jaypipes: Yeah, gibi was starting to look into bumping our allocs code to use consumer gens and nrp, and noticed that they're currently locked together by an allocations_request_version kwarg that's passed all the way from the conductor to the compute/reportclient. | 16:27 |
| efried | jaypipes: I allowed as how there should be no need for them to be locked together; and started looking at that kwarg and thinking maybe we can get rid of it entirely. | 16:28 |
| efried | depending why it's there in the foist place. | 16:28 |
| jaypipes | efried: not sure I would say they are "locked together". I mean, yes, the placement's API version is passed to in the AR payload, down to the compute node. but is that causing some sort of problem per-se, for upping the microversion that the reportclient sends? | 16:28 |
| efried | jaypipes: Not a problem, no. They're "locked together" in the code as currently written. And I'm trying to figure out if it's okay to break that open. | 16:29 |
| jaypipes | efried: in other words, AFAIK, the microversion sent from the placement service in its response for GET /allocation_candidates will be the microversion that particular placement server is at when the request was sent. The scheduler uses that version in its call to PUT /allocations (claim_resources()) and sends the alternate ARs down to the compute node. | 16:31 |
| jaypipes | efried: I fail to see where a problem would enter in w.r.t. that particular version bein in the alternate ARs. | 16:31 |
| efried | jaypipes: Right. Is there any reason not to hardcode GET /a_c to 1.29 and PUT /allocs to 1.28?? | 16:31 |
| openstackgerrit | Merged openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561 | 16:33 |
| openstackgerrit | Merged openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 16:34 |
| openstackgerrit | Merged openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 16:34 |
| jaypipes | efried: you mean in the reportclient? I don't see a reason to hardcode the PUT /allocations at all if the version is being passed in the payload returned from GET /allocation_candidates | 16:34 |
| efried | I'll have to go look at the API ref again. We're passing a version back with GET /a_c ?? I wasn't aware of that. | 16:37 |
| jaypipes | efried: no, I think we're just sending the OpenStack-API-Version value in the payload for the alternate ARs. | 16:44 |
| jaypipes | efried: but I'd have to check | 16:44 |
| efried | I'll look once I'm off the phone | 16:44 |
| jaypipes | efried: for migrations, apparently we are hard-coding to 1.12 for some reason... https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L501 | 16:47 |
| efried | right | 16:48 |
| efried | I'm guessing that's based on a historical point where 1.12 was the minimum version, and never been changed since then. | 16:48 |
| jaypipes | efried: and for everything else, we're hard-coding to the granular AC version for some reason. https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L342 | 16:48 |
| cdent | jaypipes, efried: we made an explicit choice to always choose a specific version when making any request. which was a way of saying "we should never change this" but rather "we're acking the body structures here" | 16:57 |
| cdent | so if we want to change them, it's fine | 16:57 |
| cdent | but when doing so we have to check that the assumptions the callers make are okay. | 16:57 |
| *** ttsiouts has quit IRC | 17:15 | |
| *** ttsiouts has joined #openstack-placement | 17:15 | |
| jaypipes | cdent: ack, makes sense | 17:16 |
| *** ttsiouts has quit IRC | 17:19 | |
| efried | gibi: ^ | 17:24 |
| cdent | sigh | 17:37 |
| efried | ? | 17:38 |
| cdent | I think most of you probably read through the contraction typo: s/which was/which wasn't/ | 17:38 |
| efried | yes | 17:38 |
| efried | The 'but rather' gave it away. | 17:38 |
| cdent | the cdentease filter is relatively easy to learn too | 17:38 |
| cdent | there are definite patterns | 17:39 |
| * cdent makes dinner | 17:39 | |
| efried | what, make this the opposite of what it says? | 17:39 |
| efried | but only sometimes | 17:39 |
| efried | The only way that's easy to learn is by already knowing what you're going to say. | 17:39 |
| efried | Which, to be fair, is often doable. Except when it's not. | 17:39 |
| *** tssurya has joined #openstack-placement | 17:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505 | 17:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542 | 17:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file https://review.openstack.org/581562 | 17:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.65 https://review.openstack.org/573136 | 17:57 |
| *** e0ne has joined #openstack-placement | 18:14 | |
| openstackgerrit | sean mooney proposed openstack/nova master: DNM test of intel-nfv-ci-test changes https://review.openstack.org/582941 | 18:30 |
| *** ttsiouts has joined #openstack-placement | 18:31 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update queued-for-delete from the ComputeAPI during deletion/restoration https://review.openstack.org/566813 | 18:52 |
| *** e0ne has quit IRC | 19:05 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo https://review.openstack.org/583023 | 19:12 |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo https://review.openstack.org/583023 | 19:14 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Conf option cross-reference in config docs https://review.openstack.org/583025 | 19:16 |
| *** e0ne has joined #openstack-placement | 19:25 | |
| *** e0ne has quit IRC | 19:26 | |
| cdent | efried, jaypipes: I'm going to retire. If you are able to drop some thoughts on https://review.openstack.org/#/c/576927/ before my morning, I'll endeavor to address them. At the moment I think the main issues are outside the handler. There are, however, plenty more tests to make. | 19:47 |
| efried | ack, have a good night. | 19:48 |
| * cdent waves | 19:49 | |
| *** cdent has quit IRC | 19:49 | |
| jaypipes | efried: so which one of us is supposed to rebase 576236 and get address gibi and tetsuro's requests? :) | 20:01 |
| efried | jaypipes: I think we let that one (the client side) go stale for a while until we've satisfied ourselves that the other two are copacetic. | 20:02 |
| efried | jaypipes: So the question is, who's going to work those requests and in which (possibly both) of the other two? | 20:03 |
| efried | guess it is both patches. | 20:03 |
| openstackgerrit | Merged openstack/nova master: Add UUID validation for consumer_uuid https://review.openstack.org/580373 | 20:03 |
| efried | jaypipes: How are you for time/bandwidth at the moment? | 20:04 |
| jaypipes | efried: stretched pretty thin with $work items | 20:20 |
| openstackgerrit | Merged openstack/nova master: network: Retrieve tunneled status in '_get_physnet_info' https://review.openstack.org/564445 | 20:20 |
| openstackgerrit | Rick Bartra proposed openstack/nova master: Add granularity to os-services API https://review.openstack.org/578553 | 20:20 |
| efried | jaypipes: okay. I'll give you a buzz if my afternoon opens up and I get a chance to look into it; you do the same, and whoever gets there first, gets there first. Dig? | 20:20 |
| *** ttsiouts has quit IRC | 20:24 | |
| *** ttsiouts has joined #openstack-placement | 20:24 | |
| *** ttsiouts has quit IRC | 20:28 | |
| jaypipes | efried: today's afternoon or tomorrow's afternoon? | 20:31 |
| efried | jaypipes: Either/both :) | 20:31 |
| jaypipes | efried: I'm busy today tracking down rabbitmq issues in some of our prod clusters. so I def won't get to it today. I could work on it tomorrow morning though. | 20:42 |
| efried | jaypipes: Roger that. Just leave me a note here if/when you start on it. | 20:43 |
| jaypipes | efried: will do, thanks. | 20:45 |
| openstackgerrit | Merged openstack/nova master: Update HostState.instances during _consume_selected_host https://review.openstack.org/582976 | 21:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675 | 21:31 |
| openstackgerrit | MultipleCrashes proposed openstack/nova master: Retry decorator fix for instances which go into ERROR state during bulk delete https://review.openstack.org/563418 | 21:38 |
| *** tssurya has quit IRC | 22:07 | |
| openstackgerrit | Merged openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 22:32 |
| *** edmondsw has quit IRC | 22:43 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Conf option cross-reference in config docs https://review.openstack.org/583025 | 22:44 |
| efried | mriedem: Made oslo.config patch https://review.openstack.org/#/c/583064/ and Depends-On'd ^ to it, so now :oslo.config:option:`foo.bar` will show up as a link in the HTML, but as [foo]/bar in the sample. | 22:45 |
| *** edmondsw has joined #openstack-placement | 22:45 | |
| efried | ...if everything works, that is. | 22:46 |
| efried | whoops, meant that for -nova. Oh well, sue me. | 22:46 |
| *** edmondsw has quit IRC | 22:50 | |
| *** bauzas has quit IRC | 23:43 | |
| *** bauzas has joined #openstack-placement | 23:43 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!