| *** mriedem is now known as mriedem_vacay | 00:03 | |
| *** tetsuro has joined #openstack-placement | 00:27 | |
| *** gjayavelu has quit IRC | 00:28 | |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384 | 01:18 |
|---|---|---|
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case https://review.openstack.org/574240 | 01:39 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673 | 01:51 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534 | 01:54 |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Remove neutron_opts url in conf. https://review.openstack.org/576674 | 01:55 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Optimize member_of check for nested providers https://review.openstack.org/573203 | 02:05 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676 | 02:15 |
| openstackgerrit | Zhenguo Niu proposed openstack/nova master: Fix breaking out of nested loops https://review.openstack.org/424931 | 02:20 |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Add the neutron doc link https://review.openstack.org/576682 | 02:37 |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Add the neutron doc link https://review.openstack.org/576682 | 02:41 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689 | 03:03 |
| *** mriedem_vacay has quit IRC | 03:12 | |
| openstackgerrit | xulei proposed openstack/nova master: correct some logs in schduler due to max_placement_results https://review.openstack.org/576693 | 03:22 |
| *** tetsuro has quit IRC | 03:26 | |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Add information of deprecation nova-network in system-admin.rst https://review.openstack.org/576698 | 03:32 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 03:36 |
| *** Kevin_Zheng has quit IRC | 03:48 | |
| *** yikun has quit IRC | 03:50 | |
| *** vdrok has quit IRC | 03:50 | |
| *** takashin has quit IRC | 03:54 | |
| *** takashin has joined #openstack-placement | 03:55 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Add instance.lock notification https://review.openstack.org/576702 | 04:06 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Add instance.unlock notification https://review.openstack.org/576703 | 04:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709 | 04:16 |
| *** openstack has joined #openstack-placement | 04:33 | |
| *** ChanServ sets mode: +o openstack | 04:33 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712 | 04:38 |
| *** nicolasbock has joined #openstack-placement | 04:56 | |
| *** tetsuro has joined #openstack-placement | 05:12 | |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Add the neutron doc link https://review.openstack.org/576682 | 05:13 |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Add the neutron doc link https://review.openstack.org/576682 | 05:15 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Mention nova-status upgrade check CLI in upgrade doc https://review.openstack.org/576719 | 05:18 |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-config-drive https://review.openstack.org/576435 | 05:24 |
| *** yikun has joined #openstack-placement | 05:29 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Mention server status in api-ref when rebuild https://review.openstack.org/576438 | 05:32 |
| *** Kevin_Zheng has joined #openstack-placement | 05:32 | |
| *** vdrok has joined #openstack-placement | 05:35 | |
| *** yikun has quit IRC | 05:55 | |
| *** Kevin_Zheng has quit IRC | 05:58 | |
| *** vdrok has quit IRC | 05:59 | |
| *** gjayavelu has joined #openstack-placement | 06:00 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Adjust log style and remove ocata support https://review.openstack.org/576462 | 06:01 |
| openstackgerrit | zhangyangyang proposed openstack/nova master: Update information of neutron_opts url in conf https://review.openstack.org/576674 | 06:06 |
| *** e0ne has joined #openstack-placement | 06:08 | |
| *** vdrok has joined #openstack-placement | 06:20 | |
| *** yikun has joined #openstack-placement | 06:21 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Mention nova-status upgrade check CLI in upgrade doc https://review.openstack.org/576719 | 06:23 |
| *** e0ne has quit IRC | 06:24 | |
| *** Kevin_Zheng has joined #openstack-placement | 06:26 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in sec group test and functional tests https://review.openstack.org/576751 | 06:26 |
| *** tssurya has joined #openstack-placement | 06:34 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487 | 06:38 |
| *** tetsuro has quit IRC | 06:42 | |
| *** e0ne has joined #openstack-placement | 06:53 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.openstack.org/576758 | 07:02 |
| openstackgerrit | Dmitry Tantsur proposed openstack/nova master: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576580 | 07:27 |
| openstackgerrit | jichenjc proposed openstack/nova master: Mention server status in api-ref when rebuild https://review.openstack.org/576438 | 07:50 |
| *** nicolasbock has quit IRC | 07:58 | |
| *** takashin has left #openstack-placement | 08:02 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova master: NUMACell, InstanceNUMACell: Adopt 'PCPU' changes https://review.openstack.org/576021 | 08:08 |
| *** peereb has joined #openstack-placement | 08:11 | |
| *** peereb has quit IRC | 08:14 | |
| *** peereb has joined #openstack-placement | 08:15 | |
| *** peereb has quit IRC | 08:16 | |
| *** peereb has joined #openstack-placement | 08:16 | |
| *** peereb has quit IRC | 08:17 | |
| *** peereb has joined #openstack-placement | 08:18 | |
| *** peereb has quit IRC | 08:19 | |
| *** peereb has joined #openstack-placement | 08:19 | |
| *** peereb has quit IRC | 08:20 | |
| *** peereb has joined #openstack-placement | 08:21 | |
| *** peereb has quit IRC | 08:21 | |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317 | 08:23 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Add instance.lock notification https://review.openstack.org/576702 | 08:23 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703 | 08:24 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add instance.lock notification https://review.openstack.org/576702 | 08:25 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703 | 08:26 |
| *** gjayavelu has quit IRC | 08:39 | |
| *** nicolasbock has joined #openstack-placement | 08:41 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703 | 08:43 |
| *** gjayavelu has joined #openstack-placement | 08:44 | |
| *** gjayavelu has quit IRC | 08:49 | |
| *** gjayavelu has joined #openstack-placement | 09:00 | |
| *** gjayavelu has quit IRC | 09:09 | |
| *** rubasov has quit IRC | 09:25 | |
| openstackgerrit | Merged openstack/nova master: Don't heal allocations for deleted servers https://review.openstack.org/575222 | 09:25 |
| *** avolkov has joined #openstack-placement | 09:27 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Ensure pre-migrating instances are destroyed during init_host https://review.openstack.org/562284 | 09:31 |
| *** cdent has joined #openstack-placement | 09:36 | |
| *** rubasov has joined #openstack-placement | 10:50 | |
| cdent | efried, jaypipes unless someone shouts I'll go ahead and fix the test failures in the concumer generation set? | 10:55 |
| openstackgerrit | yanpuqing proposed openstack/nova master: Rename auth_uri to www_authenticate_uri https://review.openstack.org/576820 | 10:55 |
| *** e0ne has quit IRC | 10:56 | |
| *** belmoreira has joined #openstack-placement | 10:57 | |
| jaypipes | cdent: last I saw, efried had rebased and pushed a revision on that series. I haven't looked at the resulting test (failures) yet. | 11:00 |
| cdent | they failed overnight | 11:00 |
| cdent | it's conflicting with the new database context manager | 11:00 |
| cdent | so I figured I'd fix those bits and push it back up | 11:01 |
| cdent | jaypipes: unless you are feeling like violating pto, I can fix it quick and it should go back on its merry way to merging | 11:01 |
| jaypipes | cdent: nope, I don't feel like violating anything this morning :) go for it. | 11:05 |
| cdent | roger | 11:05 |
| jaypipes | ty cdent | 11:05 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 11:11 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324 | 11:11 |
| openstackgerrit | yanpuqing proposed openstack/nova master: Rename auth_uri to www_authenticate_uri https://review.openstack.org/576820 | 11:19 |
| *** jaypipes has quit IRC | 11:46 | |
| *** e0ne has joined #openstack-placement | 11:58 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle strings as booleans in nova.network.model.VIF utils https://review.openstack.org/576664 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 12:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 12:21 |
| *** edmondsw has joined #openstack-placement | 12:52 | |
| openstackgerrit | Merged openstack/nova master: Wait for network-vif-plugged before starting live migration https://review.openstack.org/558001 | 13:10 |
| efried | Thanks for the fixup cdent | 13:17 |
| cdent | ✔ | 13:18 |
| openstackgerrit | Chen proposed openstack/nova master: Specify periodic task interval in configurations https://review.openstack.org/576864 | 13:44 |
| efried | dansmith: Are you going to take a crack at the top consumer gen patch? https://review.openstack.org/#/c/565604/ | 13:47 |
| efried | dansmith: Since you've been involved with the rest of the series... | 13:48 |
| dansmith | efried: I was avoiding it because I'm not super familiar with those mechanics, but I guess I can if it needs it | 13:48 |
| efried | dansmith: thanks | 13:49 |
| cdent | efried: huzzah, raise_exc ksa fix released. thanks. | 14:10 |
| efried | ✔, as you say. | 14:10 |
| efried | cdent: You gonna go through and fix that up everywhere? | 14:10 |
| efried | guess we need requirements bumpage first. | 14:11 |
| cdent | I can, but it wasn't highest on my list. I'm laying my motivational groundwork to start on /reshaper | 14:11 |
| efried | heh | 14:12 |
| *** avolkov has quit IRC | 14:18 | |
| dansmith | efried: a few comments on that.. can you go answer them in ways that make me happy to +2? | 14:20 |
| efried | dansmith: on it. | 14:20 |
| efried | dansmith: Responded. | 14:32 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Ironic update_provider_tree: restore traits override https://review.openstack.org/576877 | 14:32 |
| cdent | so consumer gen will merge as 1.28. is there a 1.29 already in progress. I guess tetsuro's nested alloc cands? | 14:39 |
| efried | cdent: yes | 14:59 |
| cdent | thanks | 14:59 |
| efried | cdent: Which is otherwise +2, sigh. | 15:00 |
| *** ttsiouts has joined #openstack-placement | 15:02 | |
| *** belmoreira has quit IRC | 15:13 | |
| cdent | efried: question about the payload in http://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/reshape-provider-tree.html#placement-post-reshaper | 15:14 |
| cdent | you show top level inventories and allocations as lists, but the contents looks like dicts | 15:15 |
| cdent | (dicts of dicts) | 15:15 |
| cdent | and the following text says dicts too, so I'm going with dicts | 15:15 |
| cdent | which is good, becuase that's easier | 15:15 |
| efried | cdent: Yup, that's an oops. The [] should be {}. | 15:15 |
| cdent | coo | 15:16 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Ironic update_provider_tree: restore traits override https://review.openstack.org/576877 | 15:28 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Comment proposed ironic fix for removal of ironic driver workaround https://review.openstack.org/576892 | 15:28 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Comment proposed ironic fix for removal of ironic driver workaround https://review.openstack.org/576892 | 15:31 |
| *** belmoreira has joined #openstack-placement | 15:38 | |
| *** belmoreira has quit IRC | 15:38 | |
| *** gjayavelu has joined #openstack-placement | 15:48 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: RT stats deepcopy 1x/node instead of 1x/instance https://review.openstack.org/576591 | 15:54 |
| *** smcginnis is now known as swampboys | 16:00 | |
| *** swampboys is now known as smcginnis | 16:01 | |
| *** ttsiouts has quit IRC | 16:12 | |
| *** ttsiouts has joined #openstack-placement | 16:12 | |
| *** ttsiouts has quit IRC | 16:13 | |
| *** ttsiouts has joined #openstack-placement | 16:13 | |
| *** ttsiouts has quit IRC | 16:18 | |
| openstackgerrit | Zack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876 | 16:25 |
| openstackgerrit | Merged openstack/nova master: Convert ironic virt driver to update_provider_tree https://review.openstack.org/576588 | 16:42 |
| openstackgerrit | Merged openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 16:43 |
| openstackgerrit | Merged openstack/nova master: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576580 | 16:43 |
| *** e0ne has quit IRC | 17:04 | |
| openstackgerrit | Merged openstack/nova master: Optimize member_of check for nested providers https://review.openstack.org/573203 | 17:29 |
| *** tssurya has quit IRC | 17:34 | |
| *** gjayavelu has quit IRC | 17:35 | |
| cdent | efried: sanity check/true or false: In /reshaper the dict of inventories will always have at least one member, but the allocations could be empty (because there are no active instances using the inventory). I'm thinking True. dansmith, mriedem, you too | 17:36 |
| dansmith | I think true, think reshaping a compute node you've emptied before upgrade | 17:39 |
| cdent | yeah. thanks. | 17:41 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | 18:11 |
| *** e0ne has joined #openstack-placement | 18:13 | |
| *** gjayavelu has joined #openstack-placement | 18:15 | |
| openstackgerrit | Merged openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 18:23 |
| *** e0ne has quit IRC | 18:23 | |
| *** tssurya has joined #openstack-placement | 18:35 | |
| *** e0ne has joined #openstack-placement | 18:40 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1764883 https://review.openstack.org/562072 | 19:05 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Ensure pre-migrating instances are destroyed during init_host https://review.openstack.org/562284 | 19:05 |
| openstack | bug 1764883 in OpenStack Compute (nova) "Evacuation fails if the source host returns while the migration is still in progress" [Undecided,In progress] https://launchpad.net/bugs/1764883 - Assigned to Lee Yarwood (lyarwood) | 19:05 |
| *** gjayavelu has quit IRC | 19:20 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Add machine output for nova-status upgrade checks https://review.openstack.org/576944 | 19:21 |
| efried | cdent: Agree true, assuming we're going with only using /reshaper on migrations AND virt didn't lie to us about a migration being necessary. Not sure we should fail if inventories is empty though. | 19:26 |
| cdent | why would you call it with an empty inventories? | 19:28 |
| efried | I wouldn't think it would be on purpose. | 19:29 |
| efried | Just depends how much extra checking we want to do. | 19:30 |
| efried | if allocations is not None, we're going to use the /reshaper. We'll be stuffing the allocations into the payload as given, but we're going to have to derive the inventory manips. | 19:30 |
| cdent | both sides should check on purpose, I would think | 19:30 |
| efried | If we get through looking for inventory moves and we... don't find any, then we should error before calling /reshaper? | 19:31 |
| cdent | is there any situation under which allocations is None, but we still want to reshape? dansmith's response above suggests there might be? | 19:31 |
| efried | oh, definitely that. | 19:31 |
| efried | I was wondering more about the converse. | 19:31 |
| efried | well, allocations is {}, not None. | 19:32 |
| * cdent nods | 19:32 | |
| * cdent is still standing | 19:32 | |
| efried | If allocations is nonempty but inventories is empty, it would just be a POST /allocations/{u} | 19:32 |
| * cdent nods | 19:32 | |
| efried | which is something we don't yet do at virt's behest. | 19:32 |
| efried | That's not to say we never will. | 19:33 |
| efried | But I guess there's no reason to allow it until we have a use case for it? | 19:33 |
| cdent | I would think so, yeah. if you've got additional ideas, put em on the code, I gotta eat and sit or expire | 19:33 |
| efried | Just trying to contrive a scenario where virt thinks it needs to move allocations but not inventories. I imagine it could happen. | 19:33 |
| * cdent nods | 19:34 | |
| * cdent waves | 19:34 | |
| efried | cdent: Got it. | 19:34 |
| efried | cdent: vmware rebalance. | 19:34 |
| efried | put that in your noodle and bake it while you eat. ttyl | 19:34 |
| cdent | if by rebalance you mean what the drs does, that's already under the covers, until such a time as enhanced platform awareness is a thing, and even then, the resources may still be managed at the level of the cluster, not the individual hardware | 19:35 |
| cdent | is undecided | 19:35 |
| cdent | if you mean datastore adjustments, /me shrugs | 19:39 |
| cdent | i would think that's akin to a migration (thus /allocations). I think /reshaper is a tool for a narrow set of activities. At least for now. I hope. | 19:40 |
| openstackgerrit | Carlos Camacho proposed openstack/nova stable/queens: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576948 | 19:40 |
| cdent | I don't like big operations | 19:40 |
| *** cdent has quit IRC | 19:42 | |
| *** gjayavelu has joined #openstack-placement | 20:33 | |
| openstackgerrit | Merged openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324 | 20:41 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: fix PowerVM get_bootdisk_path docstring https://review.openstack.org/576977 | 20:52 |
| openstackgerrit | Merged openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876 | 20:58 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Tighten up ReportClient use of generation https://review.openstack.org/556669 | 21:10 |
| openstackgerrit | Zack Cornelius proposed openstack/nova master: Fix nits from change I676291ec0faa1dea0bd5050ef8e3426d171de4c6 https://review.openstack.org/576985 | 21:13 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Tighten up ReportClient use of generation https://review.openstack.org/556669 | 21:14 |
| *** tssurya has quit IRC | 21:18 | |
| *** e0ne has quit IRC | 21:36 | |
| *** tssurya has joined #openstack-placement | 21:39 | |
| *** tssurya has quit IRC | 21:43 | |
| *** nicolasbock has quit IRC | 21:52 | |
| openstackgerrit | Zack Cornelius proposed openstack/nova master: Implement discard for file backed memory https://review.openstack.org/576994 | 21:53 |
| *** edmondsw has quit IRC | 22:07 | |
| *** edmondsw has joined #openstack-placement | 22:10 | |
| *** edmondsw has quit IRC | 22:14 | |
| *** e0ne has joined #openstack-placement | 22:14 | |
| *** e0ne has quit IRC | 22:15 | |
| *** tssurya has joined #openstack-placement | 22:38 | |
| *** tssurya has quit IRC | 22:42 | |
| *** rpifan has joined #openstack-placement | 23:02 | |
| rpifan | hi | 23:03 |
| openstackgerrit | Merged openstack/nova master: Comment proposed ironic fix for removal of ironic driver workaround https://review.openstack.org/576892 | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!