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

Check minversion of test-proxy prior to push #8067

Open
5 tasks
scbedd opened this issue Apr 10, 2024 · 0 comments
Open
5 tasks

Check minversion of test-proxy prior to push #8067

scbedd opened this issue Apr 10, 2024 · 0 comments
Assignees
Labels
Central-EngSys This issue is owned by the Engineering System team.

Comments

@scbedd
Copy link
Member

scbedd commented Apr 10, 2024

  • Add a minimum_proxy_version.txt file under eng/ of azure-sdk-assets
  • Leverage Merge eng/ from main on assets push #8020 's pull from main ability to check get the updated file from main
  • Before a push operation, check running proxy version against this minimum_proxy_version file.
  • Error and recommend upgrade if not newer than the target version.
  • an eng/common template that can check this same file and error if the running proxy version for CI is below this minimum version.

FYI @gearama this is based on your suggestion in the just-past meeting. This is lower priority than adding a version of credscan or getting that common PR out, but we'll likely get to it pretty quick.

@scbedd scbedd self-assigned this Apr 10, 2024
@kurtzeborn kurtzeborn moved this from 🤔 Triage to 📋 Backlog in Azure SDK EngSys 🤖🧠 Aug 12, 2024
@kurtzeborn kurtzeborn added the Central-EngSys This issue is owned by the Engineering System team. label Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants