Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-12
16:53:59 mriedem i guess i can respin, remove the model change, add a FIXME and report a bug to oslo.db
16:54:03 jaypipes mriedem: maybe we just put a TODO in there?
16:54:13 mriedem yeah i'll do that quick
16:54:15 jaypipes mriedem: or I can do it. just let me know.
16:54:31 mriedem i've got it local, 2 seconds
16:54:41 jaypipes k, thx mriedem
16:55:55 simondodsley_ mriedem: cinder docs review added https://review.openstack.org/#/c/574804/
16:55:56 cfriesen mnaser: for what it's worth, for "local" filesystems we don't copy over the swap file but instead create a whole new one, so the guest sees the "right" swap size when it boots up after the resize.
16:56:30 mnaser cfriesen: ah you are right about that one, so we do kinda make the drive disappear with resize
17:07:22 openstackgerrit Merged openstack/nova master: Use instance project/user when creating RequestSpec during resize reschedule https://review.openstack.org/571245
17:12:35 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
17:12:36 openstackgerrit Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435
17:12:37 openstackgerrit Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857
17:13:21 openstackgerrit Chris Dent proposed openstack/nova master: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067
17:15:01 openstackgerrit Matt Riedemann proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958
17:15:04 mriedem jaypipes: ^ but i gave up on trying to rebase the series after https://review.openstack.org/#/c/572280/ caused merge conflicts
17:15:08 mriedem i'll leave that rebase to you
17:22:32 jaypipes mriedem: roger
17:24:13 openstackgerrit Merged openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459
17:44:44 tasker mriedem: is there something that needs to be done for https://review.openstack.org/#/c/550865/? also, can it be backported to Pike, or is that window closed? ( re: https://bugs.launchpad.net/nova/+bug/1754360 )
17:44:46 openstack Launchpad bug 1754360 in OpenStack Compute (nova) "no unquiesce for volume backed on quiesce failure" [Medium,In progress] - Assigned to Eric M Gonzalez (egrh3)
17:45:32 cdent mriedem: is "oslo_policy/policy.py:865: UserWarning: Policy placement:resource_providers:traits:update failed scope check. The token used to make the request was project scoped but the policy requires ['system'] scope. This behavior may change in the future where using the intended scope is required" something you're already aware of and I don't need to think about?
17:48:37 mriedem tasker: just requires reviews; pike is open for backports
17:49:13 mriedem tasker: and for a long time https://superuser.openstack.org/articles/extended-maintenance-openstack/
17:51:07 mriedem cdent: i'm aware of it, and slightly annoyed at how verbose that is
17:51:11 mriedem especially if it's not cached
17:51:16 openstackgerrit Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262
17:51:36 mriedem basically if you have a scope type set on a policy rule that gets used, and enable_scope=False (the default) then you get spammed with warnings
17:51:40 cdent mriedem: yeah. you kinda wanna be able to say omg, I know
17:51:50 mriedem probably needs to be a bug in oslo.policy
18:04:46 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
18:04:47 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
18:04:48 openstackgerrit Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
18:10:49 mriedem bpoulos: you moved https://review.openstack.org/#/c/479949/76..77/nova/compute/manager.py but didn't move the related unit test
18:13:46 mriedem oh nvm i guess it's here https://review.openstack.org/#/c/561262/26/nova/tests/unit/compute/test_compute_mgr.py
18:17:20 mriedem bpoulos: ok there is just 1 missing test in https://review.openstack.org/#/c/479949/ - i can add that if you like
18:22:38 bpoulos mriedem: that would be great, thanks!
18:43:55 openstackgerrit Simon Dodsley proposed openstack/nova master: Add enhanced KVM storage QoS quotas https://review.openstack.org/558530
18:45:06 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
18:45:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
18:45:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
18:45:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
18:45:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
19:13:23 openstackgerrit Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958
19:13:24 openstackgerrit Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405
19:13:25 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
19:13:26 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
19:13:27 openstackgerrit Jay Pipes proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324
19:13:44 jaypipes mriedem: ^ rebased and merge-conflict-fixed.
19:16:28 mriedem jaypipes: https://review.openstack.org/#/c/557958/29..30/nova/db/sqlalchemy/api_models.py
19:24:35 melwitt mriedem: great, thanks for swapping the runway
19:28:14 openstackgerrit Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958
19:28:15 openstackgerrit Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405
19:28:16 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
19:28:17 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
19:28:18 openstackgerrit Jay Pipes proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324
19:28:20 jaypipes mriedem: sorry about that. fixed.
19:53:14 openstackgerrit Artom Lifshitz proposed openstack/nova master: DNM: Depends on intel-nfv-ci-tests to test NUMA LM https://review.openstack.org/574872
20:00:20 openstackgerrit Merged openstack/nova master: Refactor libvirt get_memory_used_mb() https://review.openstack.org/571030
20:31:59 mriedem dansmith: i'm assuming you could have cert validation setup on kernel and ramdisk images...?
20:32:12 mriedem because this libvirt driver change isn't passing the trusted certs through in that case
20:32:19 mriedem not sure if that was intentional
20:32:22 dansmith mriedem: um
20:32:37 dansmith mriedem: if you use that kind of image, each would have its own signature
20:32:40 mriedem in general, i assume any image we download from glance could be signed and verified
20:32:50 dansmith aren't we buried deep in the image code enough that we'd verify those like anything else/
20:33:16 mriedem i think so https://review.openstack.org/#/c/561262/27/nova/virt/libvirt/driver.py@7432
20:33:28 mriedem this is the trouble part that's missing passing through trusted_certs
20:34:20 dansmith mriedem: you know I didn't write any of that code right?
20:34:49 mriedem IT'S YOUR GD CODE NOW SMITH
20:34:59 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398
20:35:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: libvirt live migration: fit NUMA topology on dest https://review.openstack.org/567242
20:35:01 openstackgerrit Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723
20:35:02 openstackgerrit Artom Lifshitz proposed openstack/nova master: DNM: Depends on intel-nfv-ci-tests to test NUMA LM https://review.openstack.org/574872
20:35:05 mriedem your prints are all over this
20:35:17 mriedem it's 1/8 my baby now too
20:35:22 dansmith I really have lost most of my context on it at this point
20:35:28 dansmith I just split this off from the main patch for them
20:35:32 mriedem i know
20:35:38 mriedem and this change could be split into 2 now too
20:35:43 mriedem virt changes and then compute manager changes
20:35:45 mriedem but i'm not going to do that
20:37:05 dansmith yeah, so I guess you're right, since they don't pass trusted_certs there,
20:37:18 dansmith but if they did what you suggest (which I'm pretty sure I had suggested already) then they'd auto-verify
20:39:30 mriedem yeah i moved the low-level calls to libvirt_utils.fetch_image to just use the in-scope instance variable and pass instance.trusted_certs rather than add a new parameter to the various _try_fetch_image* methods
20:39:37 mriedem that's how i found this bug
20:45:12 mriedem gdi, i can't really tell how to assert anything that is mocked out from ImageBackendFixture
20:50:52 efried mriedem: example?
20:51:31 mriedem don't have the energy
20:51:46 efried you want to assert that the _create_ephemeral mock was called?
20:51:46 efried self.mock_create_ephemeral))
20:51:46 efried 'nova.virt.libvirt.driver.LibvirtDriver._create_ephemeral',
20:51:46 efried self.useFixture(fixtures.MonkeyPatch(
20:51:46 efried mock.create_autospec(driver.LibvirtDriver._create_ephemeral)
20:51:46 efried self.mock_create_ephemeral = \
20:51:46 efried mriedem: You mean from something like
20:52:06 mriedem more or less yes,
20:52:07 mriedem was doing:
20:52:10 efried mriedem: You have to make the fixture save off the actual mock
20:52:13 mriedem img_backend_fixture = snaz

Earlier   Later