Skip to content
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

ManagedOSVersionChannel.Spec.Options is schemaless #779

Open
anmazzotti opened this issue Jun 24, 2024 · 1 comment
Open

ManagedOSVersionChannel.Spec.Options is schemaless #779

anmazzotti opened this issue Jun 24, 2024 · 1 comment

Comments

@anmazzotti
Copy link
Contributor

Unclear why this is a Schemaless field.
What's the reason behind it? How am I supposed to use this as a user?

Documentation doesn't inform the user about it, also it's going to be really really hard to do any kind of validation.

This is worth some investigation and definitely a cleanup.

@davidcassany davidcassany moved this to 💡 Backlog in Elemental Aug 7, 2024
@davidcassany
Copy link
Contributor

As an idea, I'd trim out from the metadata all current data and promote it to spec and I'd keep the metadata as a schemaless field that is not associated with the operator functionality in any way. I'd use the metadata field as a place to include free form OS information that is not required for the controller operation in any way. So metadata contents are not part of any contract.

Then I'd use metadata to include changelog information and make explicit this is a free form data subject to change.

We should also ensure this suits in the roadmap as this implies API changes that not backward compatible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 💡 Backlog
Development

No branches or pull requests

2 participants