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: add grafana tempo topology #3217

Merged
merged 4 commits into from
Jan 30, 2025
Merged

Conversation

35C4n0r
Copy link
Contributor

@35C4n0r 35C4n0r commented Jan 29, 2025

Closes #3187

πŸ“‘ Description

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

β„Ή Additional Information

Signed-off-by: 35C4n0r <jaykumar20march@gmail.com>
Copy link

vercel bot commented Jan 29, 2025

@35C4n0r is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. Feature A new feature Provider Providers related issues labels Jan 29, 2025
Signed-off-by: 35C4n0r <jaykumar20march@gmail.com>
@talboren
Copy link
Member

@Matvey-Kuk do you think there's an added value for you to review too here?

Copy link

vercel bot commented Jan 30, 2025

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
keep ⬜️ Ignored (Inspect) Visit Preview Jan 30, 2025 8:03am

Copy link
Member

@talboren talboren left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 30, 2025
@talboren talboren enabled auto-merge (squash) January 30, 2025 07:58
@talboren talboren merged commit ee1a386 into keephq:main Jan 30, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A new feature lgtm This PR has been approved by a maintainer Provider Providers related issues size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[πŸ”Œ Provider]: Tempo Service Map
2 participants