fix: delete appcahce dir with error (#100)

beclab · Nov 7, 2024 · 7866af8 · 7866af8
1 parent 25eef29
commit 7866af8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/controllers/appmgr_controller.go b/controllers/appmgr_controller.go
@@ -527,12 +527,13 @@ func (r *ApplicationManagerController) uninstall(ctx context.Context, appMgr *ap
 					for _, n := range nodes.Items {
 						URL := fmt.Sprintf(constants.AppDataDirURL, appMgr.Spec.AppOwner, dir)
 						c.SetHeader("X-Terminus-Node", n.Name)
+						c.SetHeader("x-bfl-user", appMgr.Spec.AppOwner)
 						res, e := c.R().Delete(URL)
 						if e != nil {
 							klog.Errorf("Failed to delete dir err=%v", e)
 						}
-						if res.StatusCode() == http.StatusOK {
-							klog.Info("Success to delete appcache dir=%s", dir)
+						if res.StatusCode() != http.StatusOK {
+							klog.Infof("delete app cache failed with: %v", res.String())
 						}
 					}
 				}

diff --git a/pkg/constants/constants.go b/pkg/constants/constants.go
@@ -86,7 +86,7 @@ const (
 
 	DependencyTypeSystem = "system"
 	DependencyTypeApp    = "application"
-	AppDataDirURL        = "http://files-service.user-space-%s/api/resources/AppData/%s"
+	AppDataDirURL        = "http://files-service.user-space-%s/api/resources/AppData/%s/"
 
 	UserSpaceDirKey   = "userspace_hostpath"
 	UserAppDataDirKey = "appcache_hostpath"