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

DRY the validator #2539

Open
mauromorales opened this issue May 7, 2024 · 1 comment
Open

DRY the validator #2539

mauromorales opened this issue May 7, 2024 · 1 comment
Labels
question Further information is requested triage Add this label to issues that should be triaged and prioretized in the next planning call

Comments

@mauromorales
Copy link
Member

There's an ongoing effort to have the validator be the single source of truth for configurations. This of course relates to the implementation to how we apply those configurations on the system. On the initial ticket #778 we decided to limit the scope of the project and one shortcut we decided to have back then was to not have a single source of truth because doing so required quite a big refactoring, so we introduced the validation schemas, which are somewhat on par with the config.Config, config.Install, etc. In order to make it harder to forget to change the latter, we added tests that will fail if the two structures don't have the same fields. The problem is that, we can still introduce new config structures on the implementation part and bypass the tests and validations this way.

Check if it would be possible at this point in time to have a single source of truth, or how much extra work it would take. If we could already implement #2538 then it would force us to always include everything on the implementation and validation but the mechanism could still feel duplicated if it stays as is. So this card would be about further steps to simplify that

@mauromorales mauromorales added the triage Add this label to issues that should be triaged and prioretized in the next planning call label May 7, 2024
@ci-robbot ci-robbot added the question Further information is requested label May 7, 2024
@ci-robbot
Copy link
Collaborator

Hello mauromorales, thank you for bringing up this topic. The issue you mentioned has been labeled with 'question' to indicate that it needs more information or clarification to proceed. In order to help you better, could you please provide a more detailed description of the problem you're experiencing and any relevant steps to reproduce it? Additionally, please mention the versions of the artifacts you are using (e.g., programming language, Kairos version, etc.). This information will be helpful in understanding the issue and determining the next steps. Remember, this is an experimental ai bot, created by @mudler and @jimmykarily to help with triaging Github issues. If you have any further questions, feel free to ask.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested triage Add this label to issues that should be triaged and prioretized in the next planning call
Projects
Status: No status
Development

No branches or pull requests

2 participants