Reduce cluster-lister-gen

Signed-off-by: Nelo-T. Wallus <[email protected]>

Author: ntnn

cmd/cluster-lister-gen/generators/lister.go

@@ -221,7 +221,6 @@ func (g *listerGenerator) Imports(_ *generator.Context) (imports []string) {
 		`kcplisters "github.com/kcp-dev/client-go/third_party/k8s.io/client-go/listers"`,
 		`"k8s.io/apimachinery/pkg/labels"`,
 		`"k8s.io/client-go/tools/cache"`,
-		`"k8s.io/client-go/listers"`,
 	)
 	return
 }
@@ -250,23 +249,6 @@ func (g *listerGenerator) GenerateType(c *generator.Context, t *types.Type, w io
 	}
 
 	sw.Do(typeClusterListerInterface, m)
-	sw.Do(typeListerStruct, m)
-
-	// no external interfaces provided, so we generate our own
-	if g.singleClusterListersPkg == "" {
-		sw.Do(typeListerInterface, m)
-	}
-
-	if !tags.NonNamespaced {
-		sw.Do(typeListerNamespaceLister, m)
-		sw.Do(namespaceListerStruct, m)
-
-		if g.singleClusterListersPkg == "" {
-			sw.Do(namespaceListerInterface, m)
-		}
-
-		sw.Do(namespaceListerConstructor, m)
-	}
 
 	sw.Do(scopedLister, m)
 
