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

internal/dag: apply TCPProxy service weights #4169

Merged
merged 3 commits into from
Nov 11, 2021

Conversation

skriss
Copy link
Member

@skriss skriss commented Nov 10, 2021

Fixes a bug where HTTPProxy TCPProxy service
weights were not being applied.

Closes #4158.

Signed-off-by: Steve Kriss krisss@vmware.com

Fixes a bug where HTTPProxy TCPProxy service
weights were not being applied.

Closes projectcontour#4158.

Signed-off-by: Steve Kriss <krisss@vmware.com>
@skriss skriss requested a review from a team as a code owner November 10, 2021 16:18
@skriss skriss requested review from tsaarni and youngnick and removed request for a team November 10, 2021 16:18
@skriss skriss added the release-note/small A small change that needs one line of explanation in the release notes. label Nov 10, 2021
Signed-off-by: Steve Kriss <krisss@vmware.com>
@skriss skriss added release-note/minor A minor change that needs about a paragraph of explanation in the release notes. and removed release-note/small A small change that needs one line of explanation in the release notes. labels Nov 10, 2021
@codecov
Copy link

codecov bot commented Nov 10, 2021

Codecov Report

Merging #4169 (43905de) into main (cb3ae35) will increase coverage by 0.05%.
The diff coverage is 100.00%.

❗ Current head 43905de differs from pull request most recent head 37ac6bd. Consider uploading reports for the commit 37ac6bd to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4169      +/-   ##
==========================================
+ Coverage   73.71%   73.76%   +0.05%     
==========================================
  Files         114      114              
  Lines        9867     9887      +20     
==========================================
+ Hits         7273     7293      +20     
  Misses       2440     2440              
  Partials      154      154              
Impacted Files Coverage Δ
internal/dag/httpproxy_processor.go 92.73% <100.00%> (+0.01%) ⬆️
internal/featuretests/v3/envoy.go 100.00% <100.00%> (ø)

Signed-off-by: Steve Kriss <krisss@vmware.com>
@skriss skriss merged commit 32235f7 into projectcontour:main Nov 11, 2021
skriss added a commit to skriss/contour that referenced this pull request Nov 11, 2021
Fixes some logical merge conflicts in
the tests from projectcontour#4169.

Signed-off-by: Steve Kriss <krisss@vmware.com>
skriss added a commit that referenced this pull request Nov 11, 2021
Fixes some logical merge conflicts in
the tests from #4169.

Signed-off-by: Steve Kriss <krisss@vmware.com>
@skriss skriss deleted the fix-4158 branch November 11, 2021 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/minor A minor change that needs about a paragraph of explanation in the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weight-based tcp load balancing not work
3 participants