| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-10 | |||
| 19:38:46 | mriedem | ? | |
| 19:38:52 | superdan | sdague: well, I just don't see the point in documenting a thing as "likely wrong" when we could just not expose it, but whatever | |
| 19:39:01 | mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/index.html | |
| 19:39:22 | superdan | mriedem: "leakypipes posted an inappropriate picture in IRC" is a pike event I guess | |
| 19:39:56 | cfriesen | superdan: I kind of feel like without either name or id, we don't really have a good starting point for "find me another flavor like this one" | |
| 19:40:01 | mriedem | there was a lot of random feature parity stuff that got in for hyper-v, xen and ironic virt drivers, but i can't really say that's one thing, unless i group them together | |
| 19:40:27 | superdan | cfriesen: you compare the resources not the id.. the id _is_ fiction | |
| 19:40:43 | mriedem | there is actually a lot in https://specs.openstack.org/openstack/nova-specs/specs/ocata/index.html which didn't make the release | |
| 19:40:54 | superdan | cfriesen: to put it in a way that mriedem and sdague will love: how do you know if your cloud provider doesn't dickpunch people by changing flavors? | |
| 19:41:03 | superdan | cfriesen: you don't, so you can never rely on it, thus we should never expose it | |
| 19:41:15 | superdan | er, I meant s/mriedem/mordred/ above | |
| 19:41:18 | mriedem | yeah... | |
| 19:41:25 | cfriesen | superdan: you'd still have to check it it matches, but it gives you something to check rather than starting from scratch and searching all the flavors. | |
| 19:41:26 | mriedem | i love a good dickpunch | |
| 19:41:29 | mriedem | makes me feel alive | |
| 19:41:48 | superdan | cfriesen: but you don't know if your provider is good, so you always have to check the resources anyway | |
| 19:42:21 | cfriesen | superdan: you can start by checking whether that particular flavor still matches, rather than starting from scratch. | |
| 19:42:24 | superdan | mriedem: we re-released device tags? :D | |
| 19:42:33 | mriedem | superdan: that was for attach | |
| 19:42:38 | mriedem | attach vif, attach volume + tags | |
| 19:42:40 | mriedem | which didn't happen | |
| 19:42:53 | mriedem | the original spec covered boot and attach | |
| 19:42:55 | superdan | mriedem: well, I was just trying to be funny anyway, so thanks for ruining it | |
| 19:43:05 | mriedem | oh | |
| 19:43:05 | superdan | mriedem: no, I thought the re-release thing was for boot | |
| 19:43:15 | superdan | mriedem: because get me a network stepped on us | |
| 19:43:22 | mriedem | i'm talking about this one https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/virt-device-role-tagging.html | |
| 19:43:31 | superdan | I'm talking about the bugfix | |
| 19:43:32 | superdan | anyway | |
| 19:43:33 | mriedem | yeah https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/fix-tag-attribute-disappearing.html | |
| 19:43:36 | mriedem | har har | |
| 19:43:59 | mriedem | nova-status upgrade check was a thing | |
| 19:44:04 | superdan | yeah | |
| 19:44:05 | mriedem | came in late, seems fun | |
| 19:44:09 | mriedem | no bp | |
| 19:44:14 | superdan | and it's good press | |
| 19:44:54 | mriedem | yeah, because no one cares about being able to soft reboot ironic instances :) | |
| 19:45:21 | superdan | heh | |
| 19:51:20 | superdan | mriedem: so this is the cellsv1 regression I found whilst removing the flavor compat, which I pulled out into a separate fix: https://review.openstack.org/#/c/432352/1 | |
| 19:51:27 | superdan | i think we need that back in newton | |
| 19:53:35 | mriedem | file a bug? | |
| 19:54:53 | mriedem | leakypipes: we don't support scheduling decisions based on shared storage in ocata right? we never leveraged the aggregates stuff yet from whati remmeber | |
| 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" :) | |