Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-24
21:51:37 mriedem think i know what's causing the spike in the functional tests failing
21:51:46 mriedem https://review.openstack.org/#/c/484154/2/nova/tests/functional/api/openstack/placement/gabbits/resource-class-in-use.yaml
21:52:00 mriedem i think we have gabbits racing that work against the same custom resource class
21:52:46 dansmith they use the same db instance in parallel?
21:52:57 mriedem creating the same custom resource class,
21:53:17 mriedem in one case it already exists i think so it returns 204
21:53:26 mriedem which fails the assertion for a 201
21:53:51 mriedem https://bugs.launchpad.net/nova/+bug/1706207
21:53:52 openstack Launchpad bug 1706207 in OpenStack Compute (nova) "resource-class-in-use_delete_resource_class fails with "AssertionError: '404' not found in ['204']" since 7/22" [High,Confirmed]
21:54:33 dansmith oh, the same placement fixture I guess?
21:54:38 mriedem yeah
21:54:41 dansmith I see
21:57:31 mriedem although these tests should be using isolated sqlite dbs
21:57:35 mriedem so i'm not sure why this should happen
22:00:23 mriedem "On a successful create return 201. Return 204 if the class already
22:00:23 mriedem exists."
22:00:37 mriedem not sure why the test is doing a PUT instead of a POST
22:01:16 mriedem i guess because the PUT handles both cases
22:01:16 openstackgerrit Dan Smith proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949
22:01:46 dansmith jaypipes: can haz: https://review.openstack.org/#/c/484949
22:02:16 dansmith mriedem: well, I thought you meant we're hitting the wrong fixture
22:02:29 dansmith like starting two but always hitting the first one from the tests or something
22:05:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
22:06:13 dansmith mriedem: I'll fix that since I have it open
22:06:46 openstackgerrit Dan Smith proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949
22:07:39 jaypipes dansmith: done
22:07:54 dansmith jaypipes: thanks
22:08:00 jaypipes pas de probleme
22:08:13 dansmith "done" about sums it up for me too
22:08:18 mriedem https://review.openstack.org/#/c/459493/ closes out the searchlight notifications bp
22:08:19 jaypipes heh
22:12:11 openstackgerrit Merged openstack/nova master: doc: Populate the 'configuration' section https://review.openstack.org/478473
22:12:42 openstackgerrit Merged openstack/nova master: doc: Populate the 'contributor' section https://review.openstack.org/478474
22:13:11 openstackgerrit Merged openstack/nova master: doc: Populate the 'reference' section https://review.openstack.org/478484
22:14:52 mriedem so there is one other set of gabbits that create a CUSTOM_GOLD resource class
22:15:10 mriedem but they both use the APIFixture which creates separate db fixtures
22:15:14 mriedem so i'm not sure how those would overlap
22:15:20 mriedem unless it's eventlet again
22:18:34 mriedem when this failed, both tests were running on the same process
22:18:35 mriedem so maybe
22:22:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Use different resource class for resource-class-in-use tests https://review.openstack.org/486782
22:22:10 mriedem see if you can stomach this ^
22:23:02 openstackgerrit Matt Riedemann proposed openstack/nova master: Use different resource class for resource-class-in-use tests https://review.openstack.org/486782
22:25:11 openstackgerrit Merged openstack/nova master: doc: Populate the 'user' section https://review.openstack.org/478475
22:25:46 openstackgerrit Merged openstack/nova master: Update ironic feature matrix https://review.openstack.org/486148
22:31:41 mriedem melwitt: simple issue in https://review.openstack.org/#/c/410946/
22:31:55 mriedem and then i'll +2 before dansmith is completely out
22:32:38 dansmith mriedem: worth trying that rc change right?
22:33:35 dansmith mriedem: jay is out so unless melwitt responds soon and is willing, we might as well slam that in
22:34:06 mriedem i'll update it
22:34:34 dansmith mriedem: I mean your resource class gabbit thing
22:34:38 mriedem oh
22:34:42 mriedem yes it's worth trying
22:34:43 mriedem won't hurt
22:34:49 mriedem slamaro
22:34:52 mriedem *slamaroo
22:35:18 dansmith slamaroo'd
22:36:58 mriedem rebuild reno for the quotas change and then will push that up
22:37:00 mriedem *rebuilding
22:37:24 cfriesen what will be the ordering of steps when adding a new cellsv2 cell? create the new cell in the central DB and then start up the services or vice versa?
22:38:53 mriedem create the cell first
22:39:00 mriedem and then start services within it
22:39:04 mriedem just like with the single cell case
22:40:18 cfriesen okay, that makes sense
22:43:01 cfriesen mriedem: I guess we need the DB server for the new cell up and running before creating the new cell in nova
22:43:30 cfriesen or at least we need to know what it will be
22:46:07 mriedem yes
22:46:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
22:46:29 mriedem because when you run nova-manage cell_v2 create_cell you need to specify the MQ and DB URLs for the new cell
22:46:40 mriedem which get inserted into the cell_mappings table in the nova_api db
22:47:01 mriedem dansmith: ^ addressed the comments in the quotas changetest_migrate_quota_limits
22:47:03 mriedem oops
22:47:07 mriedem https://review.openstack.org/410946
22:55:59 openstackgerrit Gage Hugo proposed openstack/nova master: Admin API Policy contingent on is_admin_project https://review.openstack.org/384148
23:16:59 melwitt mriedem, dansmith: thanks yall
23:17:23 melwitt I was at the dentist, being angry
23:28:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
23:29:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
23:30:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
23:38:14 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: Remove the useless fake ExtensionManager from API unittests https://review.openstack.org/486416
#openstack-nova - 2017-07-25
00:40:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730
00:55:45 openstackgerrit Merged openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
01:02:29 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
01:04:37 openstackgerrit Merged openstack/nova master: Allow wrapping of closures https://review.openstack.org/479801
01:05:57 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Flush RC_CACHE after each gabbit sequence https://review.openstack.org/486805
01:08:21 cdent dansmith: if you’re still around, this is probably a better fix for that 201 204 problem: https://review.openstack.org/#/c/486805/
01:08:30 cdent (I couldn’t sleep, got bored, read mail…)
01:08:31 alex_xu oomichi: appreciate if you can take a look at https://review.openstack.org/#/c/486414/, that patch is the goal of pike
01:15:19 mriedem cdent: i'll drop mine once yours passes functional tests
01:15:30 cdent mriedem: roger that
01:32:22 openstackgerrit Merged openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949
02:07:58 openstackgerrit Merged openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
02:28:54 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
02:29:48 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Use wsgi_intercept in PlacementFixture https://review.openstack.org/486237
02:30:26 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
02:40:57 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829
02:41:44 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Update the placement deployment instructions https://review.openstack.org/469048
02:44:43 openstackgerrit Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
02:44:44 openstackgerrit Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
02:44:44 openstackgerrit Tony Breeds proposed openstack/nova master: Move the last_bytes util method to libvirt https://review.openstack.org/472228

Earlier   Later