| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-11 | |||
| 13:25:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: add another up-call caveat for cells v2 for xenapi aggregates https://review.openstack.org/493006 | |
| 13:25:25 | mriedem | dansmith: melwitt: ^ superconductor up-call limitations for the docs - i found another one today | |
| 13:25:38 | mriedem | dtantsur: you mean the inventory? | |
| 13:25:53 | dtantsur | mriedem: yes. sorry, tired already :) | |
| 13:26:02 | dtantsur | Friday is not the best day to debug Nova :) | |
| 13:26:12 | mriedem | dtantsur: no because inv_data will not be empty https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L779 | |
| 13:26:44 | dtantsur | mriedem: right, so the warning should be gone, no? we won't try to delete it, just update? | |
| 13:27:13 | mriedem | dtantsur: i'd expect to get a 409 response from placement here https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L566 | |
| 13:27:38 | mriedem | dtantsur: plus, that doesn't really fix the bug that exists in ocata already, because your fix would only bypass the delete_inventory path iff there is a node.resource_class set | |
| 13:27:54 | mriedem | dtantsur: in other words, i think there are two bugs | |
| 13:28:01 | dtantsur | the 4th problems \o/ | |
| 13:28:13 | dtantsur | should I mark it as Related-Bug then? | |
| 13:28:16 | mriedem | the one i reported about the warnings is latent, and exists in ocata | |
| 13:28:21 | mriedem | dtantsur: that would be ok probably | |
| 13:28:40 | mriedem | i.e. in ocata, if i've got a baremetal env, i'm going to see these warnings in n-cpu every 60 seconds for all nodes | |
| 13:28:51 | mriedem | for all *consumed* nodes | |
| 13:28:54 | mriedem | which is annoying | |
| 13:29:37 | dtantsur | mriedem: wait, why? we no longer return an empty inventory for valid nodes. we return an inventory with s/vcpus/vcpus_used/. I think we can backport it to Ocata even | |
| 13:30:14 | dtantsur | jaypipes: hi, you may want to join the party :) | |
| 13:30:20 | mriedem | oh i see | |
| 13:30:27 | mriedem | i hadn't seen the latest patch https://review.openstack.org/#/c/492964/3/nova/virt/ironic/driver.py | |
| 13:30:50 | dtantsur | ah! | |
| 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 ? | |