-
Notifications
You must be signed in to change notification settings - Fork 456
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
[Feature Request] BREAKING change: Discuss CI environment secrets naming #1565
Comments
POC and implementation of this is covered in: |
And I would suggest we add possibility for consumers to use 2 subs, 2 SPs, and even 2 tenants, as you might need a validation tenant and or MG to validate changes. Maybe the |
Removing from upcoming release 0.7, will be worked on in the next one |
It was decided to hold on to the environment split until we figured out whether we can use Open ID connect or not (#1450) - even though it is only relevant for GitHub & not ADO |
Description
This discussion needs to take place before issues #1450 #1465 #1085
DEPLOYMENT_SP_ID
toDEPLOYMENT_SPN_ENTAPP_OBJID
#1465 Discuss a name consistent with the aboveThe text was updated successfully, but these errors were encountered: