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
deployment: Add stable overlay for kata-deploy.yaml #8509
Conversation
6ca24d9
to
5d8f6a7
Compare
9a4ebad
to
209bbd6
Compare
209bbd6
to
ec11c3b
Compare
/ok-to-test |
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks @zvonkok!
/test-power |
tools/packaging/kata-deploy/kata-deploy/overlays/stable/env_stable_conf.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The idea looks great @zvonkok. What are the plans to fully get rid of kata-deploy-stable.yaml, which is still mentioned in the documentation and used by the release script ?
Fixes: kata-containers#8508 Create a stable overlay for kata-deploy.yaml so we do not have to maintain two files, only one. Single source for both. This is also preparation for the helm-overlay Signed-off-by: Zvonko Kaiser <zkaiser@nvidia.com>
ec11c3b
to
1638055
Compare
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks @zvonkok !
Fixes: #8508