Skip to content

Update to Alpine 3.16 #46563

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

Merged
merged 3 commits into from
Feb 17, 2023
Merged

Update to Alpine 3.16 #46563

merged 3 commits into from
Feb 17, 2023

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Feb 9, 2023

Part of #46555

@wtgodbe wtgodbe requested a review from JamesNK February 9, 2023 22:51
@wtgodbe wtgodbe requested review from a team and dougbu as code owners February 9, 2023 22:51
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Feb 9, 2023
@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 9, 2023

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 10, 2023

Lots of test failures here, all seemingly related to the French culture - tagging individual owners

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 10, 2023

Lots of ModelBinding failures, e.g. Microsoft.AspNetCore.Mvc.ModelBinding.Binders.DecimalModelBinderTest.BindModel_ReturnsModel_IfAttemptedValueIsValid_FrenchTwelvePointFive(destinationType: typeof(decimal))

Assert.Equal() Failure
Expected: 12.5
Actual: 125

@brunolins16 could you take a look?

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 10, 2023

Microsoft.AspNetCore.Html.Test.HtmlContentBuilderExtensionsTest.Builder_AppendFormat_WithDifferentCulture

Assert.Equal() Failure
↓ (pos 13)
Expected: HtmlEncode[[1,21]] in french!
Actual: HtmlEncode[[1.21]] in french!
↑ (pos 13)

Not sure who owns this one - @JamesNK? link

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 10, 2023

2023-02-10T00:06:42.6138111Z Failed Microsoft.AspNetCore.Mvc.Formatters.NewtonsoftJsonInputFormatterTest.ReadAsync_WithReadJsonWithRequestCulture_DeserializesUsingRequestCulture(dateString: "15/05/2019", culture: "fr-FR") [66 ms]
2023-02-10T00:06:42.6138991Z Error Message:
2023-02-10T00:06:42.6139493Z Assert.False() Failure
2023-02-10T00:06:42.6139992Z Expected: False
2023-02-10T00:06:42.6140512Z Actual: True
2023-02-10T00:06:42.6141094Z Stack Trace:
2023-02-10T00:06:42.6141681Z at Microsoft.AspNetCore.Mvc.Formatters.NewtonsoftJsonInputFormatterTest.ReadAsync_WithReadJsonWithRequestCulture_DeserializesUsingRequestCulture(String dateString, String culture) in /_/src/Mvc/Mvc.NewtonsoftJson/test/NewtonsoftJsonInputFormatterTest.cs:line 514

@JamesNK I guess this is you as well - logs

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 10, 2023

The failures may be related to an issue with the docker image - dotnet/dotnet-buildtools-prereqs-docker#798

@brunolins16
Copy link
Member

brunolins16 commented Feb 10, 2023

The failures may be related to an issue with the docker image - dotnet/dotnet-buildtools-prereqs-docker#798

IDK much about but I took a look at the ModelBinding issues and all of them are related to scenarios where we are testing formatting with different cultures (French, Spanish, Russian, Traditional Chinese) and definitely could be related to this.

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 13, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 13, 2023

I believe https://dev.azure.com/dnceng-public/public/_build/results?buildId=169872&view=results will use the updated 3.16 image

@brunolins16
Copy link
Member

I believe https://dev.azure.com/dnceng-public/public/_build/results?buildId=169872&view=results will use the updated 3.16 image

If it is using 3.16, I believe the change did not fix the issue :(.

@brunolins16
Copy link
Member

@wtgodbe we were using 3.14 before, right?

Do you know which customizations we have in mcr.microsoft.com/dotnet-buildtools/prereqs:alpine-3.14-local? I am asking because the 3.16 base image is a plain alpine:3.16

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 13, 2023

I might have read the docker repo wrong, going to try the staging image (I think it hasn't been rolled out yet). If that works I'll wait until the rollout & update this PR to use the official image again

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 13, 2023

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 14, 2023

Ok, that worked. @mthalman what day does the weekly docker image rollout happen?

@wtgodbe wtgodbe added the * NO MERGE * Do not merge this PR as long as this label is present. label Feb 14, 2023
@mthalman
Copy link
Member

Wednesday or Thursday
cc @riarenas

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 16, 2023

@wtgodbe
Copy link
Member Author

wtgodbe commented Feb 17, 2023

@dougbu can you review

@wtgodbe wtgodbe removed the * NO MERGE * Do not merge this PR as long as this label is present. label Feb 17, 2023
@wtgodbe wtgodbe changed the title Update to Helix 3.16 Update to Alpine 3.16 Feb 17, 2023
@wtgodbe wtgodbe merged commit e48fb24 into dotnet:main Feb 17, 2023
@wtgodbe wtgodbe deleted the wtgodbe/Alpine branch February 17, 2023 17:51
@ghost ghost added this to the 8.0-preview2 milestone Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants