Posted	Nick	Remark
#openstack-nova - 2018-01-28
14:30:38	Spazmotic	what you two doing here on a sunday
14:30:54	mriedem	alex_xu: the functional job?
14:30:54	alex_xu	mriedem: ok, let me keeping recheck...
14:31:08	mriedem	trying to get this merged https://review.openstack.org/#/c/533608/
14:31:17	alex_xu	mriedem: tempest-full, both in check pipeline and gate pipleline
14:31:19	mriedem	but the patches on stable keep failing the cellsv1 job due to a libvirt crash that keeps happening
14:31:23	mriedem	oh
14:32:28	Spazmotic	I've got yet another XenAPI push I want to make but would rather not have 3 commits in all for vmops at the same time.. will wait and not tempt the merge conflict gods
14:33:17	mriedem	alex_xu: the tests passed http://logs.openstack.org/83/536083/8/gate/tempest-full/9d0949d/job-output.txt.gz#_2018-01-28_13_22_53_236590
14:33:32	mriedem	but the job timed out, probably b/c it's on a rax-dfw node, which i think have been slow since the intel kernel patch
14:33:41	Spazmotic	Need me to check it out?
14:33:47	mriedem	nah
14:34:26	Spazmotic	I'm Cloud Server Ops at RAX.. so if you ever need just holla holla
14:34:31	alex_xu	mriedem: sigh
14:56:20	mriedem	the cellsv1 job failures on stable are mostly on rax-ord nodes
14:56:23	mriedem	https://bugs.launchpad.net/nova/+bug/1745838
14:56:24	openstack	Launchpad bug 1745838 in OpenStack Compute (nova) "legacy-tempest-dsvm-cells constantly failing on stable pike and ocata due to libvirt connection reset" [Undecided,New]
14:56:38	mriedem	i don't know why the cellsv1 job would have so many libvirt crashes all of a sudden and just on stable and not master
14:57:20	mriedem	i almost wonder if there is something different about the way those hosts are configured for virt
14:57:33	Spazmotic	Give me about 10 minutes to finish my taxes
14:57:37	mriedem	but it looks like we're using qemu and not kvm
14:58:45	openstackgerrit	Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
14:59:15	Spazmotic	turbotax assures me the yare looking over every part of my return right now
14:59:17	Spazmotic	I'm pretty big deal
15:04:14	Spazmotic	annnd taxes done. Let's see what we got here.
15:07:00	Spazmotic	lemme hop on the VPN here and look at these nodes..
15:10:19	Spazmotic	c'mon log ive me an IP or someting here
15:12:01	mriedem	Spazmotic: thanks for looking, i've got to be afk for awhile, but if you find anything you can throw it in that bug ^
15:12:11	Spazmotic	Yeah i'll see what I can see
15:12:14	Spazmotic	have a good day
15:16:13	Spazmotic	I know these libvirt logs like I know my way around a foreign land.. where's all the XenAPI commands? hehhe
15:33:21	Spazmotic	@mriedem - Cannot really find much info about the hyp in the libvirt logs to track it down internally. Since these are running Libvirt I'm also guessing it's not running on our Xen Boxes so not too much that I can look at there. but it's possible. I'll keep looking once I wake up. .a little wiped out
16:09:10	openstackgerrit	Matt Riedemann proposed openstack/nova stable/ocata: Make sure that functional test triggered on sample changes https://review.openstack.org/536936
16:09:24	Spazmotic	Ahh i'm an idiot.. wrong test I was looking at heh
16:09:28	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: Make sure that functional test triggered on sample changes https://review.openstack.org/536934
16:18:50	Spazmotic	would be nice if I could catch one of these running and failing...
16:30:30	Spazmotic	Heading to bed, have a great night folks. See you tomorrows. 7 days until Korea, Woo~
17:24:00	efried	mriedem You around to re-+W some patches for manual (but trivial) rebases?
17:24:47	efried	Rather get 'em moving through the gate now, rather than tomorrow morning when everybody and their dog will be doing the same.
17:25:13	openstackgerrit	Eric Fried proposed openstack/nova master: report client: get_provider_tree_and_ensure_root https://review.openstack.org/521098
17:25:13	openstackgerrit	Eric Fried proposed openstack/nova master: SchedulerReportClient.set_aggregates_for_provider https://review.openstack.org/533808
17:25:14	openstackgerrit	Eric Fried proposed openstack/nova master: SchedulerReportClient._delete_provider https://review.openstack.org/535463
17:25:14	openstackgerrit	Eric Fried proposed openstack/nova master: ComputeDriver.update_provider_tree() https://review.openstack.org/521187
17:25:15	openstackgerrit	Eric Fried proposed openstack/nova master: set_{aggregates|traits}_for_provider: tolerate set https://review.openstack.org/536625
17:25:15	openstackgerrit	Eric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids: Top-down ordering https://review.openstack.org/536624
17:25:16	openstackgerrit	Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
17:25:16	openstackgerrit	Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
17:25:17	openstackgerrit	Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
17:25:17	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246
17:25:18	openstackgerrit	Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517
17:25:26	efried	mriedem If you would, please ^
17:26:33	efried	https://review.openstack.org/#/c/533808/ is bottom of the stack. The first six were previously approved.
18:15:30	maharg101	unning tox -e pip-missing-reqs in nova, reports missing requirements: taskflow, retrying, monotonic. These are not mentioned in requirements.txt or test-requirements.txt. They _are_ installed by devstack. I can't see how `tox -e pip-missing-reqs` can pass. What am I missing ?
18:15:36	maharg101	*running
18:15:54	maharg101	presumably pip-missing-reqs is advisory and doesn't actually cause CI builds to fail ?
18:48:20	openstackgerrit	Merged openstack/nova master: placement: using the dict format for the allocation in claim_resources https://review.openstack.org/536083
21:21:38	efried	maharg101 I hadn't heard of pip-missing-reqs before, but I can tell you there's a patch out there to include taskflow in global requirements :)
21:24:41	efried	maharg101 It's here: https://review.openstack.org/#/c/533779/
21:31:55	maharg101	efried: thanks, it makes sense. I'm new to openstack so thought I might be missing a step somewhere :)
21:35:48	maharg101	efried: so it would seem that the CI doesn't run all tox environments - perhaps Jenkins runs specific targets..
21:58:14	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove unnecessary arguments in notification methods https://review.openstack.org/535599
22:26:03	openstackgerrit	Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
22:39:43	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413
#openstack-nova - 2018-01-29
00:01:39	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
00:02:38	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
00:02:54	openstackgerrit	Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
00:04:18	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: trivialfix: cleanup _pack_instance_onto_cores() https://review.openstack.org/538698
00:04:19	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Not use thread alloc policy for emulator thread https://review.openstack.org/538700
00:04:19	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Add unit tests for EmulatorThreadsTestCase https://review.openstack.org/538699
00:21:57	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/538473
00:27:12	Spazmotic	Morning
01:04:05	Spazmotic	Off to the gym. Back in a bit
01:19:10	Efried_	Is there a core in the house?
01:21:42	Efried_	Could please get re+Ws on six previously-approved patches series starting
01:21:56	Efried_	at https://review.openstack.org/#/c/533808/
01:22:19	Efried_	Wow, freenode webchat is hard on teeny iPhone.
01:23:00	Efried_	Anyway, trivial manual rebases on a couple of those kicked out my +Ws
02:18:37	Spazmotic	Man that poor patchset still struggling its way through the gate efried ?
03:02:22	openstackgerrit	Spencer Yu proposed openstack/nova master: Use defusedxml.ElementTree function instead of lxml.etree https://review.openstack.org/519291
03:16:56	Spazmotic	jianghuaw, huanxie I'll be on vacation after this week, so we should find some time before the weekend to have a group chat.
03:17:33	huanxie	sure Spazmotic
03:17:57	Spazmotic	It's monday so i won't bother you two with it today and it's not something we need to push hard on at the moment yet anyway
03:18:08	Spazmotic	But I think we're closing in one a solution here that we can all agree on
03:45:22	huanxie	Agree with you Spazmotic
03:53:00	jianghuaw	Spazmotic, huanxie Looked at the patch. I have the same concerns as Huan. I think the point to use XCP version than product version is because the there is some other variants basing on Xen which will have the same issue.
03:54:49	jianghuaw	It looks good to me if we can change to use "XCP version" for a quick fix. Later we need check if we can prepare the volume SR earlier in the dest host.
03:56:10	jianghuaw	Preparing the SR earlier requires much refactor work to make it looks reasonable. But I think that's the right way which is worthy for us to look at.
04:13:46	openstackgerrit	Merged openstack/nova master: libvirt: don't attempt to live snapshot paused instances https://review.openstack.org/532214
04:15:15	openstackgerrit	Dao Cong Tien proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
04:41:34	openstackgerrit	Chen Hanxiao proposed openstack/nova master: libvirt: add Linux distribution guest only description for inject_xxx options https://review.openstack.org/528441
06:30:53	openstackgerrit	Hoang Trung Hieu proposed openstack/python-novaclient master: Zuul: Remove project name https://review.openstack.org/538777
06:44:21	openstackgerrit	Jianghua Wang proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
06:48:50	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove unnecessary arguments in notification methods https://review.openstack.org/535599
07:20:45	Spazmotic	jianghuaw, I think you're right, that was my first attempt, getting the SR prepped earlier, but would definately take some refactor and thought
07:20:54	Spazmotic	Will look at your multinode env later tonight :)
07:21:47	Spazmotic	Ah looks like you already got the whole gang to +1 it, not much for me to do there hehe
07:24:01	Spazmotic	Also, what version of XCP would you say to target for this?
07:24:11	openstackgerrit	Tuan Luong-Anh proposed openstack/nova master: Fix some reST field lists in docstrings https://review.openstack.org/538809
07:25:13	Spazmotic	huanxie, could use your input on that one too, since you two are tied in closer to the XCP versioning then I am
07:48:59	jianghuaw	Spazmotic, the platform version is 2.1.0 for XS 7.0.

Earlier   Later