-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Don't print warning message when service account token property is unset #4795
Conversation
@manusa this pull request relates to quarkusio/quarkus#30558. Feel free to reject/close this pull request if you consider I'm completely wrong :) |
Trace the warning message if the user has set the service account token system property. Otherwise, simply print a debug message if there is no file in the default location. Relates quarkusio/quarkus#30558
a31d204
to
ecef75c
Compare
Haven't checked deeply but overall seems fine.
My comment has a different scope and is more of a reflection about how configuration inference works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Kudos, SonarCloud Quality Gate passed! |
Signed-off-by: GitHub <bot@example.com>
Signed-off-by: GitHub <bot@example.com> (cherry picked from commit 80bff08) Signed-off-by: GitHub <bot@example.com>
Description
Trace the warning message if the user has set the service account token system property. Otherwise, simply print a debug message if there is no file in the default location.
Relates quarkusio/quarkus#30558
Type of change
test, version modification, documentation, etc.)
Checklist