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

[pull] master from grpc:master #575

Merged
merged 7 commits into from
Jan 30, 2025
Merged

[pull] master from grpc:master #575

merged 7 commits into from
Jan 30, 2025

Conversation

pull[bot]
Copy link

@pull pull bot commented Jan 30, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

ctiller and others added 7 commits January 30, 2025 08:47
#38544 inadvertently disabled a ton of testing for various experiments; re-enable.

Closes #38640

COPYBARA_INTEGRATE_REVIEW=#38640 from ctiller:oof c3ac42f
PiperOrigin-RevId: 721404314
…8646)

Closes #38646

COPYBARA_INTEGRATE_REVIEW=#38646 from drfloob:private-grpcpp-public-hdrs 9903584
PiperOrigin-RevId: 721428133
~(also includes #38633)~

Closes #38634

COPYBARA_INTEGRATE_REVIEW=#38634 from ctiller:protof b4a2843
PiperOrigin-RevId: 721461885
If we shutdown an inproc server before we create a call, we go boom.

Closes #38645

COPYBARA_INTEGRATE_REVIEW=#38645 from ctiller:regress f868aa9
PiperOrigin-RevId: 721468790
As per gRFC A88 (grpc/proposal#466).

Closes #38377

COPYBARA_INTEGRATE_REVIEW=#38377 from markdroth:xds_resource_timer_change 3166024
PiperOrigin-RevId: 721474111
We're supposed to support putting integers outside the status code range into the `grpc_status_code` enum, as a future proofing against ever adding a new status code, however unlikely that may be.

Unfortunately, whilst we parse a uint32_t, we didn't take steps to guarantee that the enum was sufficiently wide to store the value.

Compilers we care about have been getting this right in opt builds regardless, but for safety we should get this right going forward.

Closes #38644

COPYBARA_INTEGRATE_REVIEW=#38644 from ctiller:widen 74ee5cd
PiperOrigin-RevId: 721474835
As part of #38544 I decreased shard counts dramatically for some tests... a few will be under-provisioned now. Increasing this one by a bit as it's started flaking.

Closes #38649

COPYBARA_INTEGRATE_REVIEW=#38649 from ctiller:moar-shards 887e84e
PiperOrigin-RevId: 721481761
@pull pull bot added the ⤵️ pull label Jan 30, 2025
@pull pull bot merged commit 1614cd9 into makesoftwaresafe:master Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants