Skip to content

Commit

Permalink
External Port only if config enables it
Browse files Browse the repository at this point in the history
  • Loading branch information
psav committed Jun 16, 2022
1 parent 0efd345 commit 4a583a4
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion clowder-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ data:
"createServiceMonitor": true,
"watchStrimziResources": true,
"enableKedaResources": true,
"reconciliationMetrics": true
"reconciliationMetrics": true,
"enableExternalStrimzi": true
}
}
kind: ConfigMap
Expand Down
1 change: 1 addition & 0 deletions controllers/cloud.redhat.com/clowderconfig/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ type ClowderConfig struct {
PerProviderMetrics bool `json:"perProviderMetrics"`
ReconciliationMetrics bool `json:"reconciliationMetrics"`
DisableCloudWatchLogging bool `json:"disableCloudWatchLogging"`
EnableExternalStrimzi bool `json:"enableExternalStrimzi"`
} `json:"features"`
}

Expand Down
2 changes: 1 addition & 1 deletion controllers/cloud.redhat.com/providers/kafka/strimzi.go
Original file line number Diff line number Diff line change
Expand Up @@ -255,7 +255,7 @@ func (s *strimziProvider) configureKafkaCluster() error {

k.Spec.Kafka.Listeners = []strimzi.KafkaSpecKafkaListenersElem{listener}

if s.Env.Spec.Providers.Kafka.EnableLegacyStrimzi {
if clowderconfig.LoadedConfig.Features.EnableExternalStrimzi {
externalHost := "localhost"
externalPort := int32(9094)
externalListener := strimzi.KafkaSpecKafkaListenersElem{
Expand Down

0 comments on commit 4a583a4

Please sign in to comment.