Skip to content

Add smoke-copilot-aoai-apikey workflow for Azure OpenAI BYOK#37174

Merged
zarenner merged 5 commits into
mainfrom
copilot/smoke-copilot-aoai-apikey
Jun 5, 2026
Merged

Add smoke-copilot-aoai-apikey workflow for Azure OpenAI BYOK#37174
zarenner merged 5 commits into
mainfrom
copilot/smoke-copilot-aoai-apikey

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 5, 2026

Duplicates smoke-copilot.md as smoke-copilot-aoai-apikey.md (Smoke Copilot - AOAI (apikey)) to exercise the Copilot engine against an OpenAI-compatible Azure endpoint via BYOK using FOUNDRY_API_KEY / FOUNDRY_OPENAI_ENDPOINT.

Changes

  • New workflow .github/workflows/smoke-copilot-aoai-apikey.md — copy of smoke-copilot.md with:
    • engine.env wiring BYOK to the OpenAI-compatible endpoint:
      engine:
        id: copilot
        model: o4-mini-aw
        max-continuations: 2
        bare: true
        env:
          COPILOT_PROVIDER_BASE_URL: ${{ secrets.FOUNDRY_OPENAI_ENDPOINT }}
          COPILOT_PROVIDER_API_KEY: ${{ secrets.FOUNDRY_API_KEY }}
    • Renamed per-variant identifiers to avoid collision with the original: slash_command.name, add-labels.allowed, both close-older-keys, create-check-run.name, the issue title, and the check-run title.
  • Generated lock .github/workflows/smoke-copilot-aoai-apikey.lock.yml via ./gh-aw compile --validate.

Notes for review

  • COPILOT_PROVIDER_TYPE is intentionally left unset so the workflow exercises the default openai provider path against the Foundry endpoint; AOAI old-style URLs (azure provider type) can be covered by a separate variant later.
  • COPILOT_PROVIDER_* credential vars are allowlisted by the Copilot engine (ADR-29411), so the ${{ secrets.* }} references in engine.env pass strict-mode validation; the body text avoids ${{ secrets.* }} literals to stay within the markdown expression allowlist.
  • label_command (smoke) and remove-labels (smoke) are intentionally kept identical to the original so the existing smoke PR label triggers both variants.

Co-authored-by: zarenner <13670625+zarenner@users.noreply.github.com>
@zarenner
Copy link
Copy Markdown
Collaborator

zarenner commented Jun 5, 2026

@copilot change model to o4-mini-aw

Co-authored-by: zarenner <13670625+zarenner@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jun 5, 2026

@copilot change model to o4-mini-aw

Updated engine.model to o4-mini-aw and recompiled the lock file.

Copilot AI requested a review from zarenner June 5, 2026 19:16
Comment thread .github/workflows/smoke-copilot-aoai-apikey.md Outdated
Co-authored-by: zarenner <13670625+zarenner@users.noreply.github.com>
Copilot AI requested a review from zarenner June 5, 2026 20:41
Comment thread .github/workflows/smoke-copilot-aoai-apikey.md
Co-authored-by: zarenner <13670625+zarenner@users.noreply.github.com>
Co-authored-by: zarenner <13670625+zarenner@users.noreply.github.com>
@zarenner zarenner marked this pull request as ready for review June 5, 2026 22:26
Copilot AI review requested due to automatic review settings June 5, 2026 22:26
@zarenner zarenner merged commit af3df69 into main Jun 5, 2026
32 checks passed
@zarenner zarenner deleted the copilot/smoke-copilot-aoai-apikey branch June 5, 2026 22:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new Copilot smoke-test workflow variant that routes Copilot engine traffic to an OpenAI-compatible Azure endpoint via BYOK credentials (FOUNDRY_OPENAI_ENDPOINT / FOUNDRY_API_KEY), and wires it into the repo’s command/label routing.

Changes:

  • Added smoke-copilot-aoai-apikey workflow markdown variant with BYOK provider env wiring.
  • Added the compiled lock workflow for the new variant.
  • Updated agentic_commands.yml routing metadata to include the new slash command and smoke label routing.
Show a summary per file
File Description
.github/workflows/smoke-copilot-aoai-apikey.md New Copilot smoke workflow variant configured to use BYOK against a Foundry/AOAI OpenAI-compatible endpoint.
.github/workflows/smoke-copilot-aoai-apikey.lock.yml Generated compiled workflow for the new variant.
.github/workflows/agentic_commands.yml Adds the new command/workflow to routing so /smoke-copilot-aoai-apikey and smoke label can trigger it.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/3 changed files
  • Comments generated: 3

7. Discussion interaction: get latest discussion with `github-discussion-query` (`limit=1`, `jq=".[0]"`), extract number, then `add_comment` to that discussion.
8. Build: run `GOCACHE=/tmp/gh-aw/agent/go-cache GOMODCACHE=/tmp/gh-aw/agent/go-mod make build`.
9. Artifact upload (only if build passes): stage `./gh-aw` at `$RUNNER_TEMP/gh-aw/safeoutputs/upload-artifacts/gh-aw` and call `upload_artifact` with `path: "gh-aw"`.
10. Discussion create: call `create_discussion` in `announcements` with label `ai-generated`, title `copilot was here`, temp ID `aw_smoke_discussion`.
- Timestamp
- Pull request author and assignees

2. **Set Issue Type** (**required**): Use the `set_issue_type` safe-output tool with `issue_number: "aw_smoke1"` (the temporary ID from step 1) and `issue_type: "Bug"` to set the type of the just-created smoke test issue.
- Overall status: PASS or FAIL
- Mention the pull request author and any assignees

4. **Only if this workflow was NOT triggered by a pull_request event**: Use the `add_comment` tool to add a **fun and creative comment** to the newly created discussion (use the temporary ID `aw_smoke_discussion` from step 11) - be playful and entertaining in your comment
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.

3 participants