-
Notifications
You must be signed in to change notification settings - Fork 350
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
Use generated trait #4811
Comments
This issue has been automatically marked as stale due to 90 days of inactivity. |
Mind that #5233 reverts this feature development. Please, see the PR details to understand why this feature cannot be really developed and the alternative to check any value which may have been overridden by the operator. |
With #4764 we have simplified the managment of native builder execution. However we're not using a best practice as the operator alter the configuration
.spec
of builder trait by adding something that will be required at build time. In order to follow best practices, we should instead move the generated configuration into the.status
.Ideally, we should have a
.status.traits
parameter where we copied the.spec.traits
and additionally include any operator generated configuration.The text was updated successfully, but these errors were encountered: