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 - BUG]: Canvas App Share step is using canvasappid instead of uniquecanvasappid and fails to share app #5902

Closed
BJHF-power opened this issue Jun 20, 2023 · 2 comments · Fixed by microsoft/coe-alm-accelerator-templates#306
Assignees
Labels
alm-accelerator ALM Accelerator Components and Apps bug Something isn't working

Comments

@BJHF-power
Copy link

BJHF-power commented Jun 20, 2023

Describe the issue

In the Share Canvas App with AAD Group step of Build & Deploy pipeline, the pipeline attempts to share Canvas Apps with AAD IDs as provided in the User Deployment Settings. It does this using the canvasappid attribute of the app:

image

However, this fails with the error message attached, as the app ID that it attempts to share is associated to a different environment (in our case, the Production environment).

image

The canvasappid value is static; it doesn't change as the solution is moved between environments, and thus using it for the sharing step can cause failures. Instead, the sharing step should use the uniquecanvasappid, which is environment-specific as the solution is moved between environments, and which matches the app ID as displayed in the UI and as retrieved by Get-AdminPowerApp cmdlet:
imageimage
image

Expected Behavior

The Canvas app share step should use the uniquecanvasappid attribute which holds the valid app ID in the target environment.

What component are you experiencing the issue with?

ALM Accelerator Pipelines

What solution version are you using?

May release

Steps To Reproduce

  1. Create a new target environment
  2. do a minimal copy of the old environment into the new environment
  3. attempt to deploy a solution containing a canvas app into the new environment after the copy is complete

Anything else?

No response

@mikefactorial
Copy link
Collaborator

This will be included in the July release via microsoft/coe-alm-accelerator-templates#306

@CoEStarterKitBot
Copy link
Collaborator

@BJHF-power 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 bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants