Skip to content
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

Jump to bottom

Feature: Upgrade Quarkus to 3.6+ #263

Closed
1 task done
ewilkins-csi opened this issue Aug 7, 2024 · 5 comments
Closed
1 task done

Feature: Upgrade Quarkus to 3.6+ #263

ewilkins-csi opened this issue Aug 7, 2024 · 5 comments
Assignees
@ewilkins-csi
Labels
enhancement New feature or request
Milestone
1.9.0

Comments

@ewilkins-csi
Copy link
Contributor

ewilkins-csi commented Aug 7, 2024 •
edited
Loading

Description

In order to prepare for adding JDK 17 support to the baseline, we should update Quarkus to the latest possible version that still supports Java 11 (Quarkus 3.7 dropped support for 11).

DOD

  • Quarkus is updated to 3.6

Test Strategy/Script

  1. Check out the feature branch feature-java17
  2. Run the integration tests
mvn clean install -Pintegration-test
  1. Run the archetype test
mvn clean install -pl :foundation-archetype -Parchetype-test
@ewilkins-csi ewilkins-csi added the enhancement New feature or request label Aug 7, 2024
@ewilkins-csi ewilkins-csi mentioned this issue Aug 7, 2024
Open
36 tasks
@ewilkins-csi ewilkins-csi self-assigned this Aug 7, 2024
@ewilkins-csi ewilkins-csi added this to the 1.9.0 milestone Aug 7, 2024
@ewilkins-csi
Copy link
Contributor Author

DoD with @carter-cundiff @csun-cpointe and @nartieri

@ewilkins-csi
Copy link
Contributor Author

ewilkins-csi commented Aug 23, 2024 •
edited
Loading

After this upgrade, there were some issues with Policy Decision Point, Pipeline Invocation, and the Configuration Store webhook. It seems that the bulk or all of these issues will be resolved by upgrading to JDK 17. So instead of implementing hacks to workaround the issue, we will keep this work on a feature branch until the full upgrade is done. That will keep us from doing a bunch of work that is redundant/removed by the end of this ticket series. In light of that, testing will focus on the build ITs/archetype test.

@ewilkins-csi
Copy link
Contributor Author

OTS results:
mvn clean install -Pintegration-test was successful:
Screenshot 2024-08-26 at 10 22 02 AM

mvn clean install -pl :foundation-archetype -Parchetype-test was successful:
Screenshot 2024-08-26 at 10 05 38 AM

@csun-cpointe
Copy link
Contributor

OTS completed ✅

ewilkins-csi added a commit that referenced this issue Aug 26, 2024
@ewilkins-csi
[#263] general cleanup
e7cf0e0 
 - pull scala version up to build-parent
 - fix relative parent path warning
 - remove unused imports
 - remove unused inf recursion
 - fix package path
 - fix string utils import
 - rename smallrye graphql version property
 - move logging dependencies up to root so they can be used in build-support
 - add more Apache commons to root
 - remove unused incremental build profile
 - remove unused fields/imports
 - add plugin version to cucumberin build-support
 - de-duplicate fabric8 conf in spark-infrastructure
 - remove pom/bom import from dependencies in invocation-service
 - fix templates (bom removal and groupId fix)
 - simplify pipeline CDI with composition
ewilkins-csi added a commit that referenced this issue Aug 26, 2024
@ewilkins-csi
[#263] general cleanup
5529ad9 
 - pull scala version up to build-parent
 - fix relative parent path warning
 - remove unused imports
 - remove unused inf recursion
 - fix package path
 - fix string utils import
 - rename smallrye graphql version property
 - move logging dependencies up to root so they can be used in build-support
 - add more Apache commons to root
 - remove unused incremental build profile
 - remove unused fields/imports
 - add plugin version to cucumberin build-support
 - de-duplicate fabric8 conf in spark-infrastructure
 - remove pom/bom import from dependencies in invocation-service
 - fix templates (bom removal and groupId fix)
 - simplify pipeline CDI with composition
@csun-cpointe
Copy link
Contributor

final test passed!!

@ewilkins-csi ewilkins-csi mentioned this issue Aug 26, 2024
Open
1 task
ewilkins-csi added a commit that referenced this issue Aug 30, 2024
@ewilkins-csi
[#263] general cleanup
671fa93 
 - pull scala version up to build-parent
 - fix relative parent path warning
 - remove unused imports
 - remove unused inf recursion
 - fix package path
 - fix string utils import
 - rename smallrye graphql version property
 - move logging dependencies up to root so they can be used in build-support
 - add more Apache commons to root
 - remove unused incremental build profile
 - remove unused fields/imports
 - add plugin version to cucumberin build-support
 - de-duplicate fabric8 conf in spark-infrastructure
 - remove pom/bom import from dependencies in invocation-service
 - fix templates (bom removal and groupId fix)
 - simplify pipeline CDI with composition
ewilkins-csi added a commit that referenced this issue Sep 13, 2024
@ewilkins-csi
[#263] general cleanup
dbbaf32 
 - pull scala version up to build-parent
 - fix relative parent path warning
 - remove unused imports
 - remove unused inf recursion
 - fix package path
 - fix string utils import
 - rename smallrye graphql version property
 - move logging dependencies up to root so they can be used in build-support
 - add more Apache commons to root
 - remove unused incremental build profile
 - remove unused fields/imports
 - add plugin version to cucumberin build-support
 - de-duplicate fabric8 conf in spark-infrastructure
 - remove pom/bom import from dependencies in invocation-service
 - fix templates (bom removal and groupId fix)
 - simplify pipeline CDI with composition
csun-cpointe pushed a commit that referenced this issue Sep 18, 2024
@ewilkins-csi @csun-cpointe
[#263] general cleanup
3358ef3 
 - pull scala version up to build-parent
 - fix relative parent path warning
 - remove unused imports
 - remove unused inf recursion
 - fix package path
 - fix string utils import
 - rename smallrye graphql version property
 - move logging dependencies up to root so they can be used in build-support
 - add more Apache commons to root
 - remove unused incremental build profile
 - remove unused fields/imports
 - add plugin version to cucumberin build-support
 - de-duplicate fabric8 conf in spark-infrastructure
 - remove pom/bom import from dependencies in invocation-service
 - fix templates (bom removal and groupId fix)
 - simplify pipeline CDI with composition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ewilkins-csi ewilkins-csi

Labels
enhancement New feature or request
Projects
None yet
Milestone
1.9.0
Development

No branches or pull requests
2 participants
@ewilkins-csi @csun-cpointe