| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-11 | |||
| 13:46:12 | gibi | cdent: after I pushed the evac fix I can create a shelve/unshelve test | |
| 13:46:43 | cdent | there’s loads of random code floating around | |
| 13:49:40 | mriedem | afk for a bit | |
| 14:12:34 | openstackgerrit | Dmitry Tantsur proposed openstack/nova master: Fix reporting inventory for the Ironic driver https://review.openstack.org/492964 | |
| 14:12:35 | dtantsur | cdent, mriedem, bauzas, cleaned up version of my patch ^^^ | |
| 14:13:19 | dtantsur | aaaaand the patch using resource classes has passed CI: https://review.openstack.org/#/c/476968/ | |
| 14:13:26 | dtantsur | bauzas: you may want to check it for logging ^^^ | |
| 14:25:03 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | |
| 14:26:59 | bauzas | dtantsur: cdent: mriedem: sorry in an internal meeting atm | |
| 14:27:20 | bauzas | sdague: I could have a stupid question but worth asking | |
| 14:27:57 | bauzas | sdague: if someone reports an issue only appearing with an old microversion, I guess we still have to fix it ? | |
| 14:28:31 | bauzas | sdague: the context being that if using a recent microversion, the flag that was creating that bug is no longer supported | |
| 14:29:19 | sdague | bauzas: it depends | |
| 14:29:41 | dansmith | mriedem: is the az not available to compute via the requestspec or something like that? | |
| 14:29:54 | dansmith | mriedem: if not, that would be a simpler fix than the moving of things to conductor | |
| 14:29:58 | sdague | bauzas: can you be more specific with the bug | |
| 14:30:22 | bauzas | sdague: related point, while novaclient tries to use the latest microversion the server can support (capped by its own max version), it seems OSC just tries to use v2.1 | |
| 14:30:40 | bauzas | sdague: that's an internal BZ that was popped up | |
| 14:30:52 | sdague | yes, osc does work differently | |
| 14:31:31 | sdague | I don't know the current status of osc microversion support | |
| 14:31:52 | bauzas | sdague: but tl,dr: since 2.25 we are now no longer accepting disk_over_commit, which is generating some HTTP400 if you use it with older versions | |
| 14:32:34 | leakypipes | dansmith: I thought you were vacationing, superdan? :) | |
| 14:32:48 | sdague | bauzas: you are really going to have to get a bug filed | |
| 14:32:49 | dansmith | leakypipes: inlaw visits are not vacation | |
| 14:32:53 | leakypipes | ah | |
| 14:32:56 | leakypipes | tru nuf | |
| 14:32:57 | bauzas | sdague: yeah will do | |
| 14:33:05 | sdague | because is that a regression? | |
| 14:33:07 | bauzas | sdague: but needs to check the internal bug with novaclient first | |
| 14:33:22 | bauzas | sdague: that's a mitaka bug from waht I know | |
| 14:33:34 | bauzas | sdague: but I need to do more coverage before filling the buig | |
| 14:34:34 | dansmith | mriedem: +2 on both of those doc patches | |
| 14:34:39 | sdague | yeh, I mean, clearly mitaka is no longer upstream supported, so there is that. But the point is if there is a regression, a regression should be fixed. But if it was always that way, and new microversions make it fine, I'd err on the side that we don't touch it | |
| 14:41:19 | cfriesen | does anyone know if libvirt/qemu supports block live migration with a config drive but with instance storage on a ceph volume? | |
| 14:43:20 | bauzas | sdague: sure, that's why I need to do further tests against master before filling the bug | |
| 14:48:37 | mriedem | dansmith: during boot from volume it might be | |
| 14:48:42 | mriedem | but, | |
| 14:49:01 | mriedem | not really sure, as the az for the host wouldn't be known until we know the host, and i don't know if we update the request spec after that | |
| 14:49:06 | mriedem | that's a bauzas question | |
| 14:49:18 | mriedem | dansmith: btw, good morning sleepy head! | |
| 14:53:42 | cfriesen | question....if a compute node is using RBD for instance disks, would we expect it to use RBD for the config drive as well? | |
| 14:58:01 | dansmith | mriedem: lol | |
| 14:58:16 | leakypipes | cfriesen: I don't think so. I think config drives are always iso9660, no? and local to the compute node? | |
| 14:58:25 | dansmith | leakypipes: no | |
| 14:58:28 | dansmith | leakypipes: can be vfat | |
| 14:58:30 | leakypipes | dansmith: no? | |
| 14:58:32 | leakypipes | ah | |
| 14:58:39 | dansmith | and can also be on ceph I think | |
| 14:58:51 | leakypipes | oh, well, cfriesen I don't know anything then :) | |
| 14:59:10 | dansmith | I just came here to humble leakypipes | |
| 14:59:19 | leakypipes | Mission Accomplished. :P | |
| 15:00:53 | fried_rice | mtreinish About to restack - any update? | |
| 15:04:30 | mriedem | dansmith: leakypipes: don't forget the config drive can be on scaleio and is then 8GB | |
| 15:04:41 | dansmith | mriedem: right that's how I know :) | |
| 15:04:42 | mtreinish | fried_rice: not really, I'm still hitting the same issue | |
| 15:04:56 | bauzas | mriedem: dansmith: in a meeting atm, how can I help ? | |
| 15:05:25 | mtreinish | fried_rice: I've got another devstack setup locally and I'm trying different config permutations. I feel like I've gone back 2 months because I was debugging the same thing back when we originally added glance support | |
| 15:06:26 | fried_rice | mtreinish You're not needing to restack every time, right? Just muck with some confs, restart some services, and try the image create again? | |
| 15:06:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: delete allocation of evacuated instance https://review.openstack.org/493037 | |
| 15:06:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: test server evacuation with placement https://review.openstack.org/492548 | |
| 15:07:17 | mtreinish | fried_rice: yeah | |
| 15:07:28 | fried_rice | cool | |
| 15:07:42 | fried_rice | mtreinish Let me know if there's anything I can do to help. | |
| 15:12:55 | mtreinish | fried_rice: well I at least figured out why I thought it worked before. The config I have up there now drops the chunked data on the floor. (when you talk to uwsgi directly instead of going through the proxy) | |
| 15:13:14 | mtreinish | so the request gets a good response, but the data isn't actually used | |
| 15:13:31 | fried_rice | mtreinish Hah, so I would get a zero-size image, or better yet, a full-sized image full of zeros? | |
| 15:13:46 | mtreinish | heh, it's a zero size image | |
| 15:14:16 | mtreinish | I think the 502s from apache are it realizes the request isn't being properly handled and bails out | |
| 15:18:34 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Update api doc with latest updates in api framework https://review.openstack.org/493041 | |
| 15:21:04 | vdrok | sdague: mriedem could you also take a look at https://review.openstack.org/492579 if you have a bit of time? This is fixing https://bugs.launchpad.net/nova/ocata/+bug/1706772 we are seeing sometimes on the ocata grenade job, as placement start triggers reload of apache, so keystone becomes inaccessible and n-cpu fails to start | |
| 15:21:05 | openstack | Launchpad bug 1706772 in OpenStack Compute (nova) ocata "InternalServerError: Internal Server Error (HTTP 500) in n-cpu logs on startup with Ironic driver" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | |
| 15:24:50 | openstackgerrit | Merged openstack/nova master: Deprecate bare metal filters https://review.openstack.org/492563 | |
| 15:25:50 | sdague | vdrok: ok, as a backport | |
| 15:26:18 | vdrok | sdague: yup, thanks! | |
| 15:27:50 | openstackgerrit | Merged openstack/nova master: Update reno for stable/pike https://review.openstack.org/492982 | |
| 15:33:26 | dtantsur | mriedem: can/should I request a backport for https://review.openstack.org/492563 now? | |
| 15:36:02 | bauzas | folks, time to hang up for a couple of weeks | |
| 15:36:28 | cdent | enjoy the time bauzas | |
| 15:36:32 | bauzas | gentle reminder, I'll be back on 31st of August | |
| 15:36:45 | figleaf | bauzas: thanks for helping with https://review.openstack.org/487954 overnight | |
| 15:36:51 | dtantsur | bauzas: enjoy! | |
| 15:36:54 | figleaf | bauzas: and have a great holiday! | |
| 15:37:19 | bauzas | figleaf: dtantsur: cdent: appreciated, both your warm comments and your help during that very difficult release | |
| 15:37:28 | bauzas | see ya | |
| 15:38:20 | bauzas_afk | figleaf: uh, actually, thanks for the reminder, will remove my -W now | |
| 15:42:50 | figleaf | bauzas_afk: thx | |
| 15:49:37 | openstackgerrit | Merged openstack/nova master: doc: add another up-call caveat for cells v2 for xenapi aggregates https://review.openstack.org/493006 | |
| 15:50:22 | openstackgerrit | Merged openstack/nova master: doc: add superconductor up-call caveat for cross_az_attach=False https://review.openstack.org/493007 | |
| 15:51:12 | mriedem | dtantsur: to stable/pike? | |
| 15:51:24 | dtantsur | mriedem: yeah | |
| 15:51:56 | mriedem | um, idk | |
| 15:52:01 | mriedem | didn't realize that merged | |
| 15:52:56 | dtantsur | the problem is that these filters will be unusable with ironic in Queens | |
| 15:53:07 | dtantsur | so deprecating them in Pike is IMO fair | |
| 15:53:32 | dtantsur | I've proposed the backport in https://review.openstack.org/#/c/493052/ and will leave it up to you | |
| 15:54:37 | mriedem | dtantsur: i see in the ironic release notes that resource_class can't be changed on an active node https://docs.openstack.org/releasenotes/ironic/unreleased.html | |
| 15:54:43 | mriedem | but don't see anything saying resource_class is required | |
| 15:55:08 | dtantsur | mriedem: it's on the way: https://review.openstack.org/491773 | |
| 15:55:12 | mriedem | and wouldn't setting a resource_class be required for compute scheduling? or maybe that's just a requirement on the nova side, and not for standalone ironic | |
| 15:55:16 | mriedem | ah | |