Earlier  
Posted Nick Remark
#openstack-nova - 2017-05-25
01:43:09 openstackgerrit John Griffith proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
03:03:27 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
03:15:35 mriedem Kevin_Zheng: looking better but no release note yet :)
03:15:56 Kevin_Zheng ...ah...god, what was I thinking
03:20:01 gmann mriedem: re-thinking on quota class fix, as we are going to move quota/limit thing to keystone (as discussed in PTG) , is it worth to fix this API in microversion when there is a work around of getting default quotas from os-quota-set
03:20:18 gmann mriedem: but yes that's per projects not global default
03:26:17 mriedem gmann: it seems trivial to fix,
03:26:31 mriedem and i'd rather not assume we're going to have nova using limits from keystone in queens
03:27:14 mriedem anyway, i have to call it a day
03:27:19 mriedem ttyl
03:27:43 gmann mriedem: ok, yea trivial but it increase microversion numbers :) anyways yea keystone limits might be far.
03:34:57 openstackgerrit falseuser proposed openstack/nova master: Fix the problems that occur during rebuild operations https://review.openstack.org/467588
03:48:30 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI use os-xenapi v2 in nova https://review.openstack.org/453493
04:00:08 sayantan_ Hi. I am a little stuck. While creating an instance, I get the following error "no valid hosts found. There are no enough hosts available"
04:00:39 sayantan_ When I check the nova-compute.log I see the error "nova.virt.libvirt.driver [-] Error defining a domain with XML: <domain type="kvm">"
04:00:50 sayantan_ Any idea what is going wrong here?
06:17:22 openstackgerrit suzhengwei proposed openstack/nova master: compute_model problem https://review.openstack.org/462897
07:02:49 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
07:06:16 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
08:12:20 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Exclude deleted service records when calling hyperrvisor statistics https://review.openstack.org/467220
08:22:27 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569
08:30:08 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Fix vif plug problem https://review.openstack.org/467926
08:36:34 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: nova-compute cannot restart after manually delete VM https://review.openstack.org/467926
08:48:47 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI use os-xenapi v2 in nova https://review.openstack.org/453493
09:11:50 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
09:16:05 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
09:26:32 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
09:28:02 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818
09:32:05 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
09:32:30 alex_xu sfinucan: appreciate if you can approve this one https://review.openstack.org/#/c/461628/
09:32:54 alex_xu litao__: yea, it is on my review list
09:33:32 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/467948
09:33:37 litao__ alex_xu: thanks
09:34:10 openstackgerrit Bin Zhou proposed openstack/nova master: Insert the description of the change. https://review.openstack.org/467952
09:40:05 alex_xu litao__: np
10:27:00 openstackgerrit Lucian Petrut proposed openstack/nova master: [Trivial] Hyper-V: accept Glance vhdx images https://review.openstack.org/467369
10:58:33 openstackgerrit Sean Dague proposed openstack/nova master: remove oslo.context exact key tests https://review.openstack.org/467995
11:00:34 sdague sfinucan / bauzas ^^^
11:00:49 sdague we have unit tests that break on the new oslo.context because legacy
11:00:53 sdague we should just delete them
11:10:37 openstackgerrit Sean Dague proposed openstack/nova master: Migrate to oslo request_id middleware https://review.openstack.org/467998
11:17:15 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923
11:28:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set https://review.openstack.org/467999
11:30:40 gmann alex_xu: ^^ fix for quota class set bug, i put this as WIP till you push the spec for this
11:43:30 sdague alex_xu: oh, if you are still up https://review.openstack.org/467995
11:44:14 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
11:48:26 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Exclude deleted service records when calling hyperrvisor statistics https://review.openstack.org/467220
11:52:54 openstackgerrit Chris Dent proposed openstack/nova master: Handle new hosts for updating instance info in scheduler https://review.openstack.org/454426
11:59:55 gmann sdague: alex_xu: added api-ref for quota class set and documented the bug there. check if it looks fine. https://review.openstack.org/467938
12:00:36 gmann adding this API mystery&magic as warning :)
12:04:59 sdague cool, I'll take a look in a bit
12:19:24 mdbooth After tearing my hair out for most of the morning it transpires that os-client-config, rather than just reading environment variables, actually pops them. I mean, why would anybody want to look an an environment variable again?
12:19:55 mdbooth Of course, this breaks anything which gets a config more than once
12:20:28 mdbooth Like OVB deploy, for eg
12:30:29 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: provide interface-scoped nameserver information https://review.openstack.org/467699
12:49:21 cdent mdbooth: the universe does not want you to have hair
13:09:00 openstackgerrit Sean Dague proposed openstack/nova master: Migrate to oslo request_id middleware https://review.openstack.org/467998
13:22:46 savvas smcginnis: good morning! Were you by any chance able to reproduce that issue I reported yesterday regarding the Any Availability zone selection when creating an instance? Which resulted in cinder failing to create a volume
13:23:12 mriedem o/
13:24:16 smcginnis savvas: Unfortunately no, having some issues getting it set up at the moment.
13:24:28 smcginnis savvas: I was wondering if you could open a bug to make sure we track this?
13:24:29 mriedem dansmith: i've got super special early morning news, the stable/newton backport of the db creds special characters fix and all changes below it are passing https://review.openstack.org/#/c/466808/ https://review.openstack.org/#/c/420750/ https://review.openstack.org/#/c/420876/
13:24:46 mriedem including the change for the cell0 db name
13:24:47 smcginnis savvas: I looked through the code quick and it does look like we should be handling None correctly.
13:25:10 savvas I'd be happy to. Where should I report this?
13:26:16 sdague mriedem: did you get my email on the db test case debug?
13:26:28 mriedem sdague: reading now
13:26:39 smcginnis savvas: In the top right corner here: https://bugs.launchpad.net/cinder/+bugs
13:26:43 sdague cell1 is still sqlite
13:26:47 sdague is the tldr
13:27:54 mriedem hmm
13:28:22 mriedem that's got to all be done in a fixture,
13:28:30 mriedem and i'd think we could sub the db engine
13:29:18 sdague yeh, the cell magic switchero fixture is pretty complicated, but it's in there
13:29:56 sdague the real challenge is that because we're running this all in a single process, there was something about the context manager that was "interesting" that melwitt and dansmith worked through
13:30:52 mriedem fwiw i didn't think i was using the main cell database fixture
13:30:58 mriedem that did all the switching stuff
13:31:18 mriedem i was using the SimpleCellFixture, which i'm not sure is right to use, but i thought would be simpler to start
13:31:27 sdague mriedem: no this - https://github.com/openstack/nova/blob/76c2453682a7afa26ff2a7eb825e900af53a61d6/nova/tests/fixtures.py#L363-L532
13:31:39 mriedem right i didn't tihnk i was using that
13:31:43 sdague right, you aren't
13:31:51 sdague and because of that, it doesn't work
13:32:03 sdague because engine facade is global
13:32:05 mriedem huh, i thought SimpleCellFixture would take care of things
13:32:18 danpawlik Morning
13:32:39 mriedem this one https://github.com/openstack/nova/blob/76c2453682a7afa26ff2a7eb825e900af53a61d6/nova/tests/fixtures.py#L257
13:32:40 sdague I'm pretty sure that stopped working once there was the context manager calls
13:33:26 mriedem https://github.com/openstack/nova/blob/76c2453682a7afa26ff2a7eb825e900af53a61d6/nova/tests/fixtures.py#L328 ?
13:33:27 sdague https://github.com/openstack/nova/blob/76c2453682a7afa26ff2a7eb825e900af53a61d6/nova/compute/api.py#L2401 is basically where it goes off the rails
13:33:58 mriedem that shouldn't actually do anything https://github.com/openstack/nova/blob/76c2453682a7afa26ff2a7eb825e900af53a61d6/nova/tests/fixtures.py#L296
13:34:15 sdague the inst_map.cell_mapping is
13:34:16 sdague CellMapping(created_at=None,database_connection='sqlite:///',id=1,name='onlycell',transport_url='fake://nowhere/',updated_at=None,uuid=1d5950cd-863a-4eb6-a3ec-3583fc93fbfc)
13:34:34 mriedem sdague: yeah i think because of this https://github.com/openstack/nova/blob/76c2453682a7afa26ff2a7eb825e900af53a61d6/nova/tests/fixtures.py#L328
13:34:46 mriedem i think SingleCellSimple is too simple here
13:34:52 mriedem and why i wasn't sure it would work
13:34:54 mriedem so that confirms it
13:34:56 sdague yeh
13:35:17 mriedem well, i think i can add the cell1 mapping manually
13:35:24 mriedem we have the api db setup
13:35:26 sdague so, I guess the question is if you can pull that from your config

Earlier   Later