Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-27
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
13:59:49 openstackgerrit Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
14:29:31 alex_xu mriedem: I saw timeout in gate pipeline for the patch https://review.openstack.org/536083, should I recheck again now?
14:30:30 mriedem alex_xu: yeah
14:30:38 Spazmotic what you two doing here on a sunday
14:30:54 alex_xu mriedem: ok, let me keeping recheck...
14:30:54 mriedem alex_xu: the functional job?
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: SchedulerReportClient.set_aggregates_for_provider https://review.openstack.org/533808
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:14 openstackgerrit Eric Fried proposed openstack/nova master: ComputeDriver.update_provider_tree() https://review.openstack.org/521187
17:25:14 openstackgerrit Eric Fried proposed openstack/nova master: SchedulerReportClient._delete_provider https://review.openstack.org/535463
17:25:15 openstackgerrit Eric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids: Top-down ordering https://review.openstack.org/536624
17:25:15 openstackgerrit Eric Fried proposed openstack/nova master: set_{aggregates|traits}_for_provider: tolerate set https://review.openstack.org/536625
17:25:16 openstackgerrit Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648
17:25:16 openstackgerrit Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821
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:17 openstackgerrit Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
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 ?

Earlier   Later