Removal of the devops release PAT #8388
Conversation
|
The following pipelines have been queued for testing: |
|
The following pipelines have been queued for testing: |
|
The following pipelines have been queued for testing: |
JimSuplizio
force-pushed
the
RemDevOpsRelWIPat
branch
2 times, most recently
from
384d7a9 to
1577688
Compare
|
The following pipelines have been queued for testing: |
There was a problem hiding this comment.
The changes look good but as we talked lets wait until after releases finish after tues before merging. We should also make sure these don't break @praveenkuttappan's work.
JimSuplizio
force-pushed
the
RemDevOpsRelWIPat
branch
from
a74b99c to
83f3f2b
Compare
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#8388 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: James Suplizio <[email protected]>
This PR won't be merged until Thursday 6/13 as to not interfere with releases.
These are the code changes to remove the devops release PAT. Use the AzureCLI task and get the token from azure. The other AZ commands that the scripts use should be fine with the azureSubscription.
The LoginToAzureDevops function in DevOps-WorkItem-Helpers.ps1 is no longer necessary. The scenario for using this is that we passed the PAT to the pipeline and the variable was being used to log into azure. Being that this now needs to be run in an environment that's already logged into azure this function is completely moot.
The validate packages and update work items steps are part of the Build for Java, JS and NET. For Python these steps are in Analyze.