-
Notifications
You must be signed in to change notification settings - Fork 476
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
Align the -server.http-write-timeout
with -querier.timeout
to 2m
#3346
Conversation
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.
Thanks for picking this up :) I think we can move it into a higher level package
1e5c213
to
ebe2381
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.
LGTM <3
I'll let others have a look before merging
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, thanks! I left a couple of nits I'm going to auto apply.
Signed-off-by: Marco Pracucci <marco@pracucci.com>
…rafana#3346) * Align the `-server.http-write-timeout` with `-querier.timeout` to 2m * Add a validation for these values * Update docs * Update CHANGELOG.md * move it into a higher level package * Apply suggestions from code review * Reordered CHANGELOG entry Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com> Co-authored-by: Marco Pracucci <marco@pracucci.com>
…rafana#3346) * Align the `-server.http-write-timeout` with `-querier.timeout` to 2m * Add a validation for these values * Update docs * Update CHANGELOG.md * move it into a higher level package * Apply suggestions from code review * Reordered CHANGELOG entry Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com> Co-authored-by: Marco Pracucci <marco@pracucci.com>
* Remove 'server.http-write-timeout' from jsonnet This option now defaults to 2m: #3346 And also it's validated to not to be lower than querier.timeout, so having this 1m value breaks the jsonnet. Fixes #3824 Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com> * make check-jsonnet-tests Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com> * Update CHANGELOG.md Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com> * Remove helm-jsonnet ignore Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com> Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
What this PR does
Align the
-server.http-write-timeout
with-querier.timeout
to 2m. Add validation for these values.Which issue(s) this PR fixes or relates to
Fixes #3248
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]