| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-29 | |||
| 04:55:25 | openstackgerrit | zhangyangyang proposed openstack/nova master: Move libvirts qemu-img support to privsep https://review.openstack.org/507848 | |
| 08:09:45 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | |
| 10:24:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: setup.cfg: Explicitly set [build_sphinx] builders https://review.openstack.org/508483 | |
| 10:35:05 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Support traits in the Ironic driver https://review.openstack.org/507052 | |
| 10:55:46 | cdent | gibi: in the nova meeting last night it was mentioned that you need somewhere to host some burndown stuff. I’ve got some space that could work | |
| 11:02:39 | gibi | cdent: hi! thanks for the offer. | |
| 11:03:20 | gibi | cdent: the burndown is basically a static html + js and a python script called from cron to generate some data that the js code draws | |
| 11:03:23 | gibi | cdent: https://github.com/gibizer/nova-versioned-notification-transformation-burndown | |
| 11:04:17 | gibi | it was run on OpenShift 2.0 but that is retired and 3.0 does not support cron jobs in the free tier any more | |
| 11:04:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Fix indentation of database options https://review.openstack.org/443097 | |
| 11:04:44 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove 'db_driver' config opt https://review.openstack.org/508487 | |
| 11:04:44 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | |
| 11:05:25 | stephenfin | bauzas: Easy +W here (it's a typo fix) https://review.openstack.org/#/c/489028/ | |
| 11:06:08 | gibi | cdent: I'm happy to rework the code to fitt it to your env if needed | |
| 11:08:54 | cdent | it looks pretty straightforward, let me do a quick poke | |
| 11:14:46 | cdent | gibi: http://burndown.peermore.com/nova-notification/ | |
| 11:14:59 | cdent | how often should the cron job run? | |
| 11:17:05 | gibi | until now it run hourly | |
| 11:17:44 | gibi | so if that not a problem for you then let's do it that way | |
| 11:17:54 | gibi | thanks for moving it to your server | |
| 11:17:56 | gibi | :) | |
| 11:18:09 | cdent | yeah, hourly is fine with me | |
| 11:19:19 | gibi | cdent: I really appreciate your hosting. It saved me at least couple of days of work finding another free hosting and adapting to it. | |
| 11:20:08 | cdent | my pleasure, it’s easy enough to do, you already have all the necessary bits in the repo, very simply put together | |
| 11:20:24 | cdent | should update every hour on the hour now | |
| 11:20:54 | gibi | cdent: sometimes I have to update the to_be_transformed file in the repo (mostly due to decisions not to transform someting). So I will let you know if you need to pull from github | |
| 11:21:24 | cdent | cool, i’ve got it set up so that a pull is all that’s need | |
| 11:21:35 | gibi | awesome | |
| 11:21:36 | gibi | thanks again | |
| 11:21:41 | cdent | could probably even cron that too I suppose | |
| 11:22:01 | cdent | but for now let’s just do it manually by you telling me | |
| 11:22:08 | gibi | sure, that totall works for me | |
| 11:38:50 | sri_ | hey folks: is there any solution for take instance snapshot if instance created form volume ? | |
| 11:45:13 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Move 'ips' field from Subnet object to VIF object https://review.openstack.org/508498 | |
| 12:10:44 | gibi | cdent: I have a question and a nit in https://review.openstack.org/#/c/485209 | |
| 12:25:00 | johnthetubaguy | figleaf: I asked an upgrade question on your spec, I missed it last time :( | |
| 12:25:23 | johnthetubaguy | figleaf: its not a hard question, and I left the answer... but we usually require that to be described in the spec. | |
| 12:44:52 | johnthetubaguy | cdent: is that what happens you have *lots* of coffee and everything in the world seems fixable? | |
| 12:45:35 | openstackgerrit | Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506 | |
| 12:45:41 | figleaf | johnthetubaguy: reading and replying now | |
| 13:02:36 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: possible extra debug info for ironic problem https://review.openstack.org/508508 | |
| 13:03:01 | johnthetubaguy | jaypipes: been double checking the ironic resource provider, its current the ComputeNode uuid as the uuid and the Ironic Node uuid and the name | |
| 13:04:12 | johnthetubaguy | leakypipes: I think that's why things change when ironic does its rebalance, the uuid changes, but the name stays the same | |
| 13:04:36 | leakypipes | johnthetubaguy: on a calll, gimme a few :) | |
| 13:05:22 | johnthetubaguy | leakypipes: no problems, catch you in a bit | |
| 13:09:04 | mriedem | johnthetubaguy: "its current the ComputeNode uuid as the uuid and the Ironic Node uuid and the name" | |
| 13:09:13 | mriedem | that's generic from the resource tracker i think, | |
| 13:09:15 | mriedem | but good point | |
| 13:09:32 | mriedem | because it's just using the hypervisor_hostname for the RP name for the other drivers, which is the same as the hostname | |
| 13:09:36 | johnthetubaguy | yeah, I think its always hypervisor hostname (which can be ironic uuid) | |
| 13:09:36 | mriedem | but for ironic it's the uuid | |
| 13:09:40 | johnthetubaguy | yeah, +1 | |
| 13:09:54 | johnthetubaguy | so this bug is probably more important than I first thought: | |
| 13:10:00 | johnthetubaguy | https://bugs.launchpad.net/nova/+bug/1714248 | |
| 13:10:02 | openstack | Launchpad bug 1714248 in OpenStack Compute (nova) "Compute node HA for ironic doesn't work due to the name duplication of Resource Provider " [High,Confirmed] | |
| 13:10:09 | mriedem | we could put another field in the response from the get_inventory method that includes an optional rp_uuid field or something | |
| 13:10:25 | mriedem | to tell the RT what to use | |
| 13:10:35 | mriedem | although, | |
| 13:10:43 | mriedem | that might break how scheduling works | |
| 13:10:45 | johnthetubaguy | but then we will fail looking up the correct rabbit queue I presume? | |
| 13:10:52 | mriedem | because the scheduler will lookup a compute node from placement's response by uuid | |
| 13:11:04 | mriedem | this has nothing to do with mq | |
| 13:11:11 | mriedem | are you thinking the cell mapping stuff? | |
| 13:11:33 | johnthetubaguy | well the mapping from the placement object, to the compute node you build the instance on | |
| 13:11:39 | cdent | leakypipes: are the slides from your scheduler wars presentations on teh internets somewhere (not the video) | |
| 13:11:52 | mriedem | cdent: the slides would also be on the summit video site | |
| 13:12:18 | cdent | hmmm | |
| 13:12:24 | mriedem | https://www.openstack.org/videos/boston-2017/scheduler-wars-a-new-hope | |
| 13:12:24 | mriedem | or not | |
| 13:12:50 | mriedem | you'll have to get the licensed version from the author :) | |
| 13:13:39 | cdent | yeah, that’s where I starte4d | |
| 13:28:44 | openstack | Launchpad bug 1714248 in OpenStack Compute (nova) "Compute node HA for ironic doesn't work due to the name duplication of Resource Provider " [High,Confirmed] | |
| 13:28:44 | johnthetubaguy | fried_rice: added more details on that ironic rebalance bug from my findings this morning https://bugs.launchpad.net/nova/+bug/1714248: | |
| 13:29:32 | johnthetubaguy | added a patch with some comments in to make it clear where I believe the error occurs | |
| 13:30:07 | openstackgerrit | Elod Illes proposed openstack/nova stable/ocata: WIP: Functional test for regression bug #1713783 https://review.openstack.org/505160 | |
| 13:30:08 | openstack | bug 1713783 in OpenStack Compute (nova) ocata "After failed evacuation the recovered source compute tries to delete the instance" [High,Triaged] https://launchpad.net/bugs/1713783 | |
| 13:32:08 | fried_rice | johnthetubaguy Okay, haven't dug in fully, but... | |
| 13:32:14 | fried_rice | johnthetubaguy It is legal to rename a RP. | |
| 13:32:25 | fried_rice | Would that solve it, or do you actually need to change the RP's UUID too? | |
| 13:32:35 | johnthetubaguy | name is the same, uuid is different | |
| 13:32:36 | cdent | mriedem: in case you missed it before gibi and I got http://burndown.peermore.com/nova-notification/ going | |
| 13:32:44 | fried_rice | boo | |
| 13:32:50 | johnthetubaguy | yeah :( | |
| 13:32:52 | cdent | using his little framework would be easy to burn lots of other things down | |
| 13:33:14 | fried_rice | johnthetubaguy And the original UUID comes from where? | |
| 13:33:25 | johnthetubaguy | fried_rice: ComputeNode.uuid | |
| 13:33:35 | fried_rice | Which makes sense, I suppose. | |
| 13:33:41 | johnthetubaguy | ish, yeah | |
| 13:34:40 | fried_rice | johnthetubaguy Is there an entity in this world that stays the same on this failover deal? | |
| 13:35:17 | johnthetubaguy | fried_rice: ironic node uuid, i.e. ComputeNode.hypervisor_hostname and ResourceProvider.name | |
| 13:35:17 | superdan | finucannot: ack, I'll allow you to retain rights to conf/network | |
| 13:37:03 | fried_rice | johnthetubaguy Ohh, so hang on, it really *doesn't* make much sense for the RP UUID to be the compute host UUID. Or at least, it would make just as much sense for it to be the ironic node UUID. | |
| 13:37:39 | fried_rice | Though tbh, I'm a tad confused as to why those aren't the same thing. | |
| 13:37:45 | johnthetubaguy | fried_rice: except when get the thing back, the uuid, we want it to always be the same thing, in our case a ComputeNode | |
| 13:38:07 | johnthetubaguy | I am not 100% sure at this point | |
| 13:38:18 | fried_rice | Dangit, I've slept since we started this conversation, need to get the model straight in my head again. | |
| 13:38:55 | fried_rice | Hardware-wise, there's a set of machines. These are called "ironic nodes". | |
| 13:38:57 | johnthetubaguy | oh wait, that could be the answer, for ironic, we could mess with the ComputeNode.uuid | |
| 13:39:24 | fried_rice | Each "ironic node" gets modeled as a separate RP, right? | |
| 13:39:28 | johnthetubaguy | fried_rice: https://developer.openstack.org/api-ref/baremetal/#list-nodes | |