Azure / Azure/azure-sdk-tools

No Ability for Admin to Override Configs or Scripts to Ensure Special Spec PR Following Review Workflow

Open
#9,470 0 comments 0 reactions 0 assignees View on GitHub
Automation Tool
Dominant language
C#
Stars
135
Forks
260
Avg merge
3d 1h
Merged PRs (30d)
143

Description

Special case: A service team wants to add 3P npm package to generate sdk from tsp. But the npm package dependency is fixed: [azure-sdk-for-net/eng/emitter-package.json at main · Azure/azure-sdk-for-net](https://github.com/Azure/azure-sdk-for-net/blob/main/eng/emitter-package.json). It force thie team to generate SDK locally. It causes at least following issues:
- sdk automation always fails in codegen stage.
- They can't follow spec repo's review workflow, we cannot verify anything to ensure the generated package works and has no unapproved breaking changes.
- Running locally for all languages takes a lot of time and may encounter unexpected errors.

My proposal to reuse sdk automation pipeline as much as possible and make special spec PR follow Spec review workflow as much as possible:
Make configs and scripts overriadable by pipeline variables by admin (subset azure sdk team, maybe?)

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the SDK automation pipeline and the referenced eng/emitter-package.json dependency configuration. Determine which configs and scripts must accept administrator-controlled pipeline overrides for the special spec PR workflow. Done means the workflow can use the required package without local generation while preserving review and validation checks.

Written by the indexing model from the issue text.

Assessment

Tech stack
node.js
Domain
build-system, ci-cd
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.