-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
[Enhancement]: Service Account Resource for Managed Grafana #37808
Comments
Community NoteVoting for Prioritization
Volunteering to Work on This Issue
|
I'll try to pick this up. This requires a move to SDK v2, I'll pick that up first. |
@aristosvo I'm happy to hear that you're taking this on! Just out of technical curiosity, what introduces the requirement to move to SDK v2? I'm taking a look at the AWS SDK (assuming that is what you're referring to) and it seems like AMG service accounts are supported by it? For instance, I see that Thanks again! |
Hi @lorrrrrrrenzo 👋 The reason is explained in this issue, new resources are not accepted when using the old SDK. |
Great to know, thanks @aristosvo! |
Hi @lorrrrrrrenzo! I think this one can be closed, as #37645 is an extended duplicate of this one. In the meantime the AWS SDK Go PR is ready for review, so things are in progress 👍🏽 |
Closing this, as it's a duplicate of #37645. |
Warning This issue has been closed, meaning that any additional comments are hard for our team to see. Please assume that the maintainers will not see them. Ongoing conversations amongst community members are welcome, however, the issue will be locked after 30 days. Moving conversations to another venue, such as the AWS Provider forum, is recommended. If you have additional concerns, please open a new issue, referencing this one where needed. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Description
The recent release of Managed Grafana 10 introduces the ability to use Grafana Service Accounts and Service Account Tokens for improved authn/z. I am proposing that support for this new authentication method be added to the AWS provider, with an eye towards making sure it integrates well with the Grafana provider, which according to its documentation, already supports authentication with Service Account Tokens.
Affected Resource(s) and/or Data Source(s)
aws_grafana_workspace
aws_grafana_service_account
(new)Potential Terraform Configuration
References
https://registry.terraform.io/providers/grafana/grafana/latest/docs
https://aws.amazon.com/blogs/mt/amazon-managed-grafana-announces-support-for-grafana-version-10-4
(Look at the section "Service Accounts and new APIs" for an example of the new authentication flow)
https://grafana.com/docs/grafana/v10.4/administration/service-accounts/
Would you like to implement a fix?
None
The text was updated successfully, but these errors were encountered: