Repository navigation
[MrBot] Replace update_deps with the Azure-MessagingStore pipeline version - #474
Open
Parth Aggarwal (parth21999) wants to merge 1 commit into
Open
Parth Aggarwal (parth21999) wants to merge 1 commit into
Parth Aggarwal (parth21999) wants to merge 1 commit into
Conversation
Copy scripts/update_deps from Azure-MessagingStore commit 03122f16a9117680d1a0f9512457dcc6e9b690e3 (unchanged on its master since) into update_deps, replacing the interactive c-build-tools version. Every script and ignores.json is byte-identical to the Azure-MessagingStore blob. The only edit to the copied files is the link to the pipeline definition in dependency_updates_propagation.md, which now points to build/propagate_deps.yml in Azure-MessagingStore because the relative path does not resolve here. The Azure-MessagingStore version runs unattended in an Azure DevOps pipeline. It authenticates with SYSTEM_ACCESSTOKEN and GH_TOKEN, checks access before opening any PR, discovers the dependency graph live, and opens each PR as soon as everything it depends on has merged. Remove what only the interactive version used: the WAM/PAT sign-in, -Resume, -AutoFix, PR watcher, script update check and animation helpers, known_graph.json, and the PAT screenshots referenced by the old document. update_deps.ps1 is unrelated to propagation and stays. The copilot instructions now name the environment variables the script needs. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 2cc24a4e-c845-4bfe-99ea-c31ce12fb4bd
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Member
Author
|
/azp run Azure-C-Build-Tools-Gate |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This replaces
update_deps/with the pipeline version from Azure-MessagingStore (EBS):scripts/update_depsat commit03122f16a9117680d1a0f9512457dcc6e9b690e3, which is unchanged on Storemastersince. It is a straight copy. All 12 scripts andignores.jsonhave the same git blob IDs as in Azure-MessagingStore. This is an alternative to #473, which merges the two versions instead of replacing one.dependency_updates_propagation.mdlinked to the pipeline definition with../../build/propagate_deps.yml. That path doesn't resolve in this repo, so the link now points to the file in Azure-MessagingStore.helper_scripts/filesautofix,check_script_update,install_az_cli,pr_watch_utils,propagation_state,success_animation,watch_azure_prandwatch_github_pr;known_graph.json;images/, which only the old document used.update_deps.ps1, a standalone submodule update script that propagation doesn't use..github/copilot-instructions.md: theRequiresline now namesSYSTEM_ACCESSTOKENandGH_TOKEN.What changes for users
Interactive use goes away: no WAM/PAT sign-in (
-azure_token),-Resume,-AutoFixor-ForceBuildGraph, noknown_graph.jsonreuse, and no live PR watchers or Ctrl+C menu. The script runs the way the Azure-MessagingStoreaa_propagate_submodulespipeline runs it:SYSTEM_ACCESSTOKENis used for Azure DevOps REST, andGH_TOKENforghand GitHub pushes.extraheaderin a job-scopedGIT_CONFIG_GLOBAL.https://dev.azure.com/msazure, projectOne.To run this copy, Azure-MessagingStore's
build/propagate_deps.ymlonly needs a new script path:deps\c-build-tools\update_deps\propagate_updates.ps1instead ofscripts\update_deps\propagate_updates.ps1. The arguments and environment stay the same.Validation
Copy check: the blob IDs of the 12 scripts and
ignores.jsonmatch Azure-MessagingStore at03122f16and at its currentmaster(50e3cf77). The document differs only in the link line.Parse check: all 12 scripts parse in Windows PowerShell 5.1 and PowerShell 7. Nothing in
update_depsrefers to a removed file.Repository validator: it reports only its existing findings, none in files touched here.
End-to-end: a local harness (not part of this PR) ran the copied scripts in place the way the pipeline does: Windows PowerShell 5.1, with fake Azure DevOps and GitHub services backed by local bare git repos and no network access. All 8 scenarios passed:
-NoCloseFailedPr;SYSTEM_ACCESSTOKENandGH_TOKEN;The same commit in its Azure-MessagingStore layout gave identical results.