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

Java control plane 0.1.45 #420

Closed
wants to merge 2 commits into from
Closed

Java control plane 0.1.45 #420

wants to merge 2 commits into from

Conversation

kozjan
Copy link
Collaborator

@kozjan kozjan commented Jun 24, 2024

Removed SimpleCache class, which was a copy of envoyproxy's class.
It had custom stuff - shouldSendMissingEndpoints, it's used here and here.

Now the same stuff happens in envoyproxy - there is a allowDefaultEdsUpdate boolean which sets allowDefaultResource to true. Then during creating a response resources are mapped using defaultResource, which does pretty much the same mapping as our custom one but with additional check if resource type is ENDPOINT (compared to this).

@kozjan kozjan closed this Jun 24, 2024
@kozjan kozjan deleted the java-control-plane-0.1.45 branch June 24, 2024 09:26
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.

1 participant