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

[ALM Accelerator - Feature]: Remove Flow Activation Impersonation Logic #5772

Closed
mikefactorial opened this issue Jun 9, 2023 · 1 comment · Fixed by microsoft/coe-alm-accelerator-templates#306
Assignees
Labels
alm-accelerator ALM Accelerator Components and Apps enhancement New feature or request

Comments

@mikefactorial
Copy link
Collaborator

mikefactorial commented Jun 9, 2023

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

With the changes made for sharing connections with service principals we no longer support using impersonation for Flow Activation. We need to remove this logic and some of the additional configuration options in the deployment settings that are no longer used.

Describe the solution you'd like

Remove impersonation related logic from pipelines and remove activation user from deployment settings. The following scenarios need to be verified.

  1. Flows with connections / connection references
  2. Flows with no connections / connection references.
  3. Flows with owners specified in the deployment settings.
  4. Flows without owners specified in the deployment settings.
  5. Deploying to an environment where the user who owns the connections doesn't have access to the environment e.g. user's permissions have been removed after creating their connections.
  6. Others...

Describe alternatives you've considered

No response

Additional context?

No response

@CoEStarterKitBot
Copy link
Collaborator

@mikefactorial 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
alm-accelerator ALM Accelerator Components and Apps enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants