Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-11
20:53:58 sean-k-mooney cool am im going to go to sleep earlish tonight i think. ill redeploy the series in my test vms tomorow.
20:54:42 sean-k-mooney i think it more or less should work at this point as the last version i tested seamed pretty close
20:55:02 openstackgerrit Artom Lifshitz proposed openstack/nova master: New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
20:55:03 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: Make dest send NUMAMigrateData to the source https://review.openstack.org/634828
20:55:03 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: update NUMA-related XML on the source https://review.openstack.org/635229
20:55:14 artom sean-k-mooney, noice. There ^^
20:55:57 artom I did the change, will continue adding unit tests where I can without too much trouble, and then I'll start on the automated testing
20:57:32 sean-k-mooney oh cool i am going work on some os-vif stuff for another 30 mins or so. i shold have time to spin up a devstack install and manually live migrate a test vm in the background
21:22:42 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Filter duplicates from compute API get_migrations_sorted() https://review.openstack.org/636224
21:53:21 openstackgerrit Merged openstack/nova stable/queens: Handle IndexError in _populate_neutron_binding_profile https://review.openstack.org/635897
21:53:27 openstackgerrit Merged openstack/nova stable/queens: Add functional regression test for bug 1794996 https://review.openstack.org/623354
21:53:28 openstack bug 1794996 in OpenStack Compute (nova) queens "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,In progress] https://launchpad.net/bugs/1794996 - Assigned to Matt Riedemann (mriedem)
22:13:32 mriedem oh blarg, our NeutronFixture is not a KSA client fixture...
23:03:06 mriedem efried: hmm, i'm hoping this isn't a regression http://paste.openstack.org/show/744898/
23:03:28 mriedem this is on rollback of a failed resize where conductor reverts the allocation move by trying to swap the allocations back from the migration consumer to the instance consumer
23:03:40 mriedem the GET /allocations/$consumer_id in the code is using placement 1.28
23:03:45 mriedem which should have project_id in the response
23:04:26 mriedem i wonder if maybe allocations is {} ?
23:05:10 efried That seems possible.
23:05:55 efried reproducible? Stuff in some debug logs?
23:06:56 mriedem OH I KNOW
23:07:15 mriedem so this is the conductor MigrationTask trying to swap the allocs, but the compute service in this case already reverted them
23:07:24 mriedem so conductor is trying to move something that doesn't exist b/c it's already been moved
23:07:36 mriedem not a regression, just not robust code
23:08:58 efried seems like we've seen this before
23:09:09 efried thought we had a fix up that would ignore empty source allocations
23:10:56 mriedem this is a pretty specific part of nova so i'm guessing it's just not been an issue before,
23:11:21 mriedem especially because with normal resize, conductor rpc casts to compute and if something fails in compute, it reverts the allocations and conductor doesn't get a chance (b/c of the cast)
23:11:24 openstackgerrit Eric Fried proposed openstack/nova master: Use placement.inventory.inuse in report client https://review.openstack.org/568639
23:11:27 mriedem with cross-cell resize it's synchronous
23:11:35 mriedem so compute reverts and then conductor tries to revert as well
23:17:10 efried mriedem: here's the change I was thinking of. Slightly different but similar race https://review.openstack.org/#/c/617040/
23:18:35 mriedem yeah this isn't a race that i'm dealing with now, just new code paths that aren't robustified yet
23:18:51 mriedem laying my old cross-cell resize functional tests on top of my current series and flushing out some bugs
#openstack-nova - 2019-02-12
01:09:10 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Start functional testing for cross-cell resize https://review.openstack.org/636253
01:09:31 mriedem woooowee
01:09:35 mriedem dansmith: ^ well that's progress
01:09:57 mriedem retrofitted my previous functional tests to the existing series, flushed a bunch of bugs, and have it tested up through VERIFY_RESIZE
02:01:21 openstackgerrit Merged openstack/nova master: Plumbing for allowing the all-tenants filter with down cells https://review.openstack.org/635145
03:48:33 openstackgerrit Artom Lifshitz proposed openstack/nova master: Remove _legacy_dict methods https://review.openstack.org/636210
03:48:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add migration param to check_can_live_migrate_destination https://review.openstack.org/634605
03:48:34 openstackgerrit Artom Lifshitz proposed openstack/nova master: New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
03:48:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Introduce live_migration_claim() https://review.openstack.org/635669
03:48:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Use live_migration_claim() to check dest resources https://review.openstack.org/634606
03:48:36 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: Make dest send NUMAMigrateData to the source https://review.openstack.org/634828
03:48:36 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: update NUMA-related XML on the source https://review.openstack.org/635229
03:48:37 openstackgerrit Artom Lifshitz proposed openstack/nova master: Trivial: reorder hashes according to object_hashes.txt https://review.openstack.org/636261
03:49:10 artom Actually I should probably put the trivial one at the very bottom...
03:50:35 openstackgerrit Artom Lifshitz proposed openstack/nova master: Trivial: reorder hashes according to object_hashes.txt https://review.openstack.org/636261
03:50:36 openstackgerrit Artom Lifshitz proposed openstack/nova master: Remove _legacy_dict methods https://review.openstack.org/636210
03:50:36 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add migration param to check_can_live_migrate_destination https://review.openstack.org/634605
03:50:37 openstackgerrit Artom Lifshitz proposed openstack/nova master: New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
03:50:37 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Introduce live_migration_claim() https://review.openstack.org/635669
03:50:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Use live_migration_claim() to check dest resources https://review.openstack.org/634606
03:50:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: Make dest send NUMAMigrateData to the source https://review.openstack.org/634828
03:50:39 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: update NUMA-related XML on the source https://review.openstack.org/635229
06:07:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for vol-backed snapshots https://review.openstack.org/615084
06:43:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix resetting non-persistent fields when saving obj https://review.openstack.org/636271
07:12:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix resetting non-persistent fields when saving obj https://review.openstack.org/636271
08:06:52 openstackgerrit Merged openstack/nova master: doc: update the security groups admin doc https://review.openstack.org/634735
08:07:01 openstackgerrit Merged openstack/nova master: doc: link admin/configuration from admin home page https://review.openstack.org/634728
08:42:28 openstackgerrit Merged openstack/nova stable/queens: Fix InstanceNotFound during _destroy_evacuated_instances https://review.openstack.org/623355
09:46:11 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459
09:46:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
09:46:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
09:46:13 openstackgerrit Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310
09:46:13 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311
09:46:14 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
10:05:35 lyarwood bauzas: https://review.openstack.org/#/c/628726/ - would you mind hitting this stable backport today if you have a chance?
10:06:11 bauzas lyarwood: yup, actively working on fixing some problems with the reshape change, but sure will look
10:07:17 lyarwood bauzas: ack thanks
10:14:36 gibi lyarwood: hi! If we are at stable backports, could you check https://review.openstack.org/#/c/635921/1 and https://review.openstack.org/#/c/623358/1 ?
10:14:51 lyarwood gibi: ack can do
10:14:56 gibi thanks!
10:20:06 bauzas gibi: around ?
10:20:14 abhishekk hi all, any idea how can I print queries in logs?
10:20:20 bauzas gibi: I'm puzzled with testing your functional tests again my new revision
10:20:38 abhishekk I have tried to set default_log_levels = sqlalchemy=INFO but its not working
10:20:51 gibi bauzas: push your new revision up and I will check the functional test
10:20:56 bauzas ack
10:21:01 bauzas it will fail your change tho
10:21:06 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Reject migration requests when source is down https://review.openstack.org/623489
10:21:33 bauzas gibi: I'll just add a new change for correctly using the right mdev before tho
10:21:36 gibi bauzas: you mean it will fail the functional test? That is OK I will check that
10:21:46 lyarwood sean-k-mooney / stephenfin / mdbooth_ ^ I'd appreciate your thoughts on my bugfix above if you have time today
10:22:33 lyarwood oh and is anyone else having weird issues building releasenotes on f29?
10:23:54 stephenfin and tries building releasenotes to check
10:24:44 lyarwood stephenfin: /src/openstack/nova/releasenotes/source/rocky.rst:3436:Duplicate explicit target name: "rest api version history". - is the error I'm getting locally
10:27:45 stephenfin lyarwood: That doesn't sound like something F29 specific. That would happen if someone had defined the same target in two release notes, which it looks like someone has
10:28:01 stephenfin though it's building just fine for me...
10:28:42 lyarwood stephenfin: kk there must be some cruft in my local repo somewhere
10:29:10 stephenfin 'git clean -xfd -e .tox' <-- I should have this aliased
10:42:21 openstackgerrit Boris Bobrov proposed openstack/nova master: Fix memtune parameters according to libvirt docs https://review.openstack.org/636301
10:43:43 openstackgerrit Boris Bobrov proposed openstack/nova master: Fix memtune parameters according to libvirt docs https://review.openstack.org/636301
12:00:24 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add method to allow reset fields for root bdm in BDM obj https://review.openstack.org/614672
12:10:32 openstackgerrit Boris Bobrov proposed openstack/nova master: Add configs for AMD SEV https://review.openstack.org/636318
12:45:27 openstackgerrit Jim Rollenhagen proposed openstack/nova master: Ironic: bump minimum API version to 1.38 https://review.openstack.org/636326
12:45:46 jroll TheJulia: ^ sanity check me that this is cleanup we want to do, please
13:14:09 mriedem tssurya: i think we might have some traces in the api logs that are saying a cell is down when it's really something else
13:14:09 mriedem http://logs.openstack.org/67/620667/3/check/tempest-full/8fa0aa0/controller/logs/screen-n-api.txt.gz?level=TRACE#_Feb_12_00_25_30_615423

Earlier   Later