-
Notifications
You must be signed in to change notification settings - Fork 5.3k
H/3 stress #55098
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
H/3 stress #55098
Changes from all commits
ef146d3
1d83c9e
67af0a8
6dbf212
23ceced
26aac78
fd22b00
ca9ab3b
4051367
30769dd
84f63a4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| ARG SDK_BASE_IMAGE=mcr.microsoft.com/dotnet/nightly/sdk:5.0-buster-slim | ||
| ARG SDK_BASE_IMAGE=mcr.microsoft.com/dotnet/nightly/sdk:6.0-bullseye-slim | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we need this? Seems like buster is still latest stable...
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No we don't. I didn't realize it's "testing" and not "stable". I just took the newest without checking. I'll revert it.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We'll need the bullseye after all. Seems like buster doesn't have current 6.0 preview SDK... |
||
| FROM $SDK_BASE_IMAGE | ||
|
|
||
| RUN echo "DOTNET_SDK_VERSION="$DOTNET_SDK_VERSION | ||
|
|
@@ -7,6 +7,18 @@ RUN echo "DOTNET_VERSION="$DOTNET_VERSION | |
| WORKDIR /app | ||
| COPY . . | ||
|
|
||
| # Pulling the msquic Debian package from msquic-ci public pipeline and from a hardcoded build. | ||
| # Note that this is a temporary solution until we have properly published Linux packages. | ||
| # Also note that in order to update to a newer msquic build, you have update this link. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does this link already contain changes needed for #55291? If not, can we update it?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes it does contain them. |
||
| ARG MSQUIC_PACKAGE=libmsquic_1.5.0_amd64.deb | ||
| ARG PACKAGES_DIR=LinuxPackages | ||
| RUN wget 'https://dev.azure.com/dnceng/9ee6d478-d288-47f7-aacc-f6e6d082ae6d/_apis/build/builds/1223883/artifacts?artifactName=LinuxPackages&api-version=6.0&%24format=zip' -O "$PACKAGES_DIR".zip | ||
| RUN apt-get update | ||
| RUN apt-get install unzip | ||
| RUN unzip $PACKAGES_DIR.zip | ||
| RUN dpkg -i $PACKAGES_DIR/$MSQUIC_PACKAGE | ||
| RUN rm -rf $PACKAGES_DIR* | ||
|
|
||
| ARG CONFIGURATION=Release | ||
| RUN dotnet build -c $CONFIGURATION | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| <?xml version="1.0" encoding="utf-8"?> | ||
| <configuration> | ||
| <packageSources> | ||
| <!-- Add public nuget feed. --> | ||
| <add key="nuget.org" value="https://api.nuget.org/v3/index.json" /> | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Isn't this default? I don't mind to be explicit but I'm wondering if we need it in case we do not need custom feeds
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not if run from runtime repo, ours explicitly removes nuget.org: https://github.com/dotnet/runtime/blob/main/NuGet.config |
||
| </packageSources> | ||
| </configuration> | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,15 +1,14 @@ | ||
| <Project Sdk="Microsoft.NET.Sdk"> | ||
| <PropertyGroup> | ||
| <OutputType>Exe</OutputType> | ||
| <TargetFramework>net5.0</TargetFramework> | ||
| <TargetFramework>net6.0</TargetFramework> | ||
| <Nullable>enable</Nullable> | ||
| </PropertyGroup> | ||
| <ItemGroup> | ||
| <Compile Include="..\..\..\..\Common\tests\System\IO\Compression\CRC.cs" | ||
| Link="Utils\CRC.cs" /> | ||
| <Compile Include="..\..\..\..\Common\tests\System\IO\Compression\CRC.cs" Link="Utils\CRC.cs" /> | ||
| </ItemGroup> | ||
| <ItemGroup> | ||
| <PackageReference Include="System.CommandLine.Experimental" Version="0.3.0-alpha.19577.1" /> | ||
| <PackageReference Include="System.IO.Pipelines" Version="4.6.0" /> | ||
| <PackageReference Include="System.IO.Pipelines" Version="6.0.0-preview.5.21301.5" /> | ||
| </ItemGroup> | ||
| </Project> |
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.
We'll see, but on Linux we should be now able to run stress tests.