Skip to content

Fix Envoy inventory archive URL and HTTP status handling - #5366

Closed
phlax with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-30125649-398289295-38295866-aaa6-4e59-acd6-3344e8652679
Closed

phlax with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-30125649-398289295-38295866-aaa6-4e59-acd6-3344e8652679

Conversation

Copilot AI commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

project sync requests inventories from a nonexistent GCS bucket, leaving newer releases without inventories. It also accepts non-404 error responses as inventory content.

  • Archive URL: Use envoy-cncf-archive; preserve the /envoy/docs/v{version}/objects.inv path and ENVOY_DOCS_ARCHIVE_URL override.
  • Fetch handling: Accept only HTTP 200. Other statuses fall through to the legacy URL, returning None if neither succeeds.
  • Regression coverage: Update the expected URL and cover 200, 301, 403, 404, and 500 for both sources. Leave the package version unchanged.

@netlify

netlify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for nifty-bassi-e26446 ready!

Name Link
🔨 Latest commit a9c790a
🔍 Latest deploy log https://app.netlify.com/projects/nifty-bassi-e26446/deploys/6abfbfb6e1666e00087d5257
😎 Deploy Preview https://deploy-preview-5366--nifty-bassi-e26446.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Copilot AI changed the title [WIP] Copilot Request Fix Envoy inventory archive URL and HTTP status handling Oct 2, 2026
Copilot AI requested a review from phlax October 2, 2026 14:30
@phlax phlax closed this Oct 2, 2026
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.

2 participants