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

feat: option to log flux queries cancelled because of server shutdown #23032

Merged
merged 4 commits into from
Jan 11, 2022

Conversation

williamhbaker
Copy link
Contributor

@williamhbaker williamhbaker commented Dec 29, 2021

Closes #21286

Results in a log line like this if the server is shutdown with SIGINT while a flux query is in-progress:

2021-12-29T21:34:54.612657Z	info	Cancelling Flux query because of server shutdown	{"log_id": "0Yj1cVWW000", "service": "storage-reads", "query": "import \"http\" http.post(url: \"http://localhost:8080\") from(bucket: \"bucket\") |> range(start: 0) |> yield()"}

@williamhbaker williamhbaker requested a review from lesam December 30, 2021 17:25
cmd/influxd/launcher/cmd.go Outdated Show resolved Hide resolved
Copy link
Contributor

@lesam lesam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment

@serenibyss serenibyss requested a review from lesam January 7, 2022 17:57
@serenibyss serenibyss merged commit b02c89e into master Jan 11, 2022
@serenibyss serenibyss deleted the wb-log-queries-on-shutdown-21286 branch January 11, 2022 22:06
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.

Dump active queries to log
4 participants