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

Improvements in v3 for Eureka / ZUUL to reduce the delays between onboarding/instance updates and routing #3621

Open
Shobhajayanna opened this issue Jun 27, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request Priority: Medium size/M V3 New functions slated for V3 major release

Comments

@Shobhajayanna
Copy link
Contributor

There is an implementation in v2 and v1 to improve the communication of services & Eureka client, which is yet to be implemented in v3.

once a service updated registry in the Discovery service
the Eureka caches could be cleaned
the Gateways could be notified (reverting of GatewayNotifier)
once Gateway is notified, a gateway could call applicationEventPublisher.publishEvent(new RefreshRoutesEvent("List of services changed")); to regenerate routing issue

@balhar-jakub balhar-jakub added new New issue that has not been worked on yet V3 New functions slated for V3 major release enhancement New feature or request labels Jul 3, 2024
@balhar-jakub
Copy link
Member

The implementation would need to be different.

@pj892031
Copy link
Contributor

pj892031 commented Jul 3, 2024

This issue was created because of #3617. In the PR was removed document https://github.com/zowe/api-layer/blob/v2.x.x/docs/registry-communication.md. It described how was improved the communication in v2 (cleaning caches, direct notification, and almost immediate change of routing rules). The document describes not just the idea, but also the technology. The document is not valid anymore (GatewayNotifier, ZUUL, and Ribon were removed). Although that is possible to implement a similar approach in v3, just missing pieces should be replaced.

The aim of this issue is to consider benefits and could lead to implementing such a feature...

@EvaJavornicka EvaJavornicka added Priority: Medium size/M and removed new New issue that has not been worked on yet labels Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Priority: Medium size/M V3 New functions slated for V3 major release
Projects
Development

No branches or pull requests

4 participants