-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Check "enforcecompatibility" capability - does it make sense? #259
Comments
The spec says:
So, I think the concern about the state of the system becoming invalid due to changing this capability is addressed, no? @lailabougria thoughts? Also, right now this capability is a registry-global flag. Are we still ok with that? |
In essence it does but something still feels off. When I look at the other capabilities, this seems to be one that has very different kinds of effect on the registry as it requires a full revalidation where others are more "flag"-type settings that don't affect the content of the registry (although maybe specversion would have a similar effect?). EiIt appears like a different beast compared to the other settings. That also makes me wonder: would it make sense to have a different compatibility strategy per resource, but the resources still registered as part of the same registry? I think that's possible, and in that case it would make more sense on a resource level, which also alleviates the point about this being a different type of setting with very different effects on the registry. |
Let's discuss this on tomorrow's call - I do see your point (especially the "just a flag" vs "re-validation" aspect). |
Proposal:
Open questions:
|
2/19 - @lailabougria will think about how to spec/implement a compat check when there's no mechanism in the spec to know the order of the versions (when sticky is supported). Might adopt the proposal as "good enough" for v1 but will wait until next week to decide. |
No description provided.
The text was updated successfully, but these errors were encountered: