Skip to content

Commit

Permalink
Headless Service (#299)
Browse files Browse the repository at this point in the history
## overview
Change services to headless service as ClusterIP is not required. Apache
OSS operator also uses headless services.
  • Loading branch information
sethsaperstein-lyft committed Jan 10, 2024
1 parent 173badd commit e54fd58
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 1 addition & 1 deletion integ/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,4 @@ export INTEGRATION=true
export OPERATOR_IMAGE=flinkk8soperator:local

cd $(dirname "$0")
go test -p 1 -timeout 40m -check.vv IntegSuite
go test -p 1 -timeout 60m -check.vv IntegSuite
7 changes: 5 additions & 2 deletions pkg/controller/flink/job_manager_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -221,8 +221,11 @@ func FetchJobManagerServiceCreateObj(app *v1beta1.FlinkApplication, selector str
Labels: GetCommonAppLabels(app),
},
Spec: coreV1.ServiceSpec{
Ports: getJobManagerServicePorts(app),
Selector: serviceLabels,
Ports: getJobManagerServicePorts(app),
Selector: serviceLabels,
Type: coreV1.ServiceTypeClusterIP,
ClusterIP: "None",
ClusterIPs: []string{},
},
}
}
Expand Down

0 comments on commit e54fd58

Please sign in to comment.