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

Support custom Liquibase credentials #39361

Merged
merged 2 commits into from
Apr 17, 2024
Merged

Support custom Liquibase credentials #39361

merged 2 commits into from
Apr 17, 2024

Conversation

appiepollo14
Copy link
Contributor

@appiepollo14 appiepollo14 commented Mar 12, 2024 •
edited by gastaldi
Loading

@gastaldi
Copy link
Contributor

Can you rename the PR title (and the commit message if possible) to describe what changed? Just using Solves #<NUMBER> isn't too helpful

gsmet
gsmet requested changes Mar 12, 2024
View reviewed changes
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

This looks awesome but see my small gripe inline.

I will let others review this in details though.

.mvn/extensions.xml Outdated Show resolved Hide resolved
@appiepollo14 appiepollo14 changed the title Solves https://github.com/quarkusio/quarkus/issues/31214 Support custom Liquibase credentials Mar 12, 2024
@gsmet
Copy link
Member

gsmet commented Mar 12, 2024

As for PR title and commit message, I would go with something like:

Allow to use specific datasource credentials for Liquibase

Fixes #31214

@appiepollo14
Copy link
Contributor Author

@gastaldi refactored. Can this be merged?

gastaldi
gastaldi requested changes Mar 13, 2024
View reviewed changes
Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

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

It looks good overall, just added a minor suggestion to make the config description clearer

extensions/liquibase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseConfig.java Outdated Show resolved Hide resolved
extensions/liquibase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseConfig.java Outdated Show resolved Hide resolved
...ase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseDataSourceRuntimeConfig.java Outdated Show resolved Hide resolved
...ase/runtime/src/main/java/io/quarkus/liquibase/runtime/LiquibaseDataSourceRuntimeConfig.java Outdated Show resolved Hide resolved
gastaldi
gastaldi approved these changes Mar 13, 2024
View reviewed changes
Copy link
Contributor

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

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

LGTM, good job!

@gastaldi gastaldi requested a review from gsmet March 13, 2024 19:59
@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@gastaldi
Copy link
Contributor

@appiepollo14 the build failures seem related to your change?

@appiepollo14
Copy link
Contributor Author

@gastaldi It does. Will do some fixing.

@appiepollo14
Copy link
Contributor Author

Disabled one IT on Windows as there is no Docker engine available. Added one extra class to the PMT test. Hopefully that will solve the missing dependency.

@gastaldi is there a way to run the full CI on my fork?

@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
gsmet
gsmet approved these changes Mar 25, 2024
View reviewed changes
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

I force pushed to rebase and fix the formatting error.

This patch looks very good and is definitely something we want in.

My only concern is that we should avoid adding a dependency to Hibernate ORM in the tests.
We should rather use Agroal directly and do a database query.

Making everything intertwined in Quarkus makes it very hard to do major changes and we already got hit several times by Hibernate being used all around.

Would you have the cycles to address this concern? It could be after we merged this PR as it's already very good.

Thanks!

@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@appiepollo14
Copy link
Contributor Author

@gsmet I Will take on that challenge. However, there is still a failing Prod mode test. Please decide and reply to the following comment:

@gastaldi @gsmet this implementation is based on the Flyway implementation. To check the behavior of the extension with custom username / password, a second datasource (postgres) is introduced. This uses devservices to create a DB. The test now fail as, in opposition to the Flyway Integration tests, the PMT has no devservice PostGres database. Can be fixed maybe by reverting to a second datasource in H2, hopefully that has also a current_user default value for a table but should look into that. Other solution would be, to get rid of the PMT for the Liquibase Integration test. I prefer the second solution. Please approve / comment.

@gastaldi
Copy link
Contributor

gastaldi commented Mar 26, 2024 •
edited
Loading

What is PMT? EDIT: Figured out it means Prod Mode Test

@appiepollo14
Copy link
Contributor Author

@gastaldi Please advise

@gastaldi
Copy link
Contributor

Why do you need a second datasource?

@appiepollo14
Copy link
Contributor Author

Altered code, removed Postgres datasource in favor of H2 second database. Test should now work.

@appiepollo14
Copy link
Contributor Author

@gsmet @gastaldi removed hibernate and replaced it with agroal. It is partially working . I need te call, I can't figure out why, the endpoint for running Liquibase twice. After the second time I can call the endpoint: "created-by" and retrieve the inserted row from the database. After the first run, I get a database error stating the database is empty.

This issue was introduced when removing hibernate in favor of agroal sql statement. So I believe it has to do something the agroal datasource. Might be settings?

Hope you guys have the solution so this PR can finally be merged!

@appiepollo14
Copy link
Contributor Author

@gastaldi @gsmet
please guide me by providing an answer to the comment above. #39361 (comment)

@appiepollo14
Copy link
Contributor Author

Would love to see this pr merged. Help needed solving issue described here: #39361 (comment)

Any ideas are welcome

@quarkus-bot quarkus-bot

This comment has been minimized.

Sign in to view
@gsmet
Copy link
Member

gsmet commented Apr 17, 2024

Hey there. Sorry I was busy on other things :).

I'll have a look now.

appiepollo14 and others added 2 commits April 17, 2024 15:13
@appiepollo14 @gsmet
Allows to use specific datasource credentials for Liquibase
e11a8b5 
Fixes quarkusio#31214
@gsmet
Adjust the new Liquibase tests a bit
0258098 
Not sure exactly what was going wrong but I had to create two users to
make things work and also make sure everything was done in one go.
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Apr 17, 2024 •
edited by github-actions bot
Loading

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 0258098.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@appiepollo14
Copy link
Contributor Author

@gsmet nice fix!!

@gsmet gsmet merged commit 8fd116d into quarkusio:main Apr 17, 2024
19 checks passed
@gsmet
Copy link
Member

gsmet commented Apr 17, 2024

Merged! Thanks for your work and sorry for the delay!

@quarkus-bot quarkus-bot bot added this to the 3.11 - main milestone Apr 17, 2024
@quarkus-bot quarkus-bot bot added the kind/enhancement New feature or request label Apr 17, 2024
Dairdevil pushed a commit to Dairdevil/quarkus that referenced this pull request Apr 19, 2024
Squashed commit of the following:
fae7cca 
commit 6960d9e
Merge: 9a6644f 37fbf8a
Author: Dairdevil <[email protected]>
Date:   Fri Apr 19 14:24:14 2024 +0100

    Merge branch 'main' into feature/fabric8-split-package-resolved

commit 9a6644f
Author: Alasdair Preston <[email protected]>
Date:   Fri Apr 19 14:21:22 2024 +0100

    Remove explicitly ignored split packages

commit 37fbf8a
Merge: c85f5e2 09d77d0
Author: Guillaume Smet <[email protected]>
Date:   Fri Apr 19 11:50:57 2024 +0200

    Merge pull request quarkusio#40120 from dmlloyd/fix-40047

    Prevent double-warning when using deprecated properties

commit c85f5e2
Merge: 3afccc5 1c9f618
Author: Guillaume Smet <[email protected]>
Date:   Fri Apr 19 11:42:10 2024 +0200

    Merge pull request quarkusio#40145 from gwenneg/issue-40042

    Do not increment metrics on CaffeineCache#getIfPresent call

commit 3afccc5
Merge: 7497fef c70c45e
Author: Martin Kouba <[email protected]>
Date:   Fri Apr 19 10:11:29 2024 +0200

    Merge pull request quarkusio#40146 from mkouba/issue-39590

    WebSockets Next: configuration updates

commit c70c45e
Author: Martin Kouba <[email protected]>
Date:   Thu Apr 18 21:11:56 2024 +0200

    WebSockets Next: configuration updates

    - remove unimplemented timeout
    - add compression support/level
    - add max message size
    - also invoke error handlers for connection exceptions
    - resolves quarkusio#39590

commit 1c9f618
Author: Gwenneg Lepage <[email protected]>
Date:   Thu Apr 18 19:49:19 2024 +0200

    Do not increment metrics on CaffeineCache#getIfPresent call

