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

azure: fix image reference string #1635

Conversation

mkulke
Copy link
Contributor

@mkulke mkulke commented Dec 20, 2023

The community gallery and image definition are supposed to be stored as vars not as secret, reference is usage in the nightly image build.

AZURE_PODVM_IMAGE_DEF_NAME: "${{ vars.AZURE_PODVM_IMAGE_DEF_NAME }}"
AZURE_PODVM_IMAGE_VERSION: "${{ inputs.image-version }}"
COMMUNITY_GALLERY_PREFIX: "/CommunityGalleries/${{ vars.AZURE_COMMUNITY_GALLERY_NAME }}"

The community gallery is supposed to be stored as var not as a secret,
reference is usage in the nightly image build.

Signed-off-by: Magnus Kulke <magnuskulke@microsoft.com>
@mkulke mkulke added provider/azure Issues related to Azure CAA provider e2e-test labels Dec 20, 2023
@surajssd
Copy link
Member

Are these vars exposed though? If not the next run will fail.

@mkulke
Copy link
Contributor Author

mkulke commented Dec 21, 2023

Are these vars exposed though? If not the next run will fail.

I think so, as the nightly image builds use them atm, and they seem to work there.

@mkulke mkulke merged commit 1551894 into confidential-containers:main Dec 21, 2023
20 checks passed
@mkulke mkulke deleted the mkulke/fix-azure-e2e-communitygallery-var branch December 21, 2023 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e-test provider/azure Issues related to Azure CAA provider
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants