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

chore: Add console app to upload Docker images to ECR #791

Merged
merged 2 commits into from
Oct 4, 2023

Conversation

96malhar
Copy link
Contributor

Issue #, if available:
DOTNET-7190

Description of changes:
This PR adds a console application that does the following:

  1. Takes the project path to a test console application and a web application.
  2. Generate a Dockerfile via the DockerEngine
  3. Uses the PushContainerImageEcr recipe to push the Docker image to ECR.

This PR also adds a new GitHub action that will invoke this console app.

Note: The IAM role assumed by the GitHub action will be defined in a separate internal CR.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@96malhar 96malhar force-pushed the kmalhar/docker-image-upload branch from 0747ac5 to d3a8d0a Compare September 27, 2023 06:48
@96malhar 96malhar force-pushed the kmalhar/docker-image-upload branch from d3a8d0a to 67e5188 Compare September 27, 2023 06:56
src/AWS.Deploy.DockerImageUploader/App.cs Outdated Show resolved Hide resolved
src/AWS.Deploy.DockerImageUploader/Program.cs Outdated Show resolved Hide resolved
.github/workflows/UploadDockerImage.yml Show resolved Hide resolved
@96malhar 96malhar changed the base branch from feat/docker-image-upload to main September 29, 2023 17:11
@philasmar philasmar merged commit c9fc55f into main Oct 4, 2023
7 checks passed
@philasmar philasmar deleted the kmalhar/docker-image-upload branch October 4, 2023 18:29
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.

4 participants