-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Update Dockerfiles for 7.0.200 and remove PublishSingleFile
#4534
Conversation
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only applies the change to the published aspnet sample. Anyone building any of the other sample slim Dockerfiles will run into the same issue this is intended to address. Shouldn't it be applied across the board for all slim Dockerfiles for both aspnetapp
and dotnetapp
?
That's a good point. I can do that. |
PublishSingleFile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update this line to fix the failing test:
if (imageData.DockerfileSuffix == "windowsservercore-iis-x64") |
samples/README.md
Outdated
* [Push container Images to Azure Container Registry](push-image-to-acr.md) | ||
* [Push container Images to Docker Hub](push-image-to-dockerhub.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: lowercase images
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the simplification.
{ | ||
if (imageData.DockerfileSuffix == "windowsservercore-iis-x64") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be deleted. Rather, it should be updated to reflect the new filenames (i.e. windowsservercore-iis
).
Co-authored-by: Matt Thalman <mthalman@microsoft.com>
I took the opportunity to clean up our Dockerfiles, including applying the
PublishRelease
feature (new in 7.0.200).Replaces (most of): #4441
For: #4271