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

Allow configuring users through Terraform #1193

Closed
Matvey-Kuk opened this issue Jan 23, 2023 · 2 comments · Fixed by grafana/terraform-provider-grafana#1653
Closed

Allow configuring users through Terraform #1193

Matvey-Kuk opened this issue Jan 23, 2023 · 2 comments · Fixed by grafana/terraform-provider-grafana#1653

Comments

@Matvey-Kuk
Copy link
Contributor

Matvey-Kuk commented Jan 23, 2023

Use-case: large org onboards to OnCall. They would like to pre-fill notification policies & phone numbers.

We don't allow editing Users though API or TF, only to list: https://grafana.com/docs/oncall/latest/oncall-api-reference/users/

We allow editing Personal Notifications through API but we don't allow editing through Terraform: https://grafana.com/docs/oncall/latest/oncall-api-reference/personal_notification_rules/

@greatvovan
Copy link

OnCall user list seems to replicate Grafana users from /admin/users. It seems that we can add basic auth users (https://registry.terraform.io/providers/grafana/grafana/latest/docs/resources/user), but it will not work with OAuth SSO users, whose access list is controlled elsewhere. But I agree that it would be great to have pre-configured dummy user records that later (when users actually log in) would be joined with Grafana users. Now we need to wait until users log in at least once before we can terraform them into escalation chains.

Allowing setting a phone number through TF will open a loophole to sign up a phone number without owner's confirmation. Maybe add them in not confirmed state (unsupported currently)?

@joeyorlando
Copy link
Contributor

This will be closed soon by grafana/terraform-provider-grafana#1653

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants