Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

DO-78 ✨ add grafana #284

Merged
merged 3 commits into from
Feb 11, 2020
Merged

DO-78 ✨ add grafana #284

merged 3 commits into from
Feb 11, 2020

Conversation

dkravetz
Copy link
Contributor

References

DO-78

Summary

This PR Adds grafana and an initial setup to consume Keycloak for authentication purposes.

Are there any open tasks/blockers for the ticket?

No

A follow up PR may be necessary to finish the KeyCloak <-> Grafana integration, depending on complexity.


Reviewer checklist

Reviewer agreement:

  • Reviewers assign themselves at the start of the review.
  • Reviewers do not commit or merge the merge request.
  • Reviewers have to check and mark items in the checklist.

Merge request checklist

  • Conforms to the merge request title naming XP-XXX <a description in imperative form>.
  • Each commit conforms to the naming convention XP-XXX <a description in imperative form>.
  • Linked the ticket in the merge request title or the references section.
  • Added an informative merge request summary.

Code checklist

  • Conforms to the branch naming XP-XXX-<a_small_stub>.
  • Passed scope checks.
  • Added or updated tests if needed.
  • Added or updated code documentation if needed.
  • Conforms to Google docstring style.
  • Conforms to XAIN structlog style.

docker-compose-dev.yml Outdated Show resolved Hide resolved
@dkravetz dkravetz merged commit c60979d into development Feb 11, 2020
@dkravetz dkravetz deleted the DO-78_add_grafana branch February 11, 2020 08:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants