-
Notifications
You must be signed in to change notification settings - Fork 5k
Build HttpStress and SslStress with live-built runtime using current TFM #61689
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
Changes from all commits
bfd6dcd
72c0b81
3b70584
459611d
49338ed
cd243e6
f27e4f0
5e90f14
673b5f6
0af9f7b
5a04963
361f1e5
f87f45b
11d6325
e0a3ef8
83b0835
48a9602
280a3df
ab4072f
6557474
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -57,6 +57,7 @@ jobs: | |
export HTTPSTRESS_CLIENT_ARGS="$HTTPSTRESS_CLIENT_ARGS -http 3.0" | ||
export HTTPSTRESS_SERVER_ARGS="$HTTPSTRESS_SERVER_ARGS -http 3.0" | ||
docker-compose up --abort-on-container-exit --no-color | ||
timeoutInMinutes: 35 # In case the HTTP/3.0 run hangs, we timeout shortly after the expected 30 minute run | ||
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. I'm adding this because I experienced mysterious HTTP/3.0 hangs on some runs: 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 know what happened? I see only blank screen (no output from the run)? Either way, we shouldn't hold this PR on it and investigate it separately. 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. It stopped logging after this line, then that log disappeared for some reason after the whole job becoming cancelled. Unfortunately I wasn't able to reproduce this behavior locally. I would be surprised if this is caused by my changes, we haven't seen working stress runs on the CI for a while. |
||
displayName: Run HttpStress - HTTP 3.0 | ||
condition: and(eq(variables['buildRuntime.succeeded'], 'true'), eq(variables['buildStress.succeeded'], 'true')) | ||
|
||
|
@@ -149,4 +150,4 @@ jobs: | |
- powershell: | | ||
Set-NetFirewallProfile -Profile Domain, Public, Private -Enabled True | ||
name: enableFirewall | ||
displayName: Enable Firewall | ||
displayName: Enable Firewall |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,17 @@ | ||
<Project/> | ||
<Project> | ||
<PropertyGroup> | ||
<PackageRid>linux-x64</PackageRid> | ||
<PackageRid Condition="$([MSBuild]::IsOSPlatform('WINDOWS'))">win-x64</PackageRid> | ||
|
||
<!-- Stress projects have their own global.json, the directory above that also has it is the repository root. --> | ||
<RepositoryRoot>$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory)../, global.json))/</RepositoryRoot> | ||
|
||
<TargetingPacksTargetsLocation Condition="'$(TargetingPacksTargetsLocation)' == ''">$(RepositoryRoot)eng/targetingpacks.targets</TargetingPacksTargetsLocation> | ||
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. I will not pretend I understand all this targeting stuff, but I'm happy someone from our team does 🤣 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. I would be also happy if someone explained the terminology properly here. From what I understand, a "targeting pack" allows you to target a specific runtime / tfm in your build, in this case the one you live-build locally. |
||
<ProductVersion>7.0.0</ProductVersion> | ||
<NetCoreAppCurrent>net7.0</NetCoreAppCurrent> | ||
<NetCoreAppCurrentVersion>7.0</NetCoreAppCurrentVersion> | ||
<MicrosoftNetCoreAppFrameworkName>Microsoft.NETCore.App</MicrosoftNetCoreAppFrameworkName> | ||
<MicrosoftNetCoreAppRefPackDir Condition="'$(MicrosoftNetCoreAppRefPackDir)' == ''" >$(RepositoryRoot)artifacts/bin/microsoft.netcore.app.ref/</MicrosoftNetCoreAppRefPackDir> | ||
<MicrosoftNetCoreAppRuntimePackDir Condition="'$(MicrosoftNetCoreAppRuntimePackDir)' == ''">$(RepositoryRoot)artifacts/bin/microsoft.netcore.app.runtime.$(PackageRid)/$(Configuration)/</MicrosoftNetCoreAppRuntimePackDir> | ||
</PropertyGroup> | ||
</Project> |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,11 @@ | ||
<Project/> | ||
<Project> | ||
<Import Project="$(TargetingPacksTargetsLocation)" /> | ||
|
||
<PropertyGroup> | ||
<!-- | ||
Define this here because the SDK resets it | ||
unconditionally in Microsoft.NETCoreSdk.BundledVersions.props. | ||
--> | ||
<NETCoreAppMaximumVersion>7.0</NETCoreAppMaximumVersion> | ||
</PropertyGroup> | ||
</Project> |
Uh oh!
There was an error while loading. Please reload this page.