Skip to content

Commit 16cde44

Browse files
authored
Merge pull request kubernetes#81703 from RainbowMango/pr_fix_static_failure_for_staging_kube-aggregator
Fix static failure from package: staging/src/k8s.io/kube-aggregator
2 parents 3ae81af + 9b34fb0 commit 16cde44

File tree

5 files changed

+5
-17
lines changed

5 files changed

+5
-17
lines changed

hack/.staticcheck_failures

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -218,10 +218,6 @@ vendor/k8s.io/code-generator/cmd/client-gen/generators/fake
218218
vendor/k8s.io/code-generator/cmd/client-gen/generators/util
219219
vendor/k8s.io/code-generator/cmd/go-to-protobuf/protobuf
220220
vendor/k8s.io/component-base/metrics
221-
vendor/k8s.io/kube-aggregator/pkg/apis/apiregistration
222-
vendor/k8s.io/kube-aggregator/pkg/apis/apiregistration/v1/helper
223-
vendor/k8s.io/kube-aggregator/pkg/controllers/autoregister
224-
vendor/k8s.io/kube-aggregator/pkg/controllers/openapi/aggregator
225221
vendor/k8s.io/kubectl/pkg/cmd/annotate
226222
vendor/k8s.io/kubectl/pkg/cmd/certificates
227223
vendor/k8s.io/kubectl/pkg/cmd/config

staging/src/k8s.io/kube-aggregator/pkg/apis/apiregistration/helpers_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ import (
2424
var (
2525
a APIServiceConditionType = "A"
2626
b APIServiceConditionType = "B"
27-
c APIServiceConditionType = "C"
2827
)
2928

3029
func TestGetAPIServiceConditionByType(t *testing.T) {

staging/src/k8s.io/kube-aggregator/pkg/apis/apiregistration/v1/helper/helpers_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ import (
2626
var (
2727
a v1.APIServiceConditionType = "A"
2828
b v1.APIServiceConditionType = "B"
29-
c v1.APIServiceConditionType = "C"
3029
)
3130

3231
func TestIsAPIServiceConditionTrue(t *testing.T) {

staging/src/k8s.io/kube-aggregator/pkg/controllers/autoregister/autoregister_controller_test.go

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -51,15 +51,6 @@ func newAutoRegisterManagedModifiedAPIService(name string) *apiregistrationv1.AP
5151
}
5252
}
5353

54-
func newAutoRegisterManagedOnStartModifiedAPIService(name string) *apiregistrationv1.APIService {
55-
return &apiregistrationv1.APIService{
56-
ObjectMeta: metav1.ObjectMeta{Name: name, Labels: map[string]string{AutoRegisterManagedLabel: string("onstart")}},
57-
Spec: apiregistrationv1.APIServiceSpec{
58-
Group: "something",
59-
},
60-
}
61-
}
62-
6354
func newAPIService(name string) *apiregistrationv1.APIService {
6455
return &apiregistrationv1.APIService{
6556
ObjectMeta: metav1.ObjectMeta{Name: name},

staging/src/k8s.io/kube-aggregator/pkg/controllers/openapi/aggregator/aggregator.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,11 @@ func BuildAndRegisterAggregator(downloader *Downloader, delegationTarget server.
120120
}
121121

122122
// Install handler
123-
s.openAPIVersionedService, err = handler.RegisterOpenAPIVersionedService(
124-
specToServe, "/openapi/v2", pathHandler)
123+
s.openAPIVersionedService, err = handler.NewOpenAPIService(specToServe)
124+
if err != nil {
125+
return nil, err
126+
}
127+
err = s.openAPIVersionedService.RegisterOpenAPIVersionedService("/openapi/v2", pathHandler)
125128
if err != nil {
126129
return nil, err
127130
}

0 commit comments

Comments
 (0)