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

Create knative-sandbox repo named eventing-kafka-broker #143

Closed
Harwayne opened this issue May 28, 2020 · 9 comments
Closed

Create knative-sandbox repo named eventing-kafka-broker #143

Harwayne opened this issue May 28, 2020 · 9 comments
Assignees

Comments

@Harwayne
Copy link
Contributor

Create knative-sandbox repo named eventing-kafka-broker.

As #125 has not yet been merged, there is no defined process for creating sandbox repos. It was suggested that we create an issue here instead.

The main purpose of the new repo is to experiment with alternative implementations of integrating Kafka into knative. For example, using Java rather than Go. It is expected to be distinct enough from the existing implementation in https://github.com/knative-sandbox/eventing-kafka that it will be much easier to allow the two to evolve independently.

@matzew
Copy link
Member

matzew commented Jun 4, 2020

@evankanderson any chance we can get the repo for our student ?

@evankanderson
Copy link
Member

I can try to take this on this evening, assuming that no one else on the ToC raises an objection.

@slinkydeveloper
Copy link
Contributor

@evankanderson any updates on that one?

@evankanderson
Copy link
Member

Sorry -- was waiting on #130 to finish. I'll create this today.

@evankanderson
Copy link
Member

@slinkydeveloper or others following: you can /lgtm the website PR if you want. I think the Prow jobs need someone from test-infra to /approve

@evankanderson
Copy link
Member

/assign @evankanderson

@evankanderson
Copy link
Member

Thanks for the PRs!

I'd like to wait for knative/test-infra#2222 before starting to use the new repo, but I'll take a look at the peribolos changes.

@evankanderson
Copy link
Member

It looks like this is complete, and we have PRs that have merged.

/close

@knative-prow-robot
Copy link
Contributor

@evankanderson: Closing this issue.

In response to this:

It looks like this is complete, and we have PRs that have merged.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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

No branches or pull requests

5 participants