commit 7497fef
Merge: 572f286 8df6fcb
Author: George Gastaldi <[email protected]>
Date:   Thu Apr 18 12:05:48 2024 -0300

    Merge pull request quarkusio#40117 from Sanne/KillSecurityManager

    Remove support for the SecurityManager

commit 572f286
Merge: fc2151b e59693a
Author: Guillaume Smet <[email protected]>
Date:   Thu Apr 18 16:34:12 2024 +0200

    Merge pull request quarkusio#40141 from gsmet/fix-license-url

    Fix license URL in JReleaser config

commit e59693a
Author: Guillaume Smet <[email protected]>
Date:   Thu Apr 18 16:33:33 2024 +0200

    Fix license URL in JReleaser config

commit fc2151b
Merge: 3cfba41 9840da1
Author: George Gastaldi <[email protected]>
Date:   Thu Apr 18 10:41:02 2024 -0300

    Merge pull request quarkusio#40136 from mkouba/issue-40089

    Qute type-safe messages: add convenient way to localize enum constants

commit 3cfba41
Merge: 9b29cc7 fc12627
Author: Guillaume Smet <[email protected]>
Date:   Thu Apr 18 13:47:02 2024 +0200

    Merge pull request quarkusio#40130 from gsmet/test-release-wf

    Make test release workflow more consistent with release

commit 9b29cc7
Merge: 200652e e44d7a7
Author: Guillaume Smet <[email protected]>
Date:   Thu Apr 18 13:46:50 2024 +0200

    Merge pull request quarkusio#40135 from tsegismont/patch-2

    Update kafka doc: Emitter

commit 9840da1
Author: Martin Kouba <[email protected]>
Date:   Thu Apr 18 12:58:08 2024 +0200

    Qute type-safe messages: add convenient way to localize enum constants

    - resolves quarkusio#40089

commit 200652e
Merge: 25dac45 71ee011
Author: Stéphane Épardaud <[email protected]>
Date:   Thu Apr 18 12:24:28 2024 +0200

    Merge pull request quarkusio#39510 from wowselim/5667-update-default-values-for-bcrypt-mapper

    Update default values for salt & iteration index

commit e44d7a7
Author: Thomas Segismont <[email protected]>
Date:   Thu Apr 18 11:12:48 2024 +0200

    Update kafka doc: Emitter

    `Emitter` is not an annotation.

commit 25dac45
Merge: 621d8b2 3315867
Author: Guillaume Smet <[email protected]>
Date:   Thu Apr 18 08:36:29 2024 +0200

    Merge pull request quarkusio#40115 from gsmet/exclude-wallpapers

    Exclude wallpapers from the preview

commit 621d8b2
Merge: 8fd116d b4d152e
Author: Guillaume Smet <[email protected]>
Date:   Thu Apr 18 08:36:06 2024 +0200

    Merge pull request quarkusio#40126 from quarkusio/dependabot/gradle/devtools/gradle/com.gradle.develocity-3.17.2

    Bump com.gradle.develocity from 3.17.1 to 3.17.2 in /devtools/gradle

commit fc12627
Author: Guillaume Smet <[email protected]>
Date:   Thu Apr 18 08:34:07 2024 +0200

    Make test release workflow more consistent with release

commit b4d152e
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Wed Apr 17 19:55:00 2024 +0000

    Bump com.gradle.develocity from 3.17.1 to 3.17.2 in /devtools/gradle

    Bumps com.gradle.develocity from 3.17.1 to 3.17.2.

    ---
    updated-dependencies:
    - dependency-name: com.gradle.develocity
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit 71ee011
Author: Selim <[email protected]>
Date:   Sat Mar 16 23:44:15 2024 +0300

    update default values for salt & iteration index

commit 8fd116d
Merge: 21bb7fb 0258098
Author: Guillaume Smet <[email protected]>
Date:   Wed Apr 17 18:07:07 2024 +0200

    Merge pull request quarkusio#39361 from appiepollo14/main

    Support custom Liquibase credentials

