Merge pull request '修复多文件notebook环境记录的描述，boot_file等超过数据库字符长度限制的情况' (#3476) from file-notebook-new into V20221228

Reviewed-on: https://openi.pcl.ac.cn/OpenI/aiforge/pulls/3476
Reviewed-by: zouap <zouap@pcl.ac.cn>

options/locale/locale_en-US.ini

@@ -1063,6 +1063,8 @@ model_rename=Duplicate model name, please modify model name.
 
 notebook_file_not_exist=Notebook file does not exist.
 notebook_select_wrong=Please select a Notebook(.ipynb) file first.
+notebook_path_too_long=The total length of selected file or files path exceed 255 characters, please select a shorter path file or change the file path.
+notebook_branch_name_too_long=The total  length of branch or branches name exceed 255 characters, please select a file in other branch.
 notebook_file_no_right=You have no right to access the Notebook(.ipynb) file.
 notebook_repo_conflict=The files in different branches of the same repository can not run together.
 debug_again_fail=Fail to restart debug task, please try again later.

options/locale/locale_zh-CN.ini

@@ -1062,6 +1062,8 @@ model_rename=模型名称重复，请修改模型名称
 
 notebook_file_not_exist=Notebook文件不存在。
 notebook_select_wrong=请先选择Notebook（.ipynb）文件。
+notebook_path_too_long=选择的一个或多个Notebook文件路径总长度超过255个字符，请选择路径较短的文件或调整文件路径。
+notebook_branch_name_too_long=选择的一个或多个Notebook文件分支名总长度超过255个字符，请选择其他分支的文件。
 notebook_file_no_right=您没有这个Notebook文件的读权限。
 notebook_repo_conflict=同一个仓库的不同分支文件不能同时运行。
 debug_again_fail=再次调试失败，请稍后再试。

services/cloudbrain/cloudbrainTask/notebook.go

@@ -46,6 +46,14 @@ func FileNotebookCreate(ctx *context.Context, option api.CreateFileNotebookJobOp
 		ctx.JSON(http.StatusOK, models.BaseErrorMessageApi(ctx.Tr("repo.notebook_select_wrong")))
 		return
 	}
+	if len(getBootFile(option.File, option.OwnerName, option.ProjectName)) > 255 {
+		ctx.JSON(http.StatusOK, models.BaseErrorMessageApi(ctx.Tr("repo.notebook_path_too_long")))
+		return
+	}
+	if len(option.BranchName) > 255 {
+		ctx.JSON(http.StatusOK, models.BaseErrorMessageApi(ctx.Tr("repo.notebook_branch_name_too_long")))
+		return
+	}
 
 	isNotebookFileExist, _ := isNoteBookFileExist(ctx, option)
 	if !isNotebookFileExist {
@@ -105,14 +113,29 @@ func FileNotebookCreate(ctx *context.Context, option api.CreateFileNotebookJobOp
 					err = downloadCode(sourceRepo, getCodePath(noteBook.JobName, sourceRepo), option.BranchName)
 					if err != nil {
 						log.Error("download code failed", err)
-						ctx.JSON(http.StatusOK, models.BaseErrorMessageApi(ctx.Tr("cloudbrain.load_code_failed")))
-						return
+						if !strings.Contains(err.Error(), "already exists and is not an empty directory") {
+							ctx.JSON(http.StatusOK, models.BaseErrorMessageApi(ctx.Tr("cloudbrain.load_code_failed")))
+							return
+						}
 					}
 				}
 				if !isRepoFileMatch(option, noteBook) {
-					noteBook.BootFile += ";" + getBootFile(option.File, option.OwnerName, option.ProjectName)
-					noteBook.BranchName += ";" + option.BranchName
-					noteBook.Description += ";" + getDescription(option)
+					if len(noteBook.BootFile)+len(getBootFile(option.File, option.OwnerName, option.ProjectName))+1 <= 255 {
+						noteBook.BootFile += ";" + getBootFile(option.File, option.OwnerName, option.ProjectName)
+					} else {
+						ctx.JSON(http.StatusOK, models.BaseErrorMessageApi(ctx.Tr("cloudbrain.notebook_path_too_long")))
+						return
+					}
+					if len(noteBook.BranchName)+len(option.BranchName)+1 <= 255 {
+						noteBook.BranchName += ";" + option.BranchName
+					} else {
+						ctx.JSON(http.StatusOK, models.BaseErrorMessageApi(ctx.Tr("cloudbrain.notebook_branch_name_too_long")))
+						return
+					}
+
+					if len(noteBook.Description)+len(getDescription(option))+1 <= 256 {
+						noteBook.Description += ";" + getDescription(option)
+					}
 
 					err := models.UpdateJob(noteBook)
 					if err != nil {