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.

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

Bump org.graalvm.sdk:graal-sdk to a version >= 21.3.10 #4299

Closed
12 tasks done
tasso94 opened this issue Apr 22, 2024 · 5 comments
Closed
12 tasks done

Bump org.graalvm.sdk:graal-sdk to a version >= 21.3.10 #4299

tasso94 opened this issue Apr 22, 2024 · 5 comments
Assignees
Labels
type:task Issues that are a change to the project that is neither a feature nor a bug fix. version:7.20.10 version:7.21.7 version:7.22.2 version:7.23.0

Comments

@tasso94
Copy link
Member

tasso94 commented Apr 22, 2024

Acceptance Criteria (Required on creation)

  • Bump org.graalvm.sdk:graal-sdk to a version >= 21.3.10.
  • Warning "implementation doesn't support runtime compilation" is handled

Hints

Links

Breakdown

Bump GraalVM First Iteration

  1. bot:java-dependency-check ci:all-as
    psavidis

Bump GraalVM Second Iteration

  1. bot:java-dependency-check ci:all-as ci:jdk ci:spring-boot ci:webapp-integration
    psavidis
  2. psavidis
  3. psavidis
  4. psavidis
  5. psavidis

Dev2QA handover

  • Does this ticket need a QA test and the testing goals are not clear from the description? Add a Dev2QA handover comment
@tasso94 tasso94 added type:task Issues that are a change to the project that is neither a feature nor a bug fix. version:7.22.0 potential:7.21.4 labels Apr 22, 2024
@amardeep2006
Copy link
Contributor

Is there some consideration to upgrade graaljs javascript engine as well? The latest version is 24.0.1 and current version in Camunda seems bit old 21.1.0 (Released Apr 20, 2021)

@psavidis psavidis self-assigned this Jul 29, 2024
psavidis added a commit that referenced this issue Jul 29, 2024
psavidis added a commit that referenced this issue Jul 30, 2024
psavidis added a commit that referenced this issue Jul 30, 2024
@psavidis
Copy link
Contributor

License check passed.

Assigning to @venetrius for the review.

@psavidis
Copy link
Contributor

Main and Backport Pull Requests are merged to their respective branches.

Closing the ticket.

@psavidis
Copy link
Contributor

psavidis commented Dec 5, 2024

Assigning for the ticket for review of the main PR. The backports can continue in a second round.

@psavidis psavidis assigned yanavasileva and unassigned psavidis Dec 5, 2024
@yanavasileva yanavasileva assigned psavidis and unassigned yanavasileva Dec 6, 2024
@psavidis psavidis assigned yanavasileva and unassigned psavidis Dec 6, 2024
@yanavasileva yanavasileva assigned psavidis and unassigned yanavasileva Dec 9, 2024
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 9, 2024
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 9, 2024
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 9, 2024
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 9, 2024
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 9, 2024
psavidis added a commit that referenced this issue Dec 10, 2024
Notes

- Bumps icu:icu4j to version 71.1

Introduces:

CustomScriptEngineManager - Class that allows to parameterise behaviour needed for a given JS engine: a) After the engines are detected from the classpath b) Before the engines are created

GraalVM Warnings

The commit sets the System property to disable the respective warnings introduced by bumping GraalVM to version 21.3.10.

- The system property is always set when a GraalJS script engine is found on the classpath
- The above applies by default to all distros

Related-to: #4299
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 10, 2024
Authored-by: Petros Savvidis <[email protected]>
Co-authored-by: Yana Vasileva <[email protected]>
Related-to: camunda/camunda-bpm-platform#4299
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 10, 2024
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 10, 2024
psavidis added a commit to camunda/camunda-docs-manual that referenced this issue Dec 10, 2024
serge-krot pushed a commit to cibseven/cibseven that referenced this issue Dec 12, 2024
Notes

- Bumps icu:icu4j to version 71.1

Introduces:

CustomScriptEngineManager - Class that allows to parameterise behaviour needed for a given JS engine: a) After the engines are detected from the classpath b) Before the engines are created

GraalVM Warnings

The commit sets the System property to disable the respective warnings introduced by bumping GraalVM to version 21.3.10.

- The system property is always set when a GraalJS script engine is found on the classpath
- The above applies by default to all distros

Related-to: camunda/camunda-bpm-platform#4299
Signed-off-by: Oleg Skrypnyuk <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:task Issues that are a change to the project that is neither a feature nor a bug fix. version:7.20.10 version:7.21.7 version:7.22.2 version:7.23.0
Projects
None yet
Development

No branches or pull requests

6 participants