Skip to content

Commit

Permalink
Helm: support sync-lb-services-endpoints for sync catalog
Browse files Browse the repository at this point in the history
  • Loading branch information
jukie authored and zalimeni committed Apr 12, 2024
1 parent e575eb2 commit dd794d3
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .changelog/3905.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:improvement
helm: support sync-lb-services-endpoints flag for syncCatalog
```
3 changes: 3 additions & 0 deletions charts/consul/templates/sync-catalog-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -191,6 +191,9 @@ spec:
{{- if .Values.syncCatalog.ingress.loadBalancerIPs }}
-loadBalancer-ips=true \
{{- end }}
{{- if .Values.syncCatalog.syncLoadBalancerEndpoints }}
-sync-lb-services-endpoints=true \
{{- end }}
{{- end }}
livenessProbe:
httpGet:
Expand Down
4 changes: 4 additions & 0 deletions charts/consul/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2193,6 +2193,10 @@ syncCatalog:
# Set this to false to skip syncing ClusterIP services.
syncClusterIPServices: true

# If true, LoadBalancer service endpoints instead of ingress addresses will be synced to Consul.
# If false, LoadBalancer endpoints are not synced to Consul.
syncLoadBalancerEndpoints: false

ingress:
# Syncs the hostname from a Kubernetes Ingress resource to service registrations
# when a rule matched a service. Currently only supports host based routing and
Expand Down

0 comments on commit dd794d3

Please sign in to comment.