| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-18 | |||
| 18:35:57 | shewless | anyways.. thanks.. I'm further on now | |
| 18:53:27 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Raise general error message when volume attach fails https://review.openstack.org/320163 | |
| 19:00:16 | mriedem | https://review.openstack.org/#/c/398473/ needs another core | |
| 19:00:31 | mriedem | would get us down to 1 remaining change for ironic + resource classes | |
| 19:01:12 | mriedem | jaypipes: john is +2 on the osprofiler change now, which you were +2 on before if you are still https://review.openstack.org/#/c/254703/ | |
| 19:01:18 | mriedem | and when you get a break | |
| 19:03:48 | _gryf | EmilienM, bauzas, cdent, mriedem, doesn't placement API supposed to run on 80 port? if so, than this apache vhost is configured somehow wrong, doesn't it? http://logs.openstack.org/04/419604/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/5d95a8c/logs/apache_config/10-placement_wsgi.conf.txt.gz | |
| 19:03:56 | _gryf | or maybe it is not relevant at all? | |
| 19:04:38 | EmilienM | this is not relevant, you can run an API how you like on the port you like | |
| 19:04:48 | cdent | _gryf: how devstack configures things and how puppet configures things is not necessary going to be the same | |
| 19:04:49 | _gryf | okay | |
| 19:08:59 | openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | |
| 19:09:33 | openstackgerrit | Diana Clarke proposed openstack/nova: Fix map_cell_and_hosts help https://review.openstack.org/422171 | |
| 19:10:38 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | |
| 19:11:43 | openstackgerrit | Dean Troyer proposed openstack/python-novaclient: DNM: another OSC+SDK check https://review.openstack.org/422174 | |
| 19:21:21 | jaypipes | mriedem: done. | |
| 19:37:14 | mriedem | jroll: can you have the same vif id in a list of ports and a list of portgroups? | |
| 19:37:16 | mriedem | for the same node? | |
| 19:53:27 | mriedem | vdrok: i've got some questions in https://review.openstack.org/#/c/388756/ | |
| 19:53:32 | jroll | mriedem: no, a vif id can only be attached to one thing, AIUI | |
| 19:54:21 | mriedem | jroll: question the 2nd: can you have a port in more than one portgroup for the same node? | |
| 19:55:29 | jroll | mriedem: no | |
| 19:55:53 | mriedem | 3rd: who is your daddy and what does he do? | |
| 19:58:39 | jroll | :P | |
| 19:59:18 | vdrok | mriedem: thanks, a bit late now, will post replies tomorrow. Short version - duplicates are not possible because of the way nova generates id ( was asked by rloo in previous PS ) and cloud init does not validate parameters :) | |
| 19:59:46 | mriedem | ok | |
| 20:13:04 | wasmum | instance build from an image - running fine. needed a reboot. The original image is no longer in the image store and thus instance erroring out. Any way to resolve that? | |
| 20:13:38 | mriedem | wasmum: did you snapshot the instance? | |
| 20:13:49 | mriedem | do you have the image in the image cache on the compute host? | |
| 20:14:17 | wasmum | mriedem: image cache, tell me more? | |
| 20:16:20 | mriedem | if you're using the libvirt driver, it will store the base image on the compute host the first time it uses it so it doesn't have to re-download commonly used images that build on the same host | |
| 20:16:47 | mriedem | i believe the image cache will clean after itself after there are no instances using a given image in the cache on that host though after some period of time | |
| 20:17:16 | wasmum | where does it pull that image down too? | |
| 20:17:26 | wasmum | file location? I'll check ? | |
| 20:17:37 | mriedem | should be under the instance state path, so i think /var/lib/instances somewhere | |
| 20:17:46 | openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | |
| 20:17:46 | openstackgerrit | Sergey Nikitin proposed openstack/nova: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480 | |
| 20:17:47 | openstackgerrit | Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | |
| 20:18:14 | mriedem | see the image_cache_manager_interval config option | |
| 20:18:25 | mriedem | and image_cache_subdirectory_name | |
| 20:18:53 | mriedem | and cache_images | |
| 20:19:54 | mriedem | are you using local disk or shared? | |
| 20:20:07 | cfriesen__ | does the separation of some models out into api_models.py have any implications for upgrades? or are they still the same DB tables, just moving the python code around? | |
| 20:20:15 | mriedem | mdbooth: any idea if we have any decent documentation anywhere about the image cache manager? | |
| 20:20:32 | mriedem | cfriesen__: such as? | |
| 20:20:33 | mriedem | flavors? | |
| 20:20:54 | mriedem | cfriesen__: api_models is the API DB schema | |
| 20:21:07 | mriedem | some tables are the same or similar, like flavors / instance_types | |
| 20:21:27 | mriedem | cfriesen__: the models are not entirely the same though, i.e. the API DB doesn't use the soft delete mixin (the deleted and deleted_at columns are not in the API DB) | |
| 20:21:48 | mriedem | cfriesen__: the nova-manage db online_data_migrations is meant to move the data from the cell db to the api db | |
| 20:21:52 | mriedem | s/cell/nova/ | |
| 20:22:22 | openstackgerrit | Sergey Nikitin proposed openstack/nova: DO NOT MERGE https://review.openstack.org/422221 | |
| 20:22:24 | cfriesen__ | mriedem: I was looking at InstanceGroup in particular | |
| 20:23:53 | cfriesen__ | mriedem: a cursory check didn't see any code to migrate InstanceGroup data from the old model to the new | |
| 20:26:54 | mriedem | cfriesen: https://github.com/openstack/nova/blob/master/nova/objects/instance_group.py#L525 | |
| 20:27:25 | mriedem | cfriesen: called from here https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L612 | |
| 20:27:47 | mriedem | which is initially called from, let's all say it now, https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L763 | |
| 20:29:32 | cfriesen | mriedem: much appreciated | |
| 20:29:41 | mriedem | yw | |
| 20:36:14 | wasmum | mriedem: looks like it was built from a snapshot, snapshot was deleted | |
| 20:36:38 | wasmum | learning lessons on why not to build snowflakes | |
| 20:36:56 | mriedem | wasmum: and the image isn't cached on the compute host? | |
| 20:37:35 | wasmum | looks like it was. tried to "start" the instance using virsh and got the error of the missing backup file | |
| 20:38:04 | wasmum | looks like it was in the instances path in a "_base" directory and that must have expired? | |
| 20:38:33 | mriedem | not sure | |
| 20:38:40 | mriedem | there is a related write up from kashyap here https://kashyapc.com/2013/03/03/openstack-nova-dealing-with-unused-base-images/ | |
| 20:45:44 | openstackgerrit | Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | |
| 20:45:44 | openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | |
| 20:46:05 | wasmum | mriedem: thanks a ton | |
| 20:46:18 | mriedem | yw | |
| 20:48:38 | openstackgerrit | Sujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | |
| 20:52:12 | mriedem | edleafe: i know you're on PTO but are you satisfied https://review.openstack.org/#/c/418134/6/nova/tests/functional/db/test_compute_node.py@102 ? | |
| 20:53:03 | mriedem | i guess you're looking for a slight wrinkle there | |
| 20:53:05 | edleafe | mriedem: I was only on PTO Monday :) | |
| 20:53:09 | mriedem | we have 2 computes, you want 1 | |
| 20:53:12 | mriedem | make sure you get 1 and not 2 | |
| 20:53:36 | edleafe | mriedem: I wanted a case where only one node was returned. The current are either all or none | |
| 20:53:40 | mriedem | ok | |
| 20:53:40 | mriedem | yeah | |
| 20:54:01 | mriedem | i'll add that quick | |
| 20:55:22 | edleafe | mriedem: I was going to add it myself, but I didn't want to mess up bauzas's stuff | |
| 20:55:44 | mriedem | i've got it | |
| 20:59:40 | openstackgerrit | Matt Riedemann proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method https://review.openstack.org/418134 | |
| 20:59:41 | openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 21:00:12 | mriedem | +2 on that mofo now | |
| 21:00:50 | edleafe | mriedem: kewl | |
| 21:01:09 | mriedem | i'm trying to work on my list of 'these bp patches need another +2' by the end of hte day | |
| 21:04:46 | oomichi | alex_xu: yeah, the first one is good fo me, thanks Kevin_Zheng and alex_xu | |
| 21:05:35 | oomichi | alex_xu: https://review.openstack.org/#/c/415330/ has johnthetubaguy's -1 | |
| 21:06:25 | openstackgerrit | Ruby Loo proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | |
| 21:13:32 | mriedem | mikal: so you love the metadata api, | |
| 21:14:18 | mriedem | mikal: would changing the content of the network_data.json dict within the metadata api response constitute a bump in the API version, even though we don't filter the results to that level of granularity? i.e. https://review.openstack.org/#/c/388756/18/nova/virt/ironic/driver.py | |
| 21:14:27 | mriedem | well, | |
| 21:14:53 | mriedem | this would actually only be config drive since this is modified when the instance is created for the config drive and wouldn't be available via the metadata API, so nvm | |
| 21:21:01 | openstackgerrit | Francesco Santoro proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | |
| 21:49:55 | openstackgerrit | Pushkar Umaranikar proposed openstack/nova: [Do not merge] Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259 | |
| 21:58:16 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | |
| 22:00:45 | openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | |
| 22:32:28 | mriedem | claudiub: should https://review.openstack.org/#/c/397722/ be abandoned? | |
| 22:34:02 | EmilienM | mriedem: quick update on placement thing I reported this morning. We found a problem in HAproxy config in TripleO, we're working on it now. It's probably related. It's green for Puppet OpenStack CI. | |