| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-29 | |||
| 23:16:24 | mgagne | efried: so I'm not ready for Ocata yet but I know I will have to cross that river someday | |
| 23:16:34 | mgagne | efried: no ^ | |
| 23:17:17 | mgagne | efried: but the challenge is that once I'm ready to cross that river, it will be too late to complain about changes in Ocata I wasn't aware about. I'm playing catch up here. | |
| 23:18:41 | mgagne | efried: I don't know if you understand in what kind of boat I'm in. | |
| 23:18:54 | efried | mgagne: Well, I think the first thing that'll help you is to get placement aggregates out of your head. They're not related to host aggregates, and they're not fully implemented in Newton (or Ocata, or Pike, or Queens). | |
| 23:19:48 | efried | It sounds to me like all you need is a way to customize allocation ratios. Which can be done via placement calls, as long as there's *some* switch to turn off the automatic setting from the conf, as Jay suggests. | |
| 23:19:50 | mgagne | efried: ok, I found that it's not the answer to my problem. But I still need to educate myself on the concept | |
| 23:20:13 | mgagne | efried: yes, that's the conclusion I ended up with too | |
| 23:20:27 | efried | mgagne: Has anyone proposed a patch that would do that yet? | |
| 23:20:37 | mgagne | not that I'm aware of | |
| 23:22:32 | mriedem | efried: this? https://review.openstack.org/#/c/532924/ | |
| 23:23:40 | efried | mriedem: Certainly looks related, at a glance. | |
| 23:25:21 | efried | mgagne: Have you tried patching ^ in? | |
| 23:26:28 | mgagne | efried: I'm not running master | |
| 23:27:02 | efried | mgagne: Yeah, I get that, I'm asking if you've tried patching that into your running environment to see if it resolves the issue. | |
| 23:27:10 | efried | or rather, allows you to resolve it via conf. | |
| 23:27:40 | mgagne | I'm not sure I understand what you are asking | |
| 23:31:28 | efried | mgagne: IIUC, the point of that patch will be to backport it to where you can use it (which I guess would be Newton? Though again, I thought we were EOLing that guy) | |
| 23:31:47 | efried | mgagne: So I'm suggesting you could do that backport locally and see if it works for you. | |
| 23:31:59 | mgagne | I'm really confused. what's "it" ? =) | |
| 23:32:03 | efried | Though I suppose things have probably changed quite a bit in those areas, so the backport may not be trivial. | |
| 23:32:12 | mgagne | the bug is very technical and I don't fully understand the problem and proposed solution | |
| 23:32:15 | efried | Sorry, the patch mriedem mentioned above - https://review.openstack.org/#/c/532924/ | |
| 23:32:42 | efried | The proposed solution would allow you to set your allocation ratios via conf, end of story. | |
| 23:33:10 | efried | Nothing else would change; in particular, you would continue to be able to use your filtery thingies as you did before. | |
| 23:33:35 | mriedem | efried: we don't backport config option default changes | |
| 23:33:44 | efried | oh | |
| 23:34:01 | mriedem | since, you know, changing defaults on people in stable is kind of mean | |
| 23:34:02 | efried | what about the rest of that patch? | |
| 23:34:14 | efried | yeah, makes sense when you put it like that. | |
| 23:34:18 | mriedem | :) | |
| 23:34:24 | mriedem | i haven't looked at the rest of the patch in a few weeks | |
| 23:34:31 | mgagne | efried: what's via conf for you? Isn't already the case where you can provide allocation ratio on compute node in nova.conf? I'm totally lost | |
| 23:34:33 | mriedem | and i don't think i have the brain power to do so right now | |
| 23:35:09 | efried | mgagne: Yeah, you're right. And I'm probably just confusing the issue even more. I should stop. | |
| 23:35:50 | mgagne | efried: my issue is I'm being told to stop using aggregates (and allocation ratio in metadata) because they are broken. | |
| 23:36:19 | mgagne | efried: and to use nova.conf itself on compute node because "Config management is the solution to your problem" | |
| 23:36:29 | efried | I can understand where that's a problem. | |
| 23:37:11 | mgagne | efried: that's not what I want, this is not my flow, far from it. | |
| 23:38:42 | mgagne | efried: so that's basically it. Something I could do through an API now needs to be done through our config system and the person performing this task might not have access to our config system. | |
| 23:39:00 | mgagne | and you lose "agility" in your operations | |
| 23:44:40 | mgagne | time to leave for me, thanks for the talk ;) | |
| 23:54:34 | openstackgerrit | Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 23:54:35 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 23:54:35 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 23:55:11 | openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle IpAddressAlreadyAllocated exception https://review.openstack.org/535532 | |
| 23:56:33 | openstackgerrit | Hongbin Lu proposed openstack/nova master: Use revision_number on adding/removing security group https://review.openstack.org/535510 | |
| #openstack-nova - 2018-01-30 | |||
| 00:16:24 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 00:16:24 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 00:20:24 | openstackgerrit | melanie witt proposed openstack/nova master: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/340614 | |
| 01:24:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | |
| 01:34:09 | Spazmotic | Morning Folks | |
| 01:34:16 | Spazmotic | Jeez mriedem you still commiting over there | |
| 01:58:49 | Spazmotic | no no no no | |
| 01:59:08 | Spazmotic | IF YOU HAVE A STRUCTURED SETTLEMENT BUT YOU NEED CASH NOWWWW | |
| 01:59:28 | jgwentworth | 877 cash now | |
| 01:59:46 | Spazmotic | Why would you do that to people, core reviewers should have mortal fortitude! | |
| 02:00:18 | jgwentworth | heh | |
| 02:09:55 | Spazmotic | melwitt, Any chance you're familiar with the XenAPI drivers? :) | |
| 03:51:00 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add the nova-multiattach job https://review.openstack.org/532689 | |
| 04:09:08 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: add Linux distribution guest only description for inject_xxx options https://review.openstack.org/528441 | |
| 04:25:12 | mattoliverau | Hey nova cores. I have a doc patch I've pushed upstream to fix the nova placement-api port on OpenSuse/SLES which was incorrect. It' | |
| 04:25:47 | mattoliverau | It's been sitting there since december last year. So thought I'd do a ping here ;) | |
| 04:26:18 | mattoliverau | Only doc, so an easy review ;) | |
| 04:26:26 | mattoliverau | https://review.openstack.org/#/c/527846 | |
| 05:23:38 | mattoliverau | oh right, cool, there is another patch doing the same thing :) Nice. So long as it get's fixed then my employer will be happy, ta.. | |
| 06:48:34 | ratailor | Can anyone please review this https://review.openstack.org/#/c/530879/ | |
| 06:49:19 | ameeda | is this error caused by me ? "http://logs.openstack.org/00/526900/26/check/openstack-tox-functional/c777a93/testr_results.html.gz" | |
| 06:49:24 | ameeda | from this gerrit "https://review.openstack.org/#/c/526900/" | |
| 06:49:48 | ameeda | error "http://logs.openstack.org/14/340614/12/check/nova-tox-functional/1cd661e/testr_results.html.gz" | |
| 07:23:29 | openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | |
| 07:30:58 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: remove TODO on validation of scsi model https://review.openstack.org/525055 | |
| 07:42:06 | sticker | Has anyone reported issues with virtlogd recreating the console.log as root:root during migration or resize when append="off" is set in the dom xml (kvm) | |
| 07:42:41 | sticker | when using KVM > 2.7.0 | |
| 08:10:37 | tetsuro | #openstack-nova | |
| 08:48:27 | openstackgerrit | sahid proposed openstack/nova-specs master: update: introducing isolate emulthreads on host https://review.openstack.org/511188 | |
| 08:49:17 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.openstack.org/512553 | |
| 08:49:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.openstack.org/512623 | |
| 08:49:18 | openstack | bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Undecided,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 08:53:52 | openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | |
| 09:03:45 | lyarwood | morning all | |
| 09:14:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018 | |
| 09:24:55 | openstackgerrit | Alex Xu proposed openstack/nova master: Add server filters whitelist in server api-ref https://review.openstack.org/527042 | |
| 09:37:33 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: [WIP] Fix a bug https://review.openstack.org/539164 | |
| 09:40:27 | Spazmotic | gibi I heard rumor that you were decent with some XenAPI, If true you mind if I add you to a review to check out for me that has been looking for some cores for a couple weeks? | |
| 09:41:43 | gibi | Spazmotic: those rumors seems to be pretty off. Anyhow you can add me as reviewer but I cannot promis I will be able to get to it soon | |
| 09:41:59 | Spazmotic | Hehe as rumors go | |
| 09:42:29 | Spazmotic | I'll let it sit as it is now, sounds like you're pretty busy, so will just hope a reviewer gets to it in a ntural way :) | |
| 09:49:53 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Integrate Mypy Type Checking https://review.openstack.org/538217 | |
| 09:49:56 | bauzas | huh, 18 hours from the top change to be merged in the gate | |
| 09:50:05 | bauzas | that's going to be a long long day | |
| 09:50:17 | Spazmotic | yikes | |
| 09:50:27 | Spazmotic | Gate still bogged down after the zuul restart? | |
| 09:50:37 | bauzas | huge delay at least | |
| 09:51:32 | Spazmotic | Welp.. time for my morning begging session again. Will throw out this review if anyone gets some free cycles to look at it. https://review.openstack.org/#/c/533168/ | |
| 09:51:36 | Spazmotic | And imma go drink some coffee :) | |
| 09:52:34 | bauzas | well, looks like it's just a delay problem AFAICT http://status.openstack.org/openstack-health/#/ | |
| 09:59:50 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Invalid query parameter could lead to HTTP 500 https://review.openstack.org/539164 | |
| 10:00:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Add 'mypy' job https://review.openstack.org/539168 | |
| 10:02:47 | bauzas | Spazmotic: stupid question but does the XenServer CI covers your change ? https://review.openstack.org/#/c/533168/ | |