Skip to content

Comments

Sdk generation agent auth changes#40598

Open
praveenkuttappan wants to merge 23 commits intomainfrom
sdk_generation_agent_auth_changes
Open

Sdk generation agent auth changes#40598
praveenkuttappan wants to merge 23 commits intomainfrom
sdk_generation_agent_auth_changes

Conversation

@praveenkuttappan
Copy link
Member

This PR adds workflow instructions to create SDK generation agent and common authentication import which can be shared across GitHub agent workflow that requires Azure authentication.

@github-actions
Copy link

github-actions bot commented Feb 20, 2026

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named Protected Files has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide


Comment generated by summarize-checks workflow run.

Copy link
Contributor

Copilot AI commented Feb 20, 2026

@mikeharder I've opened a new pull request, #40599, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Member

@mikeharder mikeharder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved, though I wonder if we want to support *.lock.yaml in addition to *.lock.yml. If the lockfile is always yml, I suppose it's all we need. No matter what we do, this yaml/yml split is going to be annoying.

Copy link
Member

@mikeharder mikeharder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just supporting lock.yml is fine

- If no such PR is found, use default branch context.

5. Use the azsdk CLI at `/tmp/bin/azsdk` (installed earlier) to gather release plan metadata and required arguments:
5. Use the azsdk CLI at `$AZSDK_CLI_PATH/azsdk` (installed earlier) to gather release plan metadata and required arguments:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably easier at this point to add azsdk to $PATH instead

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.

4 participants