feat: Define a unified storage interface for storage backend integrat…

…ion (higress-group#55)
CH3CHO · Jan 31, 2024 · 440ce7c · 440ce7c
1 parent e5ea75f
commit 440ce7c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/src/apiserver/pkg/registry/file_rest.go b/src/apiserver/pkg/registry/file_rest.go
@@ -53,7 +53,7 @@ func NewFileREST(
 	newFunc func() runtime.Object,
 	newListFunc func() runtime.Object,
 	attrFunc storage.AttrFunc,
-) (rest.Storage, error) {
+) (REST, error) {
 	if attrFunc == nil {
 		if isNamespaced {
 			if isNamespaced {

diff --git a/src/apiserver/pkg/registry/nacos_rest.go b/src/apiserver/pkg/registry/nacos_rest.go
@@ -62,7 +62,7 @@ func NewNacosREST(
 	newListFunc func() runtime.Object,
 	attrFunc storage.AttrFunc,
 	dataEncryptionKey []byte,
-) rest.Storage {
+) REST {
 	if attrFunc == nil {
 		if isNamespaced {
 			if isNamespaced {

diff --git a/src/apiserver/pkg/registry/registry.go b/src/apiserver/pkg/registry/registry.go
@@ -21,9 +21,13 @@ import (
 	"k8s.io/apiserver/pkg/registry/rest"
 )
 
-// REST implements a RESTStorage for API services against etcd
-type REST struct {
+// REST defines a storage interface for integrating different storage backends with api server.
+type REST interface {
 	rest.Storage
+	rest.StandardStorage
+	rest.SingularNameProvider
+	rest.Scoper
+	rest.TableConvertor
 }
 
 // RESTInPeace is just a simple function that panics on error.