Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-26
20:44:45 jgwentworth mriedem: I've been noticing frequent timeouts on the functional test job. I was thinking, I wonder if we should split the api samples into their own job? I already feel like they should be removed from a normal 'tox -e functional' run considering they have their own tox target 'api-samples'
20:45:28 mriedem https://review.openstack.org/#/c/537933/
20:46:01 mriedem ^ increases functional job timeout, but doesn't split them up
20:47:43 jgwentworth mriedem: yeah, we should definitely do that. I was just wondering if it might help to split them up longer term
20:50:34 mriedem yeah maybe,
20:50:45 mriedem i know i don't run the full functional very often b/c of the api-samples tests
20:53:42 jgwentworth same
20:54:38 jgwentworth I was just thinking we could a) reduce developer pain for running non-api-sample functional tests b) keep the timeouts more sane in the future
22:25:01 jgwentworth the 'increase functional test job timeout' change timed out
22:25:11 jgwentworth on the py35 functional test job
22:28:04 jgwentworth and there are about a billion policy deprecated messages in the job output
22:34:08 jgwentworth I wonder if the deprecation messages are related to the timeouts
22:53:41 mriedem superdan has a patch to fix that
22:53:49 mriedem er superlance
22:53:56 mriedem https://review.openstack.org/#/c/537603/
22:54:25 jgwentworth already sent them toward the gate
22:55:01 mriedem i've been busy getting a new dev vm setup this afternoon...
22:55:16 mriedem after all the 32-bit shame from earlier
22:55:33 jgwentworth haha
23:07:53 penick Someone took my junoman nick, dang
23:16:08 mriedem welp, looks like most of the approved stuff is all happily waiting or chugging through the gate now
#openstack-nova - 2018-01-27
00:02:01 openstackgerrit melanie witt proposed openstack/nova master: Reduce policy deprecation warnings in test runs https://review.openstack.org/537603
00:02:02 openstackgerrit melanie witt proposed openstack/nova master: Remove deprecated policy items from fake_policy https://review.openstack.org/537600
00:15:51 openstackgerrit Merged openstack/nova master: Deprecate allowed_direct_url_schemes and nova.image.download.modules https://review.openstack.org/510195
00:16:10 openstackgerrit Merged openstack/nova master: Only call numa_fit_instance_to_host if necessary https://review.openstack.org/530792
00:58:59 openstackgerrit melanie witt proposed openstack/nova master: Bumping functional test job timeouts https://review.openstack.org/537933
01:20:06 Spazmotic Morning
01:22:46 Spaz-Work Alllllright time to register so I can ghost these nicks and quick messing about I suppose
01:36:44 openstackgerrit Merged openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
01:49:23 openstackgerrit Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168
03:34:13 openstackgerrit Merged openstack/nova master: placement: support traits in allocation candidates API https://review.openstack.org/535642
06:31:57 openstackgerrit Merged openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436
09:40:06 openstackgerrit Brooks Kaminski proposed openstack/nova master: XenAPI: XenServer 7 Live Migration do not assert_can_migrate on source. https://review.openstack.org/538415
09:40:46 openstackgerrit Brooks Kaminski proposed openstack/nova master: XenAPI: XenServer7 Live Migration do not assert_can_migrate https://review.openstack.org/538415
09:41:13 Spazmotic Woops :0
09:41:24 Spazmotic made it a little long on the firstp ass heh
09:41:33 Spazmotic erm... pass.. i'll just..
10:26:38 openstackgerrit Merged openstack/nova master: Initialize osprofiler in WSGI application https://review.openstack.org/519664
10:36:45 openstackgerrit Merged openstack/nova master: Fix functional tests for USE_NEUTRON https://review.openstack.org/529456
10:36:59 openstackgerrit Merged openstack/nova master: Expand on when you might want to set --max-count for map_instances https://review.openstack.org/536959
10:37:14 openstackgerrit Merged openstack/nova master: Fix the incorrect RST convention https://review.openstack.org/538208
11:32:02 openstackgerrit Merged openstack/nova master: correct referenced url in comments https://review.openstack.org/534987
11:32:11 openstackgerrit Merged openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918
11:32:22 openstackgerrit Merged openstack/nova master: Add exception to no-upcall note of cells doc https://review.openstack.org/532491
11:36:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix a comment in a notification functional test https://review.openstack.org/536262
12:01:47 openstackgerrit Merged openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
12:02:03 openstackgerrit Merged openstack/nova master: Extract instance allocation removal code https://review.openstack.org/513041
12:44:41 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042
14:13:31 openstackgerrit Merged openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728
15:28:44 openstackgerrit Merged openstack/nova master: Move remaining uses of parted to privsep. https://review.openstack.org/519483
15:28:53 openstackgerrit Merged openstack/nova master: Merge ResourceV21 obj into Resource obj https://review.openstack.org/529217
15:29:17 openstackgerrit Merged openstack/nova master: Handle images with no data https://review.openstack.org/526329
15:29:26 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/537093
15:29:44 openstackgerrit Merged openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818
15:48:30 Spazmotic Whew. Full and sleepy
15:48:37 Spazmotic Fuleepy
17:16:15 openstackgerrit Merged openstack/nova master: Make sure that functional test triggered on sample changes https://review.openstack.org/533210
17:16:31 openstackgerrit Merged openstack/nova master: Stop globally caching host states in scheduler HostManager https://review.openstack.org/533371
17:46:30 efried Any cores around to do trivial re-+Ws on patches with commit message updates for Depends-On brokenness?
18:05:35 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/538473
18:21:35 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994
18:21:56 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
19:06:06 efried mriedem Willing to +W some already-approved patches if I update the commit messages for the Depends-On thing (see ML)
19:15:35 mriedem see what in the ML? depends-on with the URL isn't working?
19:17:34 efried mriedem The #/c form isn't working, needs to be the permalink for now.
19:18:23 efried mriedem So I was gonna change https://review.openstack.org/#/c/532288/ but it'll need a re-+W
19:18:37 efried ...and the guy on top of it, when rebased.
19:20:26 mriedem ok
19:20:33 mriedem do it and send me the list
19:22:24 openstackgerrit Eric Fried proposed openstack/nova master: Implement get_traits() for the ironic virt driver https://review.openstack.org/532288
19:22:25 openstackgerrit Eric Fried proposed openstack/nova master: Add functional tests for virt driver get_traits() method https://review.openstack.org/538047
19:22:26 efried mriedem ^
19:22:52 efried both a doze
19:23:37 efried Hm, the top one kept its +W, so just https://review.openstack.org/#/c/532288/
19:24:20 mriedem done
19:24:21 mriedem thanks
19:24:26 mriedem i was wondering why that depends-on wasn't working
19:25:43 efried mriedem I had one too, though mine wasn't +W'd, so it'll keep.
19:25:50 efried but swhat prompted me to ask.
19:30:19 mriedem excellent; was just pulling this code down to play with decorating the driver capabilities as traits on the compute node resource provider
19:30:24 mriedem i.e. capabilities API
19:31:47 efried Sounds like fun. Me, I'm trying to figure out why my mock-patched driver method isn't getting called in my test that looks just like mgoddard's.
19:34:00 efried oh. I need to patch it before the compute service starts. Boo.
19:50:11 openstackgerrit Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
19:50:12 openstackgerrit Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
19:50:12 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246
19:50:13 openstackgerrit Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
19:50:13 openstackgerrit Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517
19:58:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Convert driver supported capabilities to compute node provider traits https://review.openstack.org/538498
19:58:56 openstackgerrit Matt Riedemann proposed openstack/nova master: Convert driver supported capabilities to compute node provider traits https://review.openstack.org/538498
20:51:58 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510
21:35:41 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/538473
22:05:33 openstackgerrit Merged openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
22:35:52 openstackgerrit Merged openstack/nova master: Report Client: PUT empty (not None) JSON data https://review.openstack.org/536545
22:36:07 openstackgerrit Merged openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541
22:36:56 openstackgerrit Merged openstack/nova master: log test: use fixtures.StandardLogging in setUp https://review.openstack.org/531065
23:49:32 alex_xu mriedem: thanks for taking care the traits patch
#openstack-nova - 2018-01-28
02:58:16 Spazmotic Morning folks
07:30:17 ameeda Morning
09:24:29 openstackgerrit Merged openstack/nova master: Send traits to ironic on server boot https://review.openstack.org/508116

Earlier   Later