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

DO-66 ✨ Add keycloak #283

Merged
merged 1 commit into from
Feb 11, 2020
Merged

DO-66 ✨ Add keycloak #283

merged 1 commit into from
Feb 11, 2020

Conversation

dkravetz
Copy link
Contributor

References

DO-66

Summary

This MR Adds Keycloack to the development docker-compose for OpenID Connect usage alongside Grafana

Are there any open tasks/blockers for the ticket?

No

DO-78 Will follow up adding Grafana.
A third PR may be required to finish the set-up of OpenID Connect for Grafana


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.

@rsaffi rsaffi mentioned this pull request Feb 10, 2020
10 tasks
@dkravetz dkravetz merged commit ecaa4d9 into development Feb 11, 2020
@dkravetz dkravetz deleted the DO-66_add_keycloak branch February 11, 2020 07:56
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