Skip to content

Make 100% test coverage for upstream_impl.cc#302

Merged
RomanDzhabarov merged 3 commits intomasterfrom
coverage_up
Dec 17, 2016
Merged

Make 100% test coverage for upstream_impl.cc#302
RomanDzhabarov merged 3 commits intomasterfrom
coverage_up

Conversation

@RomanDzhabarov
Copy link
Copy Markdown
Member

Cover two important cases with max_host_weight.

@RomanDzhabarov RomanDzhabarov changed the title Make 100% coverage for upstream_impl. Make 100% test coverage for upstream_impl.cc Dec 17, 2016
@RomanDzhabarov RomanDzhabarov merged commit d64e6d9 into master Dec 17, 2016
@mattklein123 mattklein123 deleted the coverage_up branch December 17, 2016 20:45
rshriram pushed a commit to rshriram/envoy that referenced this pull request Oct 30, 2018
Automatic merge from submit-queue.

[DO NOT MERGE] Auto PR to update dependencies of mixerclient

This PR will be merged automatically once checks are successful.
```release-note
none
```
liverbirdkte referenced this pull request in liverbirdkte/envoy Sep 21, 2022
Signed-off-by: LeiZhang <lei.a.zhang@intel.com>
agrawroh pushed a commit to agrawroh/envoy that referenced this pull request Jun 3, 2025
Sidecar ext authz can fail if the instance is paused or readable secondaries has been disabled because we will remove the hostname from mercury config and thus ext_authz call will fail.

Update error msg to reflect this.
mathetake pushed a commit that referenced this pull request Mar 3, 2026
**Commit Message**

Add helm install.

**Related Issues/PRs (if applicable)**

Fixes #302

---------

Signed-off-by: Loong <long0dai@foxmail.com>
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