Skip to content

feat: support auto assign node port for dp-manager #271

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.

Sign up for GitHub

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

Merged
Baoyuantop merged 2 commits into from
Mar 31, 2026
Merged

feat: support auto assign node port for dp-manager #271

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9317dcb
feat: support auto assign node port for dp-manager
Baoyuantop Mar 31, 2026
9bb5ea3
fix: address PR review comments
Baoyuantop Mar 31, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion charts/api7/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.17.47
version: 0.17.48

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
3 changes: 3 additions & 0 deletions charts/api7/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -208,14 +208,17 @@ A Helm chart for Kubernetes
| dp_manager_configuration.server.tls.host | string | `"0.0.0.0"` | |
| dp_manager_configuration.server.tls.port | int | `7943` | |
| dp_manager_service.annotations | object | `{}` | |
| dp_manager_service.autoAssignNodePort | bool | `false` | Whether to set nodePort to the same value as the HTTP/HTTPS port when dp_manager_service.type is NodePort, make sure the nodePort to be in the valid NodePort range of kubernetes service. |
| dp_manager_service.ingress.annotations | object | `{}` | |
| dp_manager_service.ingress.className | string | `""` | |
| dp_manager_service.ingress.enabled | bool | `false` | |
| dp_manager_service.ingress.hosts[0].host | string | `"dp-manager.local"` | |
| dp_manager_service.ingress.hosts[0].paths[0].path | string | `"/"` | |
| dp_manager_service.ingress.hosts[0].paths[0].pathType | string | `"ImplementationSpecific"` | |
| dp_manager_service.ingress.tls | list | `[]` | |
| dp_manager_service.nodePort | int | `nil` | The nodePort for HTTP service, only used if dp_manager_service.type is NodePort. If not set, a random port will be assigned by Kubernetes. |
| dp_manager_service.port | int | `7900` | |
| dp_manager_service.tlsNodePort | int | `nil` | The nodePort for HTTPS service, only used if dp_manager_service.type is NodePort. If not set, a random port will be assigned by Kubernetes. |
| dp_manager_service.tlsPort | int | `7943` | |
| dp_manager_service.type | string | `"ClusterIP"` | |
| fullnameOverride | string | `""` | |
Expand Down
14 changes: 14 additions & 0 deletions charts/api7/templates/dp-manager-service.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,24 @@ spec:
targetPort: http
protocol: TCP
name: http
{{- if eq .Values.dp_manager_service.type "NodePort" }}
{{- if .Values.dp_manager_service.autoAssignNodePort }}
nodePort: {{ .Values.dp_manager_service.port }}
{{- else if .Values.dp_manager_service.nodePort }}
nodePort: {{ .Values.dp_manager_service.nodePort }}
{{- end }}
{{- end }}
- port: {{ .Values.dp_manager_service.tlsPort }}
targetPort: https
protocol: TCP
name: https
{{- if eq .Values.dp_manager_service.type "NodePort" }}
{{- if .Values.dp_manager_service.autoAssignNodePort }}
nodePort: {{ .Values.dp_manager_service.tlsPort }}
{{- else if .Values.dp_manager_service.tlsNodePort }}
nodePort: {{ .Values.dp_manager_service.tlsNodePort }}
{{- end }}
{{- end }}
Comment thread
coderabbitai[bot] marked this conversation as resolved.

selector:
{{- include "api7ee3.selectorLabels" . | nindent 4 }}
Expand Down
6 changes: 6 additions & 0 deletions charts/api7/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -171,6 +171,12 @@ dp_manager_service:
annotations: {}
port: 7900
tlsPort: 7943
# -- Whether to set nodePort to the same value as the HTTP/HTTPS port when dp_manager_service.type is NodePort, make sure the nodePort to be in the valid NodePort range of kubernetes service.
autoAssignNodePort: false
# -- (int) The nodePort for HTTP service, only used if dp_manager_service.type is NodePort. If not set, a random port will be assigned by Kubernetes.
nodePort: null
# -- (int) The nodePort for HTTPS service, only used if dp_manager_service.type is NodePort. If not set, a random port will be assigned by Kubernetes.
tlsNodePort: null
ingress:
enabled: false
className: ""
Expand Down
Loading