Posted	Nick	Remark
#openstack-nova - 2017-08-23
02:40:51	gmann	if anyone change old schema then it should already catch by current tests
02:41:27	alex_xu	yea, I see now why artom want to run all the api-samples test on the latest version now
02:43:19	gmann	yea
02:45:55	artom	alex_xu, gmann, if you are are up for reviewing that, I can try to conjure the motivation to beat into shape and get it to pass Jenkins ;)
02:46:39	alex_xu	artom: thanks, I need to clear my mind first
02:47:14	artom	alex_xu, of course :) Like I said, it's 22:30, so nothing's happening tonight, that's for sure
02:47:27	artom	And it'll take me a few days, I suspect
02:47:28	alex_xu	artom: yea :)
02:47:57	gmann	artom, alex_xu yea, i also need to think more on this how can cover with minimal effort
05:59:06	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the section about extensions from the API concept doc https://review.openstack.org/496500
06:08:49	openstackgerrit	Merged openstack/nova master: update comment for dropping support https://review.openstack.org/494061
06:20:57	tobasco	cfriesen_: sorry for late reply had to leave work yesterday, thanks for testing it out, we create our instances using the api from our controlpanel perhaps there is some integration issue there, but we are sending with device_name but iirc libvirt does not honor user requested device names as it says in the log
06:21:14	tobasco	i wonder what the difference is, we use a dell backend for our cinder volumes but i doubt it's really related to cinder though
06:21:40	tobasco	cfriesen_: gonna see if i can figure something out, you dont do anything special with properties on your glance image i assume
06:50:37	openstackgerrit	Wonil Choi proposed openstack/nova master: libvirt: Fix getting a wrong guest object https://review.openstack.org/496515
06:50:42	tobasco	cfriesen_: thanks for testing it out earlier, didn't have the same problem when booting from horizon either so must be that we are sending some param when we provision using the api
06:50:55	tobasco	causing that behaviour, perhaps device_name, gonna see if i can narrow it down
07:18:03	openstackgerrit	jichenjc proposed openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798
07:41:45	openstackgerrit	Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589
08:52:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: placement: Make API history doc more consistent https://review.openstack.org/477478
08:53:04	openstackgerrit	Stephen Finucane proposed openstack/nova master: pci: Validate behavior of empty devname https://review.openstack.org/396130
08:54:32	openstackgerrit	Merged openstack/nova master: Add language for compute node configuration https://review.openstack.org/489643
08:54:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology https://review.openstack.org/466030
08:54:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology https://review.openstack.org/485553
08:54:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology https://review.openstack.org/485554
08:54:52	openstackgerrit	jichenjc proposed openstack/nova master: WIP: use conf for keystone session creation https://review.openstack.org/485121
09:26:12	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
09:27:36	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
09:29:03	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929
09:29:37	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214
09:30:18	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561
09:31:20	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications https://review.openstack.org/482622
09:32:09	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851
09:34:38	openstackgerrit	Elod Illes proposed openstack/nova master: Functional test: cold migrate to compute down https://review.openstack.org/496280
09:42:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Improve docstrings of two methods https://review.openstack.org/420298
10:02:38	ratailor	auggy, are you working on https://bugs.launchpad.net/nova/+bug/1538011
10:02:41	openstack	Launchpad bug 1538011 in OpenStack Compute (nova) "Metadata keys are not case sensitive" [Medium,In progress] - Assigned to Augustina Ragwitz (auggy)
10:15:56	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
10:28:31	stephenfin	ratailor: I don't think auggy's working on OpenStack anymore (???). You can grab it
10:28:47	ratailor	stephenfin, Thanks!
10:29:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Deprecate 'keymap' options https://review.openstack.org/483994
10:29:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Allow users to unset 'keymap' options https://review.openstack.org/496605
10:36:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Monkey patch the blockdiag extension https://review.openstack.org/476159
10:36:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Document the scheduler workflow https://review.openstack.org/475810
10:38:35	stephenfin	edleafe: Could you take a look at https://review.openstack.org/#/c/475810/ this week? I don't have the full picture so can't address cdent's comments, tbh
10:39:11	stephenfin	Feel free to take ownership if you feel like it ;)
10:55:55	openstackgerrit	Wonil Choi proposed openstack/nova master: libvirt: Fix getting a wrong guest object https://review.openstack.org/496515
11:00:41	openstackgerrit	Jianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming https://review.openstack.org/486475
11:17:10	kashyap	lyarwood: Thanks for sheperding this upstream stable/ocata change throug - https://review.openstack.org/#/c/489198/
11:17:18	kashyap	s/throug/through/
12:04:12	gmann_	alex_xu: registered BP - https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-queens
12:04:35	gmann_	alex_xu: i added this item in PTG etherpad also - https://etherpad.openstack.org/p/nova-ptg-queens
12:23:04	alex_xu	gmann_: thanks!
12:39:32	openstackgerrit	Viktor Varga proposed openstack/nova master: Remove mox from unit/conductor/test_conductor.py (1) https://review.openstack.org/496691
12:44:22	cdent	(gibi-core)++
12:45:19	alex_xu	nova api meeting is in 15 mins at #openstack-meeting-4
12:45:32	bhagyashri_s	ignored in the pike. [1]: https://review.openstack.org/#/c/428481/ [2]: https://review.openstack.org/#/c/428505/
12:45:32	bhagyashri_s	jaypipes, melwitt: Hi, I want to discuss regarding bfv case, I have done some testing on the stable/pike and what i observed is even if I boot the instance from volume then in that case it allocates the disk_gb as the flavor root_gb in both compute_nodes and allocations table but after applying the patch [1]: https://review.openstack.org/#/c/428481/ on stable/pike then it makes the correct allocations. So why these two patche [1][2] are
12:47:55	edleafe	stephenfin: sure. Would it be easier for me to revise it and push a new version?
12:49:38	stephenfin	edleafe: Probably would, yeah. cdent just left a comment saying he'd take a look too. Feel free to take ownership of the whole thing if you'd like to - I just care about the resulting doc :)
12:49:44	jaypipes	bhagyashri_s: you can propose them as backports to stable/pike. they just were not highest priority during the last two weeks
12:50:03	cdent	edleafe: if you’ve got time/brain you may take point
12:51:00	edleafe	cdent: well, since it was my poorly-worded email that started all this... :)
12:51:09	edleafe	I'll take a crack at it
12:51:26	stephenfin	jaypipes: Talking of backports, could you stick this on your review backlog? https://review.openstack.org/#/c/496605/ Something I'd like to get into Pike before final
12:52:03	bhagyashri_s	jaypipes: ok I can. Thank you :)
12:53:51	cdent	thanks edleafe
12:55:09	jaypipes	stephenfin: I'm not core on stable :)
12:56:03	stephenfin	jaypipes: Good thing it's on master (I hope :)) Master first, then backport
12:56:15	stephenfin	Only if you've time though, of course
12:58:46	stephenfin	thanks edleafe, cdent :)
13:00:26	alex_xu	nova api meeting is running at #openstack-meeting-4
13:04:56	jaypipes	stephenfin: done
13:05:12	stephenfin	jaypipes: Ta :)
13:05:15	mriedem	jaypipes: alex_xu: stephenfin: oomichi: we've got a series of fixes needed for RC2 starting here: https://review.openstack.org/#/c/496031/ - they all have a +2 already
13:05:30	stephenfin	mriedem: Sure, I'll take a look in 2
13:12:30	jaypipes	stephenfin: tá fáilte romhat
13:13:29	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Deprecate 'keymap' options https://review.openstack.org/483994
13:15:02	stephenfin	:D
13:20:29	jaypipes	mriedem: thx. my eyes are now bleeding.
13:27:01	zen369	how do i check what openstack release i'm running?
13:32:18	jaypipes	mriedem: conductor/tasks/live_migrate.py is code that runs in the cell-specific conductor, right? not the superconductor...
13:33:10	dansmith	jaypipes: what does it matter?
13:33:25	dansmith	jaypipes: however, when we have cross-cell migrations it would have to run on superconductor, and I expect it does today
13:33:52	jaypipes	dansmith: oh hey dan :) for some reason I thought you were on vacay...
13:34:00	jaypipes	dansmith: I was just checking that this code: https://review.openstack.org/#/c/496419/2/nova/conductor/tasks/live_migrate.py
13:34:10	jaypipes	dansmith: would indeed be possible to run in the cell conductor
13:34:33	jaypipes	dansmith: since I thought instance mapping information was in the API db.
13:34:54	jaypipes	dansmith: just wanted to double check that those code paths were kosher for the cell conductor.
13:35:08	dansmith	no, cell conductor can't look things up in api db
13:35:27	dansmith	all of our moves should be happening in superconductor
13:36:23	jaypipes	dansmith: ok, so is my cause for concern grounded in good reason?
13:36:35	openstackgerrit	Merged openstack/nova master: Fix quobyte test_validate_volume_no_mtab_entry https://review.openstack.org/495485
13:36:49	dansmith	jaypipes: your concern is that it can't run in cell conductor right?
13:37:02	jaypipes	dansmith: sorry, it's also the end of the day for me so I very well might just be tired :)
13:37:08	jaypipes	dansmith: yeah, that is my concern
13:37:27	dansmith	jaypipes: and I'm telling you it is running in superconductor, so I don't think there's a problem
13:37:46	jaypipes	dansmith: k, got it. I just wanted to double-check, that's all.
13:39:00	dansmith	also, the way devstack in the gate runs, it should fail if he's doing something that requires api db access
13:39:51	jaypipes	oh, good to know we have the safeguard in place. nice. ++

Earlier   Later