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

VU: cannot remove old services, cannot override paths #209

Open
2 of 3 tasks
cmars opened this issue Jun 24, 2022 · 0 comments
Open
2 of 3 tasks

VU: cannot remove old services, cannot override paths #209

cmars opened this issue Jun 24, 2022 · 0 comments

Comments

@cmars
Copy link
Contributor

cmars commented Jun 24, 2022

It's currently not possible to remove a service from VU aggregation. This is a problem if you break one service into two or more and want to aggregate them into a compiled API. The prior monolith will conflict forever with the new services because the storage implementations pull in all the old versions of the old service and it claims all the paths.

It's also not possible to selectively say:

  • Aggregate everything from Old Monolith service except /path1, /path2 and /path3 which we moved to Shiny New Service
  • Aggregate everything from Shiny New Service

It's also not possible to maintain historical continuity if the service scrape URL changes.

In order to be able to use VU in a dynamic services world, we need several things to happen:

  • Stop collating services into the result aggregate specs that have been removed from the scrape list.
  • Stop identifying services by their hostname in storage, but instead use a logical name. Logical name points at a hostname which can then change without disrupting history.
  • Path matching patterns per service to limit what gets merge from each service. We probably need both include and exclude patterns for this...
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 a pull request may close this issue.

1 participant