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

[CoE Starter Kit - Feature]: Switch to using createdBy.id instead of createdBy.displayName to identify SYSTEM created environments #6016

Closed
manuelap-msft opened this issue Jul 6, 2023 · 2 comments
Assignees
Labels
coe-starter-kit CoE Starter Kit issues enhancement New feature or request

Comments

@manuelap-msft
Copy link
Contributor

Is your feature request related to a problem? Please describe.

The Get Environments action returns the following for system and FinOps environments:

                "createdBy": {
                    "id": "SYSTEM",
                    "displayName": "SYSTEM",
                    "type": "NotSpecified"
                },

"createdBy": {
"id": "Microsoft Dynamics Deployment Service",
"displayName": "SYSTEM",

Currently we use the displayName to identify SYSTEM created environments but we've recently learned that this gets localized and there may also be plans to remove the displayName from this response altogether due to privacy reasons.

Describe the solution you'd like

We need to switch to using createdBy.id to identify system created environments.

For FinOps currently we use displayName which is SYSTEM for both cases (so both for SYSTEM environments as well as FinOps environments the displayName is the same) but when we switch to ID we'd have to handle both id=SYSTEM and id=Microsoft Dynamics Deployment Service

Describe alternatives you've considered

No response

Additional context?

No response

@manuelap-msft manuelap-msft added enhancement New feature or request coe-starter-kit CoE Starter Kit issues labels Jul 6, 2023
@manuelap-msft
Copy link
Contributor Author

Alternatively, we could use createdBy.type = NotSpecified

"createdBy": {
"id": "Microsoft Dynamics Deployment Service",
"displayName": "SYSTEM",
"type": "NotSpecified"
}

"createdBy": {
"id": "SYSTEM",
"displayName": "SYSTEM",
"type": "NotSpecified"
}

type is set to NotSpecified for all system/partner created environments and would be a "catch all" if there are other ids like Microsoft Dynamics Deployment Service out there, so we could check for if createdBy.type = NotSpecified then set maker to SYSTEM

@CoEStarterKitBot
Copy link
Collaborator

@manuelap-msft This has been fixed in the latest release. Please install the latest version of the toolkit following the instructions for installing updates. Note that if you do not remove the unmanaged layers as described there you will not receive updates from us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coe-starter-kit CoE Starter Kit issues enhancement New feature or request
Projects
Archived in project
Development

No branches or pull requests

3 participants