| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-12-05 | |||
| 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 | |
| 16:07:26 | efried | heh. wafting. | |
| 16:10:18 | donnyd | So I am hoping I can get a pointer with the right direction to go. I am having an issue where VM's just get stuck in scheduling. No errors, no logs and the hypervisor reports its up. Never seen nova do this before, there is usually something that says I couldn't build a machine because X. | |
| 16:10:44 | efried | gibi: FYI I'm having trouble with this one "unit" test (not sure how it gets away with not being in the functional set): https://opendev.org/openstack/nova/src/branch/master/nova/tests/unit/conductor/test_conductor.py#L1046 | |
| 16:10:58 | efried | donnyd: which logs have you inspected? | |