openstackgerrit | Merged openstack/os-traits master: fix tox python3 overrides https://review.openstack.org/584566 | 00:05 |
---|---|---|
*** edmondsw has joined #openstack-placement | 01:27 | |
*** lei-zh has joined #openstack-placement | 01:31 | |
*** edmondsw has quit IRC | 01:31 | |
openstackgerrit | Merged openstack/nova master: Fix missing print format error https://review.openstack.org/584277 | 02:19 |
openstackgerrit | Merged openstack/nova master: Address nits from consumer generation https://review.openstack.org/577227 | 02:19 |
*** lei-zh has quit IRC | 02:41 | |
*** lei-zh has joined #openstack-placement | 02:43 | |
openstackgerrit | melanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/540258 | 02:48 |
openstackgerrit | melanie witt proposed openstack/nova master: Add functional test for affinity with multiple cells https://review.openstack.org/585073 | 02:48 |
openstackgerrit | Tony Breeds proposed openstack/nova master: Blacklist greenlet 0.4.14 https://review.openstack.org/585016 | 03:02 |
openstackgerrit | jichenjc proposed openstack/nova master: Add zvm CI information https://review.openstack.org/533512 | 03:07 |
openstackgerrit | Merged openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931 | 03:14 |
*** edmondsw has joined #openstack-placement | 03:15 | |
*** edmondsw has quit IRC | 03:19 | |
openstackgerrit | Merged openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932 | 03:21 |
openstackgerrit | Merged openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933 | 03:21 |
openstackgerrit | Merged openstack/nova master: Use consumer generation in _heal_allocations_for_instance https://review.openstack.org/577905 | 03:21 |
openstackgerrit | Merged openstack/nova master: Update RequestSpec.instance_uuid during scheduling https://review.openstack.org/583347 | 03:48 |
*** lei-zh has quit IRC | 04:08 | |
*** e0ne has joined #openstack-placement | 04:40 | |
*** e0ne has quit IRC | 04:40 | |
*** e0ne has joined #openstack-placement | 04:53 | |
*** e0ne has quit IRC | 04:55 | |
*** edmondsw has joined #openstack-placement | 05:03 | |
*** edmondsw has quit IRC | 05:08 | |
*** lei-zh has joined #openstack-placement | 05:14 | |
openstackgerrit | Merged openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo https://review.openstack.org/583023 | 05:15 |
openstackgerrit | Merged openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 05:21 |
openstackgerrit | melanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/540258 | 05:39 |
*** tetsuro has quit IRC | 05:57 | |
openstackgerrit | Merged openstack/nova master: update tox venv env to install all requirements https://review.openstack.org/583336 | 06:21 |
openstackgerrit | Merged openstack/nova master: Enhance doc to guide user to use nova user https://review.openstack.org/583115 | 06:21 |
*** tssurya has joined #openstack-placement | 06:43 | |
openstackgerrit | Merged openstack/nova master: Report 0 root_gb in resource tracker if instance is bfv. https://review.openstack.org/584204 | 06:44 |
openstackgerrit | Merged openstack/nova master: Heal RequestSpec.is_bfv for legacy instances during moves https://review.openstack.org/583715 | 06:52 |
openstackgerrit | huanhongda proposed openstack/nova master: WIP: Fix the metadata re to match the unicode https://review.openstack.org/536236 | 07:02 |
*** tetsuro has joined #openstack-placement | 07:32 | |
*** tetsuro has quit IRC | 07:37 | |
openstackgerrit | Merged openstack/nova master: libvirt: Remove reference to transient domain when detaching devices https://review.openstack.org/584433 | 07:41 |
*** takashin has left #openstack-placement | 08:00 | |
*** e0ne has joined #openstack-placement | 08:15 | |
*** cdent has joined #openstack-placement | 08:49 | |
*** avolkov has joined #openstack-placement | 08:52 | |
*** tetsuro has joined #openstack-placement | 08:59 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | 09:29 |
*** lei-zh has quit IRC | 09:35 | |
*** stephenfin has quit IRC | 09:47 | |
*** stephenfin has joined #openstack-placement | 09:49 | |
openstackgerrit | Vladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921 | 10:09 |
*** e0ne has quit IRC | 10:27 | |
*** e0ne has joined #openstack-placement | 10:32 | |
*** edmondsw has joined #openstack-placement | 10:35 | |
*** edmondsw has quit IRC | 10:40 | |
*** e0ne has quit IRC | 10:40 | |
cdent | efried: rebased (just) /reshaper to master to remove the merge conflict and make some slight adjustments. I left some comments on your fixes to the backend but left that out of the rebase. I'm off to fetch my hopefully fixed laptop from the fixers. | 10:55 |
*** cdent has quit IRC | 10:55 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs https://review.openstack.org/567258 | 11:02 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server usage extension response into server view builder https://review.openstack.org/585262 | 11:09 |
openstackgerrit | Merged openstack/nova master: Fix confusing log message in scheduler https://review.openstack.org/583486 | 11:16 |
*** cdent has joined #openstack-placement | 11:53 | |
*** edmondsw has joined #openstack-placement | 11:57 | |
cdent | [t 3IHl] | 12:02 |
purplerbot | <efried> I remember conceding the point under protest, but not as much protest as usual, because we needed to get moving and it was going to be extra work. [2018-07-23 22:32:49.583969] [n 3IHl] | 12:02 |
cdent | there were several reasons. one was certainly expediency. But the other ways: reshaping is rare and unusual and when it is over re-getting everything is really the only sane thing to do | 12:03 |
cdent | s/ways/was/ | 12:03 |
cdent | and the fact that we didn't know how to represent the response in a sane way, especially if there were a very large number of results | 12:04 |
*** e0ne has joined #openstack-placement | 12:07 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova stable/queens: Make host_aggregate_map dictionary case-insensitive https://review.openstack.org/585334 | 12:18 |
openstackgerrit | Rajesh Tailor proposed openstack/nova stable/pike: Make host_aggregate_map dictionary case-insensitive https://review.openstack.org/585335 | 12:19 |
openstackgerrit | Merged openstack/nova master: Make host_aggregate_map dictionary case-insensitive https://review.openstack.org/498334 | 12:24 |
*** mriedem has joined #openstack-placement | 12:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317 | 12:37 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: FakeLibvirtFixture: mock get_fs_info https://review.openstack.org/579201 | 12:38 |
*** tetsuro has quit IRC | 12:41 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440 | 12:49 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441 | 12:49 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add functional tests for numa-aware-vswitches https://review.openstack.org/583288 | 12:49 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452 | 12:49 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651 | 12:49 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations https://review.openstack.org/585339 | 12:49 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations https://review.openstack.org/585339 | 12:52 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452 | 12:52 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651 | 12:52 |
openstackgerrit | Merged openstack/nova master: Fix wonky reqspec handling in conductor.unshelve_instance https://review.openstack.org/583739 | 12:53 |
*** cdent has quit IRC | 12:55 | |
efried | cdent: ack * 2 | 13:04 |
efried | cdent: Long term, having and using GET /rps?uuid=in:[list] will actually be simpler. But it's going to need another microversion, and probably won't happen real soon, because we can make do with what we've got for now. | 13:06 |
*** jaypipes has quit IRC | 13:20 | |
*** mriedem is now known as mriedem_away | 13:24 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Silence psycopg2 warnings https://review.openstack.org/585352 | 13:29 |
*** cdent has joined #openstack-placement | 13:40 | |
openstackgerrit | Merged openstack/nova master: Merge server create for keypair extension https://review.openstack.org/583203 | 13:47 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs https://review.openstack.org/567258 | 13:49 |
*** mriedem_away is now known as mriedem | 14:11 | |
*** mriedem1 has joined #openstack-placement | 14:18 | |
*** mriedem has quit IRC | 14:21 | |
*** mriedem1 is now known as mriedem | 14:24 | |
openstackgerrit | sean mooney proposed openstack/nova master: fix disk_bus handling https://review.openstack.org/584999 | 14:25 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440 | 14:39 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441 | 14:39 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add functional tests for numa-aware-vswitches https://review.openstack.org/583288 | 14:39 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations https://review.openstack.org/585339 | 14:39 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452 | 14:39 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651 | 14:39 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks https://review.openstack.org/585385 | 14:39 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | 14:40 |
*** purplerbot has quit IRC | 14:57 | |
*** purplerbot has joined #openstack-placement | 14:57 | |
*** purplerbot has quit IRC | 15:08 | |
*** purplerbot has joined #openstack-placement | 15:08 | |
openstackgerrit | Eric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider sane https://review.openstack.org/584598 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | 15:13 |
efried | rebase ^ | 15:13 |
efried | cdent: Ima work on the tests for those top two patches, propose 'em, and then if I have time try to flip the bottom two. | 15:14 |
efried | But FYI I'm off this afternoon. | 15:14 |
cdent | efried: roger that | 15:15 |
cdent | efried: anything specifc you need/want from me, or is generic attention sufficient? | 15:16 |
efried | cdent: Generic and specific | 15:16 |
efried | cdent: Sorry, I didn't know how to answer that. | 15:17 |
efried | I meant, please review the code, and be specific when you comment. | 15:17 |
cdent | noted | 15:17 |
efried | but unless you want to volunteer to write some tests, just reviews | 15:17 |
cdent | i'll see when I get there. I just got my corporate laptop back after a week of not having it so I have behind-the-vpn things to catch up on. This will cause floods of tears. | 15:19 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add nova-manage placement sync_aggregates https://review.openstack.org/575912 | 15:19 |
efried | cdent: Okay, ping me if you're ready to get started on something so we don't overlap. If I'm pto, you can assume I've uploaded anything I've got going. | 15:20 |
cdent | ✔ | 15:20 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Log trace when updating host states https://review.openstack.org/366283 | 15:33 |
openstackgerrit | Merged openstack/nova master: Blacklist greenlet 0.4.14 https://review.openstack.org/585016 | 16:06 |
*** e0ne has quit IRC | 16:10 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks https://review.openstack.org/585385 | 16:25 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475 | 16:39 |
openstackgerrit | Merged openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317 | 16:42 |
openstackgerrit | Merged openstack/nova master: FakeLibvirtFixture: mock get_fs_info https://review.openstack.org/579201 | 16:42 |
*** tssurya has quit IRC | 16:43 | |
efried | cdent: jaypipes: I take back what I said about ever needing to support empty inventories for reshape. | 16:57 |
efried | I meant clearing out inventories on all providers, which actually results in 'inventories' being a (nonempty) dict, keyed by provider UUID, of empty dicts. | 16:57 |
efried | I.e. the only way 'inventories' is itself an empty dict is if there are no providers involved in the reshape. | 16:57 |
efried | Which means the only thing you could possibly be doing is moving allocations around, in which case you should use POST /allocations instead. | 16:57 |
efried | In any case, you should not be using POST /reshaper. | 16:57 |
cdent | efried: I kinda assumed you had already come round to that understanding. | 16:58 |
efried | cdent: Not until just now, when I tried to write a test case that would fail on that condition. | 16:59 |
efried | cdent: The only way to do it in nova-land would be to not have any providers, which should never happen because the compute node is always there. | 16:59 |
efried | cdent: I'm afraid I didn't get very far at all - in fact nothing worth committing - and I have to go now. | 17:03 |
efried | I left a brief blurp on the commit message of https://review.openstack.org/#/c/585049/ about the kinds of stuff I was planning to test, in case you feel like diving in there. | 17:04 |
cdent | efried: okay, I'm in the process of reviewing what's up right now and will see when I get there, but we're getting close to the end of my day. morning may bring some activity. | 17:04 |
efried | roger that. | 17:05 |
*** efried is now known as efried_pto | 17:05 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations https://review.openstack.org/585339 | 17:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452 | 17:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651 | 17:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks https://review.openstack.org/585385 | 17:09 |
*** e0ne has joined #openstack-placement | 17:30 | |
openstackgerrit | Merged openstack/nova master: Add regression test for bug #1764883 https://review.openstack.org/562072 | 17:34 |
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 Matt Riedemann (mriedem) | 17:35 |
openstackgerrit | Adam Harwell proposed openstack/nova master: Add apply_cells to nova-manage https://review.openstack.org/568987 | 17:41 |
mriedem | huh, nothing went into the placement api-ref for microversion 1.26 | 17:42 |
cdent | mriedem: I suspect it was considered semi-invisible? | 17:44 |
mriedem | not sure what that means | 17:44 |
cdent | also mriedem: can you gaze upon eric's change at https://review.openstack.org/#/c/584648/ and my comment. It sees like he's over-engineering | 17:44 |
cdent | mriedem: sorry, being too vague: The functionality of being able to set reserved the same at total is obscure, something you're only likely to do in the report client. I'm speculating however. I don't know if there was any discussion. It was probably just an oversight that now in hindsight doesn't cost much | 17:46 |
mriedem | https://bugs.launchpad.net/nova/+bug/1783380 | 17:46 |
openstack | Launchpad bug 1783380 in OpenStack Compute (nova) "Placement version 1.26 is not documented in the API reference" [Medium,Triaged] | 17:46 |
mriedem | it's clearly important for the ironic virt driver as a client, | 17:46 |
mriedem | and per the commit message on the change that added the microversion, cyborg might also need it during fpga reprogramming, | 17:46 |
mriedem | and yeah i'm assuming it was just missed | 17:46 |
mriedem | not intentional | 17:46 |
*** e0ne has quit IRC | 18:14 | |
*** mriedem1 has joined #openstack-placement | 18:16 | |
*** mriedem has quit IRC | 18:16 | |
*** mriedem1 is now known as mriedem | 18:27 | |
*** alex_xu has quit IRC | 19:03 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Consider network NUMA affinity for move operations https://review.openstack.org/585339 | 19:04 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452 | 19:05 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651 | 19:05 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks https://review.openstack.org/585385 | 19:05 |
*** alex_xu has joined #openstack-placement | 19:07 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921 | 19:16 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: remove rocky-specific nova-scheduler min placement version https://review.openstack.org/585520 | 19:16 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Extra logs for volume detach device tags cleanup https://review.openstack.org/584032 | 19:56 |
openstackgerrit | Chris Dent proposed openstack/nova master: Use placement context in placement functional tests https://review.openstack.org/585529 | 19:56 |
cdent | efried_pto, edleafe, gibi: small extraction-relatedf cleanup ^ | 19:57 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Extra logs for volume detach device tags cleanup https://review.openstack.org/584032 | 20:05 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Extra logs for volume detach device tags cleanup https://review.openstack.org/584032 | 20:28 |
openstackgerrit | Merged openstack/nova master: Merge server create for config_drive extension https://review.openstack.org/583538 | 20:57 |
openstackgerrit | Merged openstack/nova master: Merge server create for bdm extensions https://review.openstack.org/583559 | 21:18 |
openstackgerrit | Merged openstack/nova master: Trim the fat on HostState.instances https://review.openstack.org/569247 | 21:18 |
*** edmondsw has quit IRC | 21:34 | |
*** cdent has quit IRC | 21:45 | |
*** cdent has joined #openstack-placement | 22:06 | |
*** cdent has quit IRC | 22:14 | |
*** avolkov has quit IRC | 23:17 | |
*** tetsuro has joined #openstack-placement | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!