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

Add DB to quarkus sample app for DevOps tests #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HKayed
Copy link

@HKayed HKayed commented Sep 16, 2021

No description provided.

quarkus.http.port=8081
quarkus.datasource.url = jdbc:postgresql://localhost:5432/quarkus

Choose a reason for hiding this comment

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

When trying to run the application I get the following error:

[error]: Build step io.quarkus.agroal.deployment.AgroalProcessor#build threw an exception: io.quarkus.runtime.configuration.ConfigurationException: quarkus.datasource.url and quarkus.datasource.driver have been deprecated in Quarkus 1.3 and removed in 1.9. Please use the new datasource configuration as explained in https://quarkus.io/guides/datasource.

So please change the corresponding properties

@@ -0,0 +1,12 @@
<?xml version="1.0" encoding="UTF-8"?>

Choose a reason for hiding this comment

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

What are these two xml files needed for?

Copy link
Author

Choose a reason for hiding this comment

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

@GuentherJulian Done with the investigation. persistance-units.xml is the file that is used to connect the entities created later on during tests to the database. They define connection attributes of the local db.

This file isn't complete because the configuration attributes weren't defined in the ticket hence the configuration would be done before the testing phase.

Long story short, we can remove this file but it's going to be created once again anyway.

@GuentherJulian
Copy link

The issue says that this version should be on a different branch (devonfw/devon4quarkus#15 (comment)). So please create the PR for an other branch

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