| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-31 | |||
| 01:48:07 | mriedem | efried: looks like the nrp series has a merge conflict starting here now https://review.openstack.org/#/c/521098/30 | |
| 01:48:35 | Spazmotic | Haha I tried.. something about not having a presence in South Korea so they cannot have a remote worker there or something | |
| 01:48:50 | bkero- | Yeah, it's hard to ask for permission | |
| 01:49:04 | mriedem | australia | |
| 01:49:07 | mriedem | rax is there | |
| 01:49:12 | mriedem | then at least the tz is closer | |
| 01:49:17 | Spazmotic | Well also HKG | |
| 01:49:21 | Spazmotic | But we are settling for bringing her here | |
| 01:49:24 | Spazmotic | Just waiting on VISA stuff | |
| 01:49:28 | mriedem | don't tell trump | |
| 01:49:30 | Spazmotic | dunno why i thought that was an anagram | |
| 01:49:31 | Spazmotic | it's early | |
| 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. | |