Legacy syslog bindings code cleanup#438
Conversation
With the introduction of mTLS for Syslog Drains we have changed the internal structure of the bindings. This PR cleans up the code of the legacy syslog bindings and removes the http calls to the old Syslog Binding Cache API endpoint. From now on, only the `/v2/binding` and `/v2/aggregate-bindings` API endpoints of the Syslog Binding Cache will be used.
mkocher
left a comment
There was a problem hiding this comment.
Great to see things getting cleaned up. We have some environments that have older syslog-binding-caches that do not speak /v2. We'd like to merge this and then cut a v8 line of loggregator-agent-release.
We'd like to wait a week or two to cut the v8 line so that we can double check that we're not going to pull in a v8 release into places where it shouldn't go. We're also thinking that we'd like to move otel-collector out to its own release, which would also be a breaking change. Are you OK with waiting a little bit and merging this and the otel-collector split in the same release?
|
Hi, @mkocher thank you very much for the review and the approval. I've adjusted only Syslog Agent as that is the only Syslog Binding Cache that I know of. If you have any other internal consumers, take your time to adjust them. There is no rush yo merge this. We still have to wait for the changes in the Cloud Controller to be merged qnd new capi release is cut and added in cf-deployment. Do the needful and we'll create a new release when we're ready. |
Description
With the introduction of mTLS for Syslog Drains we have changed the internal structure of the bindings. In order to enable updating without downtime of the cloud controller, the syslog binding cache and the syslog agent, we have introduced two endpoints for reading the syslog binding in the old and new format. This PR cleans up the code of the legacy syslog bindings and removes the http calls to the old Syslog Binding Cache API endpoint. From now on, only the
/v2/bindingand/v2/aggregate-bindingsAPI endpoints of the Syslog Binding Cache will be used.Type of change
Testing performed?
Checklist:
mainbranch, or relevant version branchThis PR should be merged after merging the changes introduced with the this PR in the Cloud Controller