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

Allow management-api in different port #242

Open
burmanm opened this issue Dec 2, 2021 · 7 comments
Open

Allow management-api in different port #242

burmanm opened this issue Dec 2, 2021 · 7 comments
Labels
enhancement New feature or request product-backlog Issues in the state 'product-backlog' refactoring zh:Assess/Investigate

Comments

@burmanm
Copy link
Contributor

burmanm commented Dec 2, 2021

What is missing?
Currently management-api is considered to be always in the port 8080 by the cass-operator. We should allow PodHost to define a port as well to simplify testing and potentially allow deployments where the port is different.

┆Issue is synchronized with this Jira Story by Unito
┆Issue Number: CASS-52

@burmanm burmanm added the enhancement New feature or request label Dec 2, 2021
@burmanm
Copy link
Contributor Author

burmanm commented Dec 2, 2021

And/or we should also allow management api initialization to be captured and mocked so that we could inject httpclient there.

@sync-by-unito sync-by-unito bot changed the title Allow management-api in different port K8SSAND-1417 ⁃ Allow management-api in different port Apr 4, 2022
@adejanovski adejanovski moved this to Assess/Investigate in K8ssandra Nov 8, 2022
@adejanovski adejanovski moved this from Assess/Investigate to Ready in K8ssandra Feb 3, 2023
@adejanovski adejanovski added the ready Issues in the state 'ready' label Feb 3, 2023
@Ajith-ceg
Copy link

Hi @burmanm
I have been using host networking for the multi-cluster k8ssandra deployment. Node local dns process is running on port 8080 on all the worker nodes. So, there was a port conflict with cass-management API. This feature would be of great value. May I know any updates and when this feature would be rolled out?

@burmanm
Copy link
Contributor Author

burmanm commented Aug 25, 2023

Part of the work is done in the envtesting PR. This allows the cass-operator to query the modified port, but it does not yet allow changing them from the CRD. That'll require another set of changes to expose the port settings.

@Ajith-ceg
Copy link

What is the problem with CRD? Could you pls elaborate on it?
And Good to know that work is going on this...thanks for your update @burmanm

@burmanm
Copy link
Contributor Author

burmanm commented Sep 11, 2023

What is the problem with CRD? Could you pls elaborate on it?

There's nothing in the CRD that would allow overriding ports. The ports are currently hardcoded in cass-operator. That said, I could enable changing them through the PodTemplateSpec in the containerPorts, but that would be somewhat hidden feature in that case. Maybe that would be fine.

@burmanm
Copy link
Contributor Author

burmanm commented Sep 11, 2023

Actually, there's no need to do any of that, we already supported it in the PodTemplateSpec. See the test in commit 52a430d

That would change the mgmt-api port that cass-operator calls. It does not however modify the port that management-api itself listens to. That's defined in the Dockerfile entrypoitn of mgmt-api:

https://github.com/k8ssandra/management-api-for-apache-cassandra/blob/ce77e239a69697926777866a8db7d7530b9d503a/scripts/docker-entrypoint.sh#L182

Which you will need to override for mgmt-api to listen to in a different port. That's the problem with running in hostnetwork for everything, we can't do do port forwards.

@burmanm
Copy link
Contributor Author

burmanm commented Nov 6, 2023

@adejanovski adejanovski moved this from Ready to Product Backlog in K8ssandra Sep 3, 2024
@adejanovski adejanovski added product-backlog Issues in the state 'product-backlog' and removed ready Issues in the state 'ready' labels Sep 3, 2024
@sync-by-unito sync-by-unito bot changed the title K8SSAND-1417 ⁃ Allow management-api in different port Allow management-api in different port Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request product-backlog Issues in the state 'product-backlog' refactoring zh:Assess/Investigate
Projects
No open projects
Status: Product Backlog
Development

No branches or pull requests

3 participants