commit 0258098
Author: Guillaume Smet <[email protected]>
Date:   Wed Apr 17 16:04:12 2024 +0200

    Adjust the new Liquibase tests a bit

    Not sure exactly what was going wrong but I had to create two users to
    make things work and also make sure everything was done in one go.

commit e11a8b5
Author: asjervanasten <[email protected]>
Date:   Sun Mar 3 21:07:57 2024 +0100

    Allows to use specific datasource credentials for Liquibase

    Fixes quarkusio#31214

commit 09d77d0
Author: David M. Lloyd <[email protected]>
Date:   Wed Apr 17 08:01:35 2024 -0500

    Prevent double-warning when using deprecated properties

    Fixes quarkusio#40047

commit 21bb7fb
Merge: bbdc601 7e229d4
Author: Guillaume Smet <[email protected]>
Date:   Wed Apr 17 14:37:14 2024 +0200

    Merge pull request quarkusio#40001 from gastaldi/signed_jars

    Un-sign modified dependency JARs when filtering

commit bbdc601
Merge: c58cfee b4ae2ce
Author: Guillaume Smet <[email protected]>
Date:   Wed Apr 17 14:33:14 2024 +0200

    Merge pull request quarkusio#39908 from gsmet/flaky-dev-mojo-it

    Isolate testThatNewResourcesAreServed in FlakyDevMojoIT

commit c58cfee
Merge: 7d78792 0879d5e
Author: Guillaume Smet <[email protected]>
Date:   Wed Apr 17 14:32:09 2024 +0200

    Merge pull request quarkusio#40041 from geoand/config-notify

    Notify radcortez on config issues

commit 7d78792
Merge: a8d3b17 6afb990
Author: Guillaume Smet <[email protected]>
Date:   Wed Apr 17 14:28:35 2024 +0200

    Merge pull request quarkusio#40109 from cescoffier/custom-native-it

    Introduce workflow for testing GraalVM variants

commit 8df6fcb
Author: Sanne Grinovero <[email protected]>
Date:   Wed Apr 17 12:09:51 2024 +0100

    Remove support for the SecurityManager

commit 3315867
Author: Guillaume Smet <[email protected]>
Date:   Wed Apr 17 10:42:43 2024 +0200

    Exclude wallpapers from the preview

commit a8d3b17
Merge: 7f99015 9da5774
Author: Georgios Andrianakis <[email protected]>
Date:   Wed Apr 17 09:31:48 2024 +0300

    Merge pull request quarkusio#40106 from aloubyansky/parallel-dep-resolution

    Incubating implementation of a more efficient ApplicationModel resolver for Maven projects

commit 7f99015
Merge: 1e73223 b9207f7
Author: Georgios Andrianakis <[email protected]>
Date:   Wed Apr 17 09:26:55 2024 +0300

    Merge pull request quarkusio#40024 from holly-cummins/annotation-processor-tests

    Skeleton unit tests for annotation processor

commit 1e73223
Merge: efcf654 29a0588
Author: Ozan Gunalp <[email protected]>
Date:   Wed Apr 17 00:03:52 2024 +0200

    Merge pull request quarkusio#40108 from ozangunalp/smallrye_rm_4_21

    Bump smallrye-reactive-messaging.version from 4.20.0 to 4.21.0

commit efcf654
Merge: 35c2e9f 171a793
Author: Sergey Beryozkin <[email protected]>
Date:   Tue Apr 16 22:06:44 2024 +0100

    Merge pull request quarkusio#40111 from sberyozkin/smallrye-jwt-4.5.1

    Bump smallrye-jwt version to 4.5.1

commit 35c2e9f
Merge: 82379e9 b9be05f
Author: Sergey Beryozkin <[email protected]>
Date:   Tue Apr 16 17:50:18 2024 +0100

    Merge pull request quarkusio#40086 from IvanPuntev/feature/jtw-private-key-as-string

    Adding possibility to supply the jwt private key as a string

