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

Java 17 and Camel Update #6797

Merged
merged 7 commits into from
Aug 26, 2024
Merged

Java 17 and Camel Update #6797

merged 7 commits into from
Aug 26, 2024

Conversation

clockard
Copy link
Contributor

@clockard clockard commented Aug 21, 2024 •
edited
Loading

What does this PR do?

Updates the camel version to fix CDM.
Fixed several of the csw itests that were broken on the 2.29.x branch and ignored ones not easily fixed. Follow up ticket is #6798
Updated java version to 17

Depends on codice/docker-ddf-base#4

Who is reviewing it?

Ask 2 committers to review/merge the PR and tag them here.

How should this be tested?

Good build

Notes on Review Process

Please see Notes on Review Process for further guidance on requirements for merging and abbreviated reviews.

Review Comment Legend:

  • ✏️ (Pencil) This comment is a nitpick or style suggestion, no action required for approval. This comment should provide a suggestion either as an in line code snippet or a gist.
  • ❓ (Question Mark) This comment is to gain a clearer understanding of design or code choices, clarification is required but action may not be necessary for approval.
  • ❗ (Exclamation Mark) This comment is critical and requires clarification or action before approval.

@vinamartin vinamartin self-requested a review August 21, 2024 15:52
@clockard clockard mentioned this pull request Aug 22, 2024
Open
@clockard clockard changed the title Camel Update Java 17 and Camel Update Aug 23, 2024
@clockard
Copy link
Contributor Author

build now

@cxddfbot
Copy link

Internal build has been started, your results will be available at build completion.

@cxddfbot
Copy link

Build FAILURE See the job results in legacy Jenkins UI or in Blue Ocean UI.

@clockard
Copy link
Contributor Author

build now

@cxddfbot
Copy link

Internal build has been started, your results will be available at build completion.

@cxddfbot
Copy link

Build FAILURE See the job results in legacy Jenkins UI or in Blue Ocean UI.

@clockard
Copy link
Contributor Author

build now

@cxddfbot
Copy link

Internal build has been started, your results will be available at build completion.

@cxddfbot
Copy link

Build FAILURE See the job results in legacy Jenkins UI or in Blue Ocean UI.

@clockard
Copy link
Contributor Author

build now

@cxddfbot
Copy link

Internal build has been started, your results will be available at build completion.

@cxddfbot
Copy link

Build FAILURE See the job results in legacy Jenkins UI or in Blue Ocean UI.

@clockard
Copy link
Contributor Author

build now

@cxddfbot
Copy link

Internal build has been started, your results will be available at build completion.

@cxddfbot
Copy link

Build FAILURE See the job results in legacy Jenkins UI or in Blue Ocean UI.

@clockard
Copy link
Contributor Author

build now

@cxddfbot
Copy link

Internal build has been started, your results will be available at build completion.

@cxddfbot
Copy link

Build SUCCESS See the job results in legacy Jenkins UI or in Blue Ocean UI.

@clockard clockard merged commit d65e6eb into 2.29.x Aug 26, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vinamartin vinamartin vinamartin approved these changes

@ahoffer ahoffer ahoffer approved these changes

@stustison stustison Awaiting requested review from stustison stustison is a code owner

Assignees
No one assigned
Labels
⬆️ Dependency Upgrade
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@clockard @cxddfbot @ahoffer @vinamartin