Skip to content

Commit 5c3e56e

Browse files
authored
Merge pull request #444 from CecileRobertMichon/https-probe
🏃change the APIServer health probe to HTTPS
2 parents 85d6986 + 316a2a6 commit 5c3e56e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

cloud/services/internalloadbalancers/internalloadbalancers.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ func (s *Service) Reconcile(ctx context.Context, spec interface{}) error {
5252
return errors.New("invalid internal load balancer specification")
5353
}
5454
klog.V(2).Infof("creating internal load balancer %s", internalLBSpec.Name)
55-
probeName := "tcpHTTPSProbe"
55+
probeName := "HTTPSProbe"
5656
frontEndIPConfigName := "controlplane-internal-lbFrontEnd"
5757
backEndAddressPoolName := "controlplane-internal-backEndPool"
5858
idPrefix := fmt.Sprintf("/subscriptions/%s/resourceGroups/%s/providers/Microsoft.Network/loadBalancers", s.Scope.SubscriptionID, s.Scope.ResourceGroup())
@@ -111,7 +111,8 @@ func (s *Service) Reconcile(ctx context.Context, spec interface{}) error {
111111
{
112112
Name: &probeName,
113113
ProbePropertiesFormat: &network.ProbePropertiesFormat{
114-
Protocol: network.ProbeProtocolTCP,
114+
Protocol: network.ProbeProtocolHTTPS,
115+
RequestPath: to.StringPtr("/healthz"),
115116
Port: to.Int32Ptr(s.Scope.APIServerPort()),
116117
IntervalInSeconds: to.Int32Ptr(15),
117118
NumberOfProbes: to.Int32Ptr(4),

0 commit comments

Comments
 (0)