Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-12-18
18:01:36 mriedem stephenfin: reviewed https://review.opendev.org/#/c/679502/
18:03:34 stephenfin thanks
18:21:22 openstackgerrit Luigi Toscano proposed openstack/nova stable/train: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699753
18:28:06 openstackgerrit Merged openstack/nova master: Optimization for nova-api _checks_for_create_and_rebuild https://review.opendev.org/698125
18:36:43 KeithMnemonic1 I know the holidays are coming up, but i appreciate any more reviews to help this get merged before the year ends https://review.opendev.org/691045
19:57:59 mriedem melwitt: if you're around the bottom 4 nova-net removal patches are pretty easy, starts at https://review.opendev.org/#/c/696510/
21:17:36 gmann stephenfin: did not get your comment on this - https://review.opendev.org/#/c/657698/14
22:07:47 mriedem get yer gate bugs hyeah https://bugs.launchpad.net/nova/+bug/1856902
22:07:47 openstack Launchpad bug 1856902 in OpenStack Compute (nova) "nova.tests.functional.libvirt.test_vpmem.VPMEMTests.test_create_servers_with_vpmem fails intermittently" [Medium,Confirmed]
#openstack-nova - 2019-12-19
00:06:41 openstackgerrit Merged openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048
02:27:36 luyao artom, stephenfin, sean-k-mooney: spec 'support live migration with vpmem' updated. https://review.opendev.org/695863
02:28:24 artom lyarwood, ack, hopefully I can get to it before my holiday PTO on friday
02:29:07 luyao artom: thanks
04:38:07 openstackgerrit Merged openstack/nova master: Remove now invalid cells v1 comments from conductor code https://review.opendev.org/697670
06:07:30 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Add action event fault details https://review.opendev.org/699669
06:58:54 openstackgerrit Merged openstack/nova stable/rocky: Add --dry-run option to heal_allocations CLI https://review.opendev.org/698525
06:58:59 openstackgerrit Merged openstack/nova stable/rocky: Add --instance option to heal_allocations https://review.opendev.org/698529
06:59:05 openstackgerrit Merged openstack/nova stable/rocky: Add BFV wrinkle to TestNovaManagePlacementHealAllocations https://review.opendev.org/698531
07:15:59 openstackgerrit Merged openstack/nova stable/train: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699753
07:41:17 openstackgerrit Luigi Toscano proposed openstack/nova stable/stein: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699924
12:17:13 stephenfin dtantsur: Wanna give me core on x/wsme so I can go merge a load of my own stuff? :) Maintainers seem a bit thin on the ground
13:15:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905
13:15:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for qos live migration reschedule https://review.opendev.org/699015
13:15:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Move common test method up to base class https://review.opendev.org/699064
13:15:27 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for abort and cancel live migration https://review.opendev.org/699065
13:15:27 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066
13:15:28 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix get_request_group_mapping doc https://review.opendev.org/699979
13:20:32 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905
13:20:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix get_request_group_mapping doc https://review.opendev.org/699979
13:20:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for qos live migration reschedule https://review.opendev.org/699015
13:20:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: Move common test method up to base class https://review.opendev.org/699064
13:20:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for abort and cancel live migration https://review.opendev.org/699065
13:20:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066
13:44:57 shilpasd zzzeek: hi
14:22:26 mriedem relatively simple docs bug if someone wants it https://bugs.launchpad.net/nova/+bug/1857013
14:22:26 openstack Launchpad bug 1857013 in OpenStack Compute (nova) "Configure live migrations in nova - timeout section for kvm doesn't mention live_migration_timeout_action" [Low,New]
14:25:44 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Zuul v3: use devstack-plugin-nfs-tempest-full https://review.opendev.org/699924
14:27:08 tosky mriedem: oh, do you nova people require a cherry pick footer for each item in the chain?
14:27:23 mriedem tosky: preferred
14:27:29 mriedem to see that they are going in order
14:27:38 mriedem since it was a clean cherry-pick i just fixed it using the gerrit ui
14:27:45 tosky yep, noticed
14:28:01 mriedem alex_xu: this functional test race failure is spiking http://status.openstack.org/elastic-recheck/#1856902
14:28:09 mriedem i'm inclined to skip that bug until someone can post a fix
14:28:18 mriedem stephenfin: ^ any thoughts on that?
14:36:12 shilpasd mriedem:hi
14:36:34 mriedem oh is see the bug
14:37:21 shilpasd mriedem: for https://review.opendev.org/#/c/694462/3/nova/db/sqlalchemy/api.py@2010, i have made changes suggested, but its returning list of tuples
14:37:47 mriedem stephenfin: i think the issue is the delete server helper in the libvirt functional tests is no longer waiting for the server to be gone
14:37:56 mriedem probably due to the big refactoring going on
14:38:59 mriedem shilpasd: ok, i guess you'd have to debug it
14:39:26 mriedem stephenfin: or this was just always racy and we're hitting fast nodes
14:42:49 shilpasd mriedem: can you please check 'http://paste.openstack.org/show/78779' lines 40 and 41
14:43:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix race in test_create_servers_with_vpmem https://review.opendev.org/699997
14:43:21 mriedem stephenfin: melwitt: ^
14:43:21 shilpasd mriedem: i need to iterate again to have list of uuids
14:43:46 mriedem that paste doesn't work
14:45:54 shilpasd mriedem: means its wrong approach?
14:46:09 mriedem i mean your paste url is broken
14:47:18 shilpasd http://paste.openstack.org/show/787793/
14:47:28 shilpasd corrected one
14:48:24 mriedem i don't know if filter(and_(...)) works the same as filter_by chained together, i generally have to play with sqlalchemy orm stuff when doing things like this
14:48:28 mriedem which is what i meant by debugging
14:49:38 shilpasd okay, will do that and check
14:50:02 shilpasd tried filter_by but how 'in_' parameter works with filter_by
14:50:20 shilpasd models.BlockDeviceMapping.instance_uuid.in_(instance_uuids)
14:57:04 mriedem shilpasd: if i knew the answer off the top of my head i'd tell you
14:57:08 mriedem you're going to have to play with it
14:57:11 mriedem as would i
14:59:47 shilpasd mriedem: yes, will do it
15:04:50 mriedem shilpasd: this works for me http://paste.openstack.org/show/787794/
15:07:17 shilpasd mriedem: thanks will add, this is more better that what i had
15:07:22 shilpasd will check this
15:22:38 shilpasd mriedem: it failing if we have list items more than 1, will look further how to resolve this
15:23:23 shilpasd mriedem: and regarding deletion of TC 'https://review.opendev.org/#/c/694462/3/nova/tests/unit/db/test_db_api.py@993'
15:24:06 shilpasd i have removed since observed that deleted instance are not considered in 'os-simple-tenant-usage'
15:24:09 shilpasd so removed
15:24:44 tosky mriedem: re the devstack-plugin-nfs job backport: I think it should work flawlessly up to pike, but I'm a bit unsure what to do with ocata
15:25:11 tosky because devstack/ocata lacks most of the infrastructure for zuul v3 jobs (mainly roles)
15:26:06 tosky would it be fine to copy the legacy job in-tree? After cleaning up the legacy job also in cinder, I'd like to remove it from project-config
15:29:48 stephenfin mriedem: Skip it
15:30:20 stephenfin I'm in a meeting rn but I can work on switching that test to use events instead of polling status, since that should be less racey
15:31:09 mriedem stephenfin: i already have a fix posted https://review.opendev.org/699997
15:31:44 mriedem tosky: i would just remove it from queens, pike and ocata
15:32:01 mriedem it's a non-voting experimental queue job so it doesn't get noticed anyway, and those branches are in extended maintenance mode
15:32:27 tosky mriedem: oh, ok, thanks
15:32:43 mriedem shilpasd: you shouldn't remove db api test code just because of how one caller is going to use this
15:33:08 mriedem shilpasd: you have to stop thinking about the db api layer for this as something solely for the simple tenant usage stuff, i.e. not using the active_by_window_joined method for testing it
15:33:08 stephenfin I wonder why that suddenly changed. None of my changes touched that file :/
15:33:13 stephenfin Except for 7ae1a1091326292b02d879ae5410d5e7e78d3997 but that's unrelated
15:33:21 mriedem stephenfin: maybe just hitting faster nodes
15:33:27 stephenfin Could be
15:33:30 mriedem when it fails it's on some rax nodes
15:33:38 mriedem fewer noisy neighbors right now
15:33:44 stephenfin +W'd anyway
15:33:46 stephenfin nice find
15:41:02 stephenfin mriedem_afk: Should I figure out some way to break up https://review.opendev.org/#/c/696514/ to get it over the line?
16:33:23 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Handle new is_volume_backend join column query https://review.opendev.org/694462
16:33:23 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Instance object changes for the new 'is_volume_backed' expected_attr https://review.opendev.org/694463
16:33:24 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Ignore root_gb if instance is booted from volume https://review.opendev.org/612626

Earlier   Later