@@ -289,7 +271,6 @@ type $.type|public$ClusterLister interface {
 // $.type|private$ClusterLister implements the $.type|public$ClusterLister interface.
 type $.type|private$ClusterLister struct {
 	kcplisters.ResourceClusterIndexer[*$.type|raw$]
-	indexer cache.Indexer
 }
 
 var _ $.type|public$ClusterLister = new($.type|private$ClusterLister)
@@ -302,124 +283,41 @@ var _ $.type|public$ClusterLister = new($.type|private$ClusterLister)
 $if .namespaced -$
 // - has the kcpcache.ClusterAndNamespaceIndex as an index
 $end -$
-func New$.type|public$ClusterLister(indexer cache.Indexer) *$.type|private$ClusterLister {
+func New$.type|public$ClusterLister(indexer cache.Indexer) $.type|public$ClusterLister {
 	return &$.type|private$ClusterLister{
 		kcplisters.NewCluster[*$.type|raw$](indexer, $.Resource|raw$("$.type|lowercaseSingular$")),
-		indexer,
 	}
 }
 
-// Cluster scopes the lister to one workspace, allowing users to list and get $.type|publicPlural$.
 func (l *$.type|private$ClusterLister) Cluster(clusterName logicalcluster.Name) $.listerInterface|raw$ {
 	return &$.type|private$Lister{
-		kcplisters.New[*$.type|raw$](l.indexer, clusterName, $.Resource|raw$("$.type|lowercaseSingular$")),
-		l.indexer,
-		clusterName,
+		l.ResourceClusterIndexer.Cluster(clusterName),
 	}
 }
 `
 
-var typeListerInterface = `
-$if .namespaced -$
-// $.listerInterface|raw$ can list $.type|publicPlural$ across all namespaces, or scope down to a $.namespaceListerInterface|raw$ for one namespace.
-$else -$
-// $.listerInterface|raw$ can list all $.type|publicPlural$, or get one in particular.
-$end -$
-// All objects returned here must be treated as read-only.
-type $.listerInterface|raw$ interface {
-	// List lists all $.type|publicPlural$ in the indexer.
-	// Objects returned here must be treated as read-only.
-	List(selector labels.Selector) (ret []*$.type|raw$, err error)
-$if .namespaced -$
-	// $.type|publicPlural$ returns a lister that can list and get $.type|publicPlural$ in one workspace and namespace.
-	$.type|publicPlural$(namespace string) $.namespaceListerInterface|raw$
-$else -$
-	// Get retrieves the $.type|public$ from the indexer for a given workspace and name.
-	// Objects returned here must be treated as read-only.
-	Get(name string) (*$.type|raw$, error)
-$end -$
-	$.type|public$ListerExpansion
-}
-`
+var scopedLister = `var _ $.listerInterface|raw$ = new($.type|private$Lister)
 
-var typeListerStruct = `
 // $.type|private$Lister can list all $.type|publicPlural$ inside a workspace
-// or scope down to a $.namespaceListerInterface|raw$ for one namespace.
+// or scope down to a $.namespaceListerInterface|raw$$if .namespaced$ for one namespace$end$.
 type $.type|private$Lister struct {
 	kcplisters.ResourceIndexer[*$.type|raw$]
-	indexer     cache.Indexer
-	clusterName logicalcluster.Name
 }
 
-var _ $.listerInterface|raw$ = new($.type|private$Lister)
-`
-
-var typeListerNamespaceLister = `
-// $.type|publicPlural$ returns an object that can list and get $.type|publicPlural$ in one namespace.
-func (l *$.type|private$Lister) $.type|publicPlural$(namespace string) $.namespaceListerInterface|raw$ {
-	return new$.type|public$NamespaceLister(l.ResourceIndexer, namespace)
-}
-`
-
-var namespaceListerInterface = `
-// $.namespaceListerInterface|raw$ can list all $.type|publicPlural$, or get one in particular.
-// All objects returned here must be treated as read-only.
-type $.namespaceListerInterface|raw$ interface {
-	// List lists all $.type|publicPlural$ in the indexer.
-	// Objects returned here must be treated as read-only.
-	List(selector labels.Selector) (ret []*$.type|raw$, err error)
-	// Get retrieves the $.type|public$ from the indexer for a given workspace, namespace and name.
-	// Objects returned here must be treated as read-only.
-	Get(name string) (*$.type|raw$, error)
-	$.type|public$NamespaceListerExpansion
-}
-`
-
-var namespaceListerStruct = `
-// $.type|private$NamespaceLister implements the $.namespaceListerInterface|raw$
-// interface.
-type $.type|private$NamespaceLister struct {
-	kcplisters.ResourceIndexer[*$.type|raw$]
-}
-
-var _ $.namespaceListerInterface|raw$ = new($.type|private$NamespaceLister)
-`
-
-var namespaceListerConstructor = `
-// new$.type|public$NamespaceLister returns a new $.namespaceListerInterface|raw$.
-func new$.type|public$NamespaceLister(indexer kcplisters.ResourceIndexer[*$.type|raw$], namespace string) $.namespaceListerInterface|raw$ {
-	return &$.type|private$NamespaceLister{
-		kcplisters.NewNamespaced(indexer, namespace),
-	}
-}
-`
-
-var scopedLister = `
-// New$.type|public$Lister returns a new $.listerInterface|raw$.
-// We assume that the indexer:
-// - is fed by a workspace-scoped LIST+WATCH
-// - uses cache.MetaNamespaceKeyFunc as the key function
-$if .namespaced -$
-// - has the cache.NamespaceIndex as an index
-$end -$
-func New$.type|public$Lister(indexer cache.Indexer) $.listerInterface|raw$ {
-	return &$.type|private$ScopedLister{
-		listers.New[*$.type|raw$](indexer, $.Resource|raw$("$.type|lowercaseSingular$")),
-		indexer,
+// Cluster returns an object that can list and get $.type|publicPlural$
+// in one cluster.
+func (l *$.type|private$Lister) Cluster(clusterName logicalcluster.Name) $.listerInterface|raw$ {
+	return &$.type|private$Lister{
+		l.ResourceIndexer.Cluster(clusterName),
 	}
 }
 
-// $.type|private$ScopedLister can list all $.type|publicPlural$ inside a workspace
-// or scope down to a $.namespaceListerInterface|raw$$if .namespaced$ for one namespace$end$.
-type $.type|private$ScopedLister struct {
-	listers.ResourceIndexer[*$.type|raw$]
-	indexer cache.Indexer
-}
-
 $if .namespaced -$
 // $.type|publicPlural$ returns an object that can list and get $.type|publicPlural$ in one namespace.
-func (l *$.type|private$ScopedLister) $.type|publicPlural$(namespace string) $.namespaceListerInterface|raw$ {
-	return listers.NewNamespaced(l.ResourceIndexer, namespace)
+func (l *$.type|private$Lister) $.type|publicPlural$(namespace string) $.namespaceListerInterface|raw$ {
+	return &$.type|private$Lister{
+		l.ResourceIndexer.Namespace(namespace),
+	}
 }
 $end -$
 `