-
Notifications
You must be signed in to change notification settings - Fork 183
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
Automate team subscription configuration keyvault provisioning #2572
Comments
FYI @heaths @weshaggard I'll still probably manually spin up the cognitive language tests group, but I think after that one this will be the fourth or fifth time I've done this, so I think the automation effort is worth the investment now. |
Be sure to also grant some sort of reader-type role to the service connection. I had to have @weshaggard grant me some sort of access in order to actually read the Key Vault secrets when adding a variable to the variable group. I had some sort of admin / write access to the KV, RG, and VG all, but when you click + Add on the variable group, I got an error about insufficient privileges on the service connection to the KV. |
I also mentioned one other idea to @weshaggard but it may not be practical given ambiguous times to sync between an Azure AD and DevOps: create an Azure AD group into which partner team "admins" are added then create a DevOps team based on that to grant privileges to the service connection and VG. In theory it would be a good way to manage admins for that pipeline, but in practice the delay would likely make it difficult. |
I am also considering consolidating service connections or otherwise having a simpler way of accessing the keyvaults. I don't think we necessarily need to be spinning up a new SP per keyvault. However, perhaps that will make it harder for other users to edit the variable group if they are adding new secrets to the keyvault? Since I'm admin on all of them I don't necessarily see the permission limited view that you have. |
Adding new secrets, IMO, should be a P1. P2 at worst. As new tests are added - especially for services like Cognitive that might themselves connected to many other services - new secrets for connection strings, etc. need to be added. I could almost self-service if not for the service connection issue. |
We have a way to add custom live test configuration overlays that can be managed directly by partner teams (see here and here). This process has been ad-hoc thus far as it was not common. It is both poorly documented and consists of manual steps.
More and more teams are starting to request this functionality, so it should be automated. This could either take the form of automating the ad-hoc steps via a tool/script OR adding some sort of configuration alongside the test yaml that could hint our pipeline generator to do it instead.
Provisioning steps today:
The text was updated successfully, but these errors were encountered: