Merge pull request #25266 from smarterclayton/common_storage
Automatic merge from submit-queue kube-apiserver options should be decoupled from impls A few months ago we refactored options to keep it independent of the implementations, so that it could be used in CLI tools to validate config or to generate config, without pulling in the full dependency tree of the master. This change restores that by separating server_run_options.go back to its own package. Also, options structs should never contain non-serializable types, which storagebackend.Config was doing with runtime.Codec. Split the codec out. Fix a typo on the name of the etcd2.go storage backend. Finally, move DefaultStorageMediaType to server_run_options. @nikhiljindal as per my comment in #24454, @liggitt because you and I discussed this last time
This commit is contained in:
commit
930b028d96
|
|
@ -27,6 +27,7 @@ import (
|
|||
"k8s.io/kubernetes/pkg/api/unversioned"
|
||||
"k8s.io/kubernetes/pkg/apimachinery/registered"
|
||||
"k8s.io/kubernetes/pkg/genericapiserver"
|
||||
genericoptions "k8s.io/kubernetes/pkg/genericapiserver/options"
|
||||
"k8s.io/kubernetes/pkg/storage/storagebackend"
|
||||
|
||||
// Install the testgroup API
|
||||
|
|
@ -41,7 +42,7 @@ const (
|
|||
|
||||
func newStorageFactory() genericapiserver.StorageFactory {
|
||||
config := storagebackend.Config{
|
||||
Prefix: genericapiserver.DefaultEtcdPathPrefix,
|
||||
Prefix: genericoptions.DefaultEtcdPathPrefix,
|
||||
ServerList: []string{"http://127.0.0.1:4001"},
|
||||
}
|
||||
storageFactory := genericapiserver.NewDefaultStorageFactory(config, "application/json", api.Codecs, genericapiserver.NewDefaultResourceEncodingConfig(), genericapiserver.NewResourceConfig())
|
||||
|
|
@ -49,13 +50,13 @@ func newStorageFactory() genericapiserver.StorageFactory {
|
|||
return storageFactory
|
||||
}
|
||||
|
||||
func NewServerRunOptions() *genericapiserver.ServerRunOptions {
|
||||
serverOptions := genericapiserver.NewServerRunOptions()
|
||||
func NewServerRunOptions() *genericoptions.ServerRunOptions {
|
||||
serverOptions := genericoptions.NewServerRunOptions()
|
||||
serverOptions.InsecurePort = InsecurePort
|
||||
return serverOptions
|
||||
}
|
||||
|
||||
func Run(serverOptions *genericapiserver.ServerRunOptions) error {
|
||||
func Run(serverOptions *genericoptions.ServerRunOptions) error {
|
||||
// Set ServiceClusterIPRange
|
||||
_, serviceClusterIPRange, _ := net.ParseCIDR("10.0.0.0/24")
|
||||
serverOptions.ServiceClusterIPRange = *serviceClusterIPRange
|
||||
|
|
|
|||
Loading…
Reference in New Issue