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

xds/clusterimpl: update UpdateClientConnState to handle updates synchronously #7533

Merged
merged 3 commits into from
Aug 30, 2024

fix: put methods like Close, ResolverError, ExitIdle in CallbackSeria…

5e90f8e
Select commit
Loading
Failed to load commit list.
Merged

xds/clusterimpl: update UpdateClientConnState to handle updates synchronously #7533

fix: put methods like Close, ResolverError, ExitIdle in CallbackSeria…
5e90f8e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Aug 21, 2024 in 0s

90.62% of diff hit (target 82.07%)

View this Pull Request on Codecov

90.62% of diff hit (target 82.07%)

Annotations

Check warning on line 203 in xds/internal/balancer/clusterimpl/clusterimpl.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

xds/internal/balancer/clusterimpl/clusterimpl.go#L203

Added line #L203 was not covered by tests

Check warning on line 215 in xds/internal/balancer/clusterimpl/clusterimpl.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

xds/internal/balancer/clusterimpl/clusterimpl.go#L215

Added line #L215 was not covered by tests

Check warning on line 266 in xds/internal/balancer/clusterimpl/clusterimpl.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

xds/internal/balancer/clusterimpl/clusterimpl.go#L266

Added line #L266 was not covered by tests

Check warning on line 396 in xds/internal/balancer/clusterimpl/clusterimpl.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

xds/internal/balancer/clusterimpl/clusterimpl.go#L396

Added line #L396 was not covered by tests