| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-02 | |||
| 20:01:15 | bauzas | could we just have alaski back here and just him and me +2/+W a change deprecating multiple-create API ? | |
| 20:01:55 | mriedem | oh god why would we allow attaching volumes to a resized server before it's confirmed/revert | |
| 20:01:57 | mriedem | *reverted | |
| 20:02:01 | mriedem | that's just asking for trouble | |
| 20:02:33 | sean-k-mooney | mriedem: because we did not think that is what we were allowing at the time the code merged ? | |
| 20:03:19 | melwitt | bauzas: users love the multi-create API | |
| 20:03:39 | sean-k-mooney | mriedem: i cound half of the issue with the multiple port bindingings thing by the way. ill file a bug and upload a patch tomorow | |
| 20:03:42 | mriedem | there is a forum session about the multi-create api | |
| 20:03:47 | sean-k-mooney | *found | |
| 20:03:57 | artom | mriedem, I feel like attaching anything to anything that isn't ACTIVE is asking for trouble | |
| 20:04:18 | artom | Like, we should wrap the instance decorator around any method that as 'attach' in its name. | |
| 20:04:28 | artom | *instance state decorator | |
| 20:04:49 | sean-k-mooney | melwitt: do the love the multi create api or multi create support in the client/sdk/osc | |
| 20:05:38 | sean-k-mooney | artom: i think the vm is active in this case on the dest | |
| 20:05:44 | melwitt | mriedem: is it under a more broad topic? I don't see it | |
| 20:05:45 | sean-k-mooney | artom: we just have not confimed it | |
| 20:06:02 | artom | sean-k-mooney, it's still in VERIFY_RESIZE in the API tho | |
| 20:06:07 | artom | I think is what mriedem means | |
| 20:06:10 | melwitt | sean-k-mooney: the API, I think. | |
| 20:06:17 | cfriesen | sean-k-mooney: I'd be open to something like that if people are looking for additional flexibility. making the version explicit would at least protect us if qemu ever supported a newer version | |
| 20:07:05 | sean-k-mooney | cfriesen: you can always default it to 2.0 initally so its optional | |
| 20:07:22 | mriedem | bauzas: melwitt: https://www.openstack.org/summit/berlin-2018/vote-for-speakers#/22840 | |
| 20:08:10 | bauzas | melwitt: I'm not against something elsewhere but not in the API :) | |
| 20:08:28 | bauzas | that said, now the ship has sailed... | |
| 20:08:44 | bauzas | I'm pretty sure we'd get lots of arguments if we deprecate it :) | |
| 20:08:52 | sean-k-mooney | mriedem: i would assume the answer to there first quest is ther is no sla followed by there is no test coverage for that usecase | |
| 20:09:32 | mriedem | i tested how you can kill the scheduler https://review.openstack.org/#/c/507918/ | |
| 20:09:34 | mriedem | if that helps | |
| 20:10:11 | sean-k-mooney | thats a lot of red from the ci | |
| 20:10:29 | mriedem | it's an experiment | |
| 20:10:33 | mriedem | not meant to run tempest | |
| 20:11:31 | melwitt | cool experiment. I wonder could the oslo.messaging heartbeat be leveraged in this way? | |
| 20:11:50 | sean-k-mooney | to kill the schduler proably | |
| 20:11:57 | mriedem | yes, | |
| 20:12:05 | mriedem | it passed once i set the rpc_response_timeout to 300 | |
| 20:12:18 | mriedem | the timeout stuff with the select_destinations rpc call likely needs to use the long_rpc_timeout | |
| 20:12:35 | mriedem | because if it times out, today we retry the rpc call up to max_attempts | |
| 20:12:42 | mriedem | so it just compounds the problem | |
| 20:13:01 | melwitt | yeah, sorry, I saw you increased the timeout and that worked -- that's why I asked. I wondered if we could use the heartbeat to help here | |
| 20:14:06 | mriedem | i think yes, do that and drop the old retry_select_destinations decorator | |
| 20:14:58 | melwitt | ++ | |
| 20:33:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930 | |
| 20:43:44 | openstackgerrit | Merged openstack/nova stable/ocata: unquiesce instance after quiesce failure https://review.openstack.org/605884 | |
| 21:10:19 | openstackgerrit | sean mooney proposed openstack/nova master: propagate vnic_type and profile on create port bindings https://review.openstack.org/607365 | |
| 21:15:37 | sean-k-mooney | mriedem: im goint to repropose ^ later as a bug as i would like to backport it + some other related changes just and fyi but that will partly fix macvtap sriov. | |
| 21:18:32 | sean-k-mooney | mriedem: this patch in partcalar is not super high on my list for the macvtap but per our PTG conversation ther are some other minor bug fixed need to make the codbase actully supprot what we said it would in the rocky release notes | |
| 21:19:43 | sean-k-mooney | s/for the macvtap/for backporting but/ | |
| 21:20:17 | mriedem | ok | |
| 21:20:26 | openstackgerrit | Merged openstack/nova stable/queens: nova-manage - fix online_data_migrations counts https://review.openstack.org/605839 | |
| 21:22:19 | bauzas | mriedem: heh, good news, the inventory reshape works | |
| 21:22:39 | bauzas | mriedem: now, I'm testing the allocations move | |
| 21:22:50 | mriedem | works in...functional tests? | |
| 21:22:53 | mriedem | or a real env? | |
| 21:23:09 | bauzas | mriedem: works on my devstack running on a physical machine :) | |
| 21:23:38 | mriedem | le woot | |
| 21:25:17 | bauzas | mriedem: http://paste.openstack.org/show/731303/ | |
| 21:26:12 | melwitt | that reminds me, did someone create a catchall blueprint for all of the "closing the gaps with nested resource providers" work yet? | |
| 21:26:16 | bauzas | oh oops, now you know my secret machine :p | |
| 21:26:43 | bauzas | melwitt: I already have a vgpu-rocky BP that tracks all of the VGPU effort | |
| 21:26:48 | mriedem | melwitt: probably this https://blueprints.launchpad.net/nova/+spec/use-nested-allocation-candidates | |
| 21:27:18 | bauzas | mriedem: use-nested-alloc-candidates is only one of the gaps but meh | |
| 21:27:33 | mriedem | i'll butt meh you | |
| 21:27:37 | melwitt | ok, thanks. I'm looking to collect the related blueprints and make them dependencies for the vgpu theme | |
| 21:27:56 | mriedem | you can't allocate against vgpu child providers without gibi's bp to get the scheduler stuff to work | |
| 21:28:19 | bauzas | melwitt: mriedem: btw. I reproposed an approved spec for vgpu-stein https://review.openstack.org/#/c/602474/ | |
| 21:28:44 | bauzas | mriedem: yup, I'll rebase the change once I'm done with testing the allocations migration | |
| 21:28:55 | bauzas | mriedem: for this, we don't need the nested-a-c series | |
| 21:29:10 | bauzas | nested-a-c series is only for creating a new instance | |
| 21:29:18 | bauzas | not for a reshape | |
| 21:29:37 | mriedem | sure but.... | |
| 21:29:52 | mriedem | once we model vgpu as child providers, you can't create a new instance with vgpus until that nested-a-c bp is done | |
| 21:30:03 | bauzas | yeah hence my -W | |
| 21:30:06 | melwitt | yeah, both blueprints will be part of the theme | |
| 21:30:11 | bauzas | it should really be a -2 | |
| 21:30:15 | openstackgerrit | Merged openstack/nova stable/ocata: Filter deleted computes from get_all_by_uuids() https://review.openstack.org/604451 | |
| 21:30:21 | openstackgerrit | Merged openstack/nova stable/ocata: Fix nova-status "_check_resource_providers" check https://review.openstack.org/600119 | |
| 21:30:33 | bauzas | anyway, consumer generation is merged | |
| 21:31:02 | bauzas | now, we only really need https://review.openstack.org/#/c/585672/ for the reshaper change, not all the follow-ups | |
| 21:31:04 | bauzas | mriedem: ^ | |
| 21:32:24 | melwitt | mriedem: you're right on use-nested-allocation-candidates being the catchall L53 https://etherpad.openstack.org/p/nova-ptg-stein-priorities | |
| 21:35:27 | mriedem | \o/ | |
| 21:52:12 | bauzas | mriedem: FWIW, confirmed that a flavor asking for vgpus raises a NoValidHost once the reshape is done :) | |
| 21:52:23 | bauzas | we all knew this, it's just an evidence :) | |
| 22:12:05 | dansmith | bauzas: wait, what? | |
| 22:12:19 | dansmith | oh, because we've reshaped to nested but can't yet schedule to nested? | |
| 22:14:24 | melwitt | that was my assumption | |
| 22:16:02 | openstackgerrit | Merged openstack/nova stable/ocata: Use ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa in tree https://review.openstack.org/581445 | |
| 22:16:08 | openstackgerrit | Merged openstack/nova stable/ocata: Fix DB archiver AttributeError due to wrong table name attribute used https://review.openstack.org/599885 | |
| 22:16:14 | openstackgerrit | Merged openstack/nova stable/ocata: Add unshelve instance error info to fault table https://review.openstack.org/583422 | |
| 22:16:32 | openstackgerrit | Merged openstack/nova master: Remove unnecessary redirect https://review.openstack.org/605911 | |
| 22:16:41 | openstackgerrit | Merged openstack/nova master: Replace usage of get_legacy_facade() with get_engine() https://review.openstack.org/606213 | |
| 22:16:51 | openstackgerrit | Merged openstack/nova stable/rocky: Optimize AZ lookup during schedule_and_build_instances https://review.openstack.org/604378 | |
| 22:20:09 | openstackgerrit | melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068 | |
| 22:20:42 | openstackgerrit | melanie witt proposed openstack/nova-specs master: Add a script for counting blueprints https://review.openstack.org/581914 | |
| 22:26:06 | openstackgerrit | melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068 | |
| 23:49:00 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Remove unnecessary redirect https://review.openstack.org/607400 | |
| 23:54:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Remove unnecessary redirect https://review.openstack.org/607402 | |
| 23:57:12 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Remove unnecessary redirect https://review.openstack.org/607402 | |
| #openstack-nova - 2018-10-03 | |||
| 00:56:02 | openstackgerrit | Merged openstack/nova stable/rocky: Delete instance_id_mappings record in instance_destroy https://review.openstack.org/604373 | |
| 01:09:05 | openstackgerrit | Merged openstack/nova stable/rocky: Fix stacktraces with redis caching backend https://review.openstack.org/606895 | |