| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-12-05 | |||
| 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 | |
| 14:41:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove now invalid cells v1 comments from conductor code https://review.opendev.org/697670 | |
| 14:47:49 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_KVM_AARCH64_VERSION https://review.opendev.org/695901 | |
| 14:47:49 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_QEMU_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/695915 | |
| 14:47:50 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_FILE_BACKED_VERSION https://review.opendev.org/695942 | |
| 15:12:54 | melwitt | kashyap: would you mind revisiting this review pls? https://review.opendev.org/589769 author has updated after your comments | |
| 15:18:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fup for I63c1109dcdb9132cdbc41010654c5fdb31a4fe31 https://review.opendev.org/697678 | |
| 15:37:09 | gibi | stephenfin: what kind of generic exception type you would like to see in https://review.opendev.org/#/c/696574/5/nova/objects/instance.py@1238 ? ValueError? for both cases? | |
| 15:40:23 | gibi | stephenfin: I fine not to have BuildAbortException there directly I'm just not sure if you really want a blanked ValueError for both cases or something more specific like AmbiguousResourceProviderForPciRequest and UnexpectResourceProviderNameForPciRequest for the two cases? | |
| 15:40:37 | gibi | s/blanked/blanket/ | |
| 15:50:37 | gibi | stephenfin: OK I think I going for the nicer named exceptions and you can express your opinion in the review :) | |
| 15:54:06 | efried | mriedem: Yeah, gibi, how do you feel about that? Should we start advertising (in docs etc) that you can use string suffixes in your flavors? I kinda feel like it's something we could do, but is certainly not necessary, so that could be a separate change. | |
| 15:55:59 | gibi | efried: do I understand correctly that the old numbered syntaxt still works? | |
| 15:56:06 | efried | absolutely | |
| 15:57:56 | gibi | efried: I think it is worth to add a sentence or two to the flavor doc. But that can be done in a follow up | |
| 15:58:06 | efried | yeah | |
| 15:58:10 | efried | mriedem: okay with that? | |
| 15:58:47 | mriedem | i'm fine with a fup | |
| 15:58:58 | mriedem | those existing tests don't need to change then right? | |
| 16:00:28 | efried | mriedem: they needed to change a little bit because some of the previously-bogus extra specs are now not-ignored. | |
| 16:00:41 | efried | but no, I didn't need to change as many as I did. | |
| 16:00:45 | efried | see response in the patch. | |
| 16:05:22 | mriedem | ack, will reply and +2 in a sec | |
| 16:05:32 | mriedem | waiting to push up my refactor of build_request_spec since you tickled it :) | |
| 16:05:48 | mriedem | these todos from bauzas have been wafting in here for years | |