-
Notifications
You must be signed in to change notification settings - Fork 5.3k
config: full delta xDS (including ADS) support #7293
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
Merged
Merged
Changes from all commits
Commits
Show all changes
113 commits
Select commit
Hold shift + click to select a range
6702453
WIP snapshot for motivating the various refactor PRs it entails
fredlas 87d4004
resolve merge conflict
fredlas f290ae7
quick snapshot so i can merge the detemplatization PR
fredlas 30926f8
resolve merge conflict
fredlas d483936
working for non-agg delta, agg delta should in theory work too
fredlas 860c03c
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 1df7db5
mostly working snapshot
fredlas 33b0103
aahhhhhh have to merge to get bazel working again
fredlas 916a03d
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas bc7e515
CDS integration test working again
fredlas 164ac09
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 0f7bcdb
CDS integration passes with 6729 merged
fredlas 62f42dc
lol polymorphism
fredlas 0806842
fix unit tests
fredlas 279355b
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas fd8b1be
small XdsGrpcContext comment change
fredlas d7d5b14
quick commit of beginnings of subclass stuff before merge
fredlas 6768c5c
resolve merge conflict
fredlas 67ccc26
tests partially passing
fredlas 638984e
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas d24744c
revert some test changes
fredlas 804459c
revert establishNewStream idempotency
fredlas c77d6b9
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 83dfc89
DISALLOW_COPY_AND_ASSIGN saves lives
fredlas 714d744
resolve conflict
fredlas 0b96320
snapshot
fredlas 0f8a8f1
documentation
fredlas 399f815
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas cd5134e
resolve conflict
fredlas d04b5da
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 8c4e94b
is_delta plumbing stuff to make delta better fit with ADS
fredlas e902a13
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 8d219e7
add WatchMap initial untested implementation
fredlas f133252
resolve conflict
fredlas b94c2ee
merge cleaned up WatchMap
fredlas 6624140
update to make a bit more use of WatchMap, not yet compiling
fredlas bd24aee
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 4e174c1
config: move SubscriptionCallbacks to ctor, pointers become refs
fredlas beb7a6b
Merge remote-tracking branch 'upstream/master' into OWN_callbacks
fredlas 1bcbdf6
starting to move stats to DeltaSubscriptionImpl
fredlas c56caf6
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas de02e59
whoops missed scoped RDS
fredlas c9dfcf7
merge ctor SubscriptionCallbacks and consolidate into SubscriptionStuff
fredlas 0ed44f1
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas ccd8311
fix addWatch logic with cute recursion
fredlas d4a81b1
merge conflict and get tests compiling again with is_delta arg
fredlas 5e551a5
align GrpcDeltaXdsContext::start usage with GrpcMuxImpl, most tests pass
fredlas 347a2ab
resolve conflict
fredlas 81864b9
lol resolve another conflict, but for a good cause
fredlas 02dff3e
fix subscription_impl tests, non-delta ones mocked out the code that …
fredlas 8a4f504
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas c0c3d98
allow pausing nonexistent subs, also pick up WatchMap wildcard
fredlas 34f439b
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 2a6beda
merge the switch of WatchMap from token to pointer
fredlas 6d1b5f7
integrate detokenized WatchMap, had to introduce WatchMap interface
fredlas a4a5094
resolve conflict
fredlas 6f5b052
yay delta ADS works and all tests pass
fredlas fe2f4e3
merge conflict, protect another couple maybe null adsMux pauses
fredlas b3c98e7
rename GrpcDeltaXdsContext to NewGrpcMuxImpl
fredlas 7bee783
add to version_history
fredlas e1743c8
appease spellcheck
fredlas 06cf88d
remove not-yet-used SubscriptionState interface
fredlas f54c4bc
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 540110c
add to arch_overview and config overview docs
fredlas 9ea226e
add link to xds_protocol.rst
fredlas eee089a
move arguments
fredlas 72c99d1
merge conflicts
fredlas 1907e62
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 7c298bd
move release note to 1.12
fredlas 6f465a8
clean up WatchMap, DeltaSubscriptionImpl becomes more simply RAIIy
fredlas 3a79955
merge conflict
fredlas 019c911
another merge conflict
fredlas c3628c4
add watch_map_test
fredlas 2ac7d0b
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 512f4f9
RELEASE_ASSERT for impossible broken case
fredlas 022bf3a
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 4f50d89
fix test failed by merge conflict
fredlas 861e434
fix CI complaints
fredlas 1460c15
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 23291a4
merge conflict
fredlas f21baf9
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 105f7be
another merge conflict
fredlas f37479d
merge conflict
fredlas 770f48d
merge conflict, scoped_rds_test still broken by 7451
fredlas c70e7b0
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas a972cb0
fix scoped_rds_test
fredlas acb6bc1
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 0ca1b0a
snapshot
fredlas 0093de5
merge conflict
fredlas 933fea5
fix breakage, protect another adsMux pause call against null
fredlas 102fec8
merge conflict
fredlas e45ae53
fix Cleanup i mistakenly brought inside an if, thanks very very much …
fredlas f0a6810
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 5b7489b
snapshot
fredlas 280e842
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas a7a59ca
fix typo in docs, deta to delta
fredlas c1de3f7
clang_tidy
fredlas c873f6a
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas f0e7838
merge master
fredlas 3d14b16
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 4226627
a little cleanup in new_grpc_mux_impl
fredlas aba71b0
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 60dfeab
fix merge drift revealed by fix of CONSTRUCT_ON_FIRST_USE
fredlas 1ac8fb4
merge conflict with 8152
fredlas dc2fc7c
the pedantic spellchecker has decided that unpaused isnt a word
fredlas a12d914
disable more instances of set_node_on_first_message_only
fredlas 82b08e9
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas dcf287c
rearrange update_attempt inc, assert on nonexistent sub, add nonexist…
fredlas d32b4e0
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 590e800
a couple of clang_tidy things
fredlas 9ab78de
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas 6be011e
correct and simplify pausable_ack_queue RELEASE_ASSERTs
fredlas aace5dc
Merge remote-tracking branch 'upstream/master' into ADS_incremental
fredlas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.