Skip to content

tonic-xds: allow deprecated fetch_update until MSRV >= 1.95 - #2904

Merged
YutaoMa merged 1 commit into
grpc:masterfrom
YutaoMa:yutaoma/tonic-xds-fetch-update-deprecation
Oct 1, 2026
Merged

YutaoMa merged 1 commit into
grpc:masterfrom
YutaoMa:yutaoma/tonic-xds-fetch-update-deprecation

Conversation

@YutaoMa

@YutaoMa YutaoMa commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Motivation

CI blocked because of Rust 1.99 clippy now warns on use of deprecated fetch_upate in favor of try_update. However fetch_upate is only stable since 1.95 which is above the current msrv.

Solution

#[allow(deprecated)] with TODO reminders to delete later when msrv bumps.

Rust 1.99 deprecates `fetch_update` in favor of `try_update`, which is
only stable since 1.95, above our 1.88 MSRV.
@YutaoMa
YutaoMa marked this pull request as ready for review October 1, 2026 21:27

@ejona86 ejona86 left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Seems we should pin the rust or clippy version to avoid this sort of CI-fails-suddenly problem. (Not in this PR; just how to avoid this sort of problem.)

@YutaoMa
YutaoMa merged commit b9f904c into grpc:master Oct 1, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants