| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-02 | |||
| 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 | |
| 01:09:11 | openstackgerrit | Merged openstack/nova stable/rocky: Null out instance.availability_zone on shelve offload https://review.openstack.org/606086 | |
| 01:13:36 | openstackgerrit | Merged openstack/nova stable/rocky: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/604203 | |
| 01:14:48 | openstackgerrit | Merged openstack/nova stable/pike: Follow devstack-plugin-ceph job rename https://review.openstack.org/602022 | |
| 01:14:55 | openstackgerrit | Merged openstack/nova stable/pike: Fix unit test modifying global state https://review.openstack.org/584592 | |
| 02:36:21 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/rocky: Fix aggregate members in nested alloc candidates https://review.openstack.org/607454 | |
| 04:21:31 | mnaser | has anyone seen behaviour (on queens) where doing pci passthrough for gpus, qemu-kvm process goes up but seems to be spinning at 100% cpu and the process is stuck? | |
| 04:21:46 | mnaser | strace just shows a bunch of ioctl's and poll's | |
| 04:22:43 | mnaser | nothing weird in dmesg or journals | |
| 04:23:31 | mnaser | nova side looks ok.. "Final resource view: name=<snip> phys_ram=524194MB used_ram=66560MB phys_disk=1863GB used_disk=225GB total_vcpus=48 used_vcpus=6 pci_stats=[PciDevicePool(count=7,numa_node=0,product_id='102d',tags={dev_type='type-PCI'},vendor_id='10de')]" | |
| 04:45:20 | mnaser | Hmm, I have some leads. I might push some doc changes for the pci passthrough | |
| 04:45:28 | mnaser | Dunno if it’s okay to have docs that are more hardware specific | |
| 04:45:41 | mnaser | Something along the lines of https://github.com/dholt/kvm-gpu/blob/master/README.md | |
| 05:22:06 | takashin | cd /tmp | |
| 06:00:56 | gmann | nova API office hour time | |
| 06:01:34 | gmann | #startmeeting nova api | |
| 06:01:35 | openstack | Meeting started Wed Oct 3 06:01:34 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 06:01:36 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |