Skip to content

Adapt pipeline-lib to the latest el9.7 Jenkins file#496

Open
osalyk wants to merge 5 commits intomasterfrom
osalyk/SRE-3636
Open

Adapt pipeline-lib to the latest el9.7 Jenkins file#496
osalyk wants to merge 5 commits intomasterfrom
osalyk/SRE-3636

Conversation

@osalyk
Copy link
Copy Markdown
Contributor

@osalyk osalyk commented Mar 12, 2026

No description provided.

Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Copy link
Copy Markdown
Contributor

@grom72 grom72 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -354,7 +353,6 @@
case 'NLT on EL 8':
case 'NLT on EL 8.8':
case 'NLT on EL 9':
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
case 'NLT on EL 9':

@@ -370,9 +368,7 @@
case 'Unit Test with memcheck on EL 8':
case 'Unit Test with memcheck on EL 8.8':
case 'Unit Test with memcheck on EL 9':
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
case 'Unit Test with memcheck on EL 9':

case 'Unit Test with memcheck on EL 8.8':
case 'Unit Test with memcheck on EL 9':
case 'Unit Test with memcheck on EL 9.7':
case 'Unit Test bdev with memcheck on EL 9':
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
case 'Unit Test bdev with memcheck on EL 9':
case 'Unit Test bdev with memcheck':

@@ -384,9 +380,7 @@
case 'Unit Test bdev on EL 8':
case 'Unit Test bdev on EL 8.8':
case 'Unit Test on EL 9':
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
case 'Unit Test on EL 9':

case 'Unit Test bdev on EL 8.8':
case 'Unit Test on EL 9':
case 'Unit Test on EL 9.7':
case 'Unit Test bdev on EL 9':
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
case 'Unit Test bdev on EL 9':
case 'Unit Test bdev':

@@ -426,14 +420,12 @@
case 'Functional on EL 8.8 with Valgrind':
return skip_ftest_valgrind('el8', target_branch, tags)
case 'Functional on EL 9 with Valgrind':
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
case 'Functional on EL 9 with Valgrind':

return skip_ftest_valgrind('el8', target_branch, tags)
case 'Functional on EL 9 with Valgrind':
case 'Functional on EL 9.7 with Valgrind':
return skip_ftest_valgrind('el9', target_branch, tags)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
return skip_ftest_valgrind('el9', target_branch, tags)

skip_stage_pragma('func-test') ||
skip_stage_pragma('func-test-vm') ||
stageAlreadyPassed()
case 'Fault injection testing on EL 9':
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
case 'Fault injection testing on EL 9':
case 'Fault injection testing':

@osalyk osalyk marked this pull request as ready for review March 19, 2026 08:48
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
@osalyk osalyk requested review from jzarzycki and ryon-jensen March 31, 2026 07:46
grom72
grom72 previously approved these changes Apr 8, 2026
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@grom72 grom72 self-assigned this Apr 8, 2026
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants