| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-29 | |||
| 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/ | |
| 10:05:59 | mgoddard_ | alex_xu: hi, is there documentation for usage of traits as flavor extra specs? https://review.openstack.org/#/c/536085/9 | |
| 10:09:36 | bauzas | mgoddard_: for the moment, see the relnote https://review.openstack.org/#/c/536085/9/releasenotes/notes/request-traits-in-nova-ffcb00f76229b6e9.yaml | |
| 10:09:54 | bauzas | some nova documentation could be made I agree | |
| 10:11:35 | mgoddard_ | bauzas: that would be nice. I can't see anything about custom resource classes on flavors either. | |
| 10:12:04 | mgoddard_ | bauzas: something in here: https://docs.openstack.org/nova/latest/user/flavors.html? | |
| 10:13:01 | Spazmotic | sorry bauzas was grabbing coffee | |
| 10:14:07 | bauzas | Spazmotic: no excuses. | |