Skip to content

Commit

Permalink
fix(eventbus): allow clients with istio sidecar. Fixes: argoproj#1311
Browse files Browse the repository at this point in the history
Istio-related issues affecting this behaviour:

- nats-io/nats-operator#88
- istio/istio#28623

The fix is very simple. I added the related issues to the code comment,
let me know if that's undesirable or there's something else to clarify.
  • Loading branch information
antoniomo committed Aug 14, 2021
1 parent bd73790 commit 2efa113
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion controllers/eventbus/installer/nats.go
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,11 @@ func (i *natsInstaller) buildStanService() (*corev1.Service, error) {
Spec: corev1.ServiceSpec{
ClusterIP: corev1.ClusterIPNone,
Ports: []corev1.ServicePort{
{Name: "client", Port: clientPort},
// Prefix tcp- to enable clients to connect from
// an istio-enabled namespace, following:
// https://github.com/nats-io/nats-operator/issues/88
// https://github.com/istio/istio/issues/28623
{Name: "tcp-client", Port: clientPort},
{Name: "cluster", Port: clusterPort},
{Name: "monitor", Port: monitorPort},
},
Expand Down

0 comments on commit 2efa113

Please sign in to comment.