| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-12-05 | |||
| 22:56:55 | efried | yeah, I think we wanted to be able to use it from the compute | |
| 22:56:57 | efried | but hadn't yet. | |
| 22:57:22 | sean-k-mooney | ok if we havent yet its not an upgrade issue :) | |
| 22:57:35 | sean-k-mooney | we will have the placment native way before we do | |
| 22:57:38 | efried | Like, to correlate VGPUs with the proper mdevs, and eventually to make sure we pin CPUs from the right NUMA nodes. | |
| 22:57:50 | sean-k-mooney | yep that is what i was thinking of | |
| 22:58:07 | sean-k-mooney | we will need to use the mapping when doing the physical assignment on the compute node | |
| 22:58:20 | sean-k-mooney | to make sure we assign the resocue that corralates with the allocation | |
| 22:58:22 | efried | actually, I guess bw gets around that today because the mapping is stuffed into the binding profile, so (I'm guessing) the compute pulls it from there. | |
| 22:58:33 | efried | to... figure out which NIC to assign? | |
| 22:58:49 | efried | start talking network stuff, I hear woh woh woh | |
| 22:58:51 | sean-k-mooney | in the sriov case maybe | |
| 22:58:57 | sean-k-mooney | in the ovs case it does not matter | |
| 23:00:04 | sean-k-mooney | actully in the sriov cas ethe port request has the pf netdev name and we add that to the pci_request object | |
| 23:00:17 | sean-k-mooney | so again stuffing stuff in the vif object to avoid the problem | |
| 23:01:23 | efried | I think ima rebase on the "skip the bw test" patch, and put one on top unskipping to prove it works. | |
| 23:01:31 | efried | so I don't have to recheck a zillion times on that stupid multi-network bug. | |
| 23:08:27 | openstackgerrit | Eric Fried proposed openstack/nova master: Use Placement 1.34 (string suffixes & mappings) https://review.opendev.org/696418 | |
| 23:08:27 | openstackgerrit | Eric Fried proposed openstack/nova master: refactor: RequestGroup.is_empty() and .strip_zeros() https://review.opendev.org/696991 | |
| 23:08:28 | openstackgerrit | Eric Fried proposed openstack/nova master: Tie requester_id to RequestGroup suffix https://review.opendev.org/696946 | |
| 23:08:28 | openstackgerrit | Eric Fried proposed openstack/nova master: Use provider mappings from Placement (mostly) https://review.opendev.org/696992 | |
| 23:08:29 | openstackgerrit | Eric Fried proposed openstack/nova master: Restore test_minbw_allocation_placement in nova-next job https://review.opendev.org/697586 | |
| 23:13:58 | efried | mriedem, gibi: FYI, re-restoring that minbw test via https://review.opendev.org/#/c/697586/ | |
| 23:14:30 | efried | It's at the top of that series for now so I can prove the top patch works, can move it later if necessary. | |
| 23:15:27 | efried | and with that, I'm outta here o/ | |
| 23:21:59 | openstackgerrit | sean mooney proposed openstack/nova master: Block rebuild when NUMA topology changed https://review.opendev.org/687957 | |
| 23:21:59 | openstackgerrit | sean mooney proposed openstack/nova master: Disable NUMATopologyFilter on rebuild https://review.opendev.org/689861 | |
| 23:22:00 | openstackgerrit | sean mooney proposed openstack/nova master: support pci numa affinity policies in flavor and image https://review.opendev.org/674072 | |
| 23:23:52 | sean-k-mooney | also rebased that on top of the same skip patch to avoid the rechecks | |
| #openstack-nova - 2019-12-06 | |||
| 00:41:15 | mriedem | efried: ack yeah i was thinking about pushing a revert on top of the skip that depends on the tempest fix, but you beat me to it | |
| 01:22:13 | openstackgerrit | Merged openstack/nova stable/train: compute: Use long_rpc_timeout in reserve_block_device_name https://review.opendev.org/696953 | |
| 01:49:06 | openstackgerrit | Merged openstack/nova stable/train: compute: Take an instance.uuid lock when rebooting https://review.opendev.org/696151 | |
| 01:49:13 | openstackgerrit | Merged openstack/nova stable/ocata: Add regression recreate test for bug 1830747 https://review.opendev.org/663143 | |
| 01:49:13 | openstack | bug 1830747 in OpenStack Compute (nova) ocata "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem) | |
| 01:49:18 | openstackgerrit | Merged openstack/nova stable/ocata: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/663144 | |
| 01:49:25 | openstackgerrit | Merged openstack/nova stable/rocky: Replace time.sleep(10) with service forced_down in tests https://review.opendev.org/696217 | |
| 01:49:32 | openstackgerrit | Merged openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058 | |
| 02:20:26 | openstackgerrit | Merged openstack/nova master: Extend NeutronFixture to allow live migration with ports https://review.opendev.org/696245 | |
| 02:57:52 | eandersson | mnaser and well anyone else with RabbitMQ issues - we wrote this script we are using to audit our RabbitMQ servers http://paste.openstack.org/show/787203/ | |
| 02:58:41 | eandersson | After a network partition (or rmq crash) sometimes the consumers don't get properly cleaned up on the queues. | |
| 02:59:05 | eandersson | This can cause some very frustrating issues with services that don't use RPC (e.g. nova). | |
| 02:59:13 | eandersson | Notifications would also be heavily impacted. | |
| 03:08:38 | openstackgerrit | Merged openstack/nova master: Make the binding:profile handling consistent in NeutronFixture https://review.opendev.org/696526 | |
| 03:08:47 | openstackgerrit | Merged openstack/nova master: Skip test_minbw_allocation_placement in nova-next job https://review.opendev.org/697491 | |
| 06:01:54 | openstackgerrit | Merged openstack/nova stable/train: block_device: Copy original volume_type when missing for snapshot based volumes https://review.opendev.org/696683 | |
| 06:46:55 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717 | |
| 07:46:33 | kaisers | @mriedem thanks for the CI hint, will check it out. The CI did report but it seems there's a devstack setup issue the kills the job setup quite early... | |
| 08:38:47 | gibi | efried: thanks for the https://review.opendev.org/#/c/697586/ I wanted to do that yesterday but run out of time | |
| 09:07:39 | openstackgerrit | Merged openstack/nova master: xenapi: Remove vestigial nova-network support https://review.opendev.org/696505 | |
| 09:07:45 | openstackgerrit | Merged openstack/nova master: nova-net: Flatten class hierarchy for neutron SG tests https://review.opendev.org/696506 | |
| 09:07:52 | openstackgerrit | Merged openstack/nova master: nova-net: Remove SG tests that don't apply to neutron https://review.opendev.org/696507 | |
| 10:21:24 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Regression test for bug 1849657 https://review.opendev.org/693248 | |
| 10:21:24 | openstack | bug 1849657 in OpenStack Compute (nova) train " allocation key is missing from the binding:profile of the neutron qos port when the server is created by a non-admin user" [Medium,Fix committed] https://launchpad.net/bugs/1849657 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 11:22:06 | vishalmanchanda | hi all. | |
| 11:22:27 | vishalmanchanda | I am looking at a old bug https://bugs.launchpad.net/nova/+bug/1412993. | |
| 11:22:27 | openstack | Launchpad bug 1412993 in OpenStack Compute (nova) "Nova resize for boot-from-volume instance does not resize volume" [Undecided,Invalid] - Assigned to Claudiu Belu (cbelu) | |
| 11:23:03 | vishalmanchanda | is this the expected behavior. | |
| 11:23:45 | vishalmanchanda | any help would be really appreciated. | |
| 11:26:49 | lyarwood | vishalmanchanda: yeah it's expected, the flavor doesn't dictate the size of the volume used during boot from volume | |
| 11:27:35 | lyarwood | vishalmanchanda: you can resize the volume by calling the cinder api directly even when attached for most volume backends now | |
| 11:28:35 | lyarwood | vishalmanchanda: https://docs.openstack.org/api-ref/block-storage/v3/?expanded=extend-a-volume-size-detail | |
| 11:31:29 | vishalmanchanda | lyarwood: is this generic for all volume-type or specific to some volume-types only. | |
| 11:34:48 | lyarwood | vishalmanchanda: it depends on the volume backend but most support this now | |
| 11:56:40 | vishalmanchanda | lyarwood: ok thanks for the information. | |
| 11:57:20 | vishalmanchanda | Could you tell me from which openstack release this feature is added? | |
| 11:57:53 | vishalmanchanda | I am facing this issue on rocky and RHELOSP14. | |
| 12:00:48 | lyarwood | vishalmanchanda: first introduced in Queens / 13 with rbd and some other volume backends only gaining support in Rocky / 14. | |
| 12:04:22 | vishalmanchanda | I am using ceph as backend in rocky and RHELOSP14. | |
| 12:05:04 | vishalmanchanda | but instance disk is not resizing. | |
| 12:06:00 | lyarwood | vishalmanchanda: have you rescanned the disk within the instance? | |
| 12:06:29 | lyarwood | vishalmanchanda: or is the cinder command / c-api failing? | |
| 12:07:26 | vishalmanchanda | Command is not failing but disk is not resizing acc. to flavor. | |
| 12:07:53 | lyarwood | vishalmanchanda: as I said, resizing the instance isn't going to resize any attached cinder volumes | |
| 12:07:59 | lyarwood | vishalmanchanda: you need to call cinder directly to do that | |
| 12:08:07 | lyarwood | vishalmanchanda: cinder --os-volume-api-version 3.42 extend $volume $size | |
| 12:08:24 | lyarwood | vishalmanchanda: and then rescan within the instance for it to pickup the increased block device size | |
| 12:10:57 | vishalmanchanda | lyarwood: ok thanks for the help. I will check and let you know. | |
| 12:11:10 | vishalmanchanda | Have a good day:) | |
| 12:20:29 | lyarwood | np | |
| 12:43:44 | openstackgerrit | Merged openstack/nova master: Remove unused rootwrap filters https://review.opendev.org/697134 | |
| 13:11:57 | efried | gibi: Do we still need to keep tests where allocation_request_version is less than the latest? | |
| 13:15:55 | gibi | efried: good question. I think those are created in the scheduler and scheduler is updated along with the controller services first | |
| 13:16:07 | gibi | so that should be always the latest | |
| 13:17:16 | gibi | but I remember there was some controvery along this line | |
| 13:17:31 | efried | e.g. this one https://opendev.org/openstack/nova/src/branch/master/nova/tests/unit/conductor/tasks/test_migrate.py#L730 | |
| 13:17:44 | efried | Is it possible to actually get 1.19 there? | |
| 13:19:07 | gibi | efried: no I don't think so. | |
| 13:19:23 | efried | I'll just change it to 1.34 and make the allocation requests modern. | |
| 13:19:37 | efried | There's a bunch of other tests in that suite that have 1.19 hardcoded | |
| 13:19:37 | gibi | efried: that test checks the old compute service version and that is still valid test but you can change the request version to latest | |
| 13:20:03 | efried | but that may just be because it never failed before so wasn't noticed :) | |
| 13:45:41 | mriedem | easy docs bug if someone wants to throw up a fix https://bugs.launchpad.net/nova/+bug/1855314 | |
| 13:45:41 | openstack | Launchpad bug 1855314 in OpenStack Compute (nova) "Typo in documentation of "host aggregates in nova"" [Undecided,New] | |
| 13:54:59 | mriedem | please to be getting core reviews on these stable stein backports that have a +2 already https://review.opendev.org/#/q/topic:bug/1852610+(status:open+OR+status:merged)+branch:stable/stein | |
| 13:56:59 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Switch to uses_virtio to enable iommu driver for AMD SEV https://review.opendev.org/696697 | |
| 13:57:00 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Also enable iommu for virtio controllers and video in libvirt https://review.opendev.org/684825 | |
| 13:57:00 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Create a controller for qga when SEV is used https://review.opendev.org/693072 | |
| 14:11:29 | kashyap | Probably on PTO | |
| 14:16:41 | mriedem | efried: questions in https://review.opendev.org/#/c/696418/ - also wondering about a reno | |
| 14:34:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove dict compat from populate_filter_properties https://review.opendev.org/697669 | |