Posted	Nick	Remark
#openstack-nova - 2017-02-10
19:57:17	mriedem	i'm only the key master
19:58:17	sdague	an alternative is to provide a much more extensive search interface for flavors, including, give me one like server={id}
19:59:31	openstackgerrit	Dan Smith proposed openstack/nova master: Remove flavor_* functions from db/api https://review.openstack.org/431752
19:59:31	openstackgerrit	Dan Smith proposed openstack/nova master: Remove straggling use of main db flavors in cellsv1 code https://review.openstack.org/432352
19:59:32	openstackgerrit	Dan Smith proposed openstack/nova master: Remove main_db flavor compatibility https://review.openstack.org/431722
19:59:39	openstackgerrit	melanie witt proposed openstack/nova master: Add Cells V1 -> Cells V2 step-by-step example https://review.openstack.org/429075
20:00:57	superdan	mriedem: I can make lyarwood very uncomfortable at team social events, so I think it'll be approved
20:01:00	openstackgerrit	Jay Pipes proposed openstack/nova-specs master: Record reality of placement API usage from n-sched https://review.openstack.org/432442
20:01:01	leakypipes	mriedem: ^
20:06:11	mriedem	ocata, pike and queens marketing and roadmaps are all done, whew
20:13:58	mriedem	superdan: +2
20:15:18	superdan	thanks
20:16:57	mriedem	leakypipes: see my comment in https://review.openstack.org/#/c/432442/ - up to you if you want to adjust anything there
20:17:04	mriedem	if not, i'm your huckleberry
20:18:58	cfriesen	sdague: what would "one like server={id}" imply? at least as good in all respects? what about stuff like aggregate_instance_extra_specs keys?
20:19:29	scottda	mriedem: Just saw your message re: Nova using cinder v3 as default. That seems ok to me.
20:19:46	scottda	mriedem: And we've plenty of time to test before next release
20:19:58	mriedem	scottda: the only thing is i'd still want at least one job running with cinder v2 just to make sure we don't break something there
20:20:06	mriedem	well, nvm
20:20:08	mriedem	grenade would test that
20:20:28	mriedem	scottda: let me air out the idea in the ML and see if john can poke hole sin it
20:20:46	scottda	mriedem: Sounds good.
20:21:40	smcginnis	I'd love to just have the one at some point too.
20:21:52	sdague	cfriesen: right, it's a whole other API for sure
20:21:54	openstackgerrit	Eric Fried proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387
20:25:37	openstackgerrit	Jay Pipes proposed openstack/nova-specs master: Record reality of placement API usage from n-sched https://review.openstack.org/432442
20:25:43	leakypipes	mriedem: try ^ on for size.
20:26:29	mriedem	leakypipes: only if it says "juicy" on my ass
20:31:46	leakypipes	mriedem: lol
20:34:52	superdan	mriedem: just realized if we want to remove all that flavor code in pike and the tables/models in queens we need a blocker migration
20:35:33	mriedem	yeah
20:36:08	mriedem	pretty simple isn't it? if there are any flavors with delete==0 in the main db we fail right?
20:36:13	mriedem	*deleted
20:36:15	superdan	also, we only had two actual migrations in ocata and they were small
20:36:29	superdan	mriedem: yep, but flavors, aggregates, server_groups, keypairs at least
20:37:25	superdan	oh wait
20:37:29	superdan	that was in newton
20:37:37	openstackgerrit	Pushkar Umaranikar proposed openstack/nova master: Placement api: set custom json_error_formatter in resource_class https://review.openstack.org/430463
20:37:50	openstackgerrit	Maciej Szankin proposed openstack/nova master: conf: resolved final todos in libvirt conf https://review.openstack.org/429042
20:37:51	superdan	so we did that already in the pike migration.. sweet, nevermind, I lost track
20:39:01	mriedem	ocata migration...?
20:39:38	openstackgerrit	Maciej Szankin proposed openstack/nova master: conf: resolved final todos in libvirt conf https://review.openstack.org/429042
20:39:38	superdan	mriedem: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/345_require_online_migration_completion.py
20:39:46	mriedem	yeah
20:41:41	mriedem	efried: you need to run py27 on your spec thing
20:41:44	mriedem	zuul is unhappy
20:42:13	efried	mriedem Roger that. (Who knew an RST needed python validation)
20:42:31	efried	(I'm sure it's documented, isn't it)
20:43:51	mriedem	leakypipes: https://review.openstack.org/#/c/432442/ fits, thanks
20:43:55	dims_	mriedem : "cat woke me up at 3am" :)
20:43:58	mriedem	superdan: want to get that?
20:44:17	superdan	mriedem: gross.
20:44:21	mriedem	dims_: yes, well, i was awake at 2, tossed and turned for 45 minutes, fell asleep, and then the cat was running around making noise and woke me up again
20:44:47	mriedem	then i proceeded to think about cinder v3 for some reason
20:45:03	superdan	mriedem: I have to -1 that
20:45:30	mriedem	superdan: the spec or the mid morning cinder v3?
20:45:37	superdan	mriedem: the spec
20:46:42	superdan	well, I guess maybe he's copy/pasting a bit.. grr, I will note my complaint and proceed I guess
20:49:30	mriedem	because of "behaviour" with a "u"?
20:49:54	superdan	eff yeah man
20:50:01	mriedem	you know jay eats blood pudding for breakfast right?
20:50:23	mriedem	breakfaust
20:50:36	superdan	heh
20:56:39	openstackgerrit	Eric Fried proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387
20:57:50	openstackgerrit	Pushkar Umaranikar proposed openstack/nova master: Remove unused columns from BuildRequest table in nova_api db https://review.openstack.org/422259
21:03:19	openstackgerrit	Maciej Szankin proposed openstack/nova master: conf/compute: remove invalid TODOs https://review.openstack.org/432453
21:06:56	efried	mriedem Done and done. Thanks for the feedback.
21:07:12	openstackgerrit	Maciej Szankin proposed openstack/nova master: conf/libvirt: remove invalid TODOs https://review.openstack.org/432455
21:43:06	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Fix a couple of spec names https://review.openstack.org/432469
21:52:41	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Fix paths when moving specs https://review.openstack.org/432470
21:52:42	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Move implemented specs for Ocata https://review.openstack.org/432471
21:59:59	openstackgerrit	Sujitha proposed openstack/nova master: Change os-server-tags default policy https://review.openstack.org/396420
22:00:48	mriedem	melwitt: https://review.openstack.org/#/c/429075/ looks good - leakypipes want to +W that?
22:03:03	leakypipes	mriedem: done
22:03:44	openstackgerrit	Pushkar Umaranikar proposed openstack/nova master: Placement api: set custom json_error_formatter in inventory https://review.openstack.org/432478
22:17:17	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: PowerVM Driver Integration (Pike) https://review.openstack.org/432387
22:22:21	efried	mriedem Thanks!
22:23:15	mriedem	np
22:23:27	openstackgerrit	Pushkar Umaranikar proposed openstack/nova master: Adopts keystoneauth with glance client. https://review.openstack.org/412634
22:25:09	mriedem	superdan: i'm going to start bugging you about stable/newton things, starting with https://review.openstack.org/#/c/408725/
22:28:58	superdan	mriedem: hmm, so this one is in ocata and we're just backporting it if they want to get a jump on the cleanup right?
22:30:00	mriedem	superdan: yeah, it was reportd by someone moving to newton
22:30:09	mriedem	so i fixed on ocata with the intent of backporting
22:30:14	mriedem	hence the comments in there
22:33:01	superdan	yeah, reading, I just want to get my head wrapped around it, since this is fairly strange
22:33:14	superdan	people would have run the online migrations when they upgraded to newton and wouldn't again until the next time
22:36:42	mriedem	this is to resolve the issue for anyone upgrading from mitaka before the hit the issue, unless i'm not catching your point
22:37:36	superdan	yeah, I think I get it
22:37:51	mriedem	in mitaka we had the build request mirror the instance,
22:38:04	mriedem	failed instance build in the api (i think) which didn't get deleted,
22:38:06	mriedem	something like that
22:38:15	mriedem	then in newton, we serialized the instance into the build request,
22:38:29	mriedem	but when you try to load that old build request from mitaka, it's data format is different and blows up on load
22:38:52	superdan	yeah
22:39:00	superdan	why didn't we just do the delete.filter(instance_uuid=null()).limit($limit) ?
22:39:05	superdan	instead of the pre-query?
22:39:11	superdan	I mean too late now, just thinking
22:39:20	mriedem	this https://review.openstack.org/#/c/306023/
22:39:45	mriedem	because it's sticking with the limit/count semantis in the online migrations cli
22:39:54	mriedem	we talked about just doing the delete straight up on master

Earlier   Later