Skip to content
This repository has been archived by the owner on Feb 11, 2025. It is now read-only.

feat: push docker image #352

Merged
merged 1 commit into from
Oct 27, 2022
Merged

Conversation

FrankYang0529
Copy link
Contributor

@FrankYang0529 FrankYang0529 commented Oct 26, 2022

@FrankYang0529
Copy link
Contributor Author

@microsoft-github-policy-service agree [company="default"]

@FrankYang0529
Copy link
Contributor Author

@microsoft-github-policy-service agree

@FrankYang0529 FrankYang0529 force-pushed the add-release-docker-image branch from cd3ca71 to 136818f Compare October 26, 2022 15:24
Copy link
Member

@vdice vdice left a comment

Choose a reason for hiding this comment

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

Defer to the code owners for final decision, but I'm wondering if we want to push to this repo's GitHub container registry instead (ghcr.io/deislabs/bindle)? We should be able to use the default $GITHUB_TOKEN to push images.

Signed-off-by: Frank Yang <[email protected]>
@FrankYang0529 FrankYang0529 force-pushed the add-release-docker-image branch from 136818f to 91d119e Compare October 27, 2022 13:04
@FrankYang0529
Copy link
Contributor Author

Defer to the code owners for final decision, but I'm wondering if we want to push to this repo's GitHub container registry instead (ghcr.io/deislabs/bindle)? We should be able to use the default $GITHUB_TOKEN to push images.

Yeah, I think it's good to use the default $GITHUB_TOKEN and we don't need to maintain docker credentials. Thanks. Updated it!

Copy link
Member

@vdice vdice left a comment

Choose a reason for hiding this comment

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

Nice, I appreciate how this enables forks to test/run this same workflow as well 👍

@vdice vdice merged commit 218e4f2 into deislabs:main Oct 27, 2022
@FrankYang0529 FrankYang0529 deleted the add-release-docker-image branch October 28, 2022 00:49
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.

2 participants