[Bug][Jenkins] ValidateTaskOptions not working as expected about incubator-devlake HOT 2 CLOSED

The issue you're encountering with Jenkins job imports failing for jobs not part of a folder is due to the logic in DevLake's ValidateTaskOptions function. Specifically, when a job's fullName attribute does not contain a /, the function sets the JobPath to view/all/, which is not a valid path for folderless jobs, leading to a 404 error.

Here's the relevant code snippet from ValidateTaskOptions:

if i := strings.LastIndex(op.JobFullName, `/`); i >= 0 {
    op.JobName = op.JobFullName[i+1:]
    op.JobPath = `job/` + strings.Join(strings.Split(op.JobFullName[:i], `/`), `/job/`)

    if op.Class == WORKFLOW_MULTI_BRANCH_PROJECT {
        op.JobPath = `view/all/` + op.JobPath
    }

} else {
    op.JobName = op.JobFullName
    op.JobPath = `view/all`
}

For jobs not part of a folder, the else block sets op.JobPath to view/all, which is likely causing the import to fail because the path view/all might not be valid for folderless jobs ^[1].

The historical context for setting the default JobPath to view/all/ is to provide a default view path for Jenkins jobs that are not nested within any folders. This ensures that even if a job is not part of a folder structure, it can still be accessed and managed under the view/all path, which is a common default view in Jenkins for listing all jobs ^[1].

To address this issue, you might consider modifying the else block to construct a valid JobPath similar to how it is done for jobs within folders. This would involve ensuring that the path is correctly set for folderless jobs to avoid the 404 error.

To continue talking to Dosu, mention @dosu.

from incubator-devlake.

There was an issue on our side since the view/all path was removed. After recreating it, the import finished successfully

from incubator-devlake.