Skip to content

Commit

Permalink
fix: condition to control virtualService creation
Browse files Browse the repository at this point in the history
  • Loading branch information
h1manshu98 committed May 17, 2024
1 parent 12126c4 commit 313f5a3
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/helmchart/config/override-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ service:
istio:
enabled: false
virtualService:
enabled: true
hosts:
- test.example.com
- test-2.example.com
Expand All @@ -73,7 +74,6 @@ ingress:
paths:
- path: /
pathType: ImplementationSpecific

tls: []

resource:
Expand Down
2 changes: 1 addition & 1 deletion charts/helmchart/templates/servicemonitor.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# This object requires CustomResourceDefinition (crds).
{{- if .Values.metrics.enabled }}
# This object requires CustomResourceDefinition (crds).
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
Expand Down
2 changes: 1 addition & 1 deletion charts/helmchart/templates/virtualservice.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{{- if .Values.istio.enabled -}}
{{- if .Values.istio.virtualService.enabled -}}
apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
Expand Down
1 change: 1 addition & 0 deletions charts/helmchart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ istio:
enabled: false
# -- virtualService is required when using istio.
virtualService:
enabled: false
hosts: []
# - test.example.com
# - test-2.example.com
Expand Down

0 comments on commit 313f5a3

Please sign in to comment.