Don't fail the build when AP logs upload fails (#5849)
* Fix retry of failures from catch block Signed-off-by: Shirang Jia <shiranj@amazon.com> * Make the build unstable when AP load fails Signed-off-by: Shirang Jia <shiranj@amazon.com> * Remove test codes Signed-off-by: Shirang Jia <shiranj@amazon.com>
This commit is contained in:
Vendored
+6
-4
@@ -451,10 +451,12 @@ def ArchiveArtifactsOnS3(String artifactsSource, String s3Prefix="", boolean rec
|
||||
}
|
||||
|
||||
def UploadAPLogs(String platformName, String jobName, String workspace, Map params) {
|
||||
dir("${workspace}/${ENGINE_REPOSITORY_NAME}") {
|
||||
projects = params.CMAKE_LY_PROJECTS.split(",")
|
||||
projects.each{ project ->
|
||||
ArchiveArtifactsOnS3("${project}/user/log", "ap_logs/${platformName}/${jobName}/${project}", true)
|
||||
catchError(message: "Error archiving AssetProcessor logs (this won't fail the build)", buildResult: 'UNSTABLE', stageResult: 'FAILURE') {
|
||||
dir("${workspace}/${ENGINE_REPOSITORY_NAME}") {
|
||||
projects = params.CMAKE_LY_PROJECTS.split(",")
|
||||
projects.each{ project ->
|
||||
ArchiveArtifactsOnS3("${project}/user/log", "ap_logs/${platformName}/${jobName}/${project}", true)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user