Skip to content

Add RestAPISpecsDocsRepos to branch cleanup and update the delete scripts to output limits and telemetry data#8830

Merged
JimSuplizio merged 3 commits intoAzure:mainfrom
JimSuplizio:UpdateBranchCleanup
Aug 15, 2024
Merged

Add RestAPISpecsDocsRepos to branch cleanup and update the delete scripts to output limits and telemetry data#8830
JimSuplizio merged 3 commits intoAzure:mainfrom
JimSuplizio:UpdateBranchCleanup

Conversation

@JimSuplizio
Copy link
Copy Markdown
Contributor

@JimSuplizio JimSuplizio commented Aug 13, 2024

  1. Add MicrosoftDocs/AzureRestPreview to the cleanup scripts. There were about 130k branches that were deleted prior to this change. The number of branches in the repository is now at a manageable level.
  2. We have a lot of repositories we're deleting branches from and we're also making gh API calls other than branch cleanup. Output some telemetry.
  3. If one of the API commands fails, exit the script, don't continue to try and process. The reason being is that if we hit a rate limit we can't continue processing.

The changes in here, right now, are going to be used for information gathering. There's currently no data of how many API calls are being made when the pipeline runs. Further updates will probably be made depending on the findings.

@JimSuplizio JimSuplizio self-assigned this Aug 13, 2024
@JimSuplizio JimSuplizio requested a review from a team as a code owner August 13, 2024 21:18
@azure-sdk
Copy link
Copy Markdown
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@JimSuplizio JimSuplizio requested a review from weshaggard August 13, 2024 23:12
@azure-sdk
Copy link
Copy Markdown
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

Comment thread eng/common/scripts/Delete-RemoteBranches.ps1 Outdated
Comment thread eng/common/scripts/Delete-RemoteBranches.ps1
Comment thread eng/common/scripts/Helpers/git-helpers.ps1 Outdated
Comment thread eng/common/scripts/Helpers/git-helpers.ps1
@azure-sdk
Copy link
Copy Markdown
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

azure-sdk added a commit to Azure/azure-sdk-for-js that referenced this pull request Aug 15, 2024
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#8830 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: James Suplizio <jasupliz@microsoft.com>
@JimSuplizio JimSuplizio merged commit 51d0611 into Azure:main Aug 15, 2024
@JimSuplizio JimSuplizio deleted the UpdateBranchCleanup branch August 15, 2024 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants