Skip to content

Commit fce9ab2

Browse files
authored
Merge pull request #29 from ashtongraves/main
Removes unused values from helm chart
2 parents a29384d + 7f42a7a commit fce9ab2

File tree

8 files changed

+17
-114
lines changed

8 files changed

+17
-114
lines changed

supported/osg-htc/osg-hosted-ce/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@ apiVersion: v1
33
appVersion: "V5-branch"
44
description: OSG Hosted Compute Entrypoint
55
name: osg-hosted-ce
6-
version: 4.8.0
6+
version: 4.9.0

supported/osg-htc/osg-hosted-ce/README.md

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -242,12 +242,6 @@ To avoid these rate limits, it's possible to bootstrap the certificate request p
242242

243243
slate secret create <YOUR HOST KEY SECRET NAME> --cluster <YOUR CLUSTER> --group <YOUR GROUP> --from-file host.key=<PATH TO KEYFILE>
244244

245-
1. Update your values file to use the host key secret that you've created:
246-
247-
HostCredentials:
248-
HostKeySecret: <YOUR HOST KEY SECRET NAME>
249-
HostCertSecret: null
250-
251245
1. Upon successful startup of the Hosted CE app, the Let's Encrypt host certificate can be found in the instance logs:
252246

253247
slate instance logs <YOUR INSTANCE NAME> --container osg-hosted-ce --max-lines 0
@@ -258,12 +252,6 @@ To avoid these rate limits, it's possible to bootstrap the certificate request p
258252

259253
slate secret create <YOUR HOST CERT SECRET NAME> --cluster <YOUR CLUSTER> --group <YOUR GROUP> --from-file host.cert=<PATH TO CERT FILE>
260254

261-
1. Update your values file to use the host key secret that you've created:
262-
263-
HostCredentials:
264-
HostKeySecret: <YOUR HOST KEY SECRET NAME>
265-
HostCertSecret: <YOUR HOST CERT SECRET NAME>
266-
267255
### Developer
268256
Simply disable this. It is in place for the purpose of OSG Internal Testbed hosts, and is not intended for use with production CEs.
269257

@@ -329,8 +317,7 @@ BoscoOverrides:
329317
GitKeySecret: null
330318
331319
HostCredentials:
332-
HostKeySecret: null
333-
HostCertSecret: null
320+
HostCertKeySecret: null
334321
335322
Developer:
336323
Enabled: false

supported/osg-htc/osg-hosted-ce/templates/_helpers.tpl

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,6 @@ If release name contains chart name it will be used as a full name.
2020
{{- end -}}
2121
{{- end -}}
2222

23-
{{- define "namespace" -}}
24-
{{- .Release.Namespace | trimPrefix "slate-vo-" | printf " %s" -}}
25-
{{- end -}}
26-
2723
{{/*
2824
Create chart name and version as used by the chart label.
2925
*/}}

supported/osg-htc/osg-hosted-ce/templates/configmap.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@ data:
7272
{{ end }}
7373

7474
SLATE_HOSTED_CE = True
75-
SCHEDD_ATTRS = $(SCHEDD_ATTRS) SLATE_HOSTED_CE
75+
OSG_HOSTED_CE = True
76+
SCHEDD_ATTRS = $(SCHEDD_ATTRS) SLATE_HOSTED_CE OSG_HOSTED_CE
7677

7778
# Templatize IDTOKEN generation for glidein -> CE collector advertising (SOFTWARE-5556)
7879
{{- range $index, $map := .Values.SciTokenRemoteUserMapping }}
@@ -174,15 +175,15 @@ data:
174175
apiVersion: v1
175176
kind: ConfigMap
176177
metadata:
177-
name: osg-hosted-ce-{{ .Values.Instance }}-slate-scitokens
178+
name: osg-hosted-ce-{{ .Values.Instance }}-scitokens
178179
labels:
179180
app: osg-hosted-ce
180181
instance: {{ .Values.Instance }}
181182
release: {{ .Release.Name }}
182183
app.kubernetes.io/part-of: {{ .Chart.Name }}
183184
app.kubernetes.io/instance: {{ .Release.Name }}
184185
data:
185-
50-slate-scitokens.conf: |+
186+
50-scitokens.conf: |+
186187
{{- range $index, $map := .Values.SciTokenRemoteUserMapping }}
187188
{{- range $url, $user := $map }}
188189
SCITOKENS /^{{ $url | replace "/" "\\/" | replace "." "\\." | replace "-" "\\-" }}{{ if not ( $url | contains "," ) }},{{ end }}/ {{ $user }}

supported/osg-htc/osg-hosted-ce/templates/deployment.yaml

Lines changed: 5 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ metadata:
66
app: {{ template "osg-hosted-ce.name" . }}
77
release: {{ .Release.Name }}
88
instance: {{ .Values.Instance }}
9-
instanceID: {{ .Values.SLATE.Instance.ID | quote }}
109
app.kubernetes.io/part-of: {{ .Chart.Name }}
1110
app.kubernetes.io/instance: {{ .Release.Name }}
1211
spec:
@@ -53,9 +52,9 @@ spec:
5352
- name: osg-hosted-ce-{{ .Values.Instance }}-htcondor-ce-configuration
5453
configMap:
5554
name: osg-hosted-ce-{{ .Values.Instance }}-htcondor-ce-configuration
56-
- name: osg-hosted-ce-{{ .Values.Instance }}-slate-scitokens
55+
- name: osg-hosted-ce-{{ .Values.Instance }}-scitokens
5756
configMap:
58-
name: osg-hosted-ce-{{ .Values.Instance }}-slate-scitokens
57+
name: osg-hosted-ce-{{ .Values.Instance }}-scitokens
5958
- name: bosco-ssh-private-key-volume
6059
secret:
6160
secretName: {{ .Values.RemoteCluster.PrivateKeySecret }}
@@ -88,25 +87,6 @@ spec:
8887
- key: tls.key
8988
path: hostkey.pem
9089
mode: 0o400
91-
{{ else }}
92-
{{ if and .Values.HostCredentials.HostCertSecret .Values.HostCredentials.HostKeySecret }}
93-
- name: osg-hosted-ce-hostcert-volume
94-
secret:
95-
secretName: {{ .Values.HostCredentials.HostCertSecret }}
96-
items:
97-
- key: host.cert
98-
path: hostcert.pem
99-
mode: 256
100-
{{ end }}
101-
{{ if .Values.HostCredentials.HostKeySecret }}
102-
- name: osg-hosted-ce-hostkey-volume
103-
secret:
104-
secretName: {{ .Values.HostCredentials.HostKeySecret }}
105-
items:
106-
- key: host.key
107-
path: hostkey.pem
108-
mode: 256
109-
{{ end }}
11090
{{ end }}
11191
{{ if .Values.BoscoOverrides.Enabled }}
11292
{{ if .Values.BoscoOverrides.GitKeySecret }}
@@ -162,17 +142,6 @@ spec:
162142
{{ if .Values.HostCredentials.HostCertKeySecret }}
163143
- name: osg-hosted-ce-hostcertkey-volume
164144
mountPath: /etc/grid-security-orig.d
165-
{{ else }}
166-
{{ if and .Values.HostCredentials.HostCertSecret .Values.HostCredentials.HostKeySecret }}
167-
- name: osg-hosted-ce-hostcert-volume
168-
mountPath: /etc/grid-security/hostcert.pem
169-
subPath: hostcert.pem
170-
{{ end }}
171-
{{ if .Values.HostCredentials.HostKeySecret }}
172-
- name: osg-hosted-ce-hostkey-volume
173-
mountPath: /etc/grid-security/hostkey.pem
174-
subPath: hostkey.pem
175-
{{ end }}
176145
{{ end }}
177146
{{ if .Values.Persistence.LogVolume}}
178147
- name: log-volume
@@ -186,21 +155,14 @@ spec:
186155
{{ end }}
187156
{{ end }}
188157
{{ if .Values.SciTokenRemoteUserMapping }}
189-
- name: osg-hosted-ce-{{ .Values.Instance }}-slate-scitokens
190-
mountPath: /etc/condor-ce/mapfiles.d/50-slate-scitokens.conf
191-
subPath: 50-slate-scitokens.conf
158+
- name: osg-hosted-ce-{{ .Values.Instance }}-scitokens
159+
mountPath: /etc/condor-ce/mapfiles.d/50-scitokens.conf
160+
subPath: 50-scitokens.conf
192161
{{ end }}
193162
ports:
194163
- name: htcondor-ce
195164
containerPort: 9619
196165
protocol: TCP
197-
{{ if or .Values.Dashboard.Enabled
198-
( and (not .Values.HostCredentials.HostCertKeySecret)
199-
(not .Values.HostCredentials.HostCertSecret) ) }}
200-
- name: http # needed to get a cert issued from Let's Encrypt
201-
containerPort: 80
202-
protocol: TCP
203-
{{ end }}
204166
env:
205167
{{ if eq .Values.Networking.ServiceType "HostNetwork" }}
206168
- name: _CONDOR_NETWORK_HOSTNAME
@@ -225,10 +187,6 @@ spec:
225187
{{ else }}
226188
value: "false"
227189
{{ end }}
228-
{{ if .Values.Dashboard.Enabled }}
229-
- name: ENABLE_CE_VIEW
230-
value: "true"
231-
{{ end }}
232190
{{ if .Values.Debug.ContinueOnError }}
233191
- name: HOSTED_CE_CONTINUE_ON_ERROR
234192
value: "true"

