Skip to content

Commit

Permalink
Add ingress class name (#24)
Browse files Browse the repository at this point in the history
* feat: add ingress class name

* Bumped Chart to 0.1.3
- Added `ingress.className` to values.yaml (default `""`)
- Bumped chart version in Readme
- Fixed error in ingress template

* Small change to Workflow

---------

Co-authored-by: Francesco Zimbolo <[email protected]>
  • Loading branch information
halkeye and Haloghen authored Jun 21, 2024
1 parent 0422d21 commit 5033f77
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/chart-workflow.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ on:
paths:
- charts/**
- .github/workflows/**
pull_request:
pull_request_target:
paths:
- charts/**
- .github/workflows/**
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ The Applications we offer at the moment are listed down here. Watch this website
- `Prometheus PVE [email protected]`
- `Prometheus Haproxy [email protected]`
- `[email protected]`
- `[email protected].2`
- `[email protected].3`

### License

Expand Down
2 changes: 1 addition & 1 deletion charts/foundry-vtt/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: foundry-vtt
description: Foundry Virtual Tabletop
type: application
version: 0.1.2
version: 0.1.3
appVersion: "v10.288"
keywords:
- game
Expand Down
3 changes: 3 additions & 0 deletions charts/foundry-vtt/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ metadata:
{{ toYaml .Values.ingress.annotations | indent 4 }}
{{- end }}
spec:
{{- if .Values.ingress.ingressClassName }}
ingressClassName: {{ .Values.ingress.ingressClassName }}
{{- end }}
rules:
- host: {{ .Values.ingress.hostname }}
http:
Expand Down
2 changes: 2 additions & 0 deletions charts/foundry-vtt/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -255,6 +255,7 @@ container:
## @param ingress.pathType Type of Path
## @param ingress.tls Enable or Disable TLS
## @param ingress.configurationSnippet Extra NGINX Configuration to add as Annotation
## @param ingress.className Optional Ingress class name
ingress:
enabled: false
hostname: foundry.domain.com
Expand All @@ -267,3 +268,4 @@ ingress:
# configurationSnippet: |
# more_set_input_headers "X-Forwarded-Host: {{ .Values.foundryvtt.hostname }}";
tls: false
className: ""

0 comments on commit 5033f77

Please sign in to comment.