Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-10
19:34:20 sdague mordred: honestly, it is better to get the horizon expectations reset
19:34:21 mordred yah. I see that ship as having sailed
19:34:23 superdan that's also punching ops in the face
19:34:39 mordred fwiw, I do not feel _strongly_ about name or id. I have a contract with shade users that I'll always be providing an id key in the flavor dict of a server, but I can easily just plop a none in there if it goes away from the server record
19:35:08 superdan mordred: in some sense, your flavor id becomes the uuid of the instance itself after boot
19:35:09 mordred so, in the realm of "mordred will scream and yell and rant for years" and "mordred will just sigh and have a beer" 0 this is WAY much more in the beer stage than the other
19:35:19 superdan mordred: if we later allow you to "buy an extra gig of ram".. that's what it'll be
19:35:22 mordred superdan: I can see that argument
19:35:57 sdague superdan: so, while it could some day
19:36:02 sdague it really doesn't today
19:36:14 superdan sdague: what doesn't?
19:36:35 sdague it feels like there is definitely a common case of "I want another like that one"
19:36:52 superdan sdague: that's even worse, because we know that id can be wrong
19:36:57 superdan if you teach people to ask for another just like it,
19:36:58 sdague and if people didn't rename all their flavors, or hot resize
19:37:02 sdague then it can work
19:37:03 superdan they're going to get a 0disk 64mb instance
19:37:22 superdan didn't johnthetubaguy tell me that RAX has renamed flavors before?
19:37:32 sdague it feels like the moment you hot added anything to the instance, the flavor id flips to uuid of the instance
19:37:46 sdague so we actively decouple there
19:38:08 superdan sdague: but if you change/delete a flavor, it's the same thing, and we can't really query for any instances that need changing if you alter the flavor
19:38:13 superdan that's hugely expensive
19:38:24 sdague superdan: sure, and that's why we document it as "possibly fiction"
19:38:26 superdan bigly expensive I mean
19:38:29 mriedem SO
19:38:34 mriedem besides placement and cells v2,
19:38:45 mriedem what's the #3 newsworthy feature delivered in ocata
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.

Earlier   Later