Skip to content

[Test] Dedup vs2019 test in the Windows test#33385

Merged
veblush merged 2 commits intogrpc:masterfrom
veblush:kokoro-windows
Jun 9, 2023
Merged

[Test] Dedup vs2019 test in the Windows test#33385
veblush merged 2 commits intogrpc:masterfrom
veblush:kokoro-windows

Conversation

@veblush
Copy link
Copy Markdown
Contributor

@veblush veblush commented Jun 9, 2023

cmake_ninja_vs2019 and default are using the same cmake_ninja_vs2019 so having two tests are waste so this is removing cmake_ninja_vs2019 leaving default which does cmake_ninja_vs2019.

This change can cut the space consumption by half and with 250GB disc,

  • Pre-test: 267,770,322,944 bytes free
  • Post-test: 134,499,295,232 bytes free

@veblush veblush changed the title Show infos [Test] Dedup vs2019 test in the Windows test Jun 9, 2023
@veblush veblush requested a review from yijiem June 9, 2023 20:51
@veblush veblush marked this pull request as ready for review June 9, 2023 20:51
@veblush veblush enabled auto-merge (squash) June 9, 2023 20:51
@veblush veblush merged commit 6b39495 into grpc:master Jun 9, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Jun 9, 2023
veblush added a commit that referenced this pull request Jun 9, 2023
veblush added a commit that referenced this pull request Jun 9, 2023
mario-vimal pushed a commit to mario-vimal/grpc that referenced this pull request Jun 15, 2023
`cmake_ninja_vs2019` and `default` are using the same
`cmake_ninja_vs2019` so having two tests are waste so this is removing
`cmake_ninja_vs2019` leaving `default` which does `cmake_ninja_vs2019`.

This change can cut the space consumption by half and with 250GB disc, 

- Pre-test: 267,770,322,944 bytes free 
- Post-test: 134,499,295,232 bytes free
mario-vimal pushed a commit to mario-vimal/grpc that referenced this pull request Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test bloat/none imported Specifies if the PR has been imported to the internal repository 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.

2 participants