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.

Sign up for GitHub

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

Jump to bottom

Pixiu Config #242

Merged
merged 63 commits into from
Aug 29, 2021
Merged

Pixiu Config #242

merged 63 commits into from
Aug 29, 2021

Conversation

ztelur
Copy link
Contributor

@ztelur ztelur commented Aug 22, 2021 •
edited
Loading

What this PR does:

  • merge the pr Make filter configurable & Centralized management from mark4z
  • create Plugin, HttpFilter(such as ratelimit), NetworkFilter(such as httpconnectionmanager) and Adapter(such as cloud cloud adapter to control cluster and router ) abstraction, all extension will registered at pluginregistry.go
  • move api_config and dubbo/http client to remote filter and api_config filter which offer special route strategy and dubbo proxy
  • create RouteManager、ClusterManager and common router strategy base on model.Router
  • split server run to init and run two phase
  • add dubbo/http、http/simple into integration test and assert response more strictly

Special notes for your reviewer:

ztelur and others added 30 commits July 10, 2021 17:48
@ztelur
delete old samples/admin
1a2c8ff
@ztelur
recovery
b63e352
@xiaoliu10
Merge branch 'develop' into develop
6f1193f
@xiaoliu10
Merge branch 'develop' into develop
b16f10b
@mark4z
filter manager
be5bfae
@mark4z
filter manager
32cadd7
@mark4z
filter manager
6e3ccfc
@ztelur
Merge branch 'apache:develop' into develop
d1bcc58
@mark4z
Merge branch 'develop' of https://github.com/apache/dubbo-go-pixiu in…
b491863 
…to filter
@mark4z
filter manager
6fddf27
@mark4z
filter manager
6850a0c
@mark4z
acl & ratelimit
05b0460
@mark4z
auth
9eb232e
@mark4z
metirc
52a9804
@mark4z
metirc
63b4823
@ztelur
Merge branch 'apache:develop' into develop
20cc4b5
@ztelur
update
704df39
@ztelur
Merge branch 'config' into filter
4cb543a
@mark4z
all filter refactor succ
586e0a5
@mark4z
ready to load filters
a309a6a
@ztelur
update
a149063
@ztelur
update
0771b88
@ztelur
update
32e767a
@ztelur
update
71b1fd5
@ztelur
filter
0ed2ec0
@ztelur
update
04ec65d
@ztelur
update
d831eef
@ztelur
update
16c6dba
@ztelur
update
648dcb7
@ztelur
Merge branch 'config' of https://github.com/ztelur/dubbo-go-pixiu int…
b31ee68 
…o config
@ztelur ztelur marked this pull request as ready for review August 25, 2021 15:08
AlexStocks
AlexStocks reviewed Aug 26, 2021
View reviewed changes
configs/api_config.yaml Outdated Show resolved Hide resolved
mark4z
mark4z approved these changes Aug 27, 2021
View reviewed changes
Copy link
Member

@mark4z mark4z left a comment

Choose a reason for hiding this comment

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

Nice job

@AlexStocks AlexStocks merged commit 8b17845 into apache:develop Aug 29, 2021
@mark4z mark4z mentioned this pull request Aug 29, 2021
Closed
@ztelur ztelur added this to the v0.4.0 milestone Oct 20, 2021
mark4z pushed a commit that referenced this pull request Nov 7, 2021
@AlexStocks
Merge pull request #242 from ztelur/config
db3167f 
Pixiu Config

Former-commit-id: 8b17845
tydhot pushed a commit to tydhot/dubbo-go-pixiu that referenced this pull request Nov 10, 2021
@AlexStocks
Merge pull request apache#242 from ztelur/config
809221e 
Pixiu Config

Former-commit-id: db3167f [formerly 8b17845]
Former-commit-id: 51873d2
bobtthp pushed a commit to bobtthp/dubbo-go-pixiu that referenced this pull request Dec 12, 2022
@AlexStocks
Merge pull request apache#242 from ztelur/config
3b404d0 
Pixiu Config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexStocks AlexStocks AlexStocks approved these changes

@xiaoliu10 xiaoliu10 xiaoliu10 approved these changes

@mark4z mark4z mark4z approved these changes

@williamfeng323 williamfeng323 Awaiting requested review from williamfeng323

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.4.0
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@ztelur @codecov-commenter @AlexStocks @xiaoliu10 @mark4z