Tuesday, 2018-07-24

openstackgerritMerged openstack/os-traits master: fix tox python3 overrides  https://review.openstack.org/58456600:05
*** edmondsw has joined #openstack-placement01:27
*** lei-zh has joined #openstack-placement01:31
*** edmondsw has quit IRC01:31
openstackgerritMerged openstack/nova master: Fix missing print format error  https://review.openstack.org/58427702:19
openstackgerritMerged openstack/nova master: Address nits from consumer generation  https://review.openstack.org/57722702:19
*** lei-zh has quit IRC02:41
*** lei-zh has joined #openstack-placement02:43
openstackgerritmelanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/54025802:48
openstackgerritmelanie witt proposed openstack/nova master: Add functional test for affinity with multiple cells  https://review.openstack.org/58507302:48
openstackgerritTony Breeds proposed openstack/nova master: Blacklist greenlet 0.4.14  https://review.openstack.org/58501603:02
openstackgerritjichenjc proposed openstack/nova master: Add zvm CI information  https://review.openstack.org/53351203:07
openstackgerritMerged openstack/nova master: Add VIFMigrateData.get_dest_vif  https://review.openstack.org/56693103:14
*** edmondsw has joined #openstack-placement03:15
*** edmondsw has quit IRC03:19
openstackgerritMerged openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call  https://review.openstack.org/56693203:21
openstackgerritMerged openstack/nova master: libvirt: use dest host port bindings during pre_live_migration  https://review.openstack.org/56693303:21
openstackgerritMerged openstack/nova master: Use consumer generation in _heal_allocations_for_instance  https://review.openstack.org/57790503:21
openstackgerritMerged openstack/nova master: Update RequestSpec.instance_uuid during scheduling  https://review.openstack.org/58334703:48
*** lei-zh has quit IRC04:08
*** e0ne has joined #openstack-placement04:40
*** e0ne has quit IRC04:40
*** e0ne has joined #openstack-placement04:53
*** e0ne has quit IRC04:55
*** edmondsw has joined #openstack-placement05:03
*** edmondsw has quit IRC05:08
*** lei-zh has joined #openstack-placement05:14
openstackgerritMerged openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo  https://review.openstack.org/58302305:15
openstackgerritMerged openstack/nova master: ironic: add instance_uuid before any other spawn activity  https://review.openstack.org/56372205:21
openstackgerritmelanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/54025805:39
*** tetsuro has quit IRC05:57
openstackgerritMerged openstack/nova master: update tox venv env to install all requirements  https://review.openstack.org/58333606:21
openstackgerritMerged openstack/nova master: Enhance doc to guide user to use nova user  https://review.openstack.org/58311506:21
*** tssurya has joined #openstack-placement06:43
openstackgerritMerged openstack/nova master: Report 0 root_gb in resource tracker if instance is bfv.  https://review.openstack.org/58420406:44
openstackgerritMerged openstack/nova master: Heal RequestSpec.is_bfv for legacy instances during moves  https://review.openstack.org/58371506:52
openstackgerrithuanhongda proposed openstack/nova master: WIP: Fix the metadata re to match the unicode  https://review.openstack.org/53623607:02
*** tetsuro has joined #openstack-placement07:32
*** tetsuro has quit IRC07:37
openstackgerritMerged openstack/nova master: libvirt: Remove reference to transient domain when detaching devices  https://review.openstack.org/58443307:41
*** takashin has left #openstack-placement08:00
*** e0ne has joined #openstack-placement08:15
*** cdent has joined #openstack-placement08:49
*** avolkov has joined #openstack-placement08:52
*** tetsuro has joined #openstack-placement08:59
openstackgerritChris Dent proposed openstack/nova master: [placement] Add /reshaper handler for POST  https://review.openstack.org/57692709:29
*** lei-zh has quit IRC09:35
*** stephenfin has quit IRC09:47
*** stephenfin has joined #openstack-placement09:49
openstackgerritVladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed  https://review.openstack.org/51792110:09
*** e0ne has quit IRC10:27
*** e0ne has joined #openstack-placement10:32
*** edmondsw has joined #openstack-placement10:35
*** edmondsw has quit IRC10:40
*** e0ne has quit IRC10:40
cdentefried: 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 IRC10:55
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs  https://review.openstack.org/56725811:02
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server usage extension response into server view builder  https://review.openstack.org/58526211:09
openstackgerritMerged openstack/nova master: Fix confusing log message in scheduler  https://review.openstack.org/58348611:16
*** cdent has joined #openstack-placement11:53
*** edmondsw has joined #openstack-placement11: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
cdentthere 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 do12:03
cdents/ways/was/12:03
cdentand 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 results12:04
*** e0ne has joined #openstack-placement12:07
openstackgerritRajesh Tailor proposed openstack/nova stable/queens: Make host_aggregate_map dictionary case-insensitive  https://review.openstack.org/58533412:18
openstackgerritRajesh Tailor proposed openstack/nova stable/pike: Make host_aggregate_map dictionary case-insensitive  https://review.openstack.org/58533512:19
openstackgerritMerged openstack/nova master: Make host_aggregate_map dictionary case-insensitive  https://review.openstack.org/49833412:24
*** mriedem has joined #openstack-placement12:28
openstackgerritMatt Riedemann proposed openstack/nova master: Add method to get cpu traits  https://review.openstack.org/56031712:37
openstackgerritMatt Riedemann proposed openstack/nova master: FakeLibvirtFixture: mock get_fs_info  https://review.openstack.org/57920112:38
*** tetsuro has quit IRC12:41
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444012:49
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field  https://review.openstack.org/56444112:49
openstackgerritStephen Finucane proposed openstack/nova master: Add functional tests for numa-aware-vswitches  https://review.openstack.org/58328812:49
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445212:49
openstackgerritStephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series  https://review.openstack.org/58265112:49
openstackgerritStephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations  https://review.openstack.org/58533912:49
openstackgerritStephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations  https://review.openstack.org/58533912:52
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445212:52
openstackgerritStephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series  https://review.openstack.org/58265112:52
openstackgerritMerged openstack/nova master: Fix wonky reqspec handling in conductor.unshelve_instance  https://review.openstack.org/58373912:53
*** cdent has quit IRC12:55
efriedcdent: ack * 213:04
efriedcdent: 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 IRC13:20
*** mriedem is now known as mriedem_away13:24
openstackgerritStephen Finucane proposed openstack/nova master: tox: Silence psycopg2 warnings  https://review.openstack.org/58535213:29
*** cdent has joined #openstack-placement13:40
openstackgerritMerged openstack/nova master: Merge server create for keypair extension  https://review.openstack.org/58320313:47
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs  https://review.openstack.org/56725813:49
*** mriedem_away is now known as mriedem14:11
*** mriedem1 has joined #openstack-placement14:18
*** mriedem has quit IRC14:21
*** mriedem1 is now known as mriedem14:24
openstackgerritsean mooney proposed openstack/nova master: fix disk_bus handling  https://review.openstack.org/58499914:25
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444014:39
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field  https://review.openstack.org/56444114:39
openstackgerritStephen Finucane proposed openstack/nova master: Add functional tests for numa-aware-vswitches  https://review.openstack.org/58328814:39
openstackgerritStephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations  https://review.openstack.org/58533914:39
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445214:39
openstackgerritStephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series  https://review.openstack.org/58265114:39
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks  https://review.openstack.org/58538514:39
openstackgerritBalazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client  https://review.openstack.org/58366714:40
*** purplerbot has quit IRC14:57
*** purplerbot has joined #openstack-placement14:57
*** purplerbot has quit IRC15:08
*** purplerbot has joined #openstack-placement15:08
openstackgerritEric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST  https://review.openstack.org/57692715:13
openstackgerritEric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories  https://review.openstack.org/58503315:13
openstackgerritEric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider sane  https://review.openstack.org/58459815:13
openstackgerritEric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer  https://review.openstack.org/58459915:13
openstackgerritEric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree  https://review.openstack.org/58464815:13
openstackgerritEric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump  https://review.openstack.org/58503415:13
openstackgerritEric Fried proposed openstack/nova master: WIP: Report client: update_from_provider_tree w/reshape  https://review.openstack.org/58504915:13
openstackgerritEric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees  https://review.openstack.org/57623615:13
efriedrebase ^15:13
efriedcdent: 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
efriedBut FYI I'm off this afternoon.15:14
cdentefried: roger that15:15
cdentefried: anything specifc you need/want from me, or is generic attention sufficient?15:16
efriedcdent: Generic and specific15:16
efriedcdent: Sorry, I didn't know how to answer that.15:17
efriedI meant, please review the code, and be specific when you comment.15:17
cdentnoted15:17
efriedbut unless you want to volunteer to write some tests, just reviews15:17
cdenti'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
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-manage placement sync_aggregates  https://review.openstack.org/57591215:19
efriedcdent: 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
cdent15:20
openstackgerritMatt Riedemann proposed openstack/nova master: Log trace when updating host states  https://review.openstack.org/36628315:33
openstackgerritMerged openstack/nova master: Blacklist greenlet 0.4.14  https://review.openstack.org/58501616:06
*** e0ne has quit IRC16:10
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks  https://review.openstack.org/58538516:25
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder  https://review.openstack.org/58547516:39
openstackgerritMerged openstack/nova master: Add method to get cpu traits  https://review.openstack.org/56031716:42
openstackgerritMerged openstack/nova master: FakeLibvirtFixture: mock get_fs_info  https://review.openstack.org/57920116:42
*** tssurya has quit IRC16:43
efriedcdent: jaypipes: I take back what I said about ever needing to support empty inventories for reshape.16:57
efriedI 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
efriedI.e. the only way 'inventories' is itself an empty dict is if there are no providers involved in the reshape.16:57
efriedWhich means the only thing you could possibly be doing is moving allocations around, in which case you should use POST /allocations instead.16:57
efriedIn any case, you should not be using POST /reshaper.16:57
cdentefried: I kinda assumed you had already come round to that understanding.16:58
efriedcdent: Not until just now, when I tried to write a test case that would fail on that condition.16:59
efriedcdent: 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
efriedcdent: I'm afraid I didn't get very far at all - in fact nothing worth committing - and I have to go now.17:03
efriedI 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
cdentefried: 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
efriedroger that.17:05
*** efried is now known as efried_pto17:05
openstackgerritStephen Finucane proposed openstack/nova master: Consider network NUMA affinity for move operations  https://review.openstack.org/58533917:09
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445217:09
openstackgerritStephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series  https://review.openstack.org/58265117:09
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks  https://review.openstack.org/58538517:09
*** e0ne has joined #openstack-placement17:30
openstackgerritMerged openstack/nova master: Add regression test for bug #1764883  https://review.openstack.org/56207217:34
openstackbug 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
openstackgerritAdam Harwell proposed openstack/nova master: Add apply_cells to nova-manage  https://review.openstack.org/56898717:41
mriedemhuh, nothing went into the placement api-ref for microversion 1.2617:42
cdentmriedem: I suspect it was considered semi-invisible?17:44
mriedemnot sure what that means17:44
cdentalso mriedem: can you gaze upon eric's change at https://review.openstack.org/#/c/584648/ and my comment. It sees like he's over-engineering17:44
cdentmriedem: 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 much17:46
mriedemhttps://bugs.launchpad.net/nova/+bug/178338017:46
openstackLaunchpad bug 1783380 in OpenStack Compute (nova) "Placement version 1.26 is not documented in the API reference" [Medium,Triaged]17:46
mriedemit's clearly important for the ironic virt driver as a client,17:46
mriedemand per the commit message on the change that added the microversion, cyborg might also need it during fpga reprogramming,17:46
mriedemand yeah i'm assuming it was just missed17:46
mriedemnot intentional17:46
*** e0ne has quit IRC18:14
*** mriedem1 has joined #openstack-placement18:16
*** mriedem has quit IRC18:16
*** mriedem1 is now known as mriedem18:27
*** alex_xu has quit IRC19:03
openstackgerritMatt Riedemann proposed openstack/nova master: Consider network NUMA affinity for move operations  https://review.openstack.org/58533919:04
openstackgerritMatt Riedemann proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445219:05
openstackgerritMatt Riedemann proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series  https://review.openstack.org/58265119:05
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add functional tests for rebuild with NUMA networks  https://review.openstack.org/58538519:05
*** alex_xu has joined #openstack-placement19:07
openstackgerritMatt Riedemann proposed openstack/nova master: ironic: Report resources as reserved when needed  https://review.openstack.org/51792119:16
openstackgerritMatt Riedemann proposed openstack/nova master: doc: remove rocky-specific nova-scheduler min placement version  https://review.openstack.org/58552019:16
openstackgerritArtom Lifshitz proposed openstack/nova master: Extra logs for volume detach device tags cleanup  https://review.openstack.org/58403219:56
openstackgerritChris Dent proposed openstack/nova master: Use placement context in placement functional tests  https://review.openstack.org/58552919:56
cdentefried_pto, edleafe, gibi: small extraction-relatedf cleanup ^19:57
openstackgerritArtom Lifshitz proposed openstack/nova master: Extra logs for volume detach device tags cleanup  https://review.openstack.org/58403220:05
openstackgerritMatt Riedemann proposed openstack/nova master: Extra logs for volume detach device tags cleanup  https://review.openstack.org/58403220:28
openstackgerritMerged openstack/nova master: Merge server create for config_drive extension  https://review.openstack.org/58353820:57
openstackgerritMerged openstack/nova master: Merge server create for bdm extensions  https://review.openstack.org/58355921:18
openstackgerritMerged openstack/nova master: Trim the fat on HostState.instances  https://review.openstack.org/56924721:18
*** edmondsw has quit IRC21:34
*** cdent has quit IRC21:45
*** cdent has joined #openstack-placement22:06
*** cdent has quit IRC22:14
*** avolkov has quit IRC23:17
*** tetsuro has joined #openstack-placement23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!