Skip to content

devops: push Docker image to MCR#299

Merged
aslushnikov merged 5 commits intomicrosoft:masterfrom
mxschmitt:devops/push-image-mcr
Nov 30, 2020
Merged

devops: push Docker image to MCR#299
aslushnikov merged 5 commits intomicrosoft:masterfrom
mxschmitt:devops/push-image-mcr

Conversation

@mxschmitt
Copy link
Copy Markdown
Contributor

Closes #290

DOCKER_PASSWORD environment variable needs to be set.

@mxschmitt mxschmitt force-pushed the devops/push-image-mcr branch from b099d0b to 157ff93 Compare November 14, 2020 13:25
@mxschmitt
Copy link
Copy Markdown
Contributor Author

I don't get it why its failing, maybe someone has an idea?

@mxschmitt
Copy link
Copy Markdown
Contributor Author

@pavelfeldman not sure why its failing, only the video test, hmm.

@@ -0,0 +1,43 @@
name: "devrelease:docker"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Curious, why do we need this one?

Copy link
Copy Markdown
Contributor Author

@mxschmitt mxschmitt Nov 24, 2020

Choose a reason for hiding this comment

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

We can also remove the canary builds. They were useful for testing and Lushnikov and me used them a lot so I think it brings value.

Copy link
Copy Markdown
Contributor

@aslushnikov aslushnikov left a comment

Choose a reason for hiding this comment

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

Marking as "request changes" to not merge accidentally since this the tests are actually failing in the container.

@mxschmitt is looking into this

@mxschmitt mxschmitt force-pushed the devops/push-image-mcr branch from 7f1f9d1 to 0e7327f Compare November 27, 2020 19:34
@aslushnikov aslushnikov merged commit 35bdc3d into microsoft:master Nov 30, 2020
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.

Publish docker image to MCR

3 participants