| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-11 | |||
| 13:30:51 | jaypipes | dtantsur: what kind of party? ;) | |
| 13:30:58 | dtantsur | jaypipes: an ironic party! | |
| 13:31:07 | dtantsur | leakypipes: https://etherpad.openstack.org/p/nova-ironic-resource-class-migration | |
| 13:32:05 | mriedem | dtantsur: yeah that might just work | |
| 13:32:15 | dtantsur | okay, let's wait for Jenkins | |
| 13:34:07 | mriedem | ok so the *_used values for inventory will/should match in placement what we have consumed for allocations on that node provider | |
| 13:34:20 | mriedem | so we shouldn't try to remove any inventory, and thus avoid the 409, | |
| 13:34:29 | mriedem | and create the custom resource class when it's added to the node, | |
| 13:34:41 | cdent | that’s the hoe | |
| 13:34:42 | mriedem | and take the node out of scheduling decisions since inventory == allocation | |
| 13:34:43 | cdent | hope | |
| 13:34:49 | mriedem | who you callin a ho | |
| 13:34:50 | mriedem | ?! | |
| 13:34:53 | dtantsur | LOOOL | |
| 13:35:11 | cdent | hoe for capitalism | |
| 13:35:14 | dtantsur | but yeah, this is the plan | |
| 13:42:49 | cdent | gibi is an evolved tool user. on https://review.openstack.org/#/c/491529/ are you saying you think you found a bug in shelve/unshelve itself, or in the tests? | |
| 13:44:59 | cdent | dtantsur: because it has been shortcutted: tox -epy27 ironic | |
| 13:45:12 | gibi | cdent: I think it is in the shelve/unshelve | |
| 13:45:24 | gibi | cdent: but I'm still busy with the evacuation fix | |
| 13:45:26 | cdent | gibi: go you. your powers are strong. | |
| 13:45:51 | dtantsur | awesome, thanks cdent | |
| 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 | |