Skip to content

Commit 1d475d3

Browse files
authored
fix: rm private ingress (#1712)
1 parent c76c31b commit 1d475d3

File tree

6 files changed

+9
-44
lines changed

6 files changed

+9
-44
lines changed

helmfile.d/snippets/defaults.yaml

-9
Original file line numberDiff line numberDiff line change
@@ -333,14 +333,6 @@ environments:
333333
enabled: false
334334
block: false
335335
owasp: true
336-
private:
337-
enabled: false
338-
autoscaling:
339-
enabled: true
340-
minReplicas: 2
341-
maxReplicas: 10
342-
service:
343-
annotations: {}
344336
tracing:
345337
enabled: false
346338
samplingRatio: "0.01"
@@ -927,7 +919,6 @@ environments:
927919
ingress:
928920
platformClass:
929921
className: platform
930-
network: public
931922
classes: []
932923
alerts:
933924
receivers:

tests/fixtures/env/apps/ingress-nginx-platform.yaml

-4
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,3 @@ apps:
1818
requests:
1919
cpu: 100m
2020
memory: 192Mi
21-
service:
22-
annotations:
23-
- key: service.beta.kubernetes.io/do-loadbalancer-enable-proxy-protocol
24-
value: true

tests/fixtures/env/apps/ingress-nginx-private.yaml

-20
This file was deleted.

tests/fixtures/env/settings.yaml

+3-6
Original file line numberDiff line numberDiff line change
@@ -26,16 +26,13 @@ e2e:
2626
enabled: false
2727
ingress:
2828
classes:
29-
- className: private
30-
entrypoint: ''
31-
loadBalancerSubnet: subnet
32-
network: private
3329
- className: net-a
3430
entrypoint: ''
3531
loadBalancerIP: 11.0.0.1
36-
loadBalancerRG: myrg
37-
network: public
3832
sourceIpAddressFiltering: 10.0.0.0/24
33+
annotations:
34+
- key: service.beta.kubernetes.io/do-loadbalancer-enable-proxy-protocol
35+
value: true
3936
platformClass:
4037
entrypoint: ''
4138
kms:

values-schema.yaml

+4-2
Original file line numberDiff line numberDiff line change
@@ -448,7 +448,6 @@ definitions:
448448
type: string
449449
enum:
450450
- public
451-
- private
452451
default: public
453452
loadBalancerIP:
454453
title: Load balancer IP address
@@ -458,7 +457,10 @@ definitions:
458457
title: Source IP address filtering
459458
type: string
460459
description: 'Whitelist source IP address ranges. The value is a comma separated list of CIDRs, e.g. 10.0.0.0/24,172.10.0.1'
461-
460+
annotations:
461+
properties:
462+
annotations:
463+
$ref: '#/definitions/annotations'
462464
ipV4Address:
463465
type: string
464466
pattern: ^(?:[0-9]{1,3}\.){3}[0-9]{1,3}$

values/ingress-nginx/ingress-nginx.gotmpl

+2-3
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
{{/* appDefaults - each ingress-nginx Helm realse will use default values defined in defaults.yaml in app.ingress-nginx context*/}}
66
{{- $appDefaults := $v | get "apps.ingress-nginx" }}
77
{{- $n := deepCopy $appDefaults | merge $app }}
8-
{{- $privateNetwork := eq "private" ($ingress | get "network" "public") }}
98

109
nameOverride: {{ .Release.Name }}
1110

@@ -124,11 +123,11 @@ controller:
124123
{{- end }}
125124
annotations:
126125
dummy: 'true'
127-
{{- with $n | get "service.annotations" nil }}
126+
{{- with $ingress | get "annotations" nil }}
128127
{{- range . }}
129128
{{ .key }}: "{{ .value }}"
130129
{{- end }}
131-
{{- end }}
130+
{{- end }}
132131
publishService:
133132
enabled: true
134133

0 commit comments

Comments
 (0)