Switch to GroupMetaFactory in federation and clientset testgroup

cmd/libs/go2idl/client-gen/test_apis/testgroup/install/install.go

@@ -19,104 +19,31 @@ limitations under the License.
 package install
 
 import (
-	"fmt"
+	"k8s.io/apimachinery/pkg/apimachinery/announced"
-
+	"k8s.io/apimachinery/pkg/apimachinery/registered"
-	"github.com/golang/glog"
-
-	"k8s.io/apimachinery/pkg/api/meta"
-	"k8s.io/apimachinery/pkg/apimachinery"
 	"k8s.io/apimachinery/pkg/runtime"
-	"k8s.io/apimachinery/pkg/runtime/schema"
-	"k8s.io/apimachinery/pkg/util/sets"
 	"k8s.io/kubernetes/cmd/libs/go2idl/client-gen/test_apis/testgroup"
 	"k8s.io/kubernetes/cmd/libs/go2idl/client-gen/test_apis/testgroup/v1"
 	"k8s.io/kubernetes/pkg/api"
 )
 
-const importPrefix = "k8s.io/kubernetes/cmd/libs/go2idl/client-gen/test_apis/testgroup"
-
-var accessor = meta.NewAccessor()
-
-const groupName = "testgroup.k8s.io"
-
-// availableVersions lists all known external versions for this group from most preferred to least preferred
-var availableVersions = []schema.GroupVersion{{Group: groupName, Version: "v1"}}
-
 func init() {
-	externalVersions := availableVersions
+	Install(api.GroupFactoryRegistry, api.Registry, api.Scheme)
-	api.Registry.RegisterVersions(availableVersions)
-
-	if err := api.Registry.EnableVersions(externalVersions...); err != nil {
-		glog.V(4).Infof("%v", err)
-		return
-	}
-	if err := enableVersions(externalVersions); err != nil {
-		glog.V(4).Infof("%v", err)
-		return
-	}
 }
 
