Skip to content

Upstreaming envoy changes#142

Merged
sjmiller609 merged 2 commits intomainfrom
envoy
Feb 4, 2026
Merged

Upstreaming envoy changes#142
sjmiller609 merged 2 commits intomainfrom
envoy

Conversation

@sjmiller609
Copy link
Contributor

@sjmiller609 sjmiller609 commented Feb 4, 2026

Note

High Risk
Adds an in-container Envoy proxy with dynamic xDS config, new external apt repo install, and runtime certificate trust-store/NSS DB modifications; misconfig or package issues can break container startup or alter outbound traffic behavior.

Overview
Adds an optional Envoy forward-proxy to both chromium-headful and chromium-headless images, managed by supervisord and initialized during container startup.

The images now install Envoy from apt.envoyproxy.io, ship an xDS bootstrap.yaml template, and run init-envoy.sh (triggered from wrapper.sh) to render config from INST_NAME/METRO_NAME/XDS_SERVER/XDS_JWT, generate and trust a self-signed proxy cert (system + Chromium NSS DB), start the envoy supervisor program, and validate proxy readiness on 127.0.0.1:3128.

Written by Cursor Bugbot for commit e2c0df4. This will update automatically on new commits. Configure here.

@sjmiller609 sjmiller609 requested a review from rgarcia February 4, 2026 19:54
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Copy link
Contributor

@rgarcia rgarcia left a comment

Choose a reason for hiding this comment

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

TestPlaywrightDaemonRecovery failing ... might be flaky but worth double checking

@sjmiller609 sjmiller609 merged commit 25f3499 into main Feb 4, 2026
7 of 8 checks passed
@sjmiller609 sjmiller609 deleted the envoy branch February 4, 2026 21:13
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