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

Fix/275 fails #586

Merged
merged 2 commits into from
Apr 1, 2022
Merged

Fix/275 fails #586

merged 2 commits into from
Apr 1, 2022

Conversation

fedinskiy
Copy link
Contributor

@fedinskiy fedinskiy commented Mar 31, 2022

Summary

Fix for two bugs both found when running the TS in OpenShift. One is related to usage of old consul config and to this bug: quarkusio/quarkus#24277, another is due to the property, which was removed from pom.xml

Please select the relevant options.

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

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

@fedinskiy fedinskiy requested a review from pjgg March 31, 2022 16:07
@fedinskiy
Copy link
Contributor Author

Since this branch fixes two bugs, it contains two commits

properties/pom.xml Outdated Show resolved Hide resolved
Consul config was relocated from quarkus to quarkusverse
Bug fixes(including the one for failuer on Openshift[1]) are only accessible in quarkusverse artifacts

[1] quarkusio/quarkus#24277
@rsvoboda rsvoboda requested a review from pjgg April 1, 2022 08:35
@rsvoboda rsvoboda merged commit 6e47e3d into quarkus-qe:main Apr 1, 2022
@fedinskiy fedinskiy deleted the fix/275_fails branch April 1, 2022 09:55
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.

3 participants