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.

Sign up for GitHub

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

Jump to bottom

workload tracing for grafana #317

Merged
merged 3 commits into from Apr 8, 2024
Merged

workload tracing for grafana #317

merged 3 commits into from Apr 8, 2024

Conversation

PietroPasotti
Copy link
Contributor

@PietroPasotti PietroPasotti commented Apr 5, 2024 •
edited by sed-i

image

Cool!

To test:

juju deploy cos-lite --channel=edge --trust
juju deploy ./this-tempo-charm.charm tempo
juju relate tempo-k8s:tracing grafana
juju relate tempo-k8s:grafana-source grafana
juju relate tempo-k8s:grafana-dashboard grafana

Go to the grafana dashboard / the tempo datasource

dstathis
dstathis approved these changes Apr 5, 2024
View reviewed changes
src/charm.py Show resolved Hide resolved
src/charm.py Outdated Show resolved Hide resolved
@PietroPasotti PietroPasotti merged commit b1af4e6 into main Apr 8, 2024
13 checks passed
@PietroPasotti PietroPasotti deleted the workload-tracing branch April 8, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dstathis dstathis dstathis approved these changes

@Abuelodelanada Abuelodelanada Awaiting requested review from Abuelodelanada Abuelodelanada is a code owner

@IbraAoad IbraAoad Awaiting requested review from IbraAoad IbraAoad is a code owner

@lucabello lucabello Awaiting requested review from lucabello lucabello is a code owner

@mmkay mmkay Awaiting requested review from mmkay mmkay is a code owner

@sed-i sed-i Awaiting requested review from sed-i sed-i is a code owner

@simskij simskij Awaiting requested review from simskij simskij is a code owner

Assignees
No one assigned
Labels
Libraries: Out of sync
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@PietroPasotti @dstathis