-
Notifications
You must be signed in to change notification settings - Fork 226
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
Add support for using Oxia as the metadata store for Pulsar and BookKeeper #544
Conversation
next step: disable zk and modify the init script to integrate pulsar with oxia. |
what's done:
|
@merlimat I've made changes accordingly. I can see that Oxia is working, however there are problems.
The problem is that the bookkeeper init job fails. the last error message in logs is:
in conf/bookkeeper.conf there's
Connection works from the pod:
I wonder how the problem can be fixed? /cc @mattisonchao |
Last logs on coordinator:
|
@yuweisung If you'd like to debug issues in GitHub Actions, you can ssh into the running build job VM when you open a PR to your own fork. ssh authentication will happen with your public keys at https://github.com/yuweisung.keys . |
… there are in other components
Attempt to fix io.grpc.StatusRuntimeException: UNIMPLEMENTED: unknown service io.streamnative.oxia.proto.OxiaClient Since found this error in the logs: rpc error: code = Code(110) desc = oxia: namespace not found Failed to add client for shards assignments notifications
This reverts commit 39319b5.
… affinity and securityContext
- org.apache.bookkeeper.stream.storage.impl.cluster.ZkClusterInitializer is the only implementation
This reverts commit 7740a2d.
One gap in Pulsar is the lack of a org.apache.pulsar.packages.management.core.PackagesStorage implementation that works with Oxia. The org.apache.pulsar.packages.management.storage.bookkeeper.BookKeeperPackagesStorage implementation uses the DistributedLog library which requires Zookeeper. error message:
I have disabled function tests for Oxia in CI to make the tests pass. We can merge this PR without functions support for Oxia. |
Feature: add oxia template
Motivation
Adding oxia templates in helmchart.
Modifications
Just add the template with the correct variables.
The oxia component is disabled in default values.yaml.
Verifying this change