commit 82379e9
Merge: ce2e87f 2710238
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Apr 16 19:50:10 2024 +0300

    Merge pull request quarkusio#40033 from geoand/runner-cl-improvements

    Improve loading of classes in RunnerClassLoader

commit ce2e87f
Merge: 32f1a21 561212e
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 17:46:41 2024 +0200

    Merge pull request quarkusio#40074 from MichalMaler/logging-QE-review

    Applying the QE feedback to the Logging guide

commit 32f1a21
Merge: f0e6a40 2c0d16b
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 17:45:49 2024 +0200

    Merge pull request quarkusio#40096 from mkouba/issue-40083

    ArC: skip warning about invalid startup for producer methods

commit 171a793
Author: Sergey Beryozkin <[email protected]>
Date:   Tue Apr 16 16:27:35 2024 +0100

    Bump smallrye-jwt version to 4.5.1

commit b9be05f
Author: Ivan Puntev <[email protected]>
Date:   Mon Apr 15 19:26:10 2024 +0300

    Adding possibility to supply the jwt private key as a string.

commit 6afb990
Author: Clement Escoffier <[email protected]>
Date:   Mon Apr 15 09:46:54 2024 +0200

    Introduce workflow for testing GraalVM variants

    This commit introduces a new workflow tailored for
     testing GraalVM variants. Derived from the
     existing incremental workflow, it focuses on
     executing native integration tests with selected
      GraalVM variants such as CE, EE, Mandrel, Liberica,
       and versions 17, 21, and 22.

    This workflow is designed specifically for
    testing purposes and should be run on-demand
    rather than being triggered with every PR.
     It's recommended to execute this workflow from
      a fork to conserve QuarkusIO CI resources.

commit 29a0588
Author: Ozan Gunalp <[email protected]>
Date:   Tue Apr 16 16:32:42 2024 +0200

    Bump smallrye-reactive-messaging.version from 4.20.0 to 4.21.0

commit f0e6a40
Merge: 4e04f52 e430046
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 15:38:08 2024 +0200

    Merge pull request quarkusio#40107 from jedla97/update-yaml-config

    Update datasource yaml config in docs

commit 4e04f52
Merge: 96007c9 af00b5f
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 15:09:34 2024 +0200

    Merge pull request quarkusio#40105 from amusarra/amusarra-patch-1

    Docs: update deploying-to-openshift.adoc for build option

commit e430046
Author: Jakub Jedlicka <[email protected]>
Date:   Tue Apr 16 14:54:16 2024 +0200

    Update datasource yaml config in docs

commit 9da5774
Author: Alexey Loubyansky <[email protected]>
Date:   Thu Jan 11 08:31:14 2024 +0100

    Incubating implementation of a more efficient ApplicationModel resolver
    for Maven projects

commit af00b5f
Author: Antonio Musarra <[email protected]>
Date:   Tue Apr 16 14:18:03 2024 +0200

    Update deploying-to-openshift.adoc

    Replace the build option -Dquarkus.kubernetes.deploy=true with -Dquarkus.openshift.deploy=true

commit 96007c9
Merge: 0bfcab2 d56a625
Author: George Gastaldi <[email protected]>
Date:   Tue Apr 16 08:19:06 2024 -0300

    Merge pull request quarkusio#40101 from mkouba/qute-i18n-enum-test

    Qute: type-safe messages - add test for localized enum

commit 0bfcab2
Merge: d040fff a329189
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 12:26:48 2024 +0200

    Merge pull request quarkusio#40038 from karesti/infinispan-15.0.1

    Updates to Infinispan 15.0.1.Final

commit d040fff
Merge: 8612234 5d07fbd
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 12:26:32 2024 +0200

    Merge pull request quarkusio#40018 from cescoffier/fix-native-its

    Fix various IT modules and add them to the CI matrices

commit 8612234
Merge: cb01f5a b1cc21b
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Apr 16 13:09:45 2024 +0300

    Merge pull request quarkusio#40094 from geoand/quarkusio#40052

    Support `FileUpload` as multipart type in REST Client

