-
Notifications
You must be signed in to change notification settings - Fork 121
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
[8.13.x] RHPAM-4817: Update backend Dockerfiles and configuration to Quarkus 2.13.8 #868
Conversation
- Update Dockerfiles. - Explicitly allow all origins to fix a CORS issue caused by quarkusio/quarkus#33439.
(tests) - optaweb-vehicle-routing job Test results:
Those are the test failures: none See console log: Console Logs[2023-10-17T13:10:29.433Z] [INFO] Will not upload any artifacts in CLI environment[2023-10-17T13:10:29.433Z] [Pipeline] } [Pipeline] // withCredentials [Pipeline] } [2023-10-17T13:10:29.494Z] Deleting 1 temporary files [Pipeline] // configFileProvider [Pipeline] } [Pipeline] // script Post stage [Pipeline] junit [2023-10-17T13:10:29.576Z] Recording test results [2023-10-17T13:10:31.909Z] [Checks API] No suitable checks publisher found. [Pipeline] archiveArtifacts [2023-10-17T13:10:31.923Z] Archiving artifacts [Pipeline] } [Pipeline] // withEnv [Pipeline] } [Pipeline] // stage [Pipeline] stage [Pipeline] { (Sonar analysis) [Pipeline] tool [Pipeline] envVarsForTool [Pipeline] tool [Pipeline] envVarsForTool [Pipeline] tool [Pipeline] envVarsForTool [Pipeline] withEnv [Pipeline] { [Pipeline] script [Pipeline] { [Pipeline] } [Pipeline] // script [Pipeline] } [Pipeline] // withEnv [Pipeline] } [Pipeline] // stage [Pipeline] stage [Pipeline] { (check space after build) Stage "check space after build" skipped due to earlier failure(s) [Pipeline] } [Pipeline] // stage [Pipeline] stage [Pipeline] { (Declarative: Post Actions) [Pipeline] script [Pipeline] { [Pipeline] sh [2023-10-17T13:10:35.241Z] + wget --no-check-certificate -qO - 'https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/1.13.x/job/pullrequest/job/optaweb-vehicle-routing.tests.optaweb-vehicle-routing/13/api/json?depth=0' [Pipeline] readJSON [Pipeline] sh |
LGTM |
thnx for the fix @yurloc fyi @rgdoliveira, we probably need this for the build too, right? |
@mareknovotny TBH I don't know, I mean, I dont see any direct reference to Quarkus version in this PR, so maybe this can be merged only for RHPAM 7.13.5 and skipped from the respin? (cc @yurloc ). Also we dont need this to be backported to 8.13.x-blue branch, as optaplanner projects were dropped from there. |
@rgdoliveira The PR title may have been a bit confusing, sorry for that. I've updated it. What this PR actually does is that it updates backend Dockerfiles and configuration ( Quarkus upgrades:
This PR is needed to make the QE CI pipeline green. @dupliaka please confirm. |
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.
Looks Good to me
JIRA