Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-12-05
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: refactor: RequestGroup.is_empty() and .strip_zeros() https://review.opendev.org/696991
23:08:27 openstackgerrit Eric Fried proposed openstack/nova master: Use Placement 1.34 (string suffixes & mappings) https://review.opendev.org/696418
23:08:28 openstackgerrit Eric Fried proposed openstack/nova master: Use provider mappings from Placement (mostly) https://review.opendev.org/696992
23:08:28 openstackgerrit Eric Fried proposed openstack/nova master: Tie requester_id to RequestGroup suffix https://review.opendev.org/696946
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: Disable NUMATopologyFilter on rebuild https://review.opendev.org/689861
23:21:59 openstackgerrit sean mooney proposed openstack/nova master: Block rebuild when NUMA topology changed https://review.opendev.org/687957
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 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:13 openstackgerrit Merged openstack/nova stable/ocata: Add regression recreate test for bug 1830747 https://review.opendev.org/663143
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 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)
10:21:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Regression test for bug 1849657 https://review.opendev.org/693248
11:22:06 vishalmanchanda hi all.
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:22:27 vishalmanchanda I am looking at a old bug https://bugs.launchpad.net/nova/+bug/1412993.
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 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:19:37 efried There's a bunch of other tests in that suite that have 1.19 hardcoded
13:20:03 efried but that may just be because it never failed before so wasn't noticed :)
13:45:41 openstack Launchpad bug 1855314 in OpenStack Compute (nova) "Typo in documentation of "host aggregates in nova"" [Undecided,New]
13:45:41 mriedem easy docs bug if someone wants to throw up a fix https://bugs.launchpad.net/nova/+bug/1855314
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: Create a controller for qga when SEV is used https://review.opendev.org/693072
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
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_QEMU_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/695915
14:47:49 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_KVM_AARCH64_VERSION https://review.opendev.org/695901
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

Earlier   Later