Skip to content

ojdk 25 does not do parallel Install anymore #1

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

Open
andrlos wants to merge 3 commits into
base: main
Choose a base branch
from
Open

ojdk 25 does not do parallel Install anymore #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1f85eb4
Update parallel_install_test.sh
andrlos Mar 4, 2026
cec46f3
Update README.md
andrlos Mar 4, 2026
9bd44ff
Update parallel_install_test.sh
andrlos Mar 4, 2026
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
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@ Parallel Install Test for rpms
For testing parallel install ability of legacy RH builds.

## How to run
User needs to first export BUILD_OS_VERSION (number) and BUILD_OS_NAME (el or f) for the test to work.
User needs to first export BUILD_OS_VERSION (number), BUILD_OS_NAME (el or f) and JDK_VERSION (just the numeric value of major version) for the test to work.
Execution requires two arguments for two folders of rpms we want to install. The get installed in the order of the arguments. Running the test should then be just a simple execution of the shell script.
User can also influence where the produced logs will be stored by exporting TMPRESULTS variable.
15 changes: 13 additions & 2 deletions parallel_install_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,10 +61,21 @@ function run_parallel_test {

RUNNING=false
Copy link
Copy Markdown

@judovana judovana Mar 5, 2026

Choose a reason for hiding this comment

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

pls add comment that all fedoras have it removed. As it is missing in conditions, it can be confusing.

Copy link
Copy Markdown

@judovana judovana Mar 5, 2026

Choose a reason for hiding this comment

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

Otherwise lkooks good now,. TY!


if [[ ( x${BUILD_OS_NAME} == xel && ${BUILD_OS_VERSION} > 6 ) || x${BUILD_OS_NAME} == xf ]]; then
# parallel install is on el7-9
if [[ x${BUILD_OS_NAME} == xel && ${BUILD_OS_VERSION} -gt 6 && ${BUILD_OS_VERSION} -lt 10 ]]; then
RUNNING=true
fi

# not on el10 for ojdk21
if [[ ${BUILD_OS_VERSION} -ge 10 && ${JDK_VERSION} -ge 21 ]]; then
RUNNING=false
fi

# not on ojdk25 at all
if [[ ${JDK_VERSION} -ge 25 ]]; then
RUNNING=false
fi

FAILED=0
PASSED=0
IGNORED=0
Expand All @@ -86,7 +97,7 @@ fi
echo "" > $TMPRESULTS/parallel_install_log.txt

let "PASSED+=1"
echo "Appending dummy test, so there is at elast one test always running in suite" >> $TMPRESULTS/parallel_install_log.txt
echo "Appending dummy test, so there is at least one test always running in suite" >> $TMPRESULTS/parallel_install_log.txt
TEST=$(printXmlTest "tps" "dummyTestToPrventTotalFailure" "0" "" "")
BODY+="$TEST
" # new line to improve clarity, also is used in TPS/tesultsToJtregs.sh
Expand Down