supported/osg-htc/osg-hosted-ce/templates/networkpolicy.yaml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,5 @@ spec:
1313
- ports:
1414
- protocol: TCP
1515
port: 9619
16-
{{ if or .Values.Dashboard.Enabled
17-
( and (not .Values.HostCredentials.HostCertKeySecret)
18-
(not .Values.HostCredentials.HostCertSecret) ) }}
19-
- protocol: TCP
20-
port: 80
21-
{{ end }}
2216
egress:
2317
- {}

supported/osg-htc/osg-hosted-ce/templates/service.yaml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,6 @@ spec:
2222
port: 9619
2323
targetPort: htcondor-ce
2424
protocol: TCP
25-
{{ if or .Values.Dashboard.Enabled
26-
( and (not .Values.HostCredentials.HostCertKeySecret)
27-
(not .Values.HostCredentials.HostCertSecret) ) }}
28-
- name: http # needed to get a cert issued from Let's Encrypt (since the admin's not supplying a cert/key as a secret)
29-
port: 80
30-
targetPort: http
31-
protocol: TCP
32-
{{ end }}
3325
{{ if .Values.Networking.RequestIP }}
3426
loadBalancerIP: {{ .Values.Networking.RequestIP }}
3527
{{ end }}

supported/osg-htc/osg-hosted-ce/values.yaml

