Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-10
19:54:55 mriedem *remember
19:55:04 leakypipes mriedem: that is correct.
19:55:55 leakypipes mriedem: this merged: https://review.openstack.org/#/c/407309/. but it's only the beginning of what is needed.
19:56:20 leakypipes mriedem: just wrapping up that ocata scheduler-db-filters spec patch now...
19:56:32 superdan mriedem: I will if you're cool with that going back
19:56:57 mriedem superdan: yeah that's fine
19:57:04 mriedem i'm not the one you have to convince
19:57:08 mriedem lyarwood is the gate keeper
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,

Earlier   Later