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

Move to new builder images introduced in Quarkus 2.14 #1004

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

rsvoboda
Copy link
Member

@rsvoboda rsvoboda commented Jan 19, 2023

Move to new builder images introduced in Quarkus 2.14

PR with the change - quarkusio/quarkus#27997

NEW is active - https://quay.io/repository/quarkus/ubi-quarkus-mandrel-builder-image?tab=tags
OLD is without update for 3 months - https://quay.io/repository/quarkus/ubi-quarkus-mandrel?tab=tags

  • Bug fix (non-breaking change which fixes an issue)
  • Dependency update
  • Refactoring
  • Backport
  • New scenario (non-breaking change which adds functionality)
  • This change requires a documentation update
  • 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

@rsvoboda rsvoboda requested a review from mjurc January 19, 2023 09:33
@@ -665,8 +665,7 @@
<quarkus.package.type>native</quarkus.package.type>
<quarkus.native.container-build>true</quarkus.native.container-build>
<quarkus.native.native-image-xmx>4g</quarkus.native.native-image-xmx>
<!-- please keep in mind that lifecycle-application module also defines quarkus.native.builder-image property -->
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line was dropped because 677520f removed the property from lifecycle app

@rsvoboda
Copy link
Member Author

PR - Linux - JVM build fail is not related to the change. There is something fishy with CLI tests.

@mjurc mjurc merged commit 420a92e into quarkus-qe:main Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants