Skip to content

Allow GitHub domains in Daily Go Test Parallelizer - #59736

Merged
pelikhan merged 4 commits into
mainfrom
copilot/deep-report-add-missing-github-network-preset
Sep 9, 2026
Merged

Allow GitHub domains in Daily Go Test Parallelizer#59736
pelikhan merged 4 commits into
mainfrom
copilot/deep-report-add-missing-github-network-preset

Conversation

Copilot AI commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Daily Go Test Parallelizer blocked GitHub raw-content and web requests because its network configuration omitted the github ecosystem preset.

  • Network access
    • Add github to network.allowed.
    • Regenerate the compiled workflow lock file.
network:
  allowed:
    - defaults
    - github
    - go
    - node

Copilot AI and others added 3 commits September 9, 2026 13:30
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
This reverts commit bc2e167.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add missing github network preset to Daily Go Test Parallelizer Allow GitHub domains in Daily Go Test Parallelizer Sep 9, 2026
Copilot AI requested a review from pelikhan September 9, 2026 13:39
@pelikhan
pelikhan marked this pull request as ready for review September 9, 2026 14:01
Copilot AI balanced review requested due to automatic review settings September 9, 2026 14:01
@pelikhan
pelikhan merged commit ba17838 into main Sep 9, 2026
@pelikhan
pelikhan deleted the copilot/deep-report-add-missing-github-network-preset branch September 9, 2026 14:01
Copilot stopped reviewing on behalf of pelikhan due to an error September 9, 2026 14:02

Copilot AI 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.

Note

Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.

Pull request overview

This PR expands the workflow’s network allowlist to include GitHub-related domains, likely to support fetching resources from GitHub during the daily Go test parallelizer run.

Changes:

  • Added github to the workflow network allowed list.
  • Updated the generated lock workflow to reflect the expanded allowed domain set (GitHub hostnames + model list changes).
Show a summary per file
File Description
.github/workflows/daily-go-test-parallelizer.md Adds github to the allowed network domain groups.
.github/workflows/daily-go-test-parallelizer.lock.yml Regenerates the locked workflow to include GitHub domains in the firewall allowlist/env.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/2 changed files
  • Comments generated: 2
  • Review effort level: Lite

env:
GH_AW_SAFE_OUTPUTS: ${{ steps.set-runtime-paths.outputs.GH_AW_SAFE_OUTPUTS }}
GH_AW_ALLOWED_DOMAINS: "api.npms.io,api.snapcraft.io,archive.ubuntu.com,azure.archive.ubuntu.com,bun.sh,cdn.jsdelivr.net,crl.geotrust.com,crl.globalsign.com,crl.identrust.com,crl.sectigo.com,crl.thawte.com,crl.usertrust.com,crl.verisign.com,crl3.digicert.com,crl4.digicert.com,crls.ssl.com,deb.nodesource.com,deno.land,esm.sh,get.pnpm.io,go.dev,golang.org,googleapis.deno.dev,googlechromelabs.github.io,goproxy.io,json-schema.org,json.schemastore.org,jsr.io,keyserver.ubuntu.com,nodejs.org,npm.pkg.github.com,npmjs.com,npmjs.org,ocsp.digicert.com,ocsp.geotrust.com,ocsp.globalsign.com,ocsp.identrust.com,ocsp.sectigo.com,ocsp.ssl.com,ocsp.thawte.com,ocsp.usertrust.com,ocsp.verisign.com,packagecloud.io,packages.cloud.google.com,packages.microsoft.com,pkg.go.dev,ppa.launchpad.net,proxy.golang.org,registry.bower.io,registry.npmjs.com,registry.npmjs.org,registry.yarnpkg.com,repo.yarnpkg.com,s.symcb.com,s.symcd.com,security.ubuntu.com,skimdb.npmjs.com,storage.googleapis.com,sum.golang.org,telemetry.vercel.com,ts-crl.ws.symantec.com,ts-ocsp.ws.symantec.com,www.googleapis.com,www.npmjs.com,www.npmjs.org,yarnpkg.com"
GH_AW_ALLOWED_DOMAINS: "*.githubusercontent.com,api.npms.io,api.snapcraft.io,archive.ubuntu.com,azure.archive.ubuntu.com,bun.sh,cdn.jsdelivr.net,codeload.github.com,crl.geotrust.com,crl.globalsign.com,crl.identrust.com,crl.sectigo.com,crl.thawte.com,crl.usertrust.com,crl.verisign.com,crl3.digicert.com,crl4.digicert.com,crls.ssl.com,deb.nodesource.com,deno.land,docs.github.com,esm.sh,get.pnpm.io,github-cloud.githubusercontent.com,github-cloud.s3.amazonaws.com,github.blog,github.com,github.githubassets.com,go.dev,golang.org,googleapis.deno.dev,googlechromelabs.github.io,goproxy.io,json-schema.org,json.schemastore.org,jsr.io,keyserver.ubuntu.com,lfs.github.com,nodejs.org,npm.pkg.github.com,npmjs.com,npmjs.org,objects.githubusercontent.com,ocsp.digicert.com,ocsp.geotrust.com,ocsp.globalsign.com,ocsp.identrust.com,ocsp.sectigo.com,ocsp.ssl.com,ocsp.thawte.com,ocsp.usertrust.com,ocsp.verisign.com,packagecloud.io,packages.cloud.google.com,packages.microsoft.com,patch-diff.githubusercontent.com,patchdiff.githubusercontent.com,pkg.go.dev,ppa.launchpad.net,proxy.golang.org,raw.githubusercontent.com,registry.bower.io,registry.npmjs.com,registry.npmjs.org,registry.yarnpkg.com,repo.yarnpkg.com,s.symcb.com,s.symcd.com,security.ubuntu.com,skimdb.npmjs.com,storage.googleapis.com,sum.golang.org,telemetry.vercel.com,ts-crl.ws.symantec.com,ts-ocsp.ws.symantec.com,www.googleapis.com,www.npmjs.com,www.npmjs.org,yarnpkg.com"
Comment on lines 20 to 25
network:
allowed:
- defaults
- github
- go
- node
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.

[deep-report] Add missing github network preset to Daily Go Test Parallelizer (183+46 blocked requests/day)

3 participants