-// TODO: enableVersions should be centralized rather than spread in each API
+// Install registers the API group and adds types to a scheme
-// group.
+func Install(groupFactoryRegistry announced.APIGroupFactoryRegistry, registry *registered.APIRegistrationManager, scheme *runtime.Scheme) {
-// We can combine api.Registry.RegisterVersions, api.Registry.EnableVersions and
+	if err := announced.NewGroupMetaFactory(
-// api.Registry.RegisterGroup once we have moved enableVersions there.
+		&announced.GroupMetaFactoryArgs{
-func enableVersions(externalVersions []schema.GroupVersion) error {
+			GroupName:                  testgroup.SchemeGroupVersion.Group,
-	addVersionsToScheme(externalVersions...)
+			VersionPreferenceOrder:     []string{v1.SchemeGroupVersion.Version},
-	preferredExternalVersion := externalVersions[0]
+			ImportPrefix:               "k8s.io/kubernetes/cmd/libs/go2idl/client-gen/test_apis/testgroup",
-
+			AddInternalObjectsToScheme: testgroup.AddToScheme,
-	groupMeta := apimachinery.GroupMeta{
+		},
-		GroupVersion:  preferredExternalVersion,
+		announced.VersionToSchemeFunc{
-		GroupVersions: externalVersions,
+			v1.SchemeGroupVersion.Version: v1.AddToScheme,
-		RESTMapper:    newRESTMapper(externalVersions),
+		},
-		SelfLinker:    runtime.SelfLinker(accessor),
+	).Announce(groupFactoryRegistry).RegisterAndEnable(registry, scheme); err != nil {
-		InterfacesFor: interfacesFor,
-	}
-
-	if err := api.Registry.RegisterGroup(groupMeta); err != nil {
-		return err
-	}
-	return nil
-}
-
-func newRESTMapper(externalVersions []schema.GroupVersion) meta.RESTMapper {
-	// the list of kinds that are scoped at the root of the api hierarchy
-	// if a kind is not enumerated here, it is assumed to have a namespace scope
-	rootScoped := sets.NewString()
-
-	ignoredKinds := sets.NewString()
-
-	return meta.NewDefaultRESTMapperFromScheme(externalVersions, interfacesFor, importPrefix, ignoredKinds, rootScoped, api.Scheme)
-}
-
-// InterfacesFor returns the default Codec and ResourceVersioner for a given version
-// string, or an error if the version is not known.
-func interfacesFor(version schema.GroupVersion) (*meta.VersionInterfaces, error) {
-	switch version {
-	case v1.SchemeGroupVersion:
-		return &meta.VersionInterfaces{
-			ObjectConvertor:  api.Scheme,
-			MetadataAccessor: accessor,
-		}, nil
-	default:
-		g, _ := api.Registry.Group(groupName)
-		return nil, fmt.Errorf("unsupported storage version: %s (valid: %v)", version, g.GroupVersions)
-	}
-}
-
-func addVersionsToScheme(externalVersions ...schema.GroupVersion) {
-	// add the internal version to Scheme
-	if err := testgroup.AddToScheme(api.Scheme); err != nil {
-		// Programmer error, detect immediately
 		panic(err)
 	}
-	// add the enabled external versions to Scheme
-	for _, v := range externalVersions {
-		switch v {
-		case v1.SchemeGroupVersion:
-			if err := v1.AddToScheme(api.Scheme); err != nil {
-				// Programmer error, detect immediately
-				panic(err)
-			}
-		}
-	}
 }

federation/apis/federation/install/install.go

@@ -17,117 +17,36 @@ limitations under the License.
 package install
 
 import (
-	"fmt"
+	"k8s.io/apimachinery/pkg/apimachinery/announced"
-
+	"k8s.io/apimachinery/pkg/apimachinery/registered"
-	"github.com/golang/glog"
-
-	"k8s.io/apimachinery/pkg/api/meta"
-	"k8s.io/apimachinery/pkg/apimachinery"
 	"k8s.io/apimachinery/pkg/runtime"
-	"k8s.io/apimachinery/pkg/runtime/schema"
 	"k8s.io/apimachinery/pkg/util/sets"
+	"k8s.io/kubernetes/pkg/api"
+
 	"k8s.io/kubernetes/federation/apis/federation"
 	"k8s.io/kubernetes/federation/apis/federation/v1beta1"
-	"k8s.io/kubernetes/pkg/api"
 )
 
-const importPrefix = "k8s.io/kubernetes/federation/apis/federation"
-
-var accessor = meta.NewAccessor()
-
-// availableVersions lists all known external versions for this group from most preferred to least preferred
-var availableVersions = []schema.GroupVersion{v1beta1.SchemeGroupVersion}
-
 func init() {
-	api.Registry.RegisterVersions(availableVersions)
+	Install(api.GroupFactoryRegistry, api.Registry, api.Scheme)
-	externalVersions := []schema.GroupVersion{}
-	for _, v := range availableVersions {
-		if api.Registry.IsAllowedVersion(v) {
-			externalVersions = append(externalVersions, v)
-		}
-	}
-	if len(externalVersions) == 0 {
-		glog.V(4).Infof("No version is registered for group %v", federation.GroupName)
-		return
-	}
-
-	if err := api.Registry.EnableVersions(externalVersions...); err != nil {
-		glog.V(4).Infof("%v", err)
-		return
-	}
-	if err := enableVersions(externalVersions); err != nil {
-		glog.V(4).Infof("%v", err)
-		return
-	}
 }
 
-// TODO: enableVersions should be centralized rather than spread in each API
+// Install registers the API group and adds types to a scheme
-// group.
+func Install(groupFactoryRegistry announced.APIGroupFactoryRegistry, registry *registered.APIRegistrationManager, scheme *runtime.Scheme) {
-// We can combine api.Registry.RegisterVersions, api.Registry.EnableVersions and
+	if err := announced.NewGroupMetaFactory(
-// api.Registry.RegisterGroup once we have moved enableVersions there.
+		&announced.GroupMetaFactoryArgs{
-func enableVersions(externalVersions []schema.GroupVersion) error {
+			GroupName:                  federation.GroupName,
-	addVersionsToScheme(externalVersions...)
+			VersionPreferenceOrder:     []string{v1beta1.SchemeGroupVersion.Version},
-	preferredExternalVersion := externalVersions[0]
+			ImportPrefix:               "k8s.io/kubernetes/federation/apis/federation",
-
+			AddInternalObjectsToScheme: federation.AddToScheme,
-	groupMeta := apimachinery.GroupMeta{
+			RootScopedKinds: sets.NewString(
-		GroupVersion:  preferredExternalVersion,
-		GroupVersions: externalVersions,
-		RESTMapper:    newRESTMapper(externalVersions),
-		SelfLinker:    runtime.SelfLinker(accessor),
-		InterfacesFor: interfacesFor,
-	}
-
-	if err := api.Registry.RegisterGroup(groupMeta); err != nil {
-		return err
-	}
-	return nil
-}
-
-func newRESTMapper(externalVersions []schema.GroupVersion) meta.RESTMapper {
-	// the list of kinds that are scoped at the root of the api hierarchy
-	// if a kind is not enumerated here, it is assumed to have a namespace scope
-	rootScoped := sets.NewString(
 				"Cluster",
-	)
+			),
-
+		},
-	ignoredKinds := sets.NewString()
+		announced.VersionToSchemeFunc{
-
+			v1beta1.SchemeGroupVersion.Version: v1beta1.AddToScheme,
-	return meta.NewDefaultRESTMapperFromScheme(externalVersions, interfacesFor, importPrefix, ignoredKinds, rootScoped, api.Scheme)
+		},
-}
+	).Announce(groupFactoryRegistry).RegisterAndEnable(registry, scheme); err != nil {
-
-// interfacesFor returns the default Codec and ResourceVersioner for a given version
-// string, or an error if the version is not known.
-func interfacesFor(version schema.GroupVersion) (*meta.VersionInterfaces, error) {
-	switch version {
-	case v1beta1.SchemeGroupVersion:
-		return &meta.VersionInterfaces{
-			ObjectConvertor:  api.Scheme,
-			MetadataAccessor: accessor,
-		}, nil
-	default:
-		g, _ := api.Registry.Group(federation.GroupName)
-		return nil, fmt.Errorf("unsupported storage version: %s (valid: %v)", version, g.GroupVersions)
-	}
-}
-
-func addVersionsToScheme(externalVersions ...schema.GroupVersion) {
-	// add the internal version to Scheme
-	if err := federation.AddToScheme(api.Scheme); err != nil {
-		// Programmer error, detect immediately
 		panic(err)
 	}
-	// add the enabled external versions to Scheme
-	for _, v := range externalVersions {
-		if !api.Registry.IsEnabledVersion(v) {
-			glog.Errorf("Version %s is not enabled, so it will not be added to the Scheme.", v)
-			continue
-		}
-		switch v {
-		case v1beta1.SchemeGroupVersion:
-			if err := v1beta1.AddToScheme(api.Scheme); err != nil {
-				// Programmer error, detect immediately
-				panic(err)
-			}
-		}
-	}
 }

federation/apis/federation/install/install_test.go

@@ -25,9 +25,11 @@ import (
 	"k8s.io/kubernetes/federation/apis/federation"
 	"k8s.io/kubernetes/federation/apis/federation/v1beta1"
 	"k8s.io/kubernetes/pkg/api"
+	"k8s.io/apimachinery/pkg/api/meta"
 )
 
 func TestResourceVersioner(t *testing.T) {
+	accessor := meta.NewAccessor()
 	cluster := federation.Cluster{ObjectMeta: metav1.ObjectMeta{ResourceVersion: "10"}}
 	version, err := accessor.ResourceVersion(&cluster)
 	if err != nil {