Skip to content

Commit

Permalink
Merge pull request #216 from aandryashin/master
Browse files Browse the repository at this point in the history
Removed wrong hostCertificate implementation.
  • Loading branch information
vania-pooh authored Sep 14, 2023
2 parents 4fc2b01 + aa8ee49 commit f102283
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 14 deletions.
3 changes: 0 additions & 3 deletions boot/templates/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,5 @@ data:
GatewayPorts no
ForceCommand /sbin/nologin
{{- with .Values.boot.jumphost.hostCertificate }}
HostCertificate /etc/ssh/ssh_host_key-cert.pub
{{- end }}
TrustedUserCAKeys /etc/ssh/user_ca.pub
AuthorizedKeysFile /etc/ssh/authorized_keys
7 changes: 2 additions & 5 deletions boot/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,8 @@ spec:
{{- end }}
command:
- /boot
- -ca-secret-name
- {{ .Release.Name }}-vm-ssh-host-ca-keys
securityContext:
privileged: false
runAsUser: {{ printf "%d" (int64 .Values.boot.jumphost.user.uid) }}
Expand Down Expand Up @@ -141,11 +143,6 @@ spec:
- name: ssh-host-keys
mountPath: /etc/ssh/ssh_host_rsa_key
subPath: ssh_host_rsa_key
{{- with .Values.boot.jumphost.hostCertificate }}
- name: ssh-keys
mountPath: /etc/ssh/ssh_host_key-cert.pub
subPath: ssh_host_key-cert.pub
{{- end }}
- name: ssh-keys
mountPath: /etc/ssh/authorized_keys
subPath: authorized_keys
Expand Down
2 changes: 1 addition & 1 deletion boot/templates/generate-ssh-keys.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ metadata:
name: {{ $.Release.Name }}-generate-ssh-keys
namespace: {{ $namespace }}
annotations:
helm.sh/hook: "pre-install,pre-upgrade"
helm.sh/hook: pre-install,pre-upgrade
helm.sh/hook-delete-policy: hook-succeeded,before-hook-creation
helm.sh/hook-weight: "1"
---
Expand Down
4 changes: 0 additions & 4 deletions boot/templates/secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,6 @@ metadata:
name: {{ .Release.Name }}-ssh-keys
namespace: {{ .Release.Namespace }}
stringData:
{{- with .Values.boot.jumphost.hostCertificate }}
ssh_host_key-cert.pub:
{{- toYaml . | nindent 4 }}
{{- end }}
user_ca.pub: |
{{- range $k := .Values.boot.jumphost.trustedUserCAKeys }}
{{ $k }}
Expand Down
1 change: 1 addition & 0 deletions boot/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ spec:
- {{ . }}
{{- end }}
{{- end }}
type: {{ .Values.boot.service.type | default "LoadBalancer" }}
{{- $namespaces := .Values.boot.namespaces }}
{{- if not $namespaces }}
{{- $namespaces = list .Release.Namespace }}
Expand Down
2 changes: 1 addition & 1 deletion boot/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,11 @@ boot:
port: 2222
authorizedKeys: []
trustedUserCAKeys: []
hostCertificate:
service:
annotations:
labels:
externalIPs:
type:
serviceAccount:
annotations:
labels:
Expand Down

0 comments on commit f102283

Please sign in to comment.