-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat(about): reimplement git build info in /health #589
Conversation
44feb73
to
7408e90
Compare
75e1182
to
c2c9397
Compare
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.
Looks good to me.
eede010
to
cba3919
Compare
cba3919
to
2d3fe22
Compare
/build_test |
Workflow started at 8/29/2024, 12:12:29 PM. View Actions Run. |
No OpenAPI schema changes detected. |
No GraphQL schema changes detected. |
CI build and push: All tests pass ✅ |
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
Welcome to Cryostat! 👋
Before contributing, make sure you have:
main
branch[chore, ci, docs, feat, fix, test]
To recreate commits with GPG signature
git fetch upstream && git rebase --force --gpg-sign upstream/main
Related to #550
Description of the change:
Stores the output of
git rev-parse
inMETA-INF/gitinfo
similar to what the 2.x build did withgit describe
. Then,GET /health
includes this string in its output as a nested JSON property, rather than replacing the version string.Motivation for the change:
Easier identification of development builds and linking to exact sources for all builds.
How to manually test:
./smoketest.bash -Ok
http :8080/health