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

grpc: Send known checks to API on connect #351

Merged
merged 2 commits into from
Oct 19, 2022

Conversation

adriansr
Copy link
Contributor

Updates the changes logic to send the existing checks to the API server so that a reduced set of probes can be returned.

This results in a reduced memory and network footprint on the API side when the connection is lost or the API restarted and all probes connect at the same time.

@adriansr adriansr requested a review from a team as a code owner October 18, 2022 20:48
Updates the changes logic to send the existing checks to the API server
so that a reduced set of probes can be returned.

This results in a reduced memory and network footprint on the API side
when the connection is lost or the API restarted.
@adriansr adriansr requested a review from mem October 18, 2022 22:06
Copy link
Contributor

@mem mem left a comment

Choose a reason for hiding this comment

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

This makes sense. I think we can marge this and it should work with the existing API.

@adriansr adriansr merged commit 527b6f8 into grafana:main Oct 19, 2022
mem added a commit that referenced this pull request Nov 3, 2022
* Fix: handle connection state changes
* grpc: Send known checks to API on connect (#351)
* Chore(deps): Bump google.golang.org/grpc from 1.48.0 to 1.49.0
* Chore(deps): Bump github.com/rs/zerolog from 1.27.0 to 1.28.0
* Chore(deps): Bump github.com/prometheus/client_golang
* Enable GRPC keep-alive in connection to API server (#342)
* Update tools
* Add very simple code to publish k6 metrics and logs
* Add nfpm packaging to goreleaser config (#339)
* Add goreleaser configuration
* Add CODE_OF_CONDUCT.md
* build: prevent invalid os/arch combinations (#336)
* Add tests for cmd/test-api/internal/db
* Add minimal code to run K6 checks
* An small (and possibly incorrect) implementation of the API

Signed-off-by: Marcelo E. Magallon <marcelo.magallon@grafana.com>
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.

None yet

2 participants