Skip to content

fix(ci): write Brev credentials file so CLI authenticates on runner #1639

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
cv merged 5 commits into from
Apr 9, 2026
+15 −0
Merged

fix(ci): write Brev credentials file so CLI authenticates on runner #1639

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
7af8b06
fix(ci): write Brev credentials file so CLI authenticates on runner
cjagwani Apr 8, 2026
ae764ff
Merge branch 'main' into fix/brev-e2e-auth-credentials
cjagwani Apr 8, 2026
7079b27
fix(ci): add fail-fast auth validation and file permissions
cjagwani Apr 8, 2026
03ec494
Merge branch 'main' into fix/brev-e2e-auth-credentials
cjagwani Apr 9, 2026
cc6634c
Merge branch 'main' into fix/brev-e2e-auth-credentials
cjagwani Apr 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions .github/workflows/e2e-brev.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,12 +144,27 @@ jobs:
cache: npm

- name: Install Brev CLI
env:
BREV_API_TOKEN: ${{ inputs.brev_token || secrets.BREV_API_TOKEN }}
run: |
# Brev CLI v0.6.322+ — CPU instances use `brev search cpu | brev create`
# Startup scripts use `brev create --startup-script @file` (not brev start --cpu)
curl -fsSL -o /tmp/brev.tar.gz "https://github.com/brevdev/brev-cli/releases/download/v0.6.322/brev-cli_0.6.322_linux_amd64.tar.gz"
tar -xzf /tmp/brev.tar.gz -C /usr/local/bin brev
chmod +x /usr/local/bin/brev
# Brev CLI does not read BREV_API_TOKEN from env — it requires
# ~/.brev/credentials.json. The login call was removed in #1470
# (374a847d), breaking CI. Write the credentials file so `brev ls`
# works in the test harness's hasAuthenticatedBrev check.
if [ -z "${BREV_API_TOKEN:-}" ]; then
echo "::error::BREV_API_TOKEN is empty — cannot authenticate Brev CLI."
exit 1
fi
mkdir -p ~/.brev
umask 077
printf '{"refresh_token":"%s"}' "$BREV_API_TOKEN" > ~/.brev/credentials.json
chmod 600 ~/.brev/credentials.json
brev ls >/dev/null

- name: Install dependencies
run: npm install --ignore-scripts
Expand Down
Loading