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

[BUG] Windows Release Pipelines use inline script to upload manifest to Azure Blob Storage #1795

Closed
madalynrose opened this issue Mar 13, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@madalynrose
Copy link
Contributor

Describe the bug
Now that the service connections used in our release pipelines use federated credentials, the Azure File Copy task needed to be replaced as it does not support workload identity. The current solution is to use the Azure CLI task with the az storage blob upload command. We have tested the task with an inline script to confirm that it succeeds, but the ideal solution would use a script inside /tools/scripts/pipeline that is tracked via source control.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the "AIWin: Automated Canary Release" pipeline
  2. Click "Edit" and "Pipeline"
  3. Go to "Tasks"
  4. Click on the "Azure CLI" task

Expected behavior
The Azure CLI task references a script from release artifacts in the deploy_scripts directory, like the "Set Variables" PowerShell task does.

Actual behavior
The Azure CLI task uses an inline script.

Additional context
Creating this issue so this work is tracked.
This work will need to be done for three places:

  • AIWin: Automated Canary Release
  • AIWin: Release Promotion
    • Insider
    • Production
@madalynrose madalynrose added the bug Something isn't working label Mar 13, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the status: new This issue is new and requires triage by DRI. label Mar 13, 2024
@DaveTryon
Copy link
Contributor

Since this is just a 1 line script and we'd still need to customize the pipeline, we'll just keep the script in the pipeline and not add it to the repo.

@DaveTryon DaveTryon closed this as not planned Won't fix, can't repro, duplicate, stale Mar 18, 2024
@DaveTryon DaveTryon removed the status: new This issue is new and requires triage by DRI. label Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants