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

added support for inter-cluster tls and changing the inter-cluster port numbers #223

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

NathanSavageKaimai
Copy link
Contributor

This PR adds 2 new helper variables for the p2p protocol and port.
These are then used when templating initialize.sh in /charts/qdrant/templates/configmap.yaml
Fixes #222

Any advice or feedback would be very appreciated as I'm new to open source contributing. :)

@NathanSavageKaimai
Copy link
Contributor Author

also fixes #216

@kinoute
Copy link

kinoute commented Aug 26, 2024

Encountered this problem as well. Using your fix now until your PR is merged, thanks!

@NathanSavageKaimai
Copy link
Contributor Author

I will try and work out why the commands are timing out in the integration test

@NathanSavageKaimai
Copy link
Contributor Author

its really strange, individually the test files work but together they are failing

@bashofmann
Copy link
Collaborator

Maybe something is not correctly cleaned up. I can have a look, but earliest next week.

@NathanSavageKaimai
Copy link
Contributor Author

Here is a result log with all debugging enabled
qdrant integration log.txt

@bashofmann bashofmann self-assigned this Sep 9, 2024
@NathanSavageKaimai
Copy link
Contributor Author

hey @bashofmann have you been able to take a look at why the test may be failing?

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

Successfully merging this pull request may close these issues.

Consensus fails when enabling p2p tls
3 participants