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

ospp: Feature/traffic #496

Merged
merged 8 commits into from
Sep 16, 2022
Merged

ospp: Feature/traffic #496

merged 8 commits into from
Sep 16, 2022

Conversation

maxingg
Copy link
Contributor

@maxingg maxingg commented Sep 13, 2022

What this PR does:
Optimize the traffic filter, and make cluster config support both header and weight strategy.

Which issue(s) this PR fixes:

Fixes #402

@codecov-commenter
Copy link

codecov-commenter commented Sep 13, 2022

Codecov Report

Merging #496 (2ece16e) into develop (5301405) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #496      +/-   ##
===========================================
- Coverage    54.97%   54.90%   -0.08%     
===========================================
  Files          662      662              
  Lines        77433    77433              
===========================================
- Hits         42572    42512      -60     
- Misses       31246    31296      +50     
- Partials      3615     3625      +10     
Impacted Files Coverage Δ
security/pkg/credentialfetcher/fetcher.go 40.00% <0.00%> (-20.00%) ⬇️
pkg/kube/inject/watcher.go 48.14% <0.00%> (-8.65%) ⬇️
pkg/kube/controllers/queue.go 55.22% <0.00%> (-7.47%) ⬇️
pkg/istio-agent/xds_proxy_delta.go 48.18% <0.00%> (-7.26%) ⬇️
pkg/test/util/retry/retry.go 57.14% <0.00%> (-3.58%) ⬇️
pkg/istio-agent/xds_proxy.go 59.43% <0.00%> (-3.26%) ⬇️
pilot/pkg/xds/monitoring.go 78.57% <0.00%> (-2.86%) ⬇️
istioctl/cmd/wait.go 72.47% <0.00%> (-2.25%) ⬇️
pilot/pkg/bootstrap/server.go 53.11% <0.00%> (-1.07%) ⬇️
...g/serviceregistry/kube/controller/endpointslice.go 73.44% <0.00%> (-0.69%) ⬇️
... and 3 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

pixiu/pkg/filter/traffic/traffic.go Outdated Show resolved Hide resolved
pixiu/pkg/filter/traffic/traffic.go Outdated Show resolved Hide resolved
pixiu/pkg/filter/traffic/traffic.go Outdated Show resolved Hide resolved
pixiu/pkg/filter/traffic/traffic.go Outdated Show resolved Hide resolved
@mark4z mark4z merged commit a494b71 into apache:develop Sep 16, 2022
@PhilYue PhilYue added this to the 0.6.0 milestone Sep 18, 2022
bobtthp pushed a commit to bobtthp/dubbo-go-pixiu that referenced this pull request Dec 12, 2022
* refactor: TrafficFilter improve

* fix: constant declare with explicit type.

* fix: traffic.go follow by baerrewang's suggestion.

* traffic.go update

* fix: function name change in traffic filter.
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.

GSoC 2022: Pixiu Traffic Distribution Mode Implementation
7 participants