Skip to content

Generate a NodePort service for the gitea http.#511

Merged
punkwalker merged 1 commit intocnoe-io:mainfrom
ch007m:issue-510
Apr 3, 2025
Merged

Generate a NodePort service for the gitea http.#511
punkwalker merged 1 commit intocnoe-io:mainfrom
ch007m:issue-510

Conversation

@cmoulliard
Copy link
Copy Markdown
Contributor

Signed-off-by: cmoulliard <cmoulliard@redhat.com>
@cmoulliard cmoulliard requested a review from punkwalker March 31, 2025 13:52
@punkwalker
Copy link
Copy Markdown
Contributor

@cmoulliard Is there any specific use-case for which we need this Nodeport? The existing ClusterIP service works fine, right?

@cmoulliard
Copy link
Copy Markdown
Contributor Author

The existing ClusterIP service works fine, right?

Yes, it works fine but it is mainly used for internal usage when pod access gitea internally using the kube service address OR when the service should be managed behind an ingress host

@cmoulliard
Copy link
Copy Markdown
Contributor Author

Is there any specific use-case for which we need this Nodeport?

As mentioned this is to allow a pod to access gitea from another kind cluster.

cluster 1 : kind cluster running gitea = gitea.cnoe.localtest.me:8443
cluster 2 : kind cluster where a pod (aka a git client) will access gitea running on the cluster 1. In this case, the pod will resolve the address: gitea.cnoe.localtest.me to localhost/127.0.0.01 and will not be able to access: gitea.cnoe.localtest.me:8443

Copy link
Copy Markdown
Contributor

@punkwalker punkwalker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@punkwalker punkwalker merged commit beeb203 into cnoe-io:main Apr 3, 2025
5 checks passed
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.

2 participants