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

Switch all VS2017 builds to VS2019 #32662

Merged
merged 2 commits into from
Mar 23, 2023
Merged

Conversation

jtattermusch
Copy link
Contributor

@jtattermusch jtattermusch commented Mar 20, 2023

(part of removing support for VS2017)

Also see #32649
Also see #32615

The switch to grpc-win2019 windows workers has already happened: (cl/517400022).

Once this PR lands, I'll backport to 1.53.x branch as well (since that release removes the VS2017 support).

@jtattermusch
Copy link
Contributor Author

With the upgraded version of win2019 image from cl/518798103, all tests are passing now.

@jtattermusch jtattermusch marked this pull request as ready for review March 23, 2023 12:13
@jtattermusch jtattermusch merged commit 43c3d1f into grpc:master Mar 23, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Mar 23, 2023
jtattermusch added a commit to jtattermusch/grpc that referenced this pull request Mar 24, 2023
(part of removing support for VS2017)

Also see grpc#32649
Also see grpc#32615

The switch to grpc-win2019 windows workers has already happened:
(cl/517400022).

Once this PR lands, I'll backport to 1.53.x branch as well (since that
release removes the VS2017 support).
jtattermusch added a commit that referenced this pull request Mar 24, 2023
Backports #32678,
#32669,
#32668,
#32662

After this, all windows builds on 1.53.x should be using VS2019 and they
should be stable.
@gnossen
Copy link
Contributor

gnossen commented Mar 24, 2023

This PR seems to have broken the Windows portability test.

@jtattermusch
Copy link
Contributor Author

This PR seems to have broken the Windows portability test.

Good catch, I'm working on a fix.

jtattermusch added a commit that referenced this pull request Mar 31, 2023
…2764)

Tentative fix for b/275694647. Also see
#32662

Adhoc windows/grpc_portability run:
http://sponge/bb75e14c-fc77-45bb-89bd-4c4685746490

Once merged, I'll backport to 1.54.x and 1.53.x (these have switched to
VS2019)
jtattermusch added a commit to jtattermusch/grpc that referenced this pull request Apr 5, 2023
…pc#32764)

Tentative fix for b/275694647. Also see
grpc#32662

Adhoc windows/grpc_portability run:
http://sponge/bb75e14c-fc77-45bb-89bd-4c4685746490

Once merged, I'll backport to 1.54.x and 1.53.x (these have switched to
VS2019)
jtattermusch added a commit to jtattermusch/grpc that referenced this pull request Apr 5, 2023
…pc#32764)

Tentative fix for b/275694647. Also see
grpc#32662

Adhoc windows/grpc_portability run:
http://sponge/bb75e14c-fc77-45bb-89bd-4c4685746490

Once merged, I'll backport to 1.54.x and 1.53.x (these have switched to
VS2019)
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
(part of removing support for VS2017)

Also see grpc#32649
Also see grpc#32615

The switch to grpc-win2019 windows workers has already happened:
(cl/517400022).

Once this PR lands, I'll backport to 1.53.x branch as well (since that
release removes the VS2017 support).
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
…pc#32764)

Tentative fix for b/275694647. Also see
grpc#32662

Adhoc windows/grpc_portability run:
http://sponge/bb75e14c-fc77-45bb-89bd-4c4685746490

Once merged, I'll backport to 1.54.x and 1.53.x (these have switched to
VS2019)
wanlin31 pushed a commit that referenced this pull request May 18, 2023
(part of removing support for VS2017)

Also see #32649
Also see #32615

The switch to grpc-win2019 windows workers has already happened:
(cl/517400022).

Once this PR lands, I'll backport to 1.53.x branch as well (since that
release removes the VS2017 support).
wanlin31 pushed a commit that referenced this pull request May 18, 2023
…2764)

Tentative fix for b/275694647. Also see
#32662

Adhoc windows/grpc_portability run:
http://sponge/bb75e14c-fc77-45bb-89bd-4c4685746490

Once merged, I'll backport to 1.54.x and 1.53.x (these have switched to
VS2019)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bloat/none imported Specifies if the PR has been imported to the internal repository lang/Python per-call-memory/neutral per-channel-memory/neutral 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

4 participants