Remove broker related code, test, and build files#6890
Remove broker related code, test, and build files#6890rshriram merged 1 commit intoistio:release-1.0from
Conversation
|
So there's good news and bad news. 👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there. 😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request. Note to project maintainer: This is a terminal state, meaning the |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ayj Assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
d7a932a to
2395c07
Compare
|
CLAs look good, thanks! |
rshriram
left a comment
There was a problem hiding this comment.
Please remove vendor as well
The broker is not under active development at this time. Most of the
code is configuration related which will need to be rewritten from
scratch when we switch over to Galley/MCP. Furthermore, the CRD tests
are adding unnecessary load on the k8s api-server and causing test
flakes in some scenarios.
Broker APIs are removing in istio/api#573 (pending). This PR can be rebased to remove the vendor'd broker APIs once that PR is merged.