Skip to content
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

Reintroduce #22964 with extra fixes #26465

Merged
merged 6 commits into from
Jun 15, 2021

Conversation

jtattermusch
Copy link
Contributor

Reintroduces #22964 that was reverted by #26462.

Also reverts #26458.

The global.json file (that's being deleted in this PR) can be reintroduced once the C# base image in OSS benchmarks is updated to also contain dotnet SDK 3. (https://github.com/grpc/test-infra/blob/master/containers/pre_built_workers/csharp/Dockerfile)

@jtattermusch jtattermusch added release notes: yes Indicates if PR needs to be in release notes release notes: no Indicates if PR should not be in release notes lang/C# and removed release notes: yes Indicates if PR needs to be in release notes labels Jun 11, 2021
@jtattermusch
Copy link
Contributor Author

CC @paulosjca @gnossen @veblush

@jtattermusch
Copy link
Contributor Author

@jtattermusch
Copy link
Contributor Author

This needs a bit more work - as is the build_package job is still broken:
https://fusion2.corp.google.com/invocations/3389afca-6529-48eb-ad7b-901b1445577c/targets/github%2Fgrpc;config=default/tests

jtattermusch added a commit that referenced this pull request Jun 15, 2021
* 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
@jtattermusch
Copy link
Contributor Author

The windows artifact build issue is currently also failing on upstream/master. https://fusion2.corp.google.com/invocations/9eca03d7-1191-41d8-943f-8ef62b15f0b7/targets/github%2Fgrpc;config=default/tests so that problem is unrelated on this PR.
I'm going to merge this PR.

@jtattermusch jtattermusch merged commit 288fe13 into grpc:master Jun 15, 2021
apolcyn added a commit that referenced this pull request Jun 17, 2021
* Bump version to v1.38.0-pre1 (#26208)

* Bump version to v1.38.0-pre1

* Regenerate projects

* [Backport][v1.38.x] Add EventEngine's port.h to the podspec (#26222)

Backport of #26220 to v1.38.x. This is the only backport, since the EventEngine code was introduced after v1.37 was cut.

This file is needed for the Objective-C cocoapod build.

* fix xds resolver to add XdsClient to channel args even on errors (#26282) (#26288)

Co-authored-by: Mark D. Roth <roth@google.com>

* Backport config update timeout change to v1.38.x (#26276)

* Enable Traffic Director time tracer (#26111)

* Just update the timeout (#26223)

* Increase xds job timeouts (#26187)

* Revert grpc_xds_k8s job timeouts back to 120 mins (#26219)

* Revert grpc_xds_k8s_python timeout to 120mins (#26221)

Seems to have been inadvertently increased in #26187

Co-authored-by: Eric Gribkoff <ericgribkoff@google.com>
Co-authored-by: Yash Tibrewal <yashkt@google.com>

* Bump v1.38.x to v1.38.0  (#26317)

* Bump version to v1.38.0

* Regenerate projects

* Provide custom sponge configs to exclude manual runs from testgrid (#26283) (#26313)

* Provide custom sponge configs

* Correct the script

* Allow test scripts to produce artifacts through Docker

* Overcome the direcotry clone in our docker script

* Expose TESTGRID_EXCLUDE and GIT_ORIGIN_URL through docker run

* Make sanity tests happy

* Revert "Provide custom sponge configs to exclude manual runs from testgrid (#26283) (#26313)" (#26324)

This reverts commit 54dc182.

* better way of building protoc aarch64 artifacts (#26423)

* Backport #26430 and #26435 to v1.38.x (#26436)

* workaround #26279 at the expense of binary compatibility

* correctly tag aarch64 wheels as manylinux_2_24

* remove unnecessary comments left by #26430

* Backport #22964 into v1.38.x (#26448)

* upgrade dotnet SDK 3 for C# build

* add C# global.json

* install dotnet SDK 3 on macos

* upgrade C# docker images to debian buster

* regenerate dockerfiles

* Backport fixes from #26465 to v1.38.x (#26466)

* 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

Co-authored-by: Yash Tibrewal <yashkt@google.com>
Co-authored-by: AJ Heller <hork@google.com>
Co-authored-by: donnadionne <donnadionne@google.com>
Co-authored-by: Mark D. Roth <roth@google.com>
Co-authored-by: Lidi Zheng <lidiz@google.com>
Co-authored-by: Eric Gribkoff <ericgribkoff@google.com>
Co-authored-by: Jan Tattermusch <jtattermusch@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/C# priority/P0/RELEASE BLOCKER 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.

None yet

2 participants