Lines changed: 6 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ RemoteCluster:
2828
# login04.osgconnect.net ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQCqsciY/FmcH6yedW7DxbTwLjkwlgVcbcG43QwdaSvMuN5EHAeLRH2P6ig3H6s/4hZqJn7AU2oNsoIqjZZ8iHqGyiTB9r70Aovt9PUuQXw5qHvYiIoyw49/waJ+yzu4+UD3qfnxeHuRw5EOaXia72MNJJ4lBMoS6iw1JveFJY4rBgKZ7nta1eAGGfHADsf5lfhlIXnJPN050X6zcUjqEEKXjdUkFa3wDiVbFEIESCMwi1b6Q1OXGGFwuSNse1X38CREBzQ+NjOB97o70chZoh2jZ2O3iZZZbujYcDzpxF3RxmewFC3pDmkoOpJrfV2W58n1KdJJnyvjlqEZGXCXcAlB
2929
# login04.osgconnect.net ecdsa-sha2-nistp256 AAAAE2VjZHNhLXNoYTItbmlzdHAyNTYAAAAIbmlzdHAyNTYAAABBBCnySArMSVd0O3UniYN4vX3a/d9d9mEMnrf2z1S900t0GrxVazVGu3ObeLfWwYRc3qFXNRPGh3mKcz5QVG6gcm4=
3030
# login04.osgconnect.net ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIJBW6weAaFSdLYksn4vIvqt4+geNVQ/bUvp71kTeQsKV
31-
# SLATE secret with 'bosco.key' containing the SSH key to access LoginHost:
31+
# Secret with 'bosco.key' containing the SSH key to access LoginHost:
3232
PrivateKeySecret: lincolnb-bosco
3333
# Secret containing a signed certificate (required by some sites)
3434
# See CertificateFile in `man ssh_config`
@@ -145,9 +145,7 @@ SciTokenRemoteUserMapping:
145145
# max_wall_time = 34560
146146
# queue = opportunistic
147147

148-
# Enable persistence to HostedCE files by creating corresponding SLATE Volumes
149-
# SLATE Creates a PVC in Kubernetes which can be reclaimed on restart
150-
# The value for each volume should correspond to the volume name within SLATE
148+
# Enable persistence to HostedCE files by creating corresponding Volumes
151149
Persistence:
152150
# /var/log/condor-ce
153151
LogVolume: null
@@ -163,8 +161,9 @@ Persistence:
163161
BoscoOverrides:
164162
Enabled: false
165163
BoscoOverrideDir: null
166-
GitEndpoint: https://github.com/slateci/bosco-override-template
167-
# If GitEndpoint requires authentication, create a SLATE secret with
164+
GitEndpoint: git@github.com:opensciencegrid/hosted-ce-config.git
165+
RepoNeedsPrivKey: true
166+
# If GitEndpoint requires authentication, create a secret with
168167
# 'git.key' containing the private SSH key that can access
169168
# it. Specify the name of the secret in GitKeySecret:
170169
GitKeySecret: null
@@ -191,28 +190,13 @@ ServiceAnnotations: {}
191190

192191
HostCredentials:
193192
# Name of the secret containing a host key and certificate in
194-
# "tls.key" and "tls.crt", respectively. If defined, values of
195-
# HostCertSecret and HostKeySecret are ignored.
193+
# "tls.key" and "tls.crt", respectively.
196194
HostCertKeySecret: null
197-
# Use a pre-existing host key to request a new Let's Encrypt
198-
# certificate If HostCertSecret is also specified, the Let's Encrypt
199-
# request is skipped. Secret must contain a "host.key" key
200-
# containing the encoded host key.
201-
HostKeySecret: null
202-
# Use a pre-existing host certificate instead of requesting a new
203-
# Let'S Encrypt certificate. If HostKeySecret is not specified, a
204-
# new Let's Encrypt certificate and key are requested anyway.
205-
# Secret must contain a "host.cert" containing the encoded host
206-
# certificate.
207-
HostCertSecret: null
208195
# If set to 'true', use the Let's Encrypt staging server. This is
209196
# useful for avoiding Let's Encrypt rate limits when first setting
210197
# up a CE. NOT SUITABLE FOR PRODUCTION USE.
211198
LetsEncryptStaging: false
212199

213-
Dashboard:
214-
Enabled: false
215-
216200
# Choose which tag to use for the specified containers
217201
ContainerTags:
218202
HostedCE: release
@@ -228,12 +212,3 @@ Debug:
228212
# - Generates a test CA and self-signed host cert/key pair
229213
Developer:
230214
Enabled: false
231-
232-
# Should the Job Router use the new route syntax?
233-
JobRouterUseTransforms: false
234-
235-
### SLATE-START ###
236-
SLATE:
237-
Instance:
238-
ID: "untagged"
239-
### SLATE-END ###

0 commit comments

Comments
 (0)