Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI jekninsfile update hotfix #3038

Merged
merged 15 commits into from
Oct 28, 2024
+19 −19
Merged

CI jekninsfile update hotfix #3038

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
b25f382
removed catchError to see error with get_host_case_list script (also …
Oct 26, 2024
31fa9be
catchError is still being applied, but did fail, adding source gwsetu…
Oct 26, 2024
688e1ba
drilling down on real error that is acctually in create experment (me…
Oct 26, 2024
39c644b
system is a matrix control variable and could not be resloved in shel…
Oct 26, 2024
9b3c6e6
removed HOME_GFS where was using system which is a matrix control var…
Oct 26, 2024
cf6b48e
.system not found in readYaml object and is not even used
Oct 26, 2024
f32df3e
drilling down to error with system that looks like write to disk problem
Oct 26, 2024
e115db5
fixed issue with system name clash issue
Oct 26, 2024
df34b7d
fixed issue with system name clash issue and put build back in for fi…
Oct 26, 2024
8acaa62
took out extra source on gwsetup as was not the issue
Oct 26, 2024
4dc4b04
failed again when running run script, wtf
Oct 27, 2024
24b02e9
put create experment back
Oct 27, 2024
ae2170a
just adding more output for finding more info on sytem fail
Oct 27, 2024
02c7682
finaly found bug, system was no login in matrix when added refined pa…
Oct 27, 2024
7c75021
put build back in to after fixing system variable bug
Oct 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Loading
Loading status checks…
fixed issue with system name clash issue
Terry McGuinness committed Oct 26, 2024
commit e115db56e831526ebd609e3ea2cda0dbc8be4dca
30 changes: 13 additions & 17 deletions ci/Jenkinsfile
View file
Original file line number Diff line number Diff line change
@@ -194,24 +194,20 @@ pipeline {
def parallelStages = cases.collectEntries { caseName ->
["${caseName}": {
stage("Create ${caseName}") {
// catchError(buildResult: 'UNSTABLE', stageResult: 'FAILURE') {
script {
// env.HOME_GFS = HOMEgfs // setting path in HOMEgfs as an environment variable HOME_GFS for some systems that using the path in its .bashrc
sh(script: "sed -n '/{.*}/!p' ${CUSTOM_WORKSPACE}/gfs/ci/cases/pr/${caseName}.yaml > ${CUSTOM_WORKSPACE}/gfs/ci/cases/pr/${caseName}.yaml.tmp")
sh(script: "echo '}' >> ${CUSTOM_WORKSPACE}/gfs/ci/cases/pr/${caseName}.yaml.test.tmp")
sh(script: "ls -l ${CUSTOM_WORKSPACE}/gfs/ci/cases/pr")
def yaml_case = readYaml file: "${CUSTOM_WORKSPACE}/gfs/ci/cases/pr/${caseName}.yaml.tmp"
build_system = yaml_case.experiment.system
def HOMEgfs = "${CUSTOM_WORKSPACE}/${build_system}" // local HOMEgfs is used to populate the XML on per system basis
env.RUNTESTS = "${CUSTOM_WORKSPACE}/RUNTESTS"
try {
error_output = sh(script: "${HOMEgfs}/ci/scripts/utils/ci_utils_wrapper.sh create_experiment ${HOMEgfs}/ci/cases/pr/${caseName}.yaml", returnStdout: true).trim()
} catch (Exception error_create) {
sh(script: """${GH} pr comment ${env.CHANGE_ID} --repo ${repo_url} --body "${Case} **FAILED** to create experment on ${Machine} in BUILD# ${env.BUILD_NUMBER}\n with the error:\n\\`\\`\\`\n${error_output}\\`\\`\\`" """)
error("Case ${caseName} failed to create experment directory")
}
script {
sh(script: "sed -n '/{.*}/!p' ${CUSTOM_WORKSPACE}/gfs/ci/cases/pr/${caseName}.yaml > ${CUSTOM_WORKSPACE}/gfs/ci/cases/pr/${caseName}.yaml.tmp")
def yaml_case = readYaml file: "${CUSTOM_WORKSPACE}/gfs/ci/cases/pr/${caseName}.yaml.tmp"
build_system = yaml_case.experiment.system
def HOMEgfs = "${CUSTOM_WORKSPACE}/${build_system}" // local HOMEgfs is used to populate the XML on per system basis
env.HOME_GFS = HOMEgfs // setting path in HOMEgfs as an environment variable HOME_GFS for some systems that using the path in its .bashrc
env.RUNTESTS = "${CUSTOM_WORKSPACE}/RUNTESTS"
try {
error_output = sh(script: "${HOMEgfs}/ci/scripts/utils/ci_utils_wrapper.sh create_experiment ${HOMEgfs}/ci/cases/pr/${caseName}.yaml", returnStdout: true).trim()
} catch (Exception error_create) {
sh(script: """${GH} pr comment ${env.CHANGE_ID} --repo ${repo_url} --body "${Case} **FAILED** to create experment on ${Machine} in BUILD# ${env.BUILD_NUMBER}\n with the error:\n\\`\\`\\`\n${error_output}\\`\\`\\`" """)
error("Case ${caseName} failed to create experment directory")
}
//}
}
}

stage("Running ${caseName}") {