commit cb01f5a
Merge: 1bd3fa3 2de7152
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 11:51:33 2024 +0200

    Merge pull request quarkusio#40057 from Malandril/fix-dev-db-restart

    Fix datasource devservices restarting

commit d56a625
Author: Martin Kouba <[email protected]>
Date:   Tue Apr 16 11:30:54 2024 +0200

    Qute: type-safe messages - add test for localized enum

    - related to quarkusio#40089
    - also use the convenient TemplateInstance#setLocale() method

commit 1bd3fa3
Merge: 0b84af7 8b1094e
Author: Jan Martiska <[email protected]>
Date:   Tue Apr 16 11:30:25 2024 +0200

    Merge pull request quarkusio#40073 from jmini/patch-2

    Explain how logging can be configured in the smallrye-graphql-client guide

commit 8b1094e
Author: Jérémie Bresson <[email protected]>
Date:   Mon Apr 15 14:50:27 2024 +0200

    docs: mention logging in smallrye-graphql-client

commit 0b84af7
Merge: 516b0b8 ff346be
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 09:53:00 2024 +0200

    Merge pull request quarkusio#40088 from ynojima/patch-19

    Correct asciidoc syntax error

commit ff346be
Author: Yoshikazu Nojima <[email protected]>
Date:   Tue Apr 16 10:55:48 2024 +0900

    Correct asciidoc syntax error

commit 516b0b8
Merge: c922e41 82f7b20
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 09:51:55 2024 +0200

    Merge pull request quarkusio#40082 from gsmet/footnote-downstream

    Rewrite footnotes for downstream documentation

commit 2c0d16b
Author: Martin Kouba <[email protected]>
Date:   Tue Apr 16 09:49:06 2024 +0200

    ArC: skip warning about invalid startup for producer methods

    - resolves quarkusio#40083

commit 561212e
Author: Michal Maléř <[email protected]>
Date:   Mon Apr 15 14:30:11 2024 +0200

    Applying the QE feedback to the Logging guide

    Signed-off-by: Michal Maléř <[email protected]>

commit b1cc21b
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Apr 16 10:11:21 2024 +0300

    Update multipart support documentation

commit 3adb838
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Apr 16 09:31:28 2024 +0300

    Support FileUpload as multipart type in REST Client

    Closes: quarkusio#40052

commit c922e41
Merge: f56b5c3 3740264
Author: Guillaume Smet <[email protected]>
Date:   Tue Apr 16 09:15:46 2024 +0200

    Merge pull request quarkusio#40093 from geoand/rest-client-doc

    Remove useless section in REST Client doc

commit 3740264
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Apr 16 08:34:50 2024 +0300

    Remove useless section in REST Client doc

commit f56b5c3
Merge: 3b97e89 8afd81c
Author: Sergey Beryozkin <[email protected]>
Date:   Mon Apr 15 21:17:34 2024 +0100

    Merge pull request quarkusio#40056 from sberyozkin/oidc_token_certificate_validator

    Add OIDC TokenCertificateValidator

commit 82f7b20
Author: Guillaume Smet <[email protected]>
Date:   Mon Apr 15 18:11:25 2024 +0200

    Rewrite footnotes for downstream documentation

    The modern version of footnotes is not supported in the downstream
    tooling and we can't use the older version in our doc as it triggers
    warnings.
    Thus we rewrite the new format to the old format specifically for
    downstream doc.

commit 3b97e89
Merge: 6139e8a f0c5435
Author: George Gastaldi <[email protected]>
Date:   Mon Apr 15 12:48:43 2024 -0300

    Merge pull request quarkusio#40017 from gastaldi/qute_template_locale

    Introduce `TemplateInstance#setLocale` and `TemplateInstance#setVariant`

commit 7e229d4
Author: George Gastaldi <[email protected]>
Date:   Wed Apr 10 21:19:34 2024 -0300

    Unsign modified dependency JARs when filtering

    Add test for JarResultBuildStep#filterJarFile

commit 6139e8a
Merge: 8bd1c82 43e92f3
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Apr 15 17:17:44 2024 +0300

    Merge pull request quarkusio#40068 from geoand/markers

    Introduce markers for static and runtime init recorder methods

commit 2de7152
Author: Thomas Canava <[email protected]>
Date:   Sat Apr 13 00:27:26 2024 +0200

    Fix datasource devservices restarting quarkusio#40015

commit 8bd1c82
Merge: c2c42dd 986ad73
Author: Martin Kouba <[email protected]>
Date:   Mon Apr 15 15:53:01 2024 +0200

    Merge pull request quarkusio#40067 from mkouba/qute-windows-native-resource-fix

    Qute: fix NativeImageResourceBuildItem registration on Windows

commit 5d07fbd
Author: Clement Escoffier <[email protected]>
Date:   Thu Apr 11 15:27:00 2024 +0200

    Fix various IT modules and add them to the CI matrices

    - Fix the gRPC Test Random Port IT (cannot use injection)
    - Fix the virtual thread webauthn IT (cannot use injection)
    - Fix the opentelemetry redis instrumentation IT (still unable to retrieve the exception)

commit c2c42dd
Merge: ba15f8e 6979a5a
Author: Max Rydahl Andersen <[email protected]>
Date:   Mon Apr 15 14:59:26 2024 +0200

    Merge pull request quarkusio#39953 from ia3andy/web-doc

    Introduce Quarkus for the Web documentation

commit ba15f8e
Merge: 1e7403a 08ba1d1
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Apr 15 15:51:37 2024 +0300

    Merge pull request quarkusio#39988 from FroMage/cp-perf-manager

    Context Propagation performance issue and init issue

commit 6979a5a
Author: Andy Damevin <[email protected]>
Date:   Mon Apr 15 14:36:29 2024 +0200

    Apply suggestions from code review

    Co-authored-by: Max Rydahl Andersen <[email protected]>

commit 1e7403a
Merge: ed82c3f 2cde5a6
Author: Ladislav Thon <[email protected]>
Date:   Mon Apr 15 13:20:09 2024 +0200

    Merge pull request quarkusio#39837 from Ladicek/arc-avoid-thread-locals

    ArC: replace thread locals with fields in CreationalContextImpl

commit ed82c3f
Merge: 5871b82 75ad6a5
Author: Guillaume Smet <[email protected]>
Date:   Mon Apr 15 13:08:06 2024 +0200

    Merge pull request quarkusio#40053 from gsmet/fix-show-more-show-less

    Fix collapsing in config reference

commit f0c5435
Author: George Gastaldi <[email protected]>
Date:   Fri Apr 12 15:20:33 2024 -0300

    Introduce TemplateInstance.setVariant

commit 9f66268
Author: George Gastaldi <[email protected]>
Date:   Thu Apr 11 10:01:47 2024 -0300

    Introduce TemplateInstance.setLocale

    Co-authored-by: Martin Kouba <[email protected]>

commit c80d8c5
Author: Andy Damevin <[email protected]>
Date:   Mon Apr 15 11:06:07 2024 +0200

    Make Qute doc more intuitive

commit 43e92f3
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Apr 15 11:48:04 2024 +0300

    Replace comments about recorder method execution with markers

commit 986ad73
Author: Martin Kouba <[email protected]>
Date:   Mon Apr 15 10:45:36 2024 +0200

    Qute: fix NativeImageResourceBuildItem registration on Windows

    - previously, a NativeImageResourceBuildItem with a wrong path was
    produced for a template located in a nested directory

commit fd34e44
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Apr 15 11:44:02 2024 +0300

    Introduce markers for static and runtime init recorder methods

    These are meant to replace the comments found in various recorder
    methods

commit 8afd81c
Author: Sergey Beryozkin <[email protected]>
Date:   Fri Apr 12 20:15:21 2024 +0100

    Add OIDC TokenCertificateValidator

commit 0879d5e
Author: Georgios Andrianakis <[email protected]>
Date:   Sat Apr 13 14:42:37 2024 +0300

    Add matching config issues based on title

commit b4ae2ce
Author: Guillaume Smet <[email protected]>
Date:   Fri Apr 5 14:21:55 2024 +0200

    Isolate testThatNewResourcesAreServed in FlakyDevMojoIT

    Also modernize things a bit now that we are using Java 17+.

commit 75ad6a5
Author: Guillaume Smet <[email protected]>
Date:   Fri Apr 12 19:03:13 2024 +0200

    Fix collapsing in config reference

    I'm not sure when it got broken but the current toggling is not working
    very well at least in some browsers.
    Using good old add/remove was able to fix it so I didn't dig too much
    either.

commit 2cde5a6
Author: Ladislav Thon <[email protected]>
Date:   Tue Apr 2 16:24:31 2024 +0200

    ArC: replace thread locals with fields in CreationalContextImpl

commit 2710238
Author: Georgios Andrianakis <[email protected]>
Date:   Fri Apr 12 10:07:27 2024 +0300

    Improve loading of classes in RunnerClassLoader

    This ClassLoader is already parallel capable, so just mark it as such.
    Furthermore, we define the package without using the ClassLoader lock
    by utilizing getDefinedPackage and checking for duplicate definitions

    Co-authored-by: David M. Lloyd <[email protected]>

commit 4c76b0c
Author: Georgios Andrianakis <[email protected]>
Date:   Fri Apr 12 14:18:52 2024 +0300

    Notify radcortez on config issues

commit a329189
Author: Katia Aresti <[email protected]>
Date:   Fri Apr 12 10:55:57 2024 +0200

    Updates to Infinispan 15.0.1.Final

commit b9207f7
Author: Holly Cummins <[email protected]>
Date:   Thu Apr 11 17:22:11 2024 +0100

    Add skeleton tests for annotation processor

commit d0c14cd
Author: Holly Cummins <[email protected]>
Date:   Thu Apr 11 17:27:32 2024 +0100

    Formatting only

commit ef0592b
Author: Andy Damevin <[email protected]>
Date:   Tue Apr 9 14:59:30 2024 +0200

    Apply suggestions from code review

    Co-authored-by: Guillaume Smet <[email protected]>

commit 08ba1d1
Author: Stéphane Épardaud <[email protected]>
Date:   Wed Apr 10 15:13:31 2024 +0200

    CP: set up a temporary no-context/no-executor ContextManagerProvider during boot

    Because some extensions (only known one is spring-cloud-config-client)
    use Vert.x via Mutiny before runtime init is done. Mutiny and CP are
    both not set up yet, due to missing executor, but I'm starting CP with
    zero contexts and an executor which throws with an explicit error
    message, which should be enough to boot until runtime is properly set
    up.

commit b9cc3c2
Author: Stéphane Épardaud <[email protected]>
Date:   Tue Apr 9 15:03:56 2024 +0200

    Delay FT class init at runtime so CP is initialised

commit ad06f24
Author: Stéphane Épardaud <[email protected]>
Date:   Tue Apr 9 15:03:33 2024 +0200

    Make sure OIDC is set up after CP

    It uses Mutiny, which requires CP to be initialised

commit 3122557
Author: Andy Damevin <[email protected]>
Date:   Mon Apr 8 17:19:58 2024 +0200

    Introduce Quarkus for the Web documentation

commit eb6b1ff
Author: Stéphane Épardaud <[email protected]>
Date:   Mon Apr 8 15:17:29 2024 +0200

    Make a custom ContextManagerProvider with a single manager

    Because Quarkus doesn't care about one-per CL and Franz said this is a
    bottleneck for some reason
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gastaldi gastaldi gastaldi approved these changes

@gsmet gsmet gsmet approved these changes

Assignees
No one assigned
Labels
area/liquibase kind/enhancement New feature or request
Projects
None yet
Milestone
3.11.0.CR1
Development

Successfully merging this pull request may close these issues.

Dedicated DDL user for liquibase migrations to avoid DDL permissions for app datasource user
3 participants
@appiepollo14 @gastaldi @gsmet