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

Publish additional docker images to the registry #283

Merged
merged 3 commits into from
Apr 16, 2024
Merged

Publish additional docker images to the registry #283

merged 3 commits into from
Apr 16, 2024

Conversation

akoken
Copy link
Contributor

@akoken akoken commented Apr 14, 2024

Currently, the repository contains Dockerfiles for various Linux distributions and nanoserver, but only the default (debian) image is being published to the registry. This pull request addresses this limitation by extending the publication process to include all Docker images for supported Linux distributions and nanoserver.

Due to the lack of support for Windows in buildx, I've split the pipeline into separate ones for Linux and Windows, ensuring that all images are properly built and published. The registry should look like this after changes have been merged.

@badrishc
Copy link
Contributor

Great work! Thanks for the contribution.

@irinasp
Copy link
Contributor

irinasp commented Apr 16, 2024

Thank you for your contribution, it looks good!

@irinasp irinasp merged commit e382d3b into microsoft:main Apr 16, 2024
21 checks passed
@akoken
Copy link
Contributor Author

akoken commented Apr 16, 2024

Hi @irinasp

I think there is a problem with the registry. Some of the container images received a 403 Forbidden error, which is quite interesting. Could you please remove the existing images from the registry and trigger the pipeline again? Before doing so, you can check the permissions from here. Based on what I see in this issue, this situation is very common.

Thanks.

@github-actions github-actions bot locked and limited conversation to collaborators Jun 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants