Skip to content

Add telemetry.enterprise.githubcopilot.com to copilot engine default allowlist - #14007

Merged
pelikhan merged 3 commits into
mainfrom
copilot/add-telemetry-domain-allowlist
Feb 6, 2026
Merged

Add telemetry.enterprise.githubcopilot.com to copilot engine default allowlist#14007
pelikhan merged 3 commits into
mainfrom
copilot/add-telemetry-domain-allowlist

Conversation

CopilotAI commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Adds telemetry.enterprise.githubcopilot.com to CopilotDefaultDomains so copilot engine workflows can reach the enterprise telemetry endpoint without requiring explicit network configuration.

  • Added domain to CopilotDefaultDomains in pkg/workflow/domains.go
  • Updated TestCopilotDefaultDomains to expect the new domain
  • Recompiled all workflow lock files
Original prompt

Add telemetry.enterprise.githubcopilot.com to copilot engine by default as allowlist of domains


💡 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.

CopilotAIand others added 2 commits February 6, 2026 00:06
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
…fault allowlist
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
CopilotAI changed the title [WIP] Add telemetry.enterprise.githubcopilot.com to allowlist of domainsAdd telemetry.enterprise.githubcopilot.com to copilot engine default allowlistFeb 6, 2026
CopilotAI requested a review from MossakaFebruary 6, 2026 00:17
@github-actions

Copy link
Copy Markdown
Contributor

🔍 PR Triage Results

Category: chore | Risk: low | Priority: 41/100

Scores Breakdown

  • Impact: 25/50 - Adds telemetry domain to copilot engine default allowlist. Enables enterprise telemetry without manual configuration. Medium impact - helpful but not critical.
  • Urgency: 8/30 - Very recent PR (< 1 hour old). Improves telemetry access but not blocking. Low urgency.
  • Quality: 8/20 - Draft status, CI pending. Good description. 100 files changed (recompiled workflows). Needs CI validation.

📋 Recommended Action: defer

Draft 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

AI generated by PR Triage Agent

@pelikhan
pelikhan marked this pull request as ready for review February 6, 2026 00:47
CopilotAI review requested due to automatic review settings February 6, 2026 00:47
@github-actions

github-actionsBot commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

@github-actions

github-actionsBot commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

🎬 THE ENDSmoke ClaudeMISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions

github-actionsBot commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Changeset Generator failed. Please review the logs for details.

@github-actions

Copy link
Copy Markdown
Contributor

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions

Copy link
Copy Markdown
Contributor

Agent Container Tool Check

ToolStatusVersion
bash5.2.21
shavailable
git2.52.0
jq1.7
yq4.52.2
curl8.5.0
gh2.86.0
node20.20.0
python33.12.3
go1.24.12
java5.2.21
dotnetN/A

Result: 11/12 tools available ❌

⚠️Missing: .NET runtime is not available in the container.

AI generated by Agent Container Smoke Test

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.com to the CopilotDefaultDomains list
  • 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.

FileDescription
pkg/workflow/domains.goAdded telemetry.enterprise.githubcopilot.com to CopilotDefaultDomains array
pkg/workflow/domains_test.goUpdated 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.ymlAlso 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.

@github-actions

Copy link
Copy Markdown
Contributor

🎭 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 💨

AI generated by Smoke Copilot

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test Results

PRs Tested: #13996, #14003

✅ GitHub MCP
✅ Safe Inputs GH CLI
✅ Serena MCP (134 symbols)
✅ Playwright
✅ File ops
✅ Discussion interaction
✅ Build gh-aw
✅ Workflow dispatch

Status: PASS ✅

cc @pelikhan

AI generated by Smoke Copilot

@github-actions

Copy link
Copy Markdown
Contributor

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@pelikhan@Mossaka