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

CSS-6718 add oauth relation to k8s charm #1164

Merged

Conversation

kian99
Copy link
Contributor

@kian99 kian99 commented Feb 22, 2024

Description

This PR adds the OAuth relation library to JIMM. Note that the relation library comes form the hydra-operator.
This PR is heavily inspired by the Grafana charms integration with the same library in canonical/grafana-k8s-operator#278

Fixes CSS-6718

Engineering checklist

Check only items that apply

  • Documentation updated
  • Covered by unit tests
  • Covered by integration tests

@alesstimec alesstimec requested a review from nsklikas February 23, 2024 10:28
Copy link

@nsklikas nsklikas left a comment

Choose a reason for hiding this comment

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

LGTM

If you want to add integration tests for this you can check out https://github.com/canonical/iam-bundle/tree/main/oauth_tools. It deploys the whole iam bundle and provides some helper functions. Currently it is very rough but we are going to refactor it in the future. The interface is going to remain the same though, so it shouldn't cause you any trouble.
See the grafana tests for an example: https://github.com/canonical/grafana-k8s-operator/blob/main/tests/integration/test_grafana_oauth.py

@kian99 kian99 merged commit 9a059ea into canonical:feature-oidc Feb 27, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants