Skip to content

Commit 6c5132a

Browse files
authored
Pin ingress-nginx controller helm chart due to regression. (#245)
1 parent 3d60fd0 commit 6c5132a

File tree

2 files changed

+13
-18
lines changed

2 files changed

+13
-18
lines changed

roles/ingress-controller/defaults/main.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,7 @@ ingress_https_port: 4443
55
ingress_additional_config: {}
66
ingress_tcp_service_exposals: {}
77
ingress_udp_service_exposals: {}
8+
9+
# TODO: can be removed again
10+
# currently there seems to be a regression: https://github.com/kubernetes/ingress-nginx/issues/13598
11+
ingress_controller_chart_version: 4.12.4
Lines changed: 9 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,11 @@
11
---
2-
- name: Create namespace ingress controller
3-
k8s:
4-
definition:
5-
apiVersion: v1
6-
kind: Namespace
7-
metadata:
8-
name: ingress-nginx
9-
labels:
10-
name: ingress-nginx
11-
122
- name: Deploy nginx-ingress
13-
include_role:
14-
name: ansible-common/roles/helm-chart
15-
vars:
16-
helm_repo: "https://kubernetes.github.io/ingress-nginx"
17-
helm_chart: ingress-nginx
18-
helm_target_namespace: ingress-nginx
19-
helm_release_name: ingress-nginx
20-
helm_value_file_template: "values.yaml"
3+
kubernetes.core.helm:
4+
chart_repo_url: "https://kubernetes.github.io/ingress-nginx"
5+
chart_version: "{{ ingress_controller_chart_version | default(omit) }}"
6+
name: ingress-nginx
7+
chart_ref: ingress-nginx
8+
release_namespace: ingress-nginx
9+
create_namespace: true
10+
values: "{{ lookup('template', 'values.yaml') | from_yaml }}"
11+
wait: true

0 commit comments

Comments
 (0)