Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-18
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.
22:34:10 mriedem ok
22:34:16 EmilienM bauzas ^ for your morning
23:06:20 openstackgerrit Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494
23:07:23 openstack Launchpad bug 1655662 in OpenStack Compute (nova) "libvirt xen hvm guests can't hot plug (attach detach) default block devices" [Undecided,New]
23:07:23 mriedem anthonyper: you might be able to triage this https://bugs.launchpad.net/nova/+bug/1655662
23:16:45 mriedem melwitt: https://review.openstack.org/#/c/420451/ and down need your watchful eye
23:17:59 melwitt ack
23:26:49 melwitt mriedem: I had been thinking, what about deleting erroneously created cells that have e.g. broken urls in them? I guess the thinking is, those won't have hosts or instances mapped to them since their urls never worked?
23:27:36 mriedem deleting them how?
23:27:41 mriedem like, automatically?
23:27:59 melwitt I'm just imagining someone having a problem because they made a mistake creating some cell with a wrong url in it and they can't get rid of it from their list_cells view
23:28:01 melwitt no
23:28:32 melwitt oh, wait, I guess those would be empty
23:28:37 mriedem yeah :)
23:28:41 mriedem so, just use delete_cell ?
23:29:39 melwitt yeah. I was thinking if there's any problem where someone could get wedged. I think only in a case where they had a valid cell in the past and now want to get rid of it. but that's probably for later anyway, since it's related to idea of re-mapping cells too
23:34:34 mriedem this is also pretty easy https://review.openstack.org/#/c/420132/
23:36:27 melwitt k. I've been chasing a bug in my quota counting patch. that's my excuse for not reviewing these earlier
23:39:12 mriedem tdurakov: if you have highlights from the live migration meeting this week please post them in https://wiki.openstack.org/wiki/Meetings/Nova#Weekly_Nova_team_meeting
23:48:38 openstackgerrit Matt Riedemann proposed openstack/nova: hyperv: make sure to plug OVS VIFs after resize/migrate https://review.openstack.org/422314
#openstack-nova - 2017-01-19
00:24:11 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
00:42:50 openstackgerrit Eric Brown proposed openstack/nova: Remove references to Python 3.4 https://review.openstack.org/417997
01:00:39 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323
01:16:37 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323
01:52:58 openstackgerrit Jeremy Liu proposed openstack/nova: [3/3]Replace six.iteritems() with .items() https://review.openstack.org/415080
02:16:10 openstackgerrit Cao Xuan Hoang proposed openstack/os-vif: Remove support for py33 https://review.openstack.org/422337
02:27:52 alex_xu oomichi: thanks
02:28:02 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
02:33:21 alex_xu oomichi: johnthetubaguy have concern on https://review.openstack.org/#/c/415330/, the patch needs to update
02:34:38 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service https://review.openstack.org/376200
02:34:38 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Enable attach traits to ResourceProvider https://review.openstack.org/376201
02:34:39 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add resource provider traits API https://review.openstack.org/376202
02:34:40 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199
02:36:51 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
02:58:18 openstackgerrit ChangBo Guo(gcb) proposed openstack/nova: VMWare: Move constant power state strings to the constant.py https://review.openstack.org/368383
03:19:29 alex_xu johnthetubaguy: I just replied https://review.openstack.org/#/c/415330/16, I feel we still have problem.
03:23:45 alex_xu johnthetubaguy: also replied your question about why those sort key are bad https://review.openstack.org/#/c/415142/23
03:39:17 openstackgerrit melanie witt proposed openstack/nova: WIP Count resources to check quota for cells https://review.openstack.org/416521
03:50:07 openstackgerrit Tovin Seven proposed openstack/python-novaclient: Add profiling support to novaclient https://review.openstack.org/254699
04:03:04 tovin07_ hi Vek, I saw that this patch (https://review.openstack.org/#/c/254699/) is not auto-merged while depends-on patch is merged yesterday. I re-check it and wait for hours, nothing happens. I rebase it and lost Workflow +1, could you review that.
04:03:24 tovin07_ Thanks!
05:11:51 openstackgerrit Sergey Nikitin proposed openstack/nova: DO NOT MERGE https://review.openstack.org/422221

Earlier   Later