Skip to content

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

Merged
merged 13 commits into from
Sep 15, 2021

Conversation

vanajamukkara
Copy link
Member

@vanajamukkara vanajamukkara commented Sep 9, 2021

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

@vanajamukkara vanajamukkara changed the title Adding new certification profile and fixing the test failures to run on FMW jenkins Adding new certification profile and fixing the test failures on FMW pipeline jenkins Sep 9, 2021
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
Copy link
Member

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?

Copy link
Member Author

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

Copy link
Member

@anpanigr anpanigr left a comment

Choose a reason for hiding this comment

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

lgtm

@vanajamukkara vanajamukkara changed the title Adding new certification profile and fixing the test failures on FMW pipeline jenkins Modify certification profile and fixing the test failures on FMW pipeline jenkins Sep 14, 2021
Copy link
Member

@sankarpn sankarpn left a comment

Choose a reason for hiding this comment

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

lgtm

@rjeberhard rjeberhard merged commit a980e4a into main Sep 15, 2021
@rjeberhard rjeberhard deleted the new-profile branch January 31, 2022 14:29
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.

4 participants