Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-31
01:49:43 Spazmotic Yeah it's totes hush hush, just letting you openstack fellows know.. DON'T TELL TRUMP
01:50:14 mriedem he'd actually be in bed eating by now if it weren't for some speech he has to give tonight
01:51:17 Spazmotic I'd imagine he's still doing that thing
01:51:25 Spazmotic Just speech from bed with giant slippers
01:51:33 Spazmotic Glow of his own TV show shining on his face or something, and some chicken in a bucket.
01:51:48 Spazmotic Swinging a drumstick around
01:56:36 mriedem alex_xu: looks like your allocation candidates + traits gabbi tests are intermittently failing http://logs.openstack.org/10/539310/1/check/nova-tox-functional/a6ba562/job-output.txt.gz#_2018-01-30_21_02_40_414709
02:00:22 mriedem https://bugs.launchpad.net/nova/+bug/1746388
02:00:23 openstack Launchpad bug 1746388 in OpenStack Compute (nova) "allocation candidates with invalid traits negative functional tests are intermittently failing" [High,Confirmed]
02:01:15 mriedem alex_xu: ah nevermind efried found the issue https://review.openstack.org/#/c/537351/3/nova/api/openstack/placement/util.py
02:50:12 openstackgerrit Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
02:58:30 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add unit tests for EmulatorThreadsTestCase https://review.openstack.org/538699
02:58:31 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Not use thread alloc policy for emulator thread https://review.openstack.org/538700
03:08:42 alex_xu mrodden: got it, let me fix it
03:09:04 alex_xu mrodden: sorry, I should speak to mriedem
03:12:39 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Use util.validate_query_params in list_traits https://review.openstack.org/539407
03:12:40 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: [Placement] Invalid query parameter could lead to HTTP 500 https://review.openstack.org/539408
03:18:52 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: provide VGPU_DISPLAY_HEAD inventory in compute node https://review.openstack.org/523342
03:58:50 openstackgerrit Eric Fried proposed openstack/nova master: report client: get_provider_tree_and_ensure_root https://review.openstack.org/521098
03:59:14 efried Well, that didn't work.
05:19:55 openstackgerrit Merged openstack/nova master: Add taskflow to requirements https://review.openstack.org/533779
06:00:18 openstackgerrit Rajesh Tailor proposed openstack/nova master: Allow force-delete even if task_state is not None https://review.openstack.org/530879
06:04:45 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/538473
07:20:24 openstackgerrit Radoslav Gerganov proposed openstack/nova master: Insert the description of the change. https://review.openstack.org/539432
07:59:21 openstackgerrit Nakanishi Tomotaka proposed openstack/nova master: Use Placement API to check resource usage https://review.openstack.org/528953
08:15:43 openstackgerrit Merged openstack/nova master: libvirt: QEMU native LUKS decryption for encrypted volumes https://review.openstack.org/523958
08:15:55 openstackgerrit Merged openstack/nova master: Add release note for QEMU native LUKS decryption https://review.openstack.org/537642
08:17:05 openstackgerrit Alex Xu proposed openstack/nova master: Fix nits in support traits changes https://review.openstack.org/537351
08:17:05 openstackgerrit Alex Xu proposed openstack/nova master: Mention required traits in the flavors user docs https://review.openstack.org/539300
08:17:06 openstackgerrit Alex Xu proposed openstack/nova master: Add functional tests for traits-based scheduling https://review.openstack.org/539310
08:21:21 openstackgerrit Merged openstack/nova master: ProviderTree.new_child: parent is either uuid or name https://review.openstack.org/538264
08:39:10 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Invalid query parameter could lead to HTTP 500 https://review.openstack.org/539164
09:03:46 bauzas morning Novaers
09:12:05 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469
09:18:18 Spazmotic Morning
09:36:13 Spazmotic Last day of my weekend.. then just 4 days and work and finally 2 weeks in KOrea to relax
09:44:23 Spazmotic Going to take the rest of the day off and avoid python.. have a good night all, will be afk :)
09:50:36 openstackgerrit blkart proposed openstack/nova-specs master: don't show description content in code-block https://review.openstack.org/539478
09:54:11 bauzas jianghuaw: not sure you saw https://review.openstack.org/#/c/539266/2
10:01:40 jianghuaw bauzas, yes. I saw it. Just have not got chance to check it due to having been occupied by a kolla deployment problem.
10:01:44 jianghuaw thanks for asking, surely will look at it before I leave office.
10:01:52 openstackgerrit Rajesh Tailor proposed openstack/nova master: Allow force-delete even if task_state is not None https://review.openstack.org/530879
10:04:33 bauzas jianghuaw: ideally, you could make a follow-up with the Xen notes
10:05:14 jianghuaw Sure.
10:31:09 amotoki hi, i have a question on 'host' parameters in os-aggregates and os-migrateLive API.
10:31:21 amotoki I am working on https://review.openstack.org/#/c/538472/ (in horizon).
10:32:02 amotoki in nova API there are two types of "host". the one is from os-services and the other is from os-hypervisors.
10:33:49 amotoki I checked the API reference but I am still not clear which is expected for "hosts" in os-aggregates and "host" in migrateLive (and migrate) action.
10:34:06 amotoki thanks in advance.
10:35:42 bauzas amotoki: it refers to the os-services hostname
10:36:02 amotoki bauzas: thanks. is it true for both?
10:36:26 bauzas amotoki: well, there is a confusion
10:37:09 bauzas amotoki: long story short, hypervisors provide their own hostnames, so we have a tuple (service hostname, hypervisor hostname) identifying a single node
10:37:28 bauzas for most of our virt drivers, a compute service only serves a single hypervisor
10:37:44 bauzas so there is no real problem in adressing the host itself by its service name
10:37:56 bauzas but for ironic, that paradigm isn't true
10:38:18 bauzas since a single compute service serves for all the ironic nodes
10:38:24 amotoki i think the same thing happens for vmware (vcenter) driver.
10:38:28 bauzas no
10:38:41 bauzas because nova sees the vCloud as a single node
10:39:00 bauzas how it's managed within that cloud is beyond its view
10:39:04 amotoki does nova consider vCloud as a big host?
10:39:08 bauzas exactly
10:39:19 bauzas from a scheduling perspective
10:39:34 bauzas which is what aggregates are for :)
10:41:34 amotoki aggregates refer to service host name, so how can we distinguish individual hosts? is it determined based on aggregate metadata?
10:42:43 ameeda Hello, what is the different between "self.volume_api.detach" and " self._detach_volumes" in nova manager ?
10:45:08 bauzas amotoki: sorry, was at the coffee machine
10:45:24 amotoki bauzas: no problem. thanks for your support
10:46:55 bauzas amotoki: so, the answer is "that's not supported"
10:47:17 bauzas amotoki: all the Aggregate API logic is based on services checks
10:47:53 bauzas amotoki: that means that if you use ironic and want aggregates, you will only shard your cloud by the compute services, that's it, not by your ironic deployment
10:48:04 bauzas call it a limitation
10:48:54 bauzas from my perspective, it's just that ironic introduced a new paradigm (N hypervisors for a single service) without changing the Nova API, which is why it's broken
10:49:14 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161
10:50:29 amotoki bauzas: thanks for clarification. I see the current logic and it is good to know that :)
10:51:24 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Fix typos in release notes https://review.openstack.org/539490
10:59:26 jianghuaw bauzas, thanks for creating the documents for vGPU. The patch looks very good. I will add the part for XenAPI.
11:00:13 bauzas jianghuaw: FWIW, I'm also currently working on providing a functional testcase that will use a fake libvirt driver with mdevs
11:00:56 jianghuaw bauzas, thanks for the heads-up.
11:01:25 jianghuaw What's I can help on that>
11:02:01 bauzas there are a couple of libvirt functional tests already, so I'll land that funct test battery into there
11:02:14 bauzas you could see if Xen got some
11:02:40 jianghuaw ok. will check it.
11:02:42 bauzas later, we could discuss with other folks if we would maybe have a single test battery that would check both drivers, I dunno yet
11:03:33 bauzas like a base test class that would test what's supported by both drivers, and we would just inherit from it
11:03:45 bauzas of course, the setup of that test class would be per-driver
11:03:59 jianghuaw sounds good.
11:04:05 bauzas jianghuaw: so, nothing from your side yet I guess
11:04:25 jianghuaw ok.
11:04:31 bauzas just maybe if you have time, try to see how to mock your Xen API the less invasive way
11:05:17 jianghuaw k
11:05:22 bauzas ideally, I wouldn't want to mock every single libvirt call like we do in the unittests, rather just pass a libvirt fixture that would setup my host like I want
11:05:34 bauzas hence the fakelibvirt module reading
11:44:00 openstackgerrit Merged openstack/nova master: Fix documentation nits in set_and_clear_allocations https://review.openstack.org/531001
11:44:54 openstackgerrit Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501
12:08:45 openstackgerrit Mark Goddard proposed openstack/nova master: Update docstring for get_traits virt driver method https://review.openstack.org/539505
12:08:46 openstackgerrit Mark Goddard proposed openstack/nova master: Add resource_class to fields in ironic node cache https://review.openstack.org/539506
12:08:46 openstackgerrit Mark Goddard proposed openstack/nova master: Remove unused method _parse_node_instance_info https://review.openstack.org/539507
12:08:47 openstackgerrit Mark Goddard proposed openstack/nova master: Include only required fields in ironic node cache https://review.openstack.org/539508

Earlier   Later