Local docker run by sonic16x · Pull Request #1338 · gemini-testing/testplane

sonic16x · 2026-09-30T17:04:07Z

No description provided.

pkg-pr-new · 2026-10-01T03:20:15Z

npm i https://pkg.pr.new/gemini-testing/testplane@1338

commit: 2fce926

github-actions · 2026-10-01T03:20:23Z

✅ Testplane browser-env run succeed

Report

github-actions · 2026-10-01T03:20:24Z

✅ Testplane E2E run succeed

Report

sonic16x · 2026-10-01T05:16:25Z

Extended WdProcess to allow asynchronous free and kill methods and optional methods for retrieving logs, saving session logs, and starting the CDP proxy. This lets Docker environments use the existing WebDriver process interface while ensuring cleanup completes before execution continues.

sonic16x · 2026-10-01T05:16:44Z

Added a Python proxy that runs inside the browser container and forwards HTTP and WebSocket requests to Chrome’s CDP endpoint. It connects Selenoid’s expected port 7070 to the browser’s actual debugger port and resolves the correct browser WebSocket path.

sonic16x · 2026-10-01T05:16:57Z

Added browser startup through Docker and Selenoid. The implementation checks Docker availability and image architecture, downloads missing images, generates browsers.json, creates a dedicated network and Selenoid container for each session, and waits for WebDriver readiness. It starts the CDP proxy inside the browser container when needed, collects startup diagnostics, saves session logs to <os.tmpdir()>/testplane-logs/<sessionId>.log, and removes the resources it created when the session ends.

sonic16x · 2026-10-01T05:17:23Z

Added support for gridUrl: "docker", including creating the Docker environment and connecting to its WebDriver endpoint. After a Chrome session starts, it prepares the CDP proxy; before deleting the session, it saves the logs once. Session cleanup now awaits asynchronous container removal. Startup errors include diagnostic logs, and cleanup failures are reported as warnings.

sonic16x · 2026-10-01T05:17:32Z

Added forwarding of options.docker to the browser configuration. This makes Docker mode available through launchBrowser() without running the test runner.

DudaGod

Я посмотрел локальный запуск браузеров в Docker. Ниже построчные замечания по совместимости образов, сети, безопасности и удобству запуска.

DudaGod · 2026-10-12T07:13:49Z

+        this._sessionIdForLogs = session.sessionId;
+        this._wdProcess?.setSessionId?.(session.sessionId);
+        const debuggerAddress = session.capabilities["goog:chromeOptions"]?.debuggerAddress;
+        if (debuggerAddress && !("browserWSEndpoint" in this._config && this._config.browserWSEndpoint)) {


Я бы не делал подготовку CDP обязательной при создании WebDriver-сессии. Пользовательский Chrome-образ может вернуть debuggerAddress, но не иметь devtools-адаптера, которого ожидает prepareCdp(). Тогда даже тест, использующий только WebDriver, не стартует. Можно ли инициализировать CDP только при первом обращении к нему?

DudaGod · 2026-10-12T07:13:49Z

+import { prepareDockerBrowserImage } from "./docker-image";
+import { timeDockerOperation } from "./docker-timing";
+
+const PLATFORM = "linux/amd64";


Здесь жёстко задан linux/amd64, поэтому на Apple Silicon нативный arm64-образ будет отвергнут, а multi-platform образ запустится через эмуляцию. Я бы выбирал архитектуру текущего Docker-движка, сохранив возможность явно запросить другую платформу.

DudaGod · 2026-10-12T07:13:49Z

+        await writeFile(configPath, JSON.stringify(browsers));
+        // Set before creation to also clean up when the CLI times out after creating the network.
+        networkCreated = true;
+        await timeDockerOperation(name, "create network", () => docker(["network", "create", network]));


Я бы проверил IPv6 именно в создаваемой здесь сети. Включённый IPv6 у стандартного bridge не означает, что новая пользовательская сеть тоже получит IPv6. Сейчас браузерный контейнер может не открыть IPv6-only сайт, хотя сама VM имеет к нему доступ.

DudaGod · 2026-10-12T07:13:49Z

+                network,
+                // This is the socket path on the daemon's Linux host, including Docker Desktop/Colima.
+                "--volume",
+                "/var/run/docker.sock:/var/run/docker.sock",


/var/run/docker.sock — путь на стороне Docker-демона, а не сокет из выбранного локального Docker context. У rootless Podman он обычно находится в другом месте, поэтому описанный в документации сценарий может не запустить Selenoid. Я бы добавил настройку способа подключения контроллера к движку и проверку этого сценария.

DudaGod · 2026-10-12T07:13:49Z

+                ...controllerSecurityArgs,
+                selenoidImage,
+                "-conf",
+                "/browsers.json",


Я бы передавал Selenoid -disable-privileged по умолчанию. Без этого флага он запускает браузерные контейнеры в privileged-режиме, а здесь пользователь может указать произвольный образ. Если отдельным образам нужны дополнительные привилегии, лучше включать их явно.

DudaGod · 2026-10-12T07:13:49Z

+    }
+
+    const { selenoidImage } = config || {};
+    if (typeof selenoidImage !== "string" || !selenoidImage.trim()) {


Я бы не требовал от пользователя выбирать selenoidImage для базового сценария. Его цель — указать свой совместимый браузерный образ и запустить тесты; образ контроллера выглядит внутренней деталью Testplane. Можно ли дать публичный закреплённый default и оставить это поле для переопределения?

DudaGod · 2026-10-12T07:13:49Z

+    const manager = runs.get(owner);
+    if (manager) return manager.acquire(browserId, config, options, closeBrowser);
+
+    await standaloneStopping;


Если state.manager.stop() отклонится, standaloneStopping навсегда останется отклонённым Promise. После временного сбоя Docker любой следующий standalone-запуск упадёт на этом await. Я бы очищал состояние после ошибки и добавил тест на повторный запуск.

DudaGod · 2026-10-12T07:13:49Z

+
+    it("should route Firefox sessions through /wd/hub inside the browser container", async () => {
+        const driver = await start(
+            "registry.yandex.net/selenium/firefox:148.0",


Я бы заменил адрес внутреннего registry в публичном тесте на нейтральный фикстурный адрес. Для проверки маршрута /wd/hub конкретный реестр не нужен.

sonic16x force-pushed the local-docker-run branch 2 times, most recently from 31261e5 to 4b60012 Compare October 1, 2026 03:17

sonic16x force-pushed the local-docker-run branch 2 times, most recently from 0cdb86f to d364192 Compare October 1, 2026 05:05

sonic16x commented Oct 1, 2026

View reviewed changes

sonic16x force-pushed the local-docker-run branch from d364192 to 98520eb Compare October 1, 2026 05:19

rocketraccoon added 5 commits October 8, 2026 14:05

feat: local docker browsers run

de402ac

feat: add cpu limit check

5d5193b

feat: add build docker with devtools fix

84c743f

feat: add docker prepare command

2f43c28

fix: fix podman + add docs

84dc6a5

sonic16x force-pushed the local-docker-run branch from b7b963c to 84dc6a5 Compare October 8, 2026 07:05

rocketraccoon added 2 commits October 12, 2026 01:54

fix: one selenoid for all browsers

6161f18

fix: move docker logs to debug section

2fce926

DudaGod reviewed Oct 12, 2026

View reviewed changes

Conversation

sonic16x commented Sep 30, 2026

Uh oh!

pkg-pr-new Bot commented Oct 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Oct 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Testplane browser-env run succeedReport

Uh oh!

github-actions Bot commented Oct 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Testplane E2E run succeedReport

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

DudaGod left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

pkg-pr-new Bot commented Oct 1, 2026 •

edited

Loading

github-actions Bot commented Oct 1, 2026 •

edited

Loading

✅ Testplane browser-env run succeed

Report

github-actions Bot commented Oct 1, 2026 •

edited

Loading

✅ Testplane E2E run succeed

Report