Added documentation for property precedence order, per-tenant configuration and /v1/health/config endpoint. #632
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ration and /v1/health/config endpoint.
vnandwana
commented
Oct 29, 2025
|
|
||
| Includes the `user.timezone` property, which is force set to `GMT`. | ||
|
|
||
| This property cannot be overridden. |
Contributor
Author
There was a problem hiding this comment.
Overriding user.timezone results in the following warning:
Overwrite of user.timezone system property with IST may break database serialization of date. Kill Bill will overwrite to GMT
This code resets the value to GMT.
Contributor
reshmabidikar
left a comment
There was a problem hiding this comment.
I do not quite follow how this documentation fits into Aviate Health. Does this correspond to a new API endpoint that exposes all the config properties? Is this endpoint provided by the aviate plugin? If so, a couple of suggestions:
- Add the documentation for this endpoint in the API docs here
- Add a new section (maybe below the Metrics section) to the Aviate health doc and move this content to this section. Ensure that the API doc points to this documentation.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ticket -- https://github.com/killbill/technical-support/issues/216