Skip to content

Commit

Permalink
chore(security-apps): Fail deprecated gangway installations
Browse files Browse the repository at this point in the history
  • Loading branch information
hairmare committed Jan 16, 2025
1 parent 484d395 commit fd0fec0
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 42 deletions.
8 changes: 2 additions & 6 deletions charts/security-apps/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: security-apps
description: Argo CD app-of-apps config for security applications
type: application
version: 0.89.0
version: 0.89.1
home: https://github.com/adfinis/helm-charts/tree/main/charts/security-apps
sources:
- https://github.com/adfinis/helm-charts
Expand All @@ -15,10 +15,6 @@ dependencies:
version: 0.9.1
repository: https://charts.adfinis.com
annotations:
artifacthub.io/containsSecurityUpdates: "true"
artifacthub.io/changes: |
- kind: changed
description: "chore: update oauth2-proxy from 6.10.1 to 7.8.2"
links:
- name: OAuth-Proxy 7.8.2 Release
url: https://github.com/oauth2-proxy/manifests/releases/tag/oauth2-proxy-7.8.2
description: "Fail installations of deprecated gangway component"
2 changes: 1 addition & 1 deletion charts/security-apps/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 0 additions & 4 deletions charts/security-apps/ci/default-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,6 @@ dex:
enabled: true
values: {}

gangway:
enabled: true
values: {}

vault:
enabled: true
values: {}
Expand Down
32 changes: 1 addition & 31 deletions charts/security-apps/templates/gangway.yaml
Original file line number Diff line number Diff line change
@@ -1,33 +1,3 @@
{{ if .Values.gangway.enabled }}
{{ template "argoconfig.application" (list . "security-apps.gangway") }}
{{ fail "gangway is DEPRECATED, use dexK8sAuthenticator instead" }}
{{ end }}

{{- define "security-apps.gangway" -}}{{- $app := unset .Values.gangway "enabled" -}}{{- $name := default $app.destination.namespace $app.name -}}
metadata:
name: {{ template "common.fullname" . }}-{{ $name }}
spec:
{{- if $app.project }}
project: {{ $app.project | quote }}
{{- end }}
source:
repoURL: {{ $app.repoURL | quote }}
chart: {{ $app.chart | quote }}
targetRevision: {{ $app.targetRevision | quote }}
helm:
releaseName: {{ $name | quote }}
values: |-
nameOverride: {{ $name | quote }}
{{- $app.values | toYaml | nindent 8 }}
{{- if $app.destination }}
destination:
{{ $app.destination | toYaml | nindent 4 }}
{{- end }}
{{- if $app.syncPolicy }}
syncPolicy:
{{ $app.syncPolicy | toYaml | nindent 4 }}
{{- end }}
{{- if $app.ignoreDifferences }}
ignoreDifferences:
{{ $app.ignoreDifferences | toYaml | nindent 4 }}
{{- end }}
{{- end -}}

0 comments on commit fd0fec0

Please sign in to comment.