| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-09 | |||
| 11:12:01 | gibi | johnthetubaguy: also it seems there are different error handling around instance.destroy | |
| 11:12:15 | gibi | johnthetubaguy: sometimes we retry after a refresh, sometimes we don'tt | |
| 11:12:28 | gibi | johnthetubaguy: here https://review.openstack.org/#/c/437222/5/nova/conductor/manager.py | |
| 11:13:14 | johnthetubaguy | asettle: I am not seeing that placement related error anywhere in the logs | |
| 11:13:20 | johnthetubaguy | asettle: do you know where you saw it? | |
| 11:14:07 | johnthetubaguy | I see "libvirtError: Failed to connect socket to '/var/run/libvirt/libvirt-sock': Connection refused" | |
| 11:14:15 | asettle | johnthetubaguy: that placement error was specifically for the in-tree docs, unreferenced | |
| 11:14:19 | gibi | johnthetubaguy: anyhow I will ping dansmith and melwitt later with this patch | |
| 11:14:25 | asettle | The error logs I sent you were from the instructions that bmoss tried in the patch | |
| 11:14:53 | asettle | johnthetubaguy: meant to be different things, sorry for confusing you | |
| 11:15:10 | johnthetubaguy | asettle: so the bug he is hitting seems to be listed here: https://github.com/mseknibilel/OpenStack-Grizzly-Install-Guide/issues/71 | |
| 11:15:18 | johnthetubaguy | asettle: seems unrelated to placement or cells | |
| 11:16:43 | openstackgerrit | Lucian Petrut proposed openstack/nova master: Avoid using fdatasync() when fetching images https://review.openstack.org/443583 | |
| 11:16:55 | owalsh | asettle: I think the current instructions should work, I've added some nits but nothing significant | |
| 11:17:15 | asettle | Okay, this is good. I'd like to get a few more eyes on it. | |
| 11:17:25 | asettle | One of my other concerns is that nobody seemed to agree in that patch set. | |
| 11:17:32 | asettle | There was a lot of differing opinions. | |
| 11:17:39 | asettle | I might have to spin something up and test it, Brian has just gone to bed. | |
| 11:18:22 | asettle | I'm sorry I can't answer all the questions - I'm speaking on behalf of someone. | |
| 11:18:41 | cdent | asettle: you're encountering that sad reality that seems to be coming up alot lately: Half of us making this stuff haven't got enough time to actually use it and test it :( | |
| 11:19:59 | johnthetubaguy | asettle: afraid the logs don't help here, so someone is going to have to walk through the guide I guess? | |
| 11:20:31 | openstackgerrit | Lucian Petrut proposed openstack/nova master: Avoid using fdatasync() when fetching images https://review.openstack.org/443583 | |
| 11:22:16 | asettle | johnthetubaguy: volunteering? :D | |
| 11:22:23 | asettle | You can do it and I can do it and i'll meet you in the middle | |
| 11:22:23 | johnthetubaguy | asettle: I am curious what your take is on the doc change after this: https://review.openstack.org/#/c/438328/7/doc/install-guide/source/nova-controller-install.rst | |
| 11:22:55 | asettle | cdent: unfortunately, yep. I've been chatting to mriedem_away about getting a dedicated CPL for this stuff. Nova is one of the only projects that don't have one. | |
| 11:23:12 | asettle | johnthetubaguy: where specifically? | |
| 11:23:16 | johnthetubaguy | asettle: I don't think I can, I am blocking way too many people to do none of my other work I had planned, bauzas sounded like he might | |
| 11:23:35 | johnthetubaguy | asettle: I basically moved all the new instructions into the existing flow | |
| 11:24:05 | asettle | johnthetubaguy: okay that's a good start. My day is relatively clear :) | |
| 11:24:11 | johnthetubaguy | asettle: this is the bulk of the change https://review.openstack.org/#/c/438328/4..7/doc/install-guide/source/nova-controller-install.rst | |
| 11:25:00 | asettle | johnthetubaguy: okay, lemme do the thing. | |
| 11:26:20 | asettle | Let's see if I can get lucky. | |
| 11:45:35 | alex_xu | bauzas: so currently our API is free dict for hints field? | |
| 11:46:18 | alex_xu | bauzas: oops, I saw you said i have valid comment | |
| 12:22:52 | arne_r | markus_z: I addressed the issues in https://review.openstack.org/#/c/436868/ and Jenkins verified | |
| 12:24:03 | markus_z | arne_r: The zkvm CI should also pass | |
| 12:25:06 | arne_r | mriedem_away lyearwood: Is this good enough for a workflow now? https://review.openstack.org/#/c/436868/ | |
| 12:50:12 | alex_xu | bauzas: johnthetubaguy, looks like we have no quota for scheduler_hints....so is that something user can input any number of hints? | |
| 12:53:13 | alex_xu | but I guess that limited by the max length of request body | |
| 12:56:38 | mriedem | asettle: we had docs CPL volunteers in the mailing list | |
| 12:56:53 | asettle | mriedem: yes I was waiting to hear back from you regarding confirmation of that. | |
| 12:57:06 | asettle | I saw there were two volunteers - let me know who is available to provide the time | |
| 12:57:14 | mriedem | asettle: Kevin_Zheng and nicolas bock | |
| 12:57:23 | asettle | mriedem: terrific: https://wiki.openstack.org/wiki/CrossProjectLiaisons#Documentation | |
| 12:57:38 | asettle | It would be very much appreciated if you could update that table :) | |
| 12:57:53 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | |
| 12:57:53 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: Move the common code out of ResourceClassCache https://review.openstack.org/416007 | |
| 12:57:54 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199 | |
| 12:57:54 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add simple cache object for traits https://review.openstack.org/441829 | |
| 12:58:54 | mriedem | will do | |
| 12:59:03 | mriedem | nicolasbock: fyi ^ | |
| 12:59:18 | asettle | Muchos appreciated :) and now, back to the manual install that'll kill my will to live | |
| 12:59:42 | nicolasbock | mriedem, thanks | |
| 13:00:17 | mriedem | asettle: updated | |
| 13:01:01 | asettle | nicolasbock: while you're here ;) if you haven't seen my email to the dev ML, I've had to update a nova-related bug to critical. https://bugs.launchpad.net/openstack-manuals/+bug/1663485 | |
| 13:01:01 | openstack | Launchpad bug 1663485 in openstack-manuals "Document Nova v2 Cells and Placement API" [Critical,In progress] - Assigned to John Garbutt (johngarbutt) | |
| 13:01:08 | asettle | We've gotta hit the ground running | |
| 13:01:11 | mriedem | johnthetubaguy: bauzas: cdent: sfinucan: i'm not going to be able to run the full meeting this morning, got single parenting before school duties - can one of you pick up running the meeting? | |
| 13:01:16 | asettle | I need as many people as possible to help test: https://review.openstack.org/#/c/438328/ | |
| 13:01:47 | cdent | asettle: i've started a run through of the current doc | |
| 13:01:55 | cdent | mriedem: yeah, I'm sure one of us can get it | |
| 13:02:05 | nicolasbock | asettle, I'll have a look | |
| 13:02:24 | johnthetubaguy | mriedem: +1 for cdent to take the helm | |
| 13:02:33 | mriedem | heh, i see what you did there | |
| 13:02:35 | asettle | Ugh thank you! I'm on it as well. Starting completely fresh. | |
| 13:02:43 | asettle | Really appreciate it cdent and nicolasbock :) | |
| 13:03:00 | mriedem | the nova meeting agenda is up to date as of last night, so should be good to go there | |
| 13:03:01 | cdent | but yeah, sure | |
| 13:03:09 | asettle | In theory, the instructions should be right now. But I need to make sure :) | |
| 13:03:35 | johnthetubaguy | cdent: I am going to try join a TC vision thing thats happening in boston today, somehow | |
| 13:05:06 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Move detach logic from manager into driver BDM https://review.openstack.org/439520 | |
| 13:05:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: objects: Add attachment_id to BlockDeviceMapping https://review.openstack.org/437665 | |
| 13:05:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Only destroy BDMs after successful detach call https://review.openstack.org/440693 | |
| 13:05:08 | openstackgerrit | Lee Yarwood proposed openstack/nova master: db: Add attachment_id to block_device_mapping https://review.openstack.org/437597 | |
| 13:05:25 | cdent | johnthetubaguy: good luck with that, remote vision is hard | |
| 13:06:02 | johnthetubaguy | cdent: yeah | |
| 13:08:45 | bauzas | johnthetubaguy: asettle: saw the ML replies, looks like the problem is related to libvirt so | |
| 13:08:53 | johnthetubaguy | asettle: stupid question, but do we know how the packaging folks tested the packages? | |
| 13:09:23 | bauzas | mriedem: okay, I can run it | |
| 13:09:36 | johnthetubaguy | asettle: I think I always assumed they would update the install docs based on the release notes and upstream notes, maybe that was never a thing | |
| 13:11:02 | mriedem | bauzas: i think cdent is on the hook | |
| 13:11:17 | cdent | like a fish | |
| 13:11:33 | asettle | johnthetubaguy: 'how' - as in, physically how they test it, or how do we know if they did it? | |
| 13:11:45 | asettle | bauzas: yes, I hope that's all that is. But we still need to test to ensure that. | |
| 13:11:54 | johnthetubaguy | asettle: physically really | |
| 13:12:05 | bauzas | mriedem: cdent: okay, no worries, good with me | |
| 13:14:22 | asettle | johnthetubaguy: that I can't answer. We have people who test it for us. We track the work here: https://wiki.openstack.org/wiki/Documentation/OcataDocTesting | |
| 13:14:26 | asettle | As you can see, we're a bit blocked. | |
| 13:14:57 | johnthetubaguy | asettle: no worries, just missing bits of the loop here | |
| 13:15:54 | openstackgerrit | sahid proposed openstack/nova master: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | |
| 13:15:54 | openstackgerrit | sahid proposed openstack/nova master: numa: fit instance NUMA node with cpus reserved onto host NUMA node https://review.openstack.org/400875 | |
| 13:15:56 | openstackgerrit | sahid proposed openstack/nova master: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | |
| 13:16:01 | asettle | Well, we get access to the packages, we then test. Let me poke around and get some holes fixed up for you. I admit entirely I haven't been a part of the install guide process for quite some time. | |
| 13:16:03 | openstackgerrit | sahid proposed openstack/nova master: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | |
| 13:16:04 | openstackgerrit | sahid proposed openstack/nova master: numa: take into account cpus reserved https://review.openstack.org/400876 | |
| 13:16:05 | openstackgerrit | sahid proposed openstack/nova master: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877 | |
| 13:17:23 | johnthetubaguy | asettle: no worries, same here | |
| 13:21:55 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | |
| 13:24:06 | openstackgerrit | Dao Cong Tien proposed openstack/nova master: Stop using mox in unit/virt/disk/test_api.py https://review.openstack.org/342061 | |