forked from grafana/pyroscope
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] main from pyroscope-io:main #12
Open
pull
wants to merge
4,176
commits into
Mu-L:main
Choose a base branch
from
grafana:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+875,453
−49,997
Conversation
This file contains 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
petethepig
force-pushed
the
main
branch
3 times, most recently
from
December 8, 2021 02:23
cfe11d8
to
8eb788f
Compare
petethepig
force-pushed
the
main
branch
4 times, most recently
from
January 10, 2022 00:16
bc301f7
to
abf97a2
Compare
petethepig
force-pushed
the
main
branch
2 times, most recently
from
April 8, 2022 01:08
bf9d41f
to
1de9288
Compare
petethepig
force-pushed
the
main
branch
2 times, most recently
from
May 11, 2022 17:22
91689fa
to
c4b4164
Compare
petethepig
force-pushed
the
main
branch
2 times, most recently
from
July 5, 2022 22:29
f2eca4c
to
e83e482
Compare
…3428) * feat: Embed a grafana instance into pyroscope This provides an optional target `embedded-grafana` to download/configure/run a dedicated Grafana instance with Explore Profiles deployed. This is meant not to be used as production, but useful to quickly discover profiles. After this PR merges a user can run this: ``` $ docker run -p 4040:4040 -p 4041:4041 grafana/pyroscope --target all,embedded-grafana ``` And on localhost:4040 there will the classic pyroscope and on :4041, there will be Grafana with Explore Profiles. * Update explore profiles to v0.1.5 * Update contribution guide to mention embedded Grafana
I think it is a nice to have this appreciation, but we haven't updated in a while, as the nodejs of the old tools no longer maintained. Just threw some go together quickly.
Co-authored-by: grafanabot <[email protected]>
* Push is using wrong credentials * Also skip ci on the subsequent commit
This also updates grafana/eslint for compatability with node 20
* Newline after contributors * Use the PAT for pushing changes correctly Using action/checkout with token is broken for our usecase, implemented a workaround from: actions/checkout#664 (comment)
* feat(v2): add metadata label query interface * implement clients * improve error handling * implement metadata label query service * implement metadata label querier * test and fix metadata label querier * refactor metadata label compaction * introduce block/metadata package * avoid sending stubs in GetTenantStats * document metadata entry
* Restructure Pyroscope documentation and share content * Fix typo * Fix heading match for profile types * Restructure for configure server * Apply suggestions from code review Co-authored-by: Taylor C <[email protected]> * Apply suggestions from code review * Apply suggestions from code review Co-authored-by: Aleksandar Petrov <[email protected]> * Apply suggestions from code review Co-authored-by: Aleksandar Petrov <[email protected]> --------- Co-authored-by: Taylor C <[email protected]> Co-authored-by: Aleksandar Petrov <[email protected]>
Co-authored-by: Pyroscope Bot <[email protected]>
* feat: configurable symbols partitioning * fix fixtures * add CLI docs
* Added bucket-lookup-type field to s3 config * Review fixes: validation & deprecation warning * Update reference docs * Formatted code --------- Co-authored-by: Christian Simon <[email protected]>
* Update `make docs` procedure * Update `make docs` procedure --------- Co-authored-by: grafanabot <[email protected]>
We seem to have not linked up the validation for a few components.
* docs: Add v1.12 release notes * Also add documentation in changelog
Co-authored-by: Aleksandar Petrov <[email protected]>
In order to be able to parse the labelset in the classic ingest endpoints in Grafana Alloy, I am refactoring this bit of the OG code into the api package so I can use this within Grafana Alloy.
* fix(otel): compressed metric * fix TestDifferentServiceNames
* Add a metastore client admin test page * Improve page * Workaround for the mismatched server identifiers * Improve logging * Fix error on startup * Fix test * Switch metastore client retries to dskit backoff
* chore(v2): improve error handling in compaction-worker * chore(v2): add some more safety checks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
See Commits and Changes for more details.
Created by pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )