| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-30 | |||
| 11:02:45 | johnthetubaguy | yeah, I am fixing up that bug right now | |
| 11:03:07 | cdent | Unless there's something I should be doing? I'm going to go back to evaluating the number of requests the placement api is receiving. | |
| 11:03:09 | jroll | "Created resource class record via placement API for resource class CUSTOM_LOL." <- so we're halfway there :P | |
| 11:03:30 | bauzas | cool | |
| 11:03:42 | bauzas | jroll: good to know, so it's non-voting on check ? | |
| 11:04:05 | jroll | bauzas: yes, we've had a job like this non-voting on check for years :) | |
| 11:04:14 | bauzas | jroll: that I know | |
| 11:04:23 | bauzas | jroll: :) | |
| 11:04:30 | johnthetubaguy | FWIW, this is the failure: http://logs.openstack.org/96/426296/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/fd3d15e/logs/screen-n-cpu.txt.gz#_2017-01-28_00_25_20_187 | |
| 11:04:30 | jroll | :P | |
| 11:11:47 | jroll | bauzas: cdent: btw, we've an experimental job on nova/ironic that uses resource classes (triggering this code), I just have a bit of devstack plugin code left to make that work, should finish that early this week | |
| 11:12:05 | bauzas | cool | |
| 11:55:00 | artom_ | johnthetubaguy, can I ask you to take a look at https://review.openstack.org/#/c/426030/7 when you have a moment? | |
| 11:55:30 | artom_ | It's the device tag 'tag' attribute disappearing issue - you mentioned it needs a microversion in https://review.openstack.org/#/c/423952/, so a spec has been whipped up :) | |
| 11:59:46 | johnthetubaguy | artom_: ah, its that bug fix that needs a microversion, gotcha | |
| 11:59:56 | openstackgerrit | John Garbutt proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | |
| 11:59:59 | johnthetubaguy | artom_: I will try, honestly it feels like that should be in pike now | |
| 12:00:08 | johnthetubaguy | as RC1 us being cut on thursday | |
| 12:00:25 | johnthetubaguy | but not 100% sure | |
| 12:00:57 | artom_ | johnthetubaguy, I know we're short for time :( But if we can avoid a whole release cycle without device tagging... | |
| 12:01:57 | artom_ | (Yeah, I know the client can be version-pinned) | |
| 12:02:07 | artom_ | johnthetubaguy, anyways, much thanks in advance :) | |
| 12:04:25 | artom | johnthetubaguy, and for what it's worth, the patch is already up and Alex and gmann have given it one look over each: https://review.openstack.org/#/c/424759/ | |
| 12:07:38 | tdurakov | hi folks | |
| 12:07:49 | tdurakov | do we have any issues with devstack and cells? | |
| 12:08:01 | tdurakov | fresh multinode env: {"message": "Host 'node2' is not mapped to any cell", "code": 400, "created": "2017-01-30T12:04:26Z"} | |
| 12:12:58 | sfinucan | johnthetubaguy: Could I get some eyes on https://review.openstack.org/#/c/392643/ - we're seeing the referenced bug pop up in the real world | |
| 12:13:15 | sfinucan | artom: Would appreciate it if you could take a look at ^ too, please :) | |
| 12:23:46 | amrith | andreykurilin, thank you | |
| 12:32:03 | artom | sfinucan, sure, I'll get to it this afternoon :) | |
| 12:32:18 | sfinucan | artom: Thanking you kindly, sir | |
| 13:09:48 | openstackgerrit | John Garbutt proposed openstack/nova: DNM: hack ironic with resource providers https://review.openstack.org/426296 | |
| 13:32:10 | edleafe | johnthetubaguy: I see that bug and can fix it, but was waiting until after the scheduler meeting discussion for overall direction | |
| 13:41:21 | johnthetubaguy | edleafe: which bug? | |
| 13:42:31 | edleafe | johnthetubaguy: sorry, was reading scrollback | |
| 13:42:34 | edleafe | johnthetubaguy: AttributeError: 'SchedulerReportClient' object has no attribute '_compute_node_to_inventory_dict' | |
| 13:43:02 | johnthetubaguy | edleafe: ah, gotcha, pushed a fix for that, to see if the ironic tests will pass, just rebased that patch with those tests on | |
| 13:43:21 | edleafe | johnthetubaguy: ok, thanks. Still catching up | |
| 13:44:12 | johnthetubaguy | np | |
| 13:45:05 | johnthetubaguy | edleafe: I think what you have should work, although not 100% sure it gets us enough to push it into ocata | |
| 13:45:23 | edleafe | johnthetubaguy: me neither | |
| 13:45:47 | johnthetubaguy | edleafe: if we claimed the new resources we are reporting, then I think that would be good to have | |
| 13:45:52 | edleafe | johnthetubaguy: was really hoping to discuss with jaypipes before we move forward | |
| 13:46:29 | edleafe | johnthetubaguy: I'm not sure that we claim all of them - just the VCPU-rleated ones | |
| 13:46:33 | johnthetubaguy | edleafe : yup, makes sense | |
| 13:46:55 | jaypipes | edleafe: heyo | |
| 13:47:06 | edleafe | hi yerself | |
| 13:55:45 | edleafe | Nova Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt | |
| 13:57:23 | openstackgerrit | Jordan Pittier proposed openstack/nova: Fix unspecified bahavior on GET /servers/detail?tenant_id=X as admin https://review.openstack.org/426259 | |
| 13:58:29 | openstackgerrit | Maciej Szankin proposed openstack/nova: config options: remove libvirt imagecache https://review.openstack.org/378497 | |
| 13:58:37 | johnthetubaguy | edleafe: oh, I should clarify, we don't claim the new custom one right now, which seems problematic to me. | |
| 13:59:27 | edleafe | johnthetubaguy: agreed | |
| 14:55:14 | openstackgerrit | Maciej Szankin proposed openstack/nova: conf: resolved TODOs in compute configuration https://review.openstack.org/426793 | |
| 14:58:06 | johnthetubaguy | sfinucan: mdbooth: is there always a disk.mirror attribute? re https://review.openstack.org/#/c/399735 | |
| 14:58:29 | mdbooth | kashyap lyarwood: ^^^ | |
| 14:58:50 | mdbooth | johnthetubaguy: I don't recall the details now, but I do recall investigating that thoroughly and being satisfied at the time | |
| 14:59:20 | johnthetubaguy | oops, I meant to ask lyarwood as well | |
| 14:59:38 | mdbooth | johnthetubaguy: I suspect the answer is in the libvirt domain xml docs for that element | |
| 14:59:39 | kashyap | johnthetubaguy: mdbooth: Trying to remember... | |
| 15:00:01 | sfinucan | johnthetubaguy: Would it matter? | |
| 15:00:23 | johnthetubaguy | sfinucan: if its not there, we will just assume every job hasn't ever finished | |
| 15:00:24 | johnthetubaguy | I think | |
| 15:00:50 | sfinucan | johnthetubaguy: aha - I read that incorrectly https://review.openstack.org/#/c/399735/19/nova/virt/libvirt/guest.py | |
| 15:01:16 | mdbooth | johnthetubaguy: Since at least libvirt 0.9.12, possibly longer: http://libvirt.org/formatdomain.html | |
| 15:01:22 | mdbooth | Search for 'mirror' | |
| 15:01:53 | mdbooth | sfinucan: Nah, it's required as a bug fix in fact | |
| 15:02:26 | mdbooth | johnthetubaguy: MIN_LIBVIRT_VERSION = (1, 2, 1) | |
| 15:02:33 | mdbooth | IOW, it's definitely always present | |
| 15:03:06 | kashyap | mdbooth: Yes, it's been there for the duration at least Matt notes above | |
| 15:03:12 | kashyap | Err, johnthetubaguy ^ | |
| 15:03:28 | johnthetubaguy | cool, thanks folks | |
| 15:03:53 | lyarwood | johnthetubaguy, sfinucan ; I've not had time to improve the commit msg as yet btw so feel free to hold fire on this and I can make that point a little better there | |
| 15:03:53 | johnthetubaguy | "mirror" seemed odd to me, but the docs clear that up nicely, thanks | |
| 15:04:00 | dansmith | sdague: did you see the activity on this? https://review.openstack.org/#/c/424730/ | |
| 15:04:54 | johnthetubaguy | lyarwood: oh boy, I totally didn't see the notes in the commit message somehow, that seems good | |
| 15:05:21 | lyarwood | johnthetubaguy: sfinucan did ask for a little more in his review that I'm happy to add if it's still required | |
| 15:05:40 | johnthetubaguy | lyarwood: there good point, but the fix seems more important | |
| 15:05:56 | sfinucan | lyarwood, johnthetubaguy Yeah, I'm happy to let it in as is | |
| 15:06:12 | sfinucan | (hence +2 - more of a nice-to-have) | |
| 15:06:47 | kashyap | johnthetubaguy: Yeah, the word 'mirror' comes from the QEMU QMP primitive 'drive-mirror' (even _that_ is not really an accurate word, as it synchronizes as it sees fit, but not "mirrors" immediately). | |
| 15:13:22 | sdague | dansmith: I hadn't yet | |
| 15:13:32 | dansmith | sdague: okay everything else is +A behind that | |
| 15:13:33 | sdague | what was the thing that failed when it was in a dedicated section? | |
| 15:13:45 | dansmith | sdague: we start a newton-based placement | |
| 15:13:50 | dansmith | because we do it before nova has upgraded | |
| 15:14:01 | sdague | oh... right | |
| 15:14:04 | dansmith | ...yeah :) | |
| 15:15:34 | sdague | ok, well, that's fine for now then. We can separate later and figure out about earlier installing code | |
| 15:15:46 | sdague | +A | |
| 15:16:12 | dansmith | yeah, thanks | |
| 15:17:30 | jaypipes | hmm, it's always unpleasantly surprising when you run tests expecting them to fail, but they pass... | |
| 15:22:04 | mriedem | blarg | |
| 15:25:07 | mriedem | oh sweet https://review.openstack.org/#/c/424730/ | |
| 15:25:53 | johnthetubaguy | mriedem: jaypipes is looking at something for the iroinc patch | |
| 15:26:05 | mriedem | johnthetubaguy: cool | |
| 15:26:22 | jaypipes | johnthetubaguy: yeah, I'm on it. | |
| 15:26:33 | jaypipes | mriedem: trying to get it done before dentist appt at 11am. | |
| 15:26:45 | mriedem | johnthetubaguy: did we want to try and get https://review.openstack.org/#/c/335358/ in before rc1? | |
| 15:27:08 | mriedem | hate to put it off all release and then say it's too risky at the end | |
| 15:27:15 | mriedem | but i'm kind of feeling like it's too risky at this point | |