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

Disconnect the Istio Copilot Adapter #2942

Closed
tcdowney opened this issue Aug 25, 2022 · 0 comments · Fixed by #3355
Closed

Disconnect the Istio Copilot Adapter #2942

tcdowney opened this issue Aug 25, 2022 · 0 comments · Fixed by #3355
Labels
networking issues that reference network-oriented features like ASGs or container networks open for contribution unscheduled

Comments

@tcdowney
Copy link
Member

Issue

Copilot / Istio on BOSH support is deprecated and was never deemed suitable for production. Code in Cloud Controller concerning this can likely be removed.

Context

Copilot was a service that converted routing information from Cloud Controller and Diego LRPs into routing configuration that Istio and Envoy could understand. Efforts to bring Istio to CF for VMs through a BOSH release have been paused for over three years now and were formally deprecated in March 2020.

See this deprecation notice:
https://github.com/cloudfoundry-attic/istio-release#deprecated

That means the core code around starting an LRP on Diego can be simplified and no longer pass through the CopilotRunnerDelegator / Copilot Adapter layer and that this code can be removed.

Areas to look into

Some parts such as the Copilot config might need to remain in the schema as optional. I'm not sure if it's backwards compatible to just remove that. Likewise, there is a Copilot Bulk Syncer BOSH job that I'm not sure is part of any supported CF deployment manifest, but extra care should be taken with that. Otherwise, the Adapter and RunnerDelegator layers should be pretty safe to remove.


Notes

@tcdowney tcdowney changed the title Disconnect the Copilot Adapter Disconnect the Istio Copilot Adapter Aug 25, 2022
@tcdowney tcdowney added the networking issues that reference network-oriented features like ASGs or container networks label Aug 25, 2022
@tcdowney tcdowney mentioned this issue Jul 24, 2023
5 tasks
@philippthun philippthun linked a pull request Jul 25, 2023 that will close this issue
5 tasks
tcdowney added a commit that referenced this issue May 2, 2024
- The Istio Service Mesh routing layer is required for weighted routes
  to function, but development on it was discontinued in early 2020.
  See: https://github.com/cloudfoundry-attic/istio-release
- As a consequence, weighted route destinations are no longer supported.
- See this issue for more information about the removal of
Istio support from Cloud Controller: #2942
moleske pushed a commit that referenced this issue May 3, 2024
- The Istio Service Mesh routing layer is required for weighted routes
  to function, but development on it was discontinued in early 2020.
  See: https://github.com/cloudfoundry-attic/istio-release
- As a consequence, weighted route destinations are no longer supported.
- See this issue for more information about the removal of
Istio support from Cloud Controller: #2942
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
networking issues that reference network-oriented features like ASGs or container networks open for contribution unscheduled
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants