feat: rename cache methods

Signed-off-by: Matthis Holleville <matthish29@gmail.com>
k8sgpt-ai · May 25, 2023 · 470ea99 · 470ea99
1 parent 6bc94b8
commit 470ea99
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/cmd/cache/add.go b/cmd/cache/add.go
@@ -36,7 +36,7 @@ var addCmd = &cobra.Command{
 	- S3`,
 	Run: func(cmd *cobra.Command, args []string) {
 		fmt.Println(color.YellowString("Adding remote S3 based cache"))
-		err := cache.AddCache(bucketname, region)
+		err := cache.AddRemoteCache(bucketname, region)
 		if err != nil {
 			color.Red("Error: %v", err)
 			os.Exit(1)

diff --git a/pkg/cache/cache.go b/pkg/cache/cache.go
@@ -42,7 +42,7 @@ func RemoteCacheEnabled() (bool, error) {
 	return false, nil
 }
 
-func AddCache(bucketName string, region string) error {
+func AddRemoteCache(bucketName string, region string) error {
 	var cacheInfo CacheProvider
 	err := viper.UnmarshalKey("cache", &cacheInfo)
 	if err != nil {
@@ -61,7 +61,7 @@ func AddCache(bucketName string, region string) error {
 	return nil
 }
 
-func RemoveCache(bucketName string) error {
+func RemoveRemoteCache(bucketName string) error {
 	var cacheInfo CacheProvider
 	err := viper.UnmarshalKey("cache", &cacheInfo)
 	if err != nil {

diff --git a/pkg/server/config.go b/pkg/server/config.go
@@ -14,7 +14,7 @@ func (h *handler) AddConfig(ctx context.Context, i *schemav1.AddConfigRequest) (
 		return nil, errors.New("BucketName & Region are required")
 	}
 
-	err := cache.AddCache(i.Cache.BucketName, i.Cache.Region)
+	err := cache.AddRemoteCache(i.Cache.BucketName, i.Cache.Region)
 	if err != nil {
 		return &schemav1.AddConfigResponse{}, err
 	}
@@ -26,7 +26,7 @@ func (h *handler) AddConfig(ctx context.Context, i *schemav1.AddConfigRequest) (
 
 func (h *handler) RemoveConfig(ctx context.Context, i *schemav1.RemoveConfigRequest) (*schemav1.RemoveConfigResponse, error,
 ) {
-	err := cache.RemoveCache(i.Cache.BucketName)
+	err := cache.RemoveRemoteCache(i.Cache.BucketName)
 	if err != nil {
 		return &schemav1.RemoveConfigResponse{}, err
 	}