-
Notifications
You must be signed in to change notification settings - Fork 218
Modify certification profile and fixing the test failures on FMW pipeline jenkins #2528
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.
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
Conversation
kindtest.sh
Outdated
@@ -308,7 +308,7 @@ if [ ${test_filter} != "**/It*" ]; then | |||
echo "Running mvn -Dit.test=${test_filter} -Dwdt.download.url=${wdt_download_url} -Dwit.download.url=${wit_download_url} -Dwle.download.url=${wle_download_url} -DPARALLEL_CLASSES=${parallel_run} -DNUMBER_OF_THREADS=${threads} -pl integration-tests -P ${maven_profile_name} verify" | |||
time mvn -Dit.test="${test_filter}" -Dwdt.download.url="${wdt_download_url}" -Dwit.download.url="${wit_download_url}" -Dwle.download.url="${wle_download_url}" -DPARALLEL_CLASSES="${parallel_run}" -DNUMBER_OF_THREADS="${threads}" -pl integration-tests -P ${maven_profile_name} verify 2>&1 | tee "${RESULT_ROOT}/kindtest.log" || captureLogs | |||
else | |||
if [ "${maven_profile_name}" = "wls-image-cert" ] || [ "${maven_profile_name}" = "fmw-image-cert" ] || [ "${maven_profile_name}" = "kind-sequential" ]; then | |||
if [ "${maven_profile_name}" = "wls-image-cert" ] || [ "${maven_profile_name}" = "new-wls-image-cert" ] || [ "${maven_profile_name}" = "fmw-image-cert" ] || [ "${maven_profile_name}" = "kind-sequential" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the wls-cert-image still there? why not have just one cert image profile?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modified to have one profile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Modify certification profile
fixing the test failures to run on FMW pipeline jenkins
Adding checks for kubectl download in script
FMW Jenkins results -
Sequential run - https://ci-cloud.****/jenkins/wlsqa/job/wko-tmp/121/ - 1 failure, not related to changes in this PR
WKO Jenkins -
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/6321/ - 5 failures, not related to changes in this PR