Simplify Foundry configuration and clarify role assignment#446
Open
PabloZaiden wants to merge 7 commits into
Open
Simplify Foundry configuration and clarify role assignment#446PabloZaiden wants to merge 7 commits into
PabloZaiden wants to merge 7 commits into
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request updates the CAIRA skill test automation to better support Azure AI Foundry as a BYOK provider for GitHub Copilot CLI, adding workflow-side validation and OIDC-based token acquisition, plus corresponding script and documentation adjustments.
Changes:
- Add preflight validation and OIDC-based Azure login/token retrieval in the skill-test GitHub Actions workflow.
- Add a
configure_foundry_providerhelper in the skill test script to validate/export Foundry-related provider settings and improve Copilot output capture and verifier parsing. - Document the weekly workflow’s Azure authentication/token flow and required Azure role in the README.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
scripts/test-skill.sh |
Adds Foundry provider configuration/validation and improves Copilot output handling and verifier result parsing. |
.github/workflows/skill-test.yml |
Adds Foundry config validation, OIDC permissions/login, and exports a short-lived access token for Copilot CLI BYOK testing. |
README.md |
Documents the weekly workflow’s OIDC login + Foundry token acquisition + Copilot BYOK environment setup. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
This pull request introduces enhancements to the CAIRA skill test workflow, focusing on improved support for Azure AI Foundry integration, robust environment validation, and better script reliability. The main changes are grouped into workflow improvements, script enhancements, and documentation updates.
Workflow improvements for Azure AI Foundry:
.github/workflows/skill-test.yml) now validates that all required Azure and Copilot provider environment variables are set before running tests, and it ensures theCOPILOT_PROVIDER_BASE_URLmatches the expected Foundry endpoint pattern.COPILOT_PROVIDER_BEARER_TOKENfor use by Copilot CLI.id-token: writeare added to support OIDC authentication in the workflow.Script enhancements for skill testing:
scripts/test-skill.shscript now includes aconfigure_foundry_providerfunction that checks for required Foundry-related environment variables, validates endpoint formatting, and exports all necessary provider settings for Copilot CLI.Documentation update:
README.mdnow describes how the weekly GitHub Actions run authenticates with Azure, obtains a Foundry token, and configures Copilot CLI for BYOK provider testing, including the required Azure role.