| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Making consistent used of GiB and MiB in API ref https://review.openstack.org/559985 | 00:19 |
|---|---|---|
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 00:20 |
| *** cdent has joined #openstack-placement | 00:58 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Extract base functional test case from test_direct https://review.openstack.org/585777 | 01:01 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Use base test in placement functional tests https://review.openstack.org/585778 | 01:01 |
| *** cdent has quit IRC | 01:08 | |
| openstackgerrit | Merged openstack/nova master: Remove support for /os-floating-ips-bulk REST API https://review.openstack.org/582912 | 01:11 |
| openstackgerrit | Merged openstack/nova master: Remove support for /os-floating-ip-dns REST API https://review.openstack.org/582943 | 01:14 |
| openstackgerrit | Merged openstack/nova master: Remove "DEPRECATED" tag from Obsolete APIs https://review.openstack.org/583092 | 01:14 |
| openstackgerrit | Merged openstack/nova stable/pike: cleanup mapping/reqspec after archive instance https://review.openstack.org/580775 | 01:14 |
| openstackgerrit | Merged openstack/nova master: doc: remove rocky-specific nova-scheduler min placement version https://review.openstack.org/585520 | 01:14 |
| *** edmondsw has joined #openstack-placement | 01:17 | |
| *** edmondsw has quit IRC | 01:21 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: DNM? Extra logs for volume detach device tags cleanup https://review.openstack.org/584032 | 02:16 |
| openstackgerrit | Xiaohan Zhang proposed openstack/nova master: compute node local_gb_used include swap disks The ComputeNode.local_gb_used value is set in the ResourceTracker._update_usage() method: https://review.openstack.org/585928 | 02:18 |
| *** lei-zh has joined #openstack-placement | 02:29 | |
| *** openstack has joined #openstack-placement | 02:51 | |
| *** ChanServ sets mode: +o openstack | 02:51 | |
| *** edmondsw has joined #openstack-placement | 03:05 | |
| *** edmondsw has quit IRC | 03:09 | |
| *** edleafe has quit IRC | 03:14 | |
| *** edleafe has joined #openstack-placement | 03:15 | |
| *** mriedem_afk has quit IRC | 03:20 | |
| *** lei-zh has quit IRC | 03:28 | |
| *** edmondsw has joined #openstack-placement | 04:53 | |
| *** edmondsw has quit IRC | 04:58 | |
| *** e0ne has joined #openstack-placement | 05:06 | |
| *** lei-zh has joined #openstack-placement | 05:30 | |
| *** e0ne has quit IRC | 06:00 | |
| openstackgerrit | huanhongda proposed openstack/nova master: hypervisor-stats shows wrong disk usages with shared storage https://review.openstack.org/149878 | 06:22 |
| *** edmondsw has joined #openstack-placement | 06:41 | |
| *** edmondsw has quit IRC | 06:47 | |
| *** ttsiouts has joined #openstack-placement | 07:10 | |
| *** tssurya has joined #openstack-placement | 07:12 | |
| openstackgerrit | Xiaohan Zhang proposed openstack/nova master: compute node local_gb_used include swap disks https://review.openstack.org/585928 | 07:22 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Increase sleep time in ServerGroupTestV21 to avoid potential failure https://review.openstack.org/585978 | 07:31 |
| *** ttsiouts has quit IRC | 07:33 | |
| openstackgerrit | Merged openstack/nova master: Online data migration for queued_for_delete flag https://review.openstack.org/584504 | 07:52 |
| openstackgerrit | Merged openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371 | 07:53 |
| openstackgerrit | Merged openstack/nova master: Remove stevedore extensions server_create method https://review.openstack.org/583882 | 07:53 |
| *** avolkov has joined #openstack-placement | 07:56 | |
| *** ttsiouts has joined #openstack-placement | 08:10 | |
| *** avolkov has quit IRC | 08:11 | |
| *** e0ne has joined #openstack-placement | 08:17 | |
| *** edmondsw has joined #openstack-placement | 08:30 | |
| *** tetsuro has quit IRC | 08:31 | |
| *** edmondsw has quit IRC | 08:34 | |
| *** cdent has joined #openstack-placement | 08:54 | |
| *** ttsiouts has quit IRC | 09:29 | |
| *** ttsiouts has joined #openstack-placement | 09:31 | |
| *** cdent has quit IRC | 09:58 | |
| *** cdent has joined #openstack-placement | 09:58 | |
| *** ttsiouts has quit IRC | 10:10 | |
| *** lei-zh has quit IRC | 10:11 | |
| openstackgerrit | Merged openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921 | 10:11 |
| *** edmondsw has joined #openstack-placement | 10:18 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 10:18 |
| *** edmondsw has quit IRC | 10:23 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs https://review.openstack.org/567258 | 10:42 |
| *** edmondsw has joined #openstack-placement | 10:49 | |
| *** takashin_ has joined #openstack-placement | 10:57 | |
| *** takashin has quit IRC | 10:58 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Retry allocation writes server side https://review.openstack.org/586048 | 11:05 |
| cdent | efried, gibi : you may find that ^ interesting, especially the fact that I've made rp conflict exception different from consumer. It probably needs rather close review as it is a bit weird and hard to test. | 11:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add traits parameter https://review.openstack.org/578048 | 11:07 |
| cdent | efried, gibi : I was inspired to escalate my attention to that because allocation rp generation conflicts happen a lot with clustered virt drivers | 11:15 |
| *** ttsiouts has joined #openstack-placement | 11:15 | |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 11:27 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Provide a useful message in the case of 500-error https://review.openstack.org/586056 | 12:00 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Provide a useful message in the case of 500-error https://review.openstack.org/586056 | 12:01 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | 12:23 |
| openstackgerrit | OpenStack Release Bot proposed openstack/os-traits stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/586101 | 12:35 |
| openstackgerrit | OpenStack Release Bot proposed openstack/os-traits stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/586102 | 12:35 |
| openstackgerrit | OpenStack Release Bot proposed openstack/os-traits master: Update reno for stable/rocky https://review.openstack.org/586103 | 12:35 |
| openstackgerrit | OpenStack Release Bot proposed openstack/osc-placement stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/586113 | 12:36 |
| *** openstackgerrit has quit IRC | 12:36 | |
| *** mriedem has joined #openstack-placement | 12:36 | |
| efried | patchbot down? | 13:15 |
| mriedem | we should add the patchbot to the nova channel btw | 13:20 |
| efried | sorry, maybe I don't mean patchbot. Whichever one tells us about patches being proposed. I just rebased and pushed the reshaper series and it didn't show up. Must be recent, though, cause there's messages from the patchbot 45 mins ago. | 13:21 |
| cdent | efried: "openstackgerrit left the chat room. (Killed (Sigyn (Spam is off topic on freenode.)))" | 13:22 |
| efried | whoops | 13:23 |
| efried | There was a lot of spam last night; wonder if they lowered the threshold. | 13:23 |
| cdent | the spam is still flowing today, so they are probably being defensive | 13:24 |
| cdent | efried: anything in particular to which I should draw my attention today | 13:24 |
| efried | looks like they're aware in -infra. I'll just sit back. | 13:24 |
| *** mriedem is now known as mriedem_afk | 13:25 | |
| efried | cdent: Sounds like we're going to punt reshaper. | 13:25 |
| cdent | mriedem: you too might find https://review.openstack.org/586048 interesting, bug fix. It turns out the bug is more serious than I though (especially for vmware) | 13:25 |
| efried | cdent: But if you want to look at the latest updates (if you haven't already). | 13:25 |
| cdent | efried: punt all the way to stein? no FFE? | 13:25 |
| cdent | is there discussion somewhere? | 13:25 |
| efried | cdent: That's what mriedem_afk thinks. in -nova. I think we have a meeting soon, though, so we should discuss there. | 13:26 |
| efried | yeah, half an hour. I'll put it on the agenda. | 13:26 |
| cdent | hmm. I'm not sure I really mind it being punted in any sense other than "but but we're so close" | 13:28 |
| cdent | it's not a feature that I'm personally super wed to | 13:28 |
| cdent | and I guess there's no FFU tool on the immediate horizon | 13:28 |
| cdent | efried: I think we should try to merge the server side, even if we don't merge the client side | 13:43 |
| cdent | otherwise we're causing ourselves some roadblocks for extraction (if we can continue to consider that a thing) | 13:43 |
| efried | cdent: how so? | 13:45 |
| *** ttsiouts has quit IRC | 13:47 | |
| cdent | efried: code checked out of /nova that we want checked into /placement | 13:48 |
| efried | cdent: Sorry, I'm still not seeing why having a pending API change would affect that. | 13:49 |
| cdent | efried: it's an ordering thing | 13:49 |
| *** ttsiouts has joined #openstack-placement | 13:50 | |
| cdent | if we have pending changes to nova/ap/openstack/placement and at the start of stein we go ahead and pull the trigger and move all that code to placement we'll have extra merege conflicts to get those pending changes in | 13:50 |
| cdent | which people won't like | 13:51 |
| cdent | so we'll say "let's wait until all the pending stuff merges before we do the cut over" | 13:51 |
| cdent | which will eventually mean: forget it, let's wait until T | 13:51 |
| cdent | at which point we'll never do it | 13:51 |
| efried | mm. Well, I would hope we would merge the reshaper series very early, like as soon as stein is branched. | 13:51 |
| cdent | so would I, but it would be even easier if we just merged the api now and simply didn't use it | 13:52 |
| efried | Is that, taking the risk that we'll find something broken and have to burn another microversion to fix it, the lesser evil? | 13:52 |
| cdent | and "hope" is frequently for naught | 13:52 |
| cdent | I don't really care about burning microversions | 13:53 |
| *** mriedem_afk is now known as mriedem | 13:53 | |
| cdent | and ideally when placement is its own thing we can throw off some of the nova-imposed overhead for needing a spec when everyone agrees | 13:53 |
| mriedem | efried: -nova has the one that shows things proposed, | 13:56 |
| mriedem | i would also like the one that's in the meeting channel where you dump a gerrit link and it reports the title | 13:56 |
| efried | mriedem: it's gerritbot that does the proposed things, and it's in both of these channels. patchbot is in neither, afaik. | 14:03 |
| *** avolkov has joined #openstack-placement | 14:03 | |
| *** diga has joined #openstack-placement | 14:16 | |
| *** tssurya has quit IRC | 14:23 | |
| *** avolkov has quit IRC | 14:32 | |
| *** takashin_ has left #openstack-placement | 14:50 | |
| *** takashin has joined #openstack-placement | 14:50 | |
| *** avolkov has joined #openstack-placement | 14:58 | |
| *** takashin has left #openstack-placement | 15:03 | |
| cdent | mriedem, efried : I'm doing a simple repeat of the analysis I did last year at http://lists.openstack.org/pipermail/openstack-dev/2017-January/110953.html | 15:16 |
| cdent | today instead of 5 requests per iteration (when no instances) we have ten | 15:16 |
| mriedem | :( | 15:16 |
| cdent | it's effectively for the same reason: we call _udpate at the start and end of _update_available _resource | 15:16 |
| mriedem | and this time we actually care about aggregates b/c of shared storage providers, | 15:17 |
| * cdent nods | 15:17 | |
| mriedem | although that's not really supported yet | 15:17 |
| mriedem | and i'm guessing traits is now in there too | 15:17 |
| cdent | yes | 15:17 |
| cdent | but the redundancies are: the two _update calls and the fact that the thing that _udpate calls requests inventory twice | 15:18 |
| cdent | so there are 4 calls to inventory, and then 2 of rps, traits, aggregates and 1 of allocations | 15:18 |
| cdent | wait that's 11. | 15:18 |
| * cdent looks at data | 15:19 | |
| cdent | yeah, 11 | 15:19 |
| * cdent sighs | 15:19 | |
| cdent | anyway, I'll write it up and we can think about it | 15:21 |
| *** ttsiouts has quit IRC | 15:24 | |
| *** avolkov has quit IRC | 15:31 | |
| *** avolkov has joined #openstack-placement | 15:33 | |
| mriedem | cdent: for shared storage buckets https://review.openstack.org/#/c/149878/ | 15:55 |
| cdent | hmmm, interesting, will look at that more closely asap | 15:56 |
| mriedem | it's 3 years old :) | 15:56 |
| mriedem | but coincides with my desire long-term to change the compute hypervisor-stats API to just be a proxy to placement | 15:56 |
| cdent | one would hope that could be done | 15:57 |
| *** edmondsw has quit IRC | 15:57 | |
| *** edmondsw has joined #openstack-placement | 15:57 | |
| *** edmondsw has quit IRC | 15:59 | |
| *** e0ne has quit IRC | 16:02 | |
| cdent | mriedem, efried : https://anticdent.org/novas-use-of-placement.html | 16:15 |
| efried | ack | 16:15 |
| efried | cdent: s/claim/claims/ | 16:17 |
| cdent | noted | 16:17 |
| efried | "sort of noise" but probably the main reason it's not easy to simplify | 16:17 |
| cdent | that claim is correct, the actor is processes | 16:18 |
| efried | cdent: ah, see it now, thx | 16:19 |
| cdent | the other interaction are the conductor and nova-manage commands | 16:19 |
| cdent | as you'll see as you get further in "this discussion" is _very_ focussed | 16:19 |
| efried | cdent: Could be read as, "The nova-scheduler processes(v) [request candidates](n)" or "The [nova-scheduler processes](n) request(v) candidates" | 16:20 |
| efried | that was msy confusion. | 16:20 |
| * cdent nods | 16:20 | |
| efried | cdent: Having a hard time parsing, "Can get_provider_tree_and_ensure_root tries to manage inventory twice be rationalized for simple cases?" | 16:22 |
| cdent | that's a "typo" | 16:22 |
| cdent | can the way | 16:22 |
| efried | ah | 16:26 |
| cdent | efried: does it otherwise make sense? | 16:31 |
| *** diga has quit IRC | 16:37 | |
| *** openstackgerrit has joined #openstack-placement | 16:49 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Retry allocation writes server side https://review.openstack.org/586048 | 16:49 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Use base test in placement functional tests https://review.openstack.org/585778 | 16:54 |
| *** e0ne has joined #openstack-placement | 17:20 | |
| *** e0ne has quit IRC | 17:37 | |
| *** edmondsw has joined #openstack-placement | 17:48 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add additional functional tests for NUMA networks https://review.openstack.org/585385 | 17:49 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add additional functional tests for NUMA networks https://review.openstack.org/585385 | 17:55 |
| openstackgerrit | Merged openstack/nova master: [placement] Extract base functional test case from test_direct https://review.openstack.org/585777 | 18:19 |
| *** e0ne has joined #openstack-placement | 18:22 | |
| openstackgerrit | karim proposed openstack/nova master: Updated AggregateImagePropertiesIsolation filter illustration https://review.openstack.org/586317 | 18:29 |
| efried | cdent: Yeah, and we should totally figure out how to trim it down. | 18:30 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix accumulated nits from port binding for live migration series https://review.openstack.org/583994 | 18:57 |
| mriedem | nutrislim | 19:08 |
| mriedem | https://www.youtube.com/watch?v=BORkjEKVVSc | 19:08 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651 | 19:15 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add additional functional tests for NUMA networks https://review.openstack.org/585385 | 19:15 |
| openstackgerrit | Merged openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 19:35 |
| openstackgerrit | Merged openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 19:45 |
| *** cdent has quit IRC | 19:59 | |
| *** e0ne has quit IRC | 20:13 | |
| *** e0ne has joined #openstack-placement | 20:59 | |
| *** e0ne has quit IRC | 21:00 | |
| *** edmondsw has quit IRC | 21:43 | |
| *** edmondsw has joined #openstack-placement | 21:45 | |
| *** edmondsw has quit IRC | 21:49 | |
| *** mriedem is now known as mriedem_afk | 22:14 | |
| *** avolkov has quit IRC | 22:38 | |
| *** mriedem_afk is now known as mriedem | 22:47 | |
| *** knikolla has quit IRC | 23:01 | |
| openstackgerrit | Merged openstack/nova master: Stop using HostAPI.service_delete https://review.openstack.org/582680 | 23:50 |
| *** takashin has joined #openstack-placement | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!