Skip to content

Create BrowserStation auth secret in quickstart - #4

Open
mike-vincent wants to merge 2 commits into
ReinforceNow:mainfrom
mike-vincent:browserstation-auth-secret
Open

Create BrowserStation auth secret in quickstart#4
mike-vincent wants to merge 2 commits into
ReinforceNow:mainfrom
mike-vincent:browserstation-auth-secret

Conversation

@mike-vincent

Copy link
Copy Markdown

Adds the BrowserStation API key Kubernetes secret during quickstart so rayservice.yaml can resolve BROWSERSTATION_API_KEY when gateway auth is enabled.

Acceptance criteria:

  • quickstart creates/updates browserstation-auth before applying rayservice.yaml
  • bash -n scripts/quickstart.sh passes

…m k8s secret)

The gateway (uvicorn app.main:app on ray-head) reads BROWSERSTATION_API_KEY and
enforces X-API-Key on /browsers* when set, but the deploy never injected it, so
the endpoint was open (health 200 with any key). Inject it from the
browserstation-auth k8s secret so POST/GET/DELETE /browsers require the key.
Verified: no-key -> 401, with-key -> browser created; driver connects over CDP.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds a BROWSERSTATION_API_KEY environment variable to the Ray service configuration, sourced from a Kubernetes secret, and updates the quickstart script to conditionally create this secret if an API key is provided. The review feedback correctly points out that since the API key is optional, the missing secret will cause pod startup failures. It suggests marking the secret reference as optional to prevent this issue.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread rayservice.yaml
Comment on lines +21 to +25
- name: BROWSERSTATION_API_KEY
valueFrom:
secretKeyRef:
name: browserstation-auth
key: BROWSERSTATION_API_KEY

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Since the --api-key parameter in quickstart.sh is optional, the browserstation-auth secret will not be created if the user runs the quickstart without providing an API key. This will cause the Ray head pod to fail to start with a CreateContainerConfigError because of the missing secret reference. Marking the secretKeyRef as optional: true allows the pod to start successfully when the secret is not present.

            - name: BROWSERSTATION_API_KEY
              valueFrom:
                secretKeyRef:
                  name: browserstation-auth
                  key: BROWSERSTATION_API_KEY
                  optional: true

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.

1 participant