Skip to content

Navigation Menu

Sign in
Sign up

Fix monitoring mixin and dashboards - #70

Open
rgeyer wants to merge 11 commits into
githubexporter:master from
rgeyer:monitoring-mixin
Open

Fix monitoring mixin and dashboards #70
rgeyer wants to merge 11 commits into
githubexporter:master from
rgeyer:monitoring-mixin

Conversation

@rgeyer

@rgeyer rgeyer commented Jul 21, 2021

Copy link
Copy Markdown
Contributor

This PR includes, and supercedes #64, which also included the same change as #65.

The addition in this PR over both of those previous PRs is that the monitoring dashboard now properly uses whichever Grafana datasource that is selected by the user, rather than using the global default.

v-zhuravlev reacted with thumbs up emoji
fionaliao and others added 11 commits December 10, 2020 11:08
Dashboard is currently in JSON and has test data.
Also fixed whitespace causing mixin linting to fail.
Previously, only the repo label was added to the pull request metric.
Multiple repositories can have the same repo value - the user label is
required to uniquely identify a GitHub repository.
Created a GitHub repository dashboard, which graphs exporter metrics for
a given repository. Also created an API usage dashboard to check the
if the exporter is running out of requests.
The dashboards use Grafonnet, a library for writing Grafana dashboards.
Added jsonnet-bundler integration so the Grafonnet package can be
fetched.
jb install has to be run before linting as the linter checks the
dependency files exist.
Added dashboard descriptions to README and updated running instructions so
Jsonnet libraries that are required by the build are installed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /