dry-run: Promote from Alpha to Beta
Also remove the test that used to check that the dry-run flag is forbidden. Kubernetes-commit: 5dca8e00e6d5501480244b8fd3e8aca54830042a
This commit is contained in:
parent
8615592431
commit
ab5f00eec9
|
@ -38,7 +38,6 @@ import (
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
"github.com/emicklei/go-restful"
|
"github.com/emicklei/go-restful"
|
||||||
|
|
||||||
fuzzer "k8s.io/apimachinery/pkg/api/apitesting/fuzzer"
|
fuzzer "k8s.io/apimachinery/pkg/api/apitesting/fuzzer"
|
||||||
apiequality "k8s.io/apimachinery/pkg/api/equality"
|
apiequality "k8s.io/apimachinery/pkg/api/equality"
|
||||||
apierrs "k8s.io/apimachinery/pkg/api/errors"
|
apierrs "k8s.io/apimachinery/pkg/api/errors"
|
||||||
|
@ -68,8 +67,11 @@ import (
|
||||||
"k8s.io/apiserver/pkg/endpoints/handlers/responsewriters"
|
"k8s.io/apiserver/pkg/endpoints/handlers/responsewriters"
|
||||||
"k8s.io/apiserver/pkg/endpoints/request"
|
"k8s.io/apiserver/pkg/endpoints/request"
|
||||||
genericapitesting "k8s.io/apiserver/pkg/endpoints/testing"
|
genericapitesting "k8s.io/apiserver/pkg/endpoints/testing"
|
||||||
|
"k8s.io/apiserver/pkg/features"
|
||||||
"k8s.io/apiserver/pkg/registry/rest"
|
"k8s.io/apiserver/pkg/registry/rest"
|
||||||
"k8s.io/apiserver/pkg/server/filters"
|
"k8s.io/apiserver/pkg/server/filters"
|
||||||
|
utilfeature "k8s.io/apiserver/pkg/util/feature"
|
||||||
|
utilfeaturetesting "k8s.io/apiserver/pkg/util/feature/testing"
|
||||||
)
|
)
|
||||||
|
|
||||||
type alwaysMutatingDeny struct{}
|
type alwaysMutatingDeny struct{}
|
||||||
|
@ -3865,7 +3867,9 @@ func (storage *SimpleRESTStorageWithDeleteCollection) DeleteCollection(ctx conte
|
||||||
return nil, nil
|
return nil, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestDryRun(t *testing.T) {
|
func TestDryRunDisabled(t *testing.T) {
|
||||||
|
defer utilfeaturetesting.SetFeatureGateDuringTest(t, utilfeature.DefaultFeatureGate, features.DryRun, false)()
|
||||||
|
|
||||||
tests := []struct {
|
tests := []struct {
|
||||||
path string
|
path string
|
||||||
verb string
|
verb string
|
||||||
|
|
|
@ -75,6 +75,7 @@ const (
|
||||||
|
|
||||||
// owner: @apelisse
|
// owner: @apelisse
|
||||||
// alpha: v1.12
|
// alpha: v1.12
|
||||||
|
// beta: v1.13
|
||||||
//
|
//
|
||||||
// Allow requests to be processed but not stored, so that
|
// Allow requests to be processed but not stored, so that
|
||||||
// validation, merging, mutation can be tested without
|
// validation, merging, mutation can be tested without
|
||||||
|
@ -96,5 +97,5 @@ var defaultKubernetesFeatureGates = map[utilfeature.Feature]utilfeature.FeatureS
|
||||||
APIResponseCompression: {Default: false, PreRelease: utilfeature.Alpha},
|
APIResponseCompression: {Default: false, PreRelease: utilfeature.Alpha},
|
||||||
Initializers: {Default: false, PreRelease: utilfeature.Alpha},
|
Initializers: {Default: false, PreRelease: utilfeature.Alpha},
|
||||||
APIListChunking: {Default: true, PreRelease: utilfeature.Beta},
|
APIListChunking: {Default: true, PreRelease: utilfeature.Beta},
|
||||||
DryRun: {Default: false, PreRelease: utilfeature.Alpha},
|
DryRun: {Default: true, PreRelease: utilfeature.Beta},
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue