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

Added test to validate that hibernate properties in environment variables do not log a warning unused message when they are in use #18008

Merged
merged 1 commit into from
Jun 28, 2021

Conversation

radcortez
Copy link
Member

@radcortez radcortez commented Jun 18, 2021

@quarkus-bot
Copy link

quarkus-bot bot commented Jun 18, 2021

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building 4c4d394

Status Name Step Test failures Logs Raw logs
JVM Tests - JDK 11 Build Test failures Logs Raw logs
JVM Tests - JDK 16 Build Test failures Logs Raw logs

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 #

📦 integration-tests/smallrye-config

io.quarkus.it.smallrye.config.HibernatePropertiesTest.properties line 41 - More details - Source on GitHub


⚙️ JVM Tests - JDK 16 #

📦 integration-tests/smallrye-config

io.quarkus.it.smallrye.config.HibernatePropertiesTest.properties line 41 - More details - Source on GitHub

@quarkus-bot
Copy link

quarkus-bot bot commented Jun 19, 2021

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building 1513f7e

Status Name Step Test failures Logs Raw logs
✔️ JVM Tests - JDK 11
JVM Tests - JDK 11 Windows Build Test failures Logs Raw logs
✔️ JVM Tests - JDK 16
Native Tests - Data3 Build ⚠️ Check → Logs Raw logs

Full information is available in the Build summary check run.

Test Failures

⚙️ JVM Tests - JDK 11 Windows #

📦 extensions/arc/deployment

io.quarkus.arc.test.config.ConfigPropertiesTest. - More details - Source on GitHub

@gsmet
Copy link
Member

gsmet commented Jun 21, 2021

There is an unrelated problem in Liquibase:

Error: Class initialization of com.datical.liquibase.ext.config.LiquibaseProConfiguration failed. Use the option --initialize-at-run-time=com.datical.liquibase.ext.config.LiquibaseProConfiguration to explicitly request delayed initialization of this class.

@gastaldi @geoand I saw you merged an upgrade, could this be related?

@geoand
Copy link
Contributor

geoand commented Jun 21, 2021

It certainly could

…bles do not log a warning unused message when they are in use
@gsmet
Copy link
Member

gsmet commented Jun 21, 2021

I couldn't reproduce the issue, I pushed a rebase.

@gsmet gsmet merged commit d8fbb5a into quarkusio:main Jun 28, 2021
@quarkus-bot quarkus-bot bot added this to the 2.1 - main milestone Jun 28, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Jun 28, 2021

Milestone is already set for some of the items:

We haven't automatically updated the milestones for these items.

This message is automatically generated by a bot.

@radcortez radcortez deleted the fix-17818 branch November 22, 2021 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[1.13.7] Using dotenv file and get unrecognized configuration key warn.
3 participants