Skip to content

Exclude csharp from performance test continuous runs.#26458

Merged
paulosjca merged 1 commit intogrpc:masterfrom
paulosjca:csharp
Jun 10, 2021
Merged

Exclude csharp from performance test continuous runs.#26458
paulosjca merged 1 commit intogrpc:masterfrom
paulosjca:csharp

Conversation

@paulosjca
Copy link
Copy Markdown
Contributor

This is a workaround for a breakage introduced by #22964.

This is a workaround for a breakage introduced by grpc#22964.
@paulosjca paulosjca added the release notes: no Indicates if PR should not be in release notes label Jun 10, 2021
@paulosjca paulosjca self-assigned this Jun 10, 2021
@paulosjca paulosjca enabled auto-merge (squash) June 10, 2021 16:32
Copy link
Copy Markdown
Contributor

@jtattermusch jtattermusch left a comment

Choose a reason for hiding this comment

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

LGTM.

sorry for the breakage. I'll come up with a fix tomorrow.

@paulosjca paulosjca merged commit 6744e8f into grpc:master Jun 10, 2021
jtattermusch added a commit to jtattermusch/grpc that referenced this pull request Jun 11, 2021
@paulosjca paulosjca deleted the csharp branch June 12, 2021 00:17
jtattermusch added a commit that referenced this pull request Jun 15, 2021
* Revert "Revert "Upgrade dotnet SDK 3 for C# build (#22964)" (#26462)"

This reverts commit 736d914.

* Revert "Exclude csharp from performance test continuous runs. (#26458)"

This reverts commit 6744e8f.

* switch remaining tests to C# buster image

* remove C#'s global.json (which requires dotnet SDK 3+)

* dotnet SDK3 currently breaks C# package build

* regenerate dockerfiles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants