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

Upload docker image to dev ecr repo #1995

Merged
merged 2 commits into from Jan 12, 2023
Merged

Conversation

rylev
Copy link
Member

@rylev rylev commented Jan 11, 2023

With the creation of the docs-rs-staging environment, we can now deploy master to the environment. We may want to be a bit smarter about this to allow for more targeted testing, but for now this can let me test that the environment works.

@github-actions github-actions bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Jan 11, 2023
Copy link
Member

@jyn514 jyn514 left a comment

Choose a reason for hiding this comment

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

Thanks for the work you've been doing on this!

.github/workflows/docker.yml Outdated Show resolved Hide resolved
.github/workflows/docker.yml Outdated Show resolved Hide resolved
@syphar syphar added S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Jan 11, 2023
@syphar
Copy link
Member

syphar commented Jan 11, 2023

( I don't want to slow down the ECS / docs.rs story I would also merge this and then add back our tests later)

@rylev rylev merged commit 8aa73f8 into rust-lang:master Jan 12, 2023
@rylev rylev deleted the upload-to-ecr-dev branch January 12, 2023 09:42
@github-actions github-actions bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Jan 12, 2023
@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Jan 12, 2023
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.

None yet

4 participants