Skip to content

Commit cc19422

Browse files
authored
SRE-3029 ci: NLT tests failure in recordIssues (#469)
* SRE-3029 ci: NLT tests failure in recordIssues recordIssues expect scm value otherwise it uses random repository (as multiply SCM are used in job) Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com> --------- Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
1 parent ea2924b commit cc19422

File tree

1 file changed

+14
-13
lines changed

1 file changed

+14
-13
lines changed

vars/unitTestPost.groovy

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -15,20 +15,20 @@
1515
* config['testResults'] Junit test result files.
1616
* Default 'test_results/*.xml'
1717
*
18-
* config['valgrind_pattern'] Pattern for Valgind files.
18+
* config['valgrind_pattern'] Pattern for Valgrind files.
1919
* Default: '*.memcheck.xml'
2020
*
21-
* config['valgrind_stash'] Name to stash valgrind artifacts
21+
* config['valgrind_stash'] Name to stash Valgrind artifacts
2222
* Required if more than one stage is
23-
* creating valgrind reports.
23+
* creating Valgrind reports.
2424
*/
2525

2626
// groovylint-disable DuplicateStringLiteral, VariableName
2727
void call(Map config = [:]) {
2828
Map stage_info = parseStageInfo(config)
2929
String cbcResult = currentBuild.currentResult
3030

31-
// Stash the valgrind files for later analysis
31+
// Stash the Valgrind files for later analysis
3232
String valgrind_pattern = stage_info.get('valgrind_pattern',
3333
'unit-test-*memcheck.xml')
3434
if (config['valgrind_stash']) {
@@ -96,11 +96,10 @@ void call(Map config = [:]) {
9696

9797
if (stage_info['NLT']) {
9898
String cb_result = currentBuild.result
99-
discoverGitReferenceBuild(
100-
referenceJob: config.get('referenceJobName',
101-
'daos-stack/daos/master'),
102-
scm: 'daos-stack/daos',
103-
requiredResult: 'UNSTABLE')
99+
discoverGitReferenceBuild(referenceJob: config.get('referenceJobName',
100+
'daos-stack/daos/master'),
101+
scm: 'daos-stack/daos',
102+
requiredResult: 'UNSTABLE')
104103
recordIssues enabledForFailure: true,
105104
/* ignore warning/errors from PMDK logging system */
106105
filters: [excludeFile('pmdk/.+')],
@@ -114,10 +113,12 @@ void call(Map config = [:]) {
114113
[threshold: 1, type: 'TOTAL_HIGH'],
115114
[threshold: 1, type: 'NEW_NORMAL', unstable: true],
116115
[threshold: 1, type: 'NEW_LOW', unstable: true]],
117-
name: 'Node local testing',
118-
tool: issues(pattern: 'vm_test/nlt-errors.json',
119-
name: 'NLT results',
120-
id: 'VM_test')
116+
name: 'Node local testing',
117+
tool: issues(pattern: 'vm_test/nlt-errors.json',
118+
name: 'NLT results',
119+
id: 'VM_test'),
120+
scm: 'daos-stack/daos'
121+
121122
if (cb_result != currentBuild.result) {
122123
println(
123124
"The recordIssues step changed result to ${currentBuild.result}.")

0 commit comments

Comments
 (0)