chore: rollback in case of manifest file update fails

RRashmit · RRashmit · commit 5702ac6a9792 · 2025-03-12T11:59:58.000Z
diff --git a/internal/file/file_manager_service.go b/internal/file/file_manager_service.go
@@ -317,7 +317,7 @@ func (fms *FileManagerService) ConfigApply(ctx context.Context,
 	fms.UpdateCurrentFilesOnDisk(files.ConvertToMapOfFiles(fileOverview.GetFiles()))
 	manifestFileErr := fms.UpdateManifestFile(files.ConvertToMapOfFiles(fileOverview.GetFiles()))
 	if manifestFileErr != nil {
-		return model.Error, manifestFileErr
+		return model.RollbackRequired, manifestFileErr
 	}
 
 	return model.OK, nil

Original file line number	Diff line number	Diff line change
`@@ -317,7 +317,7 @@ func (fms *FileManagerService) ConfigApply(ctx context.Context,`
`317`	`317`	`fms.UpdateCurrentFilesOnDisk(files.ConvertToMapOfFiles(fileOverview.GetFiles()))`
`318`	`318`	`manifestFileErr := fms.UpdateManifestFile(files.ConvertToMapOfFiles(fileOverview.GetFiles()))`
`319`	`319`	`if manifestFileErr != nil {`
`320`		`- return model.Error, manifestFileErr`
	`320`	`+ return model.RollbackRequired, manifestFileErr`
`321`	`321`	`}`
`322`	`322`
`323`	`323`	`return model.OK, nil`