| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-17 | |||
| 17:34:55 | cburgess | cfriesen Did you already have a bug open about the pre live timeouts on glance? | |
| 17:41:19 | deeteepee-afk | hey jroll! do you mind if i add your comments from yesterday regarding https://blueprints.launchpad.net/nova/+spec/conf-network-metadata-always-static-ip ? or if you have any other comments, i would appreciate them | |
| 17:41:42 | jroll | leakypipes: having a conversation about ironic capabilities^Wtraits, reading the spec, it sounds like standard traits go through os-traits, custom go through the api. does queens sound like a reasonable timeframe for us to move ironic capabilities into that so people can stop using the ComputeCapabilitiesFilter thing? | |
| 17:42:11 | jroll | leakypipes: (which also makes me wonder, should we go ahead and cover a flavor extra spec for custom traits when we do the custom resource class extra spec?) | |
| 17:42:42 | jroll | deeteepee-afk: yeah, go for it :) | |
| 17:42:56 | deeteepee-afk | thanks | |
| 17:42:59 | leakypipes | jroll: yes and yes | |
| 17:44:33 | jroll | leakypipes: cool - also question from yesterday, I put custom RC allocations as a dependency for the flavor changes, do you want to do that work, and do you want to write a separate spec or put it in this one? https://review.openstack.org/#/c/446570/2/specs/pike/approved/custom-resource-classes-in-flavors.rst | |
| 17:50:33 | openstackgerrit | John Garbutt proposed openstack/nova master: Add and use ProcessedNetworkRequest object https://review.openstack.org/418025 | |
| 17:50:39 | openstackgerrit | John Garbutt proposed openstack/nova master: Pass objects into _validate_requested_network_ids https://review.openstack.org/418027 | |
| 17:50:55 | openstackgerrit | John Garbutt proposed openstack/nova master: Pass objects into _create_ports_for_instance https://review.openstack.org/418512 | |
| 17:51:12 | openstackgerrit | John Garbutt proposed openstack/nova master: Stop passing network when you only need dns_domain https://review.openstack.org/419577 | |
| 17:51:22 | openstackgerrit | John Garbutt proposed openstack/nova master: Pass objects into _update_ports_for_instance https://review.openstack.org/418513 | |
| 17:51:44 | openstackgerrit | John Garbutt proposed openstack/nova master: Extract preallocate_for_instance https://review.openstack.org/418876 | |
| 18:16:24 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: rename qemu_enable_gluster_storage_driver https://review.openstack.org/447091 | |
| 18:23:09 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | |
| 18:23:51 | johnthetubaguy | cfriesen: cburgess: thats that live-migrate API spec that raj_singh is working on ^ | |
| 18:24:19 | johnthetubaguy | there is also the dependent spec thats interesting | |
| 18:25:06 | cburgess | johnthetubaguy Thanks | |
| 18:32:14 | openstackgerrit | Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | |
| 18:47:47 | openstackgerrit | Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | |
| 18:51:46 | openstackgerrit | Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | |
| 19:05:52 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - add some functional tests for placement https://review.openstack.org/446123 | |
| 19:06:00 | figleaf | cdent: if you're bored ^^ | |
| 19:08:36 | openstackgerrit | Sujitha proposed openstack/nova master: Add helper method to add additional data about policy rule. https://review.openstack.org/434842 | |
| 19:19:45 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in agents.py https://review.openstack.org/445208 | |
| 19:21:06 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 19:25:46 | cfriesen | cburgess: sorry, just noticed your message. I don't have a bug open about the pre live timeouts on glance. | |
| 19:25:59 | cburgess | cfriesen k | |
| 19:27:04 | cfriesen | cburgess: mriedem: incidentally, for converting the pre-live-timeout to RPC cast, it's actually coming from the source nova-compute, not from nova-conductor. | |
| 19:27:18 | cfriesen | gah, pre_live_migration, not timeout | |
| 19:27:33 | mriedem | yeah i wondered about that | |
| 19:27:44 | mriedem | because the computes are talking to each other rather than having coordinator orchestrate | |
| 19:27:45 | cburgess | cfriesen LOL well its fairly accurate in some situtations. | |
| 19:27:54 | cfriesen | lol | |
| 19:27:57 | mriedem | which is something tdurakov was looking into | |
| 19:28:17 | mriedem | the timeout hack bug fix doesn't really change though right? | |
| 19:28:24 | mriedem | if it's the source compute or conductor as the rpc client | |
| 19:28:47 | cburgess | mriedem Right... the source caller needs a configurable timeout. | |
| 19:29:01 | cburgess | mriedem Because today is just blocks, hits rpc_timeout, and goes "oh well" | |
| 19:29:54 | mriedem | markus_z left too early https://www.youtube.com/watch?v=bGXaJzgtvS8 | |
| 19:30:13 | mriedem | i have to assume markus loves scorpions | |
| 19:30:14 | cfriesen | mriedem: the live migration is aready a cast from nova-conductor to the source nova-compute | |
| 19:30:27 | cfriesen | mriedem: so this would be a cast from nova-compute to other nova-compute | |
| 19:31:19 | mriedem | sorry i've moved topics | |
| 19:32:11 | cfriesen | I'm running into a case where running "virsh list" on a compute node hangs, trying to figure out why libvirtd isn't responding | |
| 19:34:30 | erhudy | cfriesen: i had a problem a while back where libvirt got really cranky with nwfilters and it would hang like that | |
| 19:34:59 | erhudy | rebooting made it happy again, but it was frustrating not being able to find a root cause | |
| 19:39:51 | mriedem | superdan: leakypipes: sdague: can one of you push this through? i forgot it was still sitting out here https://review.openstack.org/#/c/432471/3 | |
| 19:40:05 | superdan | yeah | |
| 19:40:12 | mriedem | danke | |
| 19:40:15 | leakypipes | mriedem: done | |
| 19:40:23 | mriedem | that was fast | |
| 19:40:26 | mriedem | ...maybe too fast | |
| 19:40:40 | mriedem | i should have put an easter egg in there | |
| 19:40:44 | leakypipes | heh | |
| 19:50:02 | mriedem | ffs is compute_nodes.pci_stats just a json blob? | |
| 19:50:58 | mriedem | ah i suppose it is but it's turned into PciDevicePoolList | |
| 19:51:18 | mriedem | this is awful https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/pci.py#L62 | |
| 19:51:43 | mriedem | we literally take the pci_stats blob out of the db for a compute node, convert it to the object, then convert it back to a json dict and spew it out of the REST API | |
| 19:51:52 | mriedem | so, | |
| 19:52:07 | mriedem | if we ever were to add new fields to PciDevicePool, those automatically go out of the REST API | |
| 19:55:26 | mriedem | https://bugs.launchpad.net/nova/+bug/1673869 | |
| 19:55:26 | openstack | Launchpad bug 1673869 in OpenStack Compute (nova) "api-ref: os-pci API is not documented at all" [Low,Confirmed] | |
| 19:55:33 | mriedem | maybe we should just delete the API before anyone ever knew it existed | |
| 19:57:02 | alex_xu | mriedem: more bad thing, actually the os-pci API isn't enable in the nova rest api | |
| 19:57:45 | alex_xu | mriedem: https://github.com/openstack/nova/blob/master/setup.cfg#L130 | |
| 20:00:08 | mriedem | alex_xu: oh, well that is actually good | |
| 20:01:19 | alex_xu | mriedem: yea, that's ture for defect in the api | |
| 20:01:36 | mriedem | so, if the API was added in v3, but never exposed or supported | |
| 20:01:40 | mriedem | this is really just all dead code isn't it? | |
| 20:01:51 | mriedem | and we would have to decide if we want to expose it out of the REST API in a microversion | |
| 20:01:56 | mriedem | which i'd think we don't actually want to do that | |
| 20:02:52 | alex_xu | mriedem: actually there is plan to do that. it is jsut the people, who commit to do that, disappeared | |
| 20:03:04 | mriedem | sure, | |
| 20:03:09 | mriedem | but that was a long time ago | |
| 20:03:22 | alex_xu | yes, super long time ago | |
| 20:03:25 | mriedem | i'm not sure we need to be exposing pci details out of the compute REST API | |
| 20:04:21 | alex_xu | why? currently we really have no way to query what pci device on the host | |
| 20:06:52 | efried | Should that be a get_inventory() thing? | |
| 20:07:30 | mriedem | efried: unrelated | |
| 20:07:48 | mriedem | alex_xu: i'm sure there are other tools available to get that information from the host system | |
| 20:07:53 | mriedem | actual inventory reporting tools | |
| 20:08:16 | mriedem | alex_xu: i sort of consider this like https://docs.openstack.org/developer/nova/policies.html?highlight=metrics#metrics-gathering | |
| 20:08:43 | mriedem | years ago we thought it was a great idea for nova to be a metrics gatherer, but in newton we decided we're not so great at that | |
| 20:08:51 | mriedem | and shouldn't be in the business of doing it | |
| 20:09:23 | mriedem | i'll post to the ML about the os-pci api | |
| 20:10:09 | alex_xu | mriedem: thanks, i probably get your point. ++ for a email | |
| 20:19:53 | mriedem | http://lists.openstack.org/pipermail/openstack-dev/2017-March/114261.html | |
| 20:20:08 | mriedem | superdan: leakypipes: sdague: ^ you'd probably like to weigh in | |
| 20:21:47 | sdague | mriedem: if you need "... with fire!" sure | |
| 20:22:12 | mriedem | acks are always good | |
| 20:22:18 | mriedem | so i don't just look like a maniac | |
| 20:23:15 | superdan | I thought it was actually kindof important for the pci use case, | |
| 20:23:20 | superdan | but if it's not enabled by default, hopefully that is not true | |
| 20:23:36 | mriedem | apparently not important enough since icehouse for anyone to care to expose it | |
| 20:23:46 | mriedem | or they did via and out of tree extension/patch | |
| 20:23:55 | mriedem | cfriesen: i'm looking at you | |
| 20:24:13 | sdague | superdan: it's not just not enabled, it was never plumbed back in at all | |
| 20:24:14 | superdan | well, it'd be a packaging change, right? but yeah | |