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

Upgrade s2i docker version and DockerFiles #628

Merged
merged 2 commits into from
Dec 16, 2022
Merged

Upgrade s2i docker version and DockerFiles #628

merged 2 commits into from
Dec 16, 2022

Conversation

pjgg
Copy link
Contributor

@pjgg pjgg commented Dec 15, 2022

Summary

  • Upgrade default s2i docker image build to openjdk-17:latest
  • Update Quarkus DockerFiles on jvm, native and legacy images

Please check the relevant options

  • Refactoring
  • This change requires execution against OCP (use run tests phrase in comment)

Checklist:

  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@pjgg pjgg requested a review from fedinskiy December 15, 2022 11:38
@pjgg pjgg force-pushed the feat/upgrade_default_s2i_docker_image branch from 3a02ea4 to 4a056bb Compare December 15, 2022 11:40
@pjgg pjgg force-pushed the feat/upgrade_default_s2i_docker_image branch from 4a056bb to a9a77b1 Compare December 15, 2022 11:42
@pjgg pjgg changed the title Upgrades2i docker version and DockerFiles Upgrade s2i docker version and DockerFiles Dec 15, 2022
@pjgg
Copy link
Contributor Author

pjgg commented Dec 15, 2022

run tests

@pjgg
Copy link
Contributor Author

pjgg commented Dec 16, 2022

There is an error on OpenShiftStrimziOperatorKafkaWithoutRegistryMessagingIT but in my opinon is not related to this PR looks an environmental issue.

@pjgg pjgg merged commit 429c999 into quarkus-qe:main Dec 16, 2022
@jsmrcka jsmrcka mentioned this pull request Jan 4, 2023
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fedinskiy fedinskiy fedinskiy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pjgg @fedinskiy