| *** takashin has joined #openstack-placement | 00:03 | |
| *** tetsuro has joined #openstack-placement | 00:27 | |
| *** tetsuro has quit IRC | 00:32 | |
| *** tetsuro has joined #openstack-placement | 01:00 | |
| *** tetsuro has quit IRC | 01:13 | |
| *** tetsuro has joined #openstack-placement | 01:13 | |
| *** openstack has joined #openstack-placement | 01:21 | |
| *** ChanServ sets mode: +o openstack | 01:21 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166 | 02:20 |
|---|---|---|
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model https://review.openstack.org/571465 | 02:20 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534 | 02:20 |
| *** tetsuro has quit IRC | 02:34 | |
| *** tetsuro has joined #openstack-placement | 02:42 | |
| openstackgerrit | Merged openstack/nova stable/queens: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/563692 | 02:59 |
| openstackgerrit | Merged openstack/nova stable/pike: Fixed auto-convergence option name in doc https://review.openstack.org/576282 | 02:59 |
| *** tetsuro has quit IRC | 03:14 | |
| openstackgerrit | Steve Kowalik proposed openstack/nova master: Switch to oslo_messaging.ConfFixture.transport_url https://review.openstack.org/573046 | 04:13 |
| openstackgerrit | Merged openstack/nova stable/pike: [Stable Only] Add amd-ssbd and amd-no-ssb CPU flags https://review.openstack.org/577548 | 04:53 |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Check a last error of power action https://review.openstack.org/470754 | 05:11 |
| *** e0ne has joined #openstack-placement | 05:17 | |
| *** nicolasbock has joined #openstack-placement | 05:21 | |
| *** giblet is now known as gibi | 05:23 | |
| *** e0ne has quit IRC | 05:46 | |
| *** e0ne has joined #openstack-placement | 05:48 | |
| openstackgerrit | Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726 | 05:55 |
| openstackgerrit | Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726 | 06:03 |
| openstackgerrit | Merged openstack/nova stable/pike: Change consecutive build failure limit to a weigher https://review.openstack.org/573248 | 06:23 |
| *** e0ne has quit IRC | 06:37 | |
| *** belmoreira has joined #openstack-placement | 06:45 | |
| *** belmoreira has quit IRC | 06:46 | |
| *** tetsuro has joined #openstack-placement | 06:58 | |
| *** tetsuro has quit IRC | 07:03 | |
| openstackgerrit | Chason Chan proposed openstack/nova master: [WIP] Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023 | 07:12 |
| openstackgerrit | Gergely Csatari proposed openstack/nova master: Adding oslo.service to doc requirements https://review.openstack.org/577095 | 07:13 |
| *** tetsuro has joined #openstack-placement | 07:22 | |
| openstackgerrit | Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726 | 07:29 |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Proxy a few attributes of block device mapping https://review.openstack.org/554093 | 07:38 |
| *** ttsiouts has joined #openstack-placement | 08:00 | |
| *** takashin has left #openstack-placement | 08:01 | |
| *** tssurya has joined #openstack-placement | 08:02 | |
| *** e0ne has joined #openstack-placement | 08:34 | |
| *** ttsiouts has quit IRC | 08:43 | |
| *** ttsiouts has joined #openstack-placement | 08:47 | |
| *** cdent has joined #openstack-placement | 08:49 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542 | 09:00 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Log breadcrumb for known encryption bug https://review.openstack.org/577757 | 09:11 |
| *** cdent has quit IRC | 09:32 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make nova list and migration-list ignore down cells https://review.openstack.org/575734 | 09:36 |
| *** cdent has joined #openstack-placement | 09:39 | |
| *** cdent_ has joined #openstack-placement | 09:49 | |
| *** cdent has quit IRC | 09:51 | |
| *** cdent_ is now known as cdent | 09:51 | |
| *** peereb has joined #openstack-placement | 09:59 | |
| *** peereb has quit IRC | 10:01 | |
| *** peereb has joined #openstack-placement | 10:01 | |
| *** peereb has quit IRC | 10:02 | |
| *** peereb has joined #openstack-placement | 10:03 | |
| *** peereb has quit IRC | 10:04 | |
| *** peereb has joined #openstack-placement | 10:04 | |
| *** ttsiouts has quit IRC | 10:05 | |
| *** peereb has quit IRC | 10:05 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 10:41 |
| *** edmondsw has joined #openstack-placement | 10:45 | |
| *** edmondsw_ has joined #openstack-placement | 10:48 | |
| *** edmondsw has quit IRC | 10:49 | |
| openstackgerrit | sahid proposed openstack/nova master: hardware: fix memory check when hugepages requested https://review.openstack.org/532168 | 11:00 |
| *** tssurya has quit IRC | 11:04 | |
| *** cdent has quit IRC | 11:11 | |
| *** ttsiouts has joined #openstack-placement | 11:13 | |
| *** tssurya has joined #openstack-placement | 11:14 | |
| *** mriedem has joined #openstack-placement | 11:16 | |
| *** ttsiouts_ has joined #openstack-placement | 11:23 | |
| *** ttsiout__ has joined #openstack-placement | 11:25 | |
| *** ttsiouts_ has quit IRC | 11:26 | |
| *** ttsiouts has quit IRC | 11:26 | |
| *** bhagyashris has quit IRC | 11:30 | |
| *** ttsiout__ has quit IRC | 11:36 | |
| *** ttsiouts has joined #openstack-placement | 11:37 | |
| *** ttsiouts has quit IRC | 11:49 | |
| *** ttsiouts has joined #openstack-placement | 11:49 | |
| *** cdent has joined #openstack-placement | 11:53 | |
| *** edmondsw_ has quit IRC | 11:53 | |
| *** ttsiouts has quit IRC | 12:00 | |
| *** ttsiouts has joined #openstack-placement | 12:01 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: conf: libvirt: Make `/dev/urandom` the default for 'rng_dev_path' https://review.openstack.org/577385 | 12:09 |
| *** ttsiouts has quit IRC | 12:09 | |
| *** ttsiouts has joined #openstack-placement | 12:10 | |
| *** edmondsw has joined #openstack-placement | 12:30 | |
| openstackgerrit | Chason Chan proposed openstack/nova master: Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023 | 12:32 |
| *** ttsiouts has quit IRC | 12:35 | |
| *** ttsiouts has joined #openstack-placement | 12:43 | |
| openstackgerrit | Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726 | 12:47 |
| *** efried_pto is now known as efried | 12:56 | |
| efried | ō/ | 12:56 |
| cdent | welcome | 13:00 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make nova list and migration-list ignore down cells https://review.openstack.org/575734 | 13:03 |
| openstackgerrit | Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726 | 13:05 |
| *** cdent has quit IRC | 13:15 | |
| *** cdent has joined #openstack-placement | 13:23 | |
| *** cdent has quit IRC | 13:26 | |
| openstackgerrit | Merged openstack/nova stable/queens: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576948 | 13:33 |
| *** cdent has joined #openstack-placement | 13:53 | |
| *** cdent has quit IRC | 14:01 | |
| *** cdent has joined #openstack-placement | 14:01 | |
| *** takashin has joined #openstack-placement | 14:04 | |
| *** cdent has quit IRC | 14:11 | |
| *** takashin has left #openstack-placement | 14:36 | |
| openstackgerrit | Merged openstack/nova stable/pike: Ensure resource class cache when listing usages https://review.openstack.org/574327 | 14:39 |
| *** ttsiouts has quit IRC | 14:58 | |
| *** e0ne has quit IRC | 15:01 | |
| *** cdent has joined #openstack-placement | 15:01 | |
| cdent | efried, edleafe, mriedem sorry for falling off there. My phone overheated and wouldn't be my network any more so I've come home. I guess the whole meeting was talking about the limit weirdness with force_hosts? | 15:02 |
| efried | cdent: Yeah, pretty much. | 15:02 |
| efried | updated the bug with action plan. | 15:02 |
| cdent | rad | 15:02 |
| efried | do you agree with said plan? | 15:02 |
| cdent | seems reasonable enough. In our copious free time we could probably do the needed microversion this cycle, if we really wanted to. it's a where clause. but I agree there's lots of other relevant stuff wanting our attention more | 15:06 |
| *** e0ne has joined #openstack-placement | 15:08 | |
| efried | cdent: I don't think the microversion will be that simple. | 15:08 |
| efried | I anticipate some discussion on how nested/sharing works there. | 15:09 |
| cdent | cuz why? | 15:09 |
| cdent | ah | 15:09 |
| cdent | hmmm | 15:09 |
| efried | Do you have to specify all the UUIDs of all the providers that are allowed to appear in a candidate? | 15:09 |
| efried | Or just the "anchor"? | 15:09 |
| efried | Both have pros and cons of course. | 15:10 |
| cdent | I would think the "anchor" covers the "do what I mean" case | 15:10 |
| efried | For this specific scenario, I agree. | 15:10 |
| efried | That said, the concept of "anchor" is slippery outside of nova. | 15:10 |
| efried | We've had a hard time explaining it in placement thus far. | 15:10 |
| cdent | I think if we think of getting allocation candidates as getting me a suite of "targets" the only legiit target is an "anchor" | 15:10 |
| cdent | that the results include information about other things is to make the targets useful | 15:11 |
| efried | Well, the result *has* to include allocations from providers other than the anchor (in the interesting cases we're talking about here). | 15:12 |
| efried | otherwise the candidate isn't complete. | 15:12 |
| cdent | right, that's what I mean, so where root_rp_uuid in (the set of uuids in query param) | 15:14 |
| efried | except that sharing providers can be (and usually are) also roots. | 15:16 |
| efried | That would be the fault of the caller, having made a poor choice of query, *probably*. | 15:17 |
| cdent | that's a sad artifact of how we are doing things :( | 15:17 |
| cdent | being able to distinguish (in sql) root, shared, child would have been nice | 15:18 |
| efried | also not quite as simple as `where root_rp_uuid in (the set of uuids in query param)` | 15:18 |
| cdent | is this something we want to talk about right now? I assume we need to clear the decks before we actually do anything | 15:18 |
| efried | Right, all I wanted to do was give a taste of why this won't be trivial. | 15:18 |
| cdent | we can carry on the discussion asynch on the bug perhaps? | 15:18 |
| efried | And not even now. We've got bigger fisch to frei. | 15:19 |
| cdent | that it is not trivial is something we should also fix, in our copious free time | 15:19 |
| * cdent fills up all the copious free time | 15:19 | |
| *** ttsiouts has joined #openstack-placement | 15:20 | |
| efried | cdent: meanwhile, you may be interested in nova.api.openstack.placement.objects.resource_provider._anchors_for_sharing_providers and its test nova.tests.functional.api.openstack.placement.db.test_resource_provider.TestResourceProviderAggregates#test_anchors_for_sharing_providers | 15:20 |
| cdent | you act like I'm talking about reality. I'm not talking about reality. I'm talking about what should be. You should know by now that that is where I live. | 15:22 |
| cdent | also I've never been able to read any of the tests under db/* | 15:22 |
| efried | This one has a fancy ascii diagram to go with it. | 15:23 |
| cdent | you want to give me an english language translation. Or rather: just tell me what it you're wanting me to know? | 15:24 |
| * cdent has hayfever | 15:24 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Replace support matrix ext with common library https://review.openstack.org/481304 | 15:25 |
| *** ttsiouts has quit IRC | 15:26 | |
| efried | cdent: Oh, I was just showing you some of the logic we already have in place that deals with the "anchor" concept. It's slightly hairy. But having already trodden this ground, it might be easier to follow the path for newer stuff. | 15:31 |
| *** e0ne has quit IRC | 15:32 | |
| efried | cdent: Realistically, I'm not thinking the difficulty is so much in the implementation; more in the documentation (and convincing others that it's even a concept we want to expose in any form). | 15:32 |
| cdent | I "target" is perhaps a more useful name than anchor | 15:33 |
| efried | It's possible I created a doodle survey for this at some point. | 15:35 |
| efried | anyway, for all these reasons I imagine we'll need a spec, even if it's a leetle one. | 15:36 |
| cdent | well, while placement is still in nova any microversion is a spec... | 15:36 |
| cdent | even leetle ones | 15:36 |
| *** ttsiouts has joined #openstack-placement | 15:37 | |
| openstackgerrit | Chris Dent proposed openstack/nova stable/ocata: Ensure resource class cache when listing usages https://review.openstack.org/574279 | 15:49 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make nova list and migration-list ignore down cells https://review.openstack.org/575734 | 15:51 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 16:19 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 16:19 |
| openstackgerrit | Merged openstack/nova stable/pike: Metadata-API fails to retrieve avz for instances created before Pike https://review.openstack.org/571320 | 16:40 |
| openstackgerrit | Merged openstack/nova master: Switch to oslo_messaging.ConfFixture.transport_url https://review.openstack.org/573046 | 16:40 |
| *** mriedem has left #openstack-placement | 16:57 | |
| *** mriedem has joined #openstack-placement | 16:58 | |
| openstackgerrit | Merged openstack/nova master: objects: Remove legacy '_to_dict' functions https://review.openstack.org/537413 | 17:19 |
| openstackgerrit | Merged openstack/nova master: network: Unchain '_get_phynet_info' from '_get_port_vnic_info' https://review.openstack.org/564443 | 17:19 |
| *** ttsiouts has quit IRC | 17:19 | |
| *** tssurya has quit IRC | 17:24 | |
| *** e0ne has joined #openstack-placement | 17:26 | |
| *** e0ne has quit IRC | 17:40 | |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317 | 17:41 |
| cdent | efried, edleafe, mriedem : this is fun: https://bugs.launchpad.net/nova/+bug/1778576 | 17:50 |
| openstack | Launchpad bug 1778576 in OpenStack Compute (nova) "making new allocations for one consumer against multiple resource providers fails with 409" [High,New] - Assigned to Chris Dent (cdent) | 17:50 |
| *** e0ne has joined #openstack-placement | 17:53 | |
| mriedem | :( | 18:01 |
| edleafe | cdent: interesting. Looking at https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py#L1961-L1962, it seems that generation is only incremented once per consumer. | 18:03 |
| cdent | that's okay | 18:04 |
| cdent | the problem is at, (one mo) | 18:04 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Heal allocations with incomplete consumer information https://review.openstack.org/574488 | 18:04 |
| cdent | https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation.py#L304 | 18:05 |
| cdent | that's all in the same consumer, but only the first time do we want the generation to be null | 18:06 |
| edleafe | But the check on generation should only happen once per PUT, not once per RP | 18:07 |
| cdent | that may be so, but that is not how the code is currently written | 18:08 |
| cdent | https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/util.py#L622 | 18:08 |
| cdent | the "quick fix" is adapt the generation for the next time through the loop | 18:09 |
| cdent | http://paste.openstack.org/show/724244/ | 18:10 |
| edleafe | That can't be right | 18:10 |
| cdent | the first time through the generation is expected as None | 18:11 |
| edleafe | Line 318 is where the allocations are created. | 18:11 |
| edleafe | That's the only place where increment* is called, and it is called once per consumer | 18:11 |
| cdent | the next time through (in ensure_consumer) the generation is expected to be zero | 18:11 |
| cdent | the issue is with creating the consumer records | 18:12 |
| cdent | not with creating the allocations | 18:12 |
| efried | cdent: Just joining, do you have a gabbi queued up for this? | 18:12 |
| cdent | because the allocation write hasn't happened yet when the failure happens | 18:13 |
| cdent | efried: I have a translated placecat thing which shows the failure, one mo | 18:13 |
| cdent | it's the PUT of sandwich2 allocations that is the problem: http://paste.openstack.org/show/724245/ | 18:14 |
| cdent | if you run that against master it will blow up | 18:14 |
| cdent | if you run it with the previous paste it will work | 18:15 |
| cdent | I'm currently working on some changes to gabbits/allocations-1.28.yaml to more directly show it | 18:15 |
| efried | cool. (We're allowed to have a non-uuid consumer in the URI?) | 18:16 |
| edleafe | So glad we have consumer objects! | 18:16 |
| efried | or is that some gabbi magic I'm not familiar with? | 18:17 |
| cdent | efried: yes, in the past it wasn't checked at all. with consumer objects they can happen, but you get a warning. I'll need to update the fridge stuff. If it is distracting, replacing sandwich1 and sandwich2 with some real uuids should cover it | 18:17 |
| efried | I don't care, the way it is now is more readable, just wasn't aware that was allowed. | 18:18 |
| cdent | it dates back from the very early days when consumer ids were defined as "whatever the consumer says" | 18:19 |
| cdent | I think because there was some expectation that non-nova clients might have consumers that were uuid+some others stuff | 18:20 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Heal allocations with incomplete consumer information https://review.openstack.org/574488 | 18:20 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Refactor _heal_instances_in_cell https://review.openstack.org/577896 | 18:20 |
| cdent | anyway: the root of the behavior is the way the ensure_consumer works when a consumer doesn't already exist compared to when it already does | 18:21 |
| efried | cdent: So if you change an allocation against an existing consumer, to a new allocation with multiple providers, it works? | 18:21 |
| cdent | i haven't explicitly tried that yet | 18:22 |
| cdent | but that code is quite different, I intend to add tests for that too | 18:22 |
| cdent | nm: that code is _not_ different. but in that case the generation checking is not special | 18:24 |
| cdent | if the consumer already exists, then the incoming generation never needs to be null | 18:25 |
| cdent | (the code is correct for that state of affairs) | 18:25 |
| cdent | question efried and edleafe : if GET /allocation/{uuid} is a consumer that exists, but there are no allocations, should the response include a consumer_generation> | 18:56 |
| *** tssurya has joined #openstack-placement | 18:57 | |
| edleafe | cdent: you mean like | 18:58 |
| edleafe | {'allocs': [], 'cgen': 0} | 18:58 |
| cdent | s/[]/{}/ but yes | 18:59 |
| cdent | right now it only includes a generation if there are some allocations | 18:59 |
| efried | cdent: This is a hole we have. Consumers, like aggregates, *should* be deleted when their last thingy goes away. | 19:00 |
| efried | (I think we may have the same hole with aggregates, is what I meant to say) | 19:00 |
| efried | Hm. | 19:00 |
| efried | on the other hand. | 19:00 |
| cdent | right | 19:00 |
| edleafe | So a consumer can exist that doesn't consume anything? | 19:00 |
| efried | we could conceivably race that way, huh? | 19:00 |
| efried | But without an operation to remove a consumer, we'll leak them forever. | 19:01 |
| efried | so | 19:01 |
| efried | The right thing at the beginning would have been to have routes for consumer creation and deletion. | 19:02 |
| efried | But at this point, we're damned if we do, damned if we don't. | 19:02 |
| efried | I think our only option is to provide an op for consumer deletion. | 19:02 |
| cdent | the expectation is that two consumers with the same uuid are the same consumer | 19:02 |
| efried | yes of course | 19:02 |
| efried | "of course" | 19:02 |
| cdent | so why would a consumer that is not doing anything right now, not exist? | 19:02 |
| cdent | it might exist again | 19:03 |
| efried | Only because we have no way to delete him. | 19:03 |
| efried | And therefore in a busy deployment, our db will grow and grow | 19:03 |
| efried | like Audrey II. | 19:03 |
| cdent | a consumer could exist that currently has no allocations (a different version of shelved) | 19:03 |
| efried | Yes, I agree with the principle of an allocation-less consumer. | 19:04 |
| efried | So the hole we need to close now is that we need a way to delete a consumer that we definitely don't want to use anymore. | 19:04 |
| efried | (i.e. instance destroy, as opposed to shelve etc.) | 19:04 |
| cdent | okay, but that's not what I was asking about | 19:04 |
| edleafe | It doesn't make sense in Nova-land, but in general, yeah, a non-consuming consumer is fine | 19:04 |
| efried | I know cdent but your question leads to this. | 19:04 |
| cdent | what I was asking about is if I ask for the allocations of a consumer without any, I can't get its generation | 19:05 |
| cdent | which means I can't write them either | 19:05 |
| efried | cdent: To answer your question, the consumer gen should show up in every GET /allocations/{uuid}. | 19:05 |
| efried | period. | 19:05 |
| cdent | okay, we need another bug | 19:05 |
| efried | two | 19:05 |
| efried | one for the absent consumer gen in allocation-less consumer response | 19:05 |
| efried | one for the fact that consumers "leak". | 19:06 |
| *** jaypipes has joined #openstack-placement | 19:08 | |
| efried | I think aggregates still leak too - though that's going to be much less of a problem (for nova anyway) because aggs don't come and go as fast as consumers. | 19:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use consumer generation in _heal_allocations_for_instance https://review.openstack.org/577905 | 19:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703 | 19:25 |
| openstackgerrit | Merged openstack/nova stable/pike: placement: Fix HTTP error generation https://review.openstack.org/571218 | 19:27 |
| openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Testing driver-notes https://review.openstack.org/577908 | 19:28 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Heal allocations with incomplete consumer information https://review.openstack.org/574488 | 19:34 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Refactor _heal_instances_in_cell https://review.openstack.org/577896 | 19:34 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use consumer generation in _heal_allocations_for_instance https://review.openstack.org/577905 | 19:34 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Demonstrate bug in consumer generation handling https://review.openstack.org/577914 | 19:51 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Fix bug in consumer generation handling https://review.openstack.org/577915 | 19:51 |
| cdent | efried, edleafe start a bug on the consumer gen when no allocations thing: https://bugs.launchpad.net/nova/+bug/1778591 | 20:01 |
| openstack | Launchpad bug 1778591 in OpenStack Compute (nova) "GET /allocations/{uuid} on a consumer with no allocations provides no generation" [Medium,Triaged] | 20:01 |
| cdent | too much pudding, barry | 20:04 |
| cdent | I gotta go | 20:05 |
| *** cdent has quit IRC | 20:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Use instance project/user when creating RequestSpec during resize reschedule https://review.openstack.org/577918 | 20:16 |
| *** tssurya has quit IRC | 20:16 | |
| *** tssurya has joined #openstack-placement | 20:29 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Use instance project/user when creating RequestSpec during resize reschedule https://review.openstack.org/577926 | 20:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Use instance project/user when creating RequestSpec during resize reschedule https://review.openstack.org/577929 | 20:50 |
| *** e0ne has quit IRC | 20:57 | |
| *** e0ne has joined #openstack-placement | 20:57 | |
| *** e0ne has quit IRC | 20:59 | |
| *** e0ne has joined #openstack-placement | 20:59 | |
| *** tssurya has quit IRC | 21:00 | |
| *** e0ne has quit IRC | 21:00 | |
| *** stephenfin has quit IRC | 21:03 | |
| *** e0ne has joined #openstack-placement | 21:04 | |
| *** stephenfin has joined #openstack-placement | 21:05 | |
| *** e0ne has quit IRC | 21:07 | |
| *** e0ne has joined #openstack-placement | 21:07 | |
| *** e0ne has quit IRC | 21:21 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Add hostId to metadata service https://review.openstack.org/577933 | 21:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova list and migration-list ignore down cells https://review.openstack.org/575734 | 21:29 |
| *** nicolasbock has quit IRC | 21:37 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Add hostId to metadata service https://review.openstack.org/577933 | 21:39 |
| openstackgerrit | Merged openstack/nova master: Add instance.lock notification https://review.openstack.org/576702 | 21:52 |
| *** takashin has joined #openstack-placement | 21:54 | |
| *** edmondsw has quit IRC | 21:58 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (1) https://review.openstack.org/572018 | 22:00 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (2) https://review.openstack.org/573481 | 22:00 |
| *** mriedem has quit IRC | 22:18 | |
| *** edmondsw has joined #openstack-placement | 22:57 | |
| openstackgerrit | Merged openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703 | 23:01 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104 | 23:46 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106 | 23:46 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110 | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!