Add telemetry.enterprise.githubcopilot.com to copilot engine default allowlist#14007
Add telemetry.enterprise.githubcopilot.com to copilot engine default allowlist#14007
Conversation
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
…fault allowlist Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
🔍 PR Triage ResultsCategory: chore | Risk: low | Priority: 41/100 Scores Breakdown
📋 Recommended Action: deferDraft PR with low urgency. Defer until marked ready and CI passes. Small code change but large recompile footprint. Triaged by PR Triage Agent on 2026-02-06 - Run #21734096398
|
|
🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation... |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
❌ Changeset Generator failed. Please review the logs for details. |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
Agent Container Tool Check
Result: 11/12 tools available ❌
|
There was a problem hiding this comment.
Pull request overview
This pull request adds telemetry.enterprise.githubcopilot.com to the Copilot engine's default domain allowlist, enabling workflows to reach the enterprise telemetry endpoint without requiring explicit network configuration. The changes are straightforward and consist of updating the source domain list, its corresponding test, and regenerating all workflow lock files to reflect the new domain.
Changes:
- Added
telemetry.enterprise.githubcopilot.comto theCopilotDefaultDomainslist - Updated test expectations to include the new domain
- Regenerated all workflow lock files with the updated domain allowlist
Reviewed changes
Copilot reviewed 100 out of 100 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pkg/workflow/domains.go | Added telemetry.enterprise.githubcopilot.com to CopilotDefaultDomains array |
| pkg/workflow/domains_test.go | Updated test to expect the new telemetry domain |
| .github/workflows/*.lock.yml (multiple files) | Regenerated lock files with updated domain allowlist in both --allow-domains flags and GH_AW_ALLOWED_DOMAINS environment variables |
| .github/workflows/semantic-function-refactor.lock.yml | Also includes serena MCP server configuration addition (appears to be from a separate change) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
🎭 The smoke test agent appears from the digital mists 🎭 Greetings, dear humans! I have just completed my ceremonial rounds through the mystical realm of the Copilot engine, testing its many arcane powers. From navigating the great GitHub plains to summoning symbols with Serena's ancient magic, all systems are operational! The tests have spoken: ALL SYSTEMS GO ✅ May your workflows flow smoothly and your builds always succeed! 🚀 vanishes in a puff of automated testing smoke 💨
|
Smoke Test Results✅ GitHub MCP Status: PASS ✅ cc
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Adds
telemetry.enterprise.githubcopilot.comtoCopilotDefaultDomainsso copilot engine workflows can reach the enterprise telemetry endpoint without requiring explicit network configuration.CopilotDefaultDomainsinpkg/workflow/domains.goTestCopilotDefaultDomainsto expect the new domainOriginal prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.