mirror of
https://github.com/NVIDIA/OpenShell.git
synced 2026-10-25 06:42:52 +08:00
Seth Jennings
dbe36eaf85
Reject non-loopback plaintext Vault endpoints, disable redirects, and support private CA bundles without weakening hostname verification. Update Helm configuration, documentation, operator skills, and regression coverage for OSSR-002. Signed-off-by: Seth Jennings <sjenning@redhat.com>
231 lines
9.1 KiB
Smarty
231 lines
9.1 KiB
Smarty
# SPDX-FileCopyrightText: Copyright (c) 2025-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
|
|
# SPDX-License-Identifier: Apache-2.0
|
|
|
|
{{/*
|
|
Gateway pod template shared by the StatefulSet and Deployment workload shapes.
|
|
*/}}
|
|
{{- define "openshell.gatewayPodTemplate" -}}
|
|
metadata:
|
|
annotations:
|
|
# Roll the gateway workload when the rendered gateway TOML changes - the
|
|
# gateway only reads /etc/openshell/gateway.toml at startup, so without
|
|
# this annotation a `helm upgrade` that only mutates the ConfigMap would
|
|
# leave pods running with stale config.
|
|
checksum/gateway-config: {{ include (print $.Template.BasePath "/gateway-config.yaml") . | sha256sum }}
|
|
{{- with .Values.podAnnotations }}
|
|
{{- toYaml . | nindent 4 }}
|
|
{{- end }}
|
|
labels:
|
|
{{- include "openshell.labels" . | nindent 4 }}
|
|
{{- with .Values.podLabels }}
|
|
{{- toYaml . | nindent 4 }}
|
|
{{- end }}
|
|
spec:
|
|
terminationGracePeriodSeconds: {{ .Values.podLifecycle.terminationGracePeriodSeconds }}
|
|
{{- with .Values.imagePullSecrets }}
|
|
imagePullSecrets:
|
|
{{- toYaml . | nindent 4 }}
|
|
{{- end }}
|
|
serviceAccountName: {{ include "openshell.serviceAccountName" . }}
|
|
{{- if .Values.server.hostGatewayIP }}
|
|
hostAliases:
|
|
- ip: {{ .Values.server.hostGatewayIP | quote }}
|
|
hostnames:
|
|
- host.docker.internal
|
|
- host.openshell.internal
|
|
{{- end }}
|
|
{{- with .Values.podSecurityContext }}
|
|
securityContext:
|
|
{{- toYaml . | nindent 4 }}
|
|
{{- end }}
|
|
containers:
|
|
- name: openshell-gateway
|
|
securityContext:
|
|
{{- toYaml .Values.securityContext | nindent 8 }}
|
|
image: {{ include "openshell.image" . | quote }}
|
|
imagePullPolicy: {{ .Values.image.pullPolicy }}
|
|
args:
|
|
- --config
|
|
- /etc/openshell/gateway.toml
|
|
{{- if not .Values.server.externalDbSecret }}
|
|
- --db-url
|
|
- {{ .Values.server.dbUrl | quote }}
|
|
{{- end }}
|
|
env:
|
|
{{- if not (or .Values.server.credentialDrivers.kubernetesSecrets.enabled .Values.server.credentialDrivers.vault.enabled) }}
|
|
- name: {{ include "openshell.credentialStorageKeyEncryptionKeyEnvName" . }}
|
|
valueFrom:
|
|
secretKeyRef:
|
|
name: {{ include "openshell.credentialStorageKeyEncryptionKeySecretName" . }}
|
|
key: {{ include "openshell.credentialStorageKeyEncryptionKeySecretKey" . }}
|
|
{{- end }}
|
|
{{- if .Values.server.externalDbSecret }}
|
|
- name: OPENSHELL_DB_URL
|
|
valueFrom:
|
|
secretKeyRef:
|
|
name: {{ .Values.server.externalDbSecret }}
|
|
key: uri
|
|
{{- end }}
|
|
# Most gateway settings live in the ConfigMap-backed TOML file
|
|
# mounted at /etc/openshell/gateway.toml. Secret-bearing settings use
|
|
# env vars that the TOML references by name. Some process-level
|
|
# settings consumed by libraries outside gateway code also remain here.
|
|
{{- if and .Values.server.oidc.issuer .Values.server.oidc.caConfigMapName }}
|
|
# OIDC issuer custom-CA: rustls/reqwest read SSL_CERT_FILE for
|
|
# outbound TLS verification. This is a process-level env var
|
|
# consumed by the TLS stack itself, not by gateway code, so it
|
|
# cannot be represented in the gateway TOML schema.
|
|
- name: SSL_CERT_FILE
|
|
value: /etc/openshell-tls/oidc-ca/ca.crt
|
|
{{- end }}
|
|
- name: OPENSHELL_TELEMETRY_ENABLED
|
|
value: {{ .Values.server.telemetryEnabled | quote }}
|
|
{{- if .Values.server.providerTokenGrants.spiffe.enabled }}
|
|
- name: OPENSHELL_GATEWAY_SPIFFE_WORKLOAD_API_SOCKET
|
|
value: {{ .Values.server.providerTokenGrants.spiffe.workloadApiSocketPath | quote }}
|
|
{{- end }}
|
|
volumeMounts:
|
|
{{- if eq (include "openshell.workloadKind" .) "statefulset" }}
|
|
- name: openshell-data
|
|
mountPath: /var/openshell
|
|
{{- end }}
|
|
# ConfigMap directory mounts expose keys through atomic-writer symlinks,
|
|
# while the gateway intentionally rejects symlinked configuration.
|
|
# The checksum annotation above rolls pods when this subPath changes.
|
|
- name: gateway-config
|
|
mountPath: /etc/openshell/gateway.toml
|
|
subPath: gateway.toml
|
|
readOnly: true
|
|
- name: sandbox-jwt
|
|
mountPath: /etc/openshell-jwt
|
|
readOnly: true
|
|
{{- if not .Values.server.disableTls }}
|
|
- name: tls-cert
|
|
mountPath: /etc/openshell-tls/server
|
|
readOnly: true
|
|
{{- if .Values.certManager.serverIssuerRef.name }}
|
|
- name: tls-external-cert
|
|
mountPath: /etc/openshell-tls/server-external
|
|
readOnly: true
|
|
{{- end }}
|
|
{{- if eq (include "openshell.gatewayClientCaEnabled" .) "true" }}
|
|
- name: tls-client-ca
|
|
mountPath: /etc/openshell-tls/client-ca
|
|
readOnly: true
|
|
{{- end }}
|
|
{{- end }}
|
|
{{- if and .Values.server.oidc.issuer .Values.server.oidc.caConfigMapName }}
|
|
- name: oidc-ca
|
|
mountPath: /etc/openshell-tls/oidc-ca
|
|
readOnly: true
|
|
{{- end }}
|
|
{{- if and .Values.server.credentialDrivers.vault.enabled .Values.server.credentialDrivers.vault.caConfigMapName }}
|
|
- name: vault-ca
|
|
mountPath: /etc/openshell-tls/vault-ca
|
|
readOnly: true
|
|
{{- end }}
|
|
{{- if .Values.server.providerTokenGrants.spiffe.enabled }}
|
|
- name: spiffe-workload-api
|
|
mountPath: {{ dir .Values.server.providerTokenGrants.spiffe.workloadApiSocketPath | quote }}
|
|
readOnly: true
|
|
{{- end }}
|
|
ports:
|
|
- name: grpc
|
|
containerPort: {{ .Values.service.port }}
|
|
protocol: TCP
|
|
- name: health
|
|
containerPort: {{ .Values.service.healthPort }}
|
|
protocol: TCP
|
|
{{- if .Values.service.metricsPort }}
|
|
- name: metrics
|
|
containerPort: {{ .Values.service.metricsPort }}
|
|
protocol: TCP
|
|
{{- end }}
|
|
startupProbe:
|
|
httpGet:
|
|
path: /healthz
|
|
port: health
|
|
periodSeconds: {{ .Values.probes.startup.periodSeconds }}
|
|
timeoutSeconds: {{ .Values.probes.startup.timeoutSeconds }}
|
|
failureThreshold: {{ .Values.probes.startup.failureThreshold }}
|
|
livenessProbe:
|
|
httpGet:
|
|
path: /healthz
|
|
port: health
|
|
initialDelaySeconds: {{ .Values.probes.liveness.initialDelaySeconds }}
|
|
periodSeconds: {{ .Values.probes.liveness.periodSeconds }}
|
|
timeoutSeconds: {{ .Values.probes.liveness.timeoutSeconds }}
|
|
failureThreshold: {{ .Values.probes.liveness.failureThreshold }}
|
|
readinessProbe:
|
|
httpGet:
|
|
path: /readyz
|
|
port: health
|
|
initialDelaySeconds: {{ .Values.probes.readiness.initialDelaySeconds }}
|
|
periodSeconds: {{ .Values.probes.readiness.periodSeconds }}
|
|
timeoutSeconds: {{ .Values.probes.readiness.timeoutSeconds }}
|
|
failureThreshold: {{ .Values.probes.readiness.failureThreshold }}
|
|
resources:
|
|
{{- toYaml .Values.resources | nindent 8 }}
|
|
volumes:
|
|
- name: gateway-config
|
|
configMap:
|
|
name: {{ include "openshell.fullname" . }}-config
|
|
- name: sandbox-jwt
|
|
secret:
|
|
secretName: {{ include "openshell.sandboxJwtSecretName" . }}
|
|
defaultMode: {{ .Values.server.sandboxJwt.secretDefaultMode | default 0400 }}
|
|
{{- if not .Values.server.disableTls }}
|
|
- name: tls-cert
|
|
secret:
|
|
secretName: {{ .Values.server.tls.certSecretName }}
|
|
{{- if .Values.certManager.serverIssuerRef.name }}
|
|
- name: tls-external-cert
|
|
secret:
|
|
secretName: {{ include "openshell.fullname" . }}-server-external-tls
|
|
{{- end }}
|
|
{{- if eq (include "openshell.gatewayClientCaEnabled" .) "true" }}
|
|
- name: tls-client-ca
|
|
secret:
|
|
{{- if or (and .Values.pkiInitJob.enabled (not .Values.certManager.enabled)) (and .Values.certManager.enabled .Values.certManager.clientCaFromServerTlsSecret) }}
|
|
secretName: {{ .Values.server.tls.certSecretName }}
|
|
items:
|
|
- key: ca.crt
|
|
path: ca.crt
|
|
{{- else }}
|
|
secretName: {{ .Values.server.tls.clientCaSecretName }}
|
|
{{- end }}
|
|
{{- end }}
|
|
{{- end }}
|
|
{{- if and .Values.server.oidc.issuer .Values.server.oidc.caConfigMapName }}
|
|
- name: oidc-ca
|
|
configMap:
|
|
name: {{ .Values.server.oidc.caConfigMapName }}
|
|
{{- end }}
|
|
{{- if and .Values.server.credentialDrivers.vault.enabled .Values.server.credentialDrivers.vault.caConfigMapName }}
|
|
- name: vault-ca
|
|
configMap:
|
|
name: {{ .Values.server.credentialDrivers.vault.caConfigMapName }}
|
|
items:
|
|
- key: ca.crt
|
|
path: ca.crt
|
|
{{- end }}
|
|
{{- if .Values.server.providerTokenGrants.spiffe.enabled }}
|
|
- name: spiffe-workload-api
|
|
csi:
|
|
driver: csi.spiffe.io
|
|
readOnly: true
|
|
{{- end }}
|
|
{{- with .Values.nodeSelector }}
|
|
nodeSelector:
|
|
{{- toYaml . | nindent 4 }}
|
|
{{- end }}
|
|
{{- with .Values.affinity }}
|
|
affinity:
|
|
{{- toYaml . | nindent 4 }}
|
|
{{- end }}
|
|
{{- with .Values.tolerations }}
|
|
tolerations:
|
|
{{- toYaml . | nindent 4 }}
|
|
{{- end }}
|
|
{{- end }}
|