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

Provisioner: Add kbs provisioning steps in CI #1684

Conversation

kartikjoshi21
Copy link
Contributor

Fixes: #1676

Comment on lines +23 to +33
kbs-image-id:
type: string
description: prebuild kbs image
kbs-image-tag:
type: string
description: prebuild kbs image tag
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't we read all this information from versions.yaml ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes we are reading it from version.yaml and we are also providing an option to give it in CI.

@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch 13 times, most recently from dd62057 to 8b87faf Compare February 8, 2024 11:58
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch 4 times, most recently from 39fec4f to 21f609f Compare February 8, 2024 19:19
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from 21f609f to d965550 Compare February 29, 2024 12:15
@kartikjoshi21 kartikjoshi21 marked this pull request as ready for review February 29, 2024 12:16
@kartikjoshi21 kartikjoshi21 marked this pull request as draft February 29, 2024 12:17
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from d965550 to bf542f0 Compare February 29, 2024 12:18
@kartikjoshi21 kartikjoshi21 marked this pull request as ready for review February 29, 2024 12:18
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from bf542f0 to f7a4cd9 Compare March 5, 2024 06:22
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from f7a4cd9 to bd2fd38 Compare March 11, 2024 17:34
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from bd2fd38 to de18b08 Compare March 27, 2024 10:16
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from de18b08 to 2a1582f Compare April 10, 2024 10:07
.github/workflows/azure-e2e-test.yml Outdated Show resolved Hide resolved
.github/workflows/azure-e2e-test.yml Outdated Show resolved Hide resolved
.github/workflows/azure-e2e-test.yml Show resolved Hide resolved
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from 68f50be to c246e59 Compare April 23, 2024 11:18
Fixes: confidential-containers#1676
Signed-off-by: Kartik Joshi <kartikjoshi@microsoft.com>
@kartikjoshi21 kartikjoshi21 force-pushed the kartikjoshi21/kbs-ci-update branch from c246e59 to b672ac0 Compare April 24, 2024 08:29
Copy link

@huoqifeng huoqifeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! @kartikjoshi21

@surajssd surajssd merged commit 1edddea into confidential-containers:main Apr 26, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update CI workflow to deploy kbs alongside CAA
4 participants