-
Kubernetes Submit Queue authored
Automatic merge from submit-queue (batch tested with PRs 42617, 43247, 43509, 43644, 43820) [GCE] Support legacy-https and generic health checks **What this PR does / why we need it**: - Adds CRUD functions to manage `compute.HttpsHealthChecks` The legacy HTTPS healthchecks will be used by the GLBC (GCE Load balancer Controller) - Adds CRUD functions to manage `compute.HealthChecks` These are required for the internal load balancer - Removes the logic that disregards NotFound errors on DeleteHttpHealthChecks as this is useful information for callers. Here are the three known invocations within kubernetes: [gce/gce_loadbalancer.go#L457](https://github.com/kubernetes/kubernetes/blob/bc6e77d42fdb40bb82796fd8ee8ba2b04dd9a0ea/pkg/cloudprovider/providers/gce/gce_loadbalancer.go#L457): Only prints warning that HC wasn't deleted -> acceptable [gce/gce_loadbalancer.go#L465](https://github.com/kubernetes/kubernetes/blob/bc6e77d42fdb40bb82796fd8ee8ba2b04dd9a0ea/pkg/cloudprovider/providers/gce/gce_loadbalancer.go#L465): Err is ignored if not nil -> acceptable [e2e/framework/ingress_utils.go#L530](https://github.com/kubernetes/kubernetes/blob/bc6e77d42fdb40bb82796fd8ee8ba2b04dd9a0ea/test/e2e/framework/ingress_utils.go#L530): Already checks if is NotFound error -> acceptable **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: Step one of https://github.com/kubernetes/ingress/issues/494 Step one of #33483 **Special notes for your reviewer**: **Release note**: ```release-note NONE ```
289ef624