Skip to content

acc: Authenticate the latest release lookup when GITHUB_TOKEN is set - #7108

Merged
denik merged 1 commit into
mainfrom
denik/acc-github-token
Oct 19, 2026
Merged

denik merged 1 commit into
mainfrom
denik/acc-github-token

Conversation

@denik

@denik denik commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Unauthenticated GitHub API requests from shared CI runner IPs get a 403 (rate limit), which fails -useversion latest. Send GITHUB_TOKEN when it is set. Needed by #7105.

This pull request and its description were written by Isaac.

Co-authored-by: Isaac <no-reply@databricks.com>
@denik
denik marked this pull request as ready for review October 19, 2026 14:49
@denik
denik requested a review from a team as a code owner October 19, 2026 14:49
@denik
denik requested a review from janniklasrose October 19, 2026 14:49
@denik
denik enabled auto-merge October 19, 2026 14:54
@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 3c07c06

Run: 39271735790

Env ✅​pass 🙈​skip Time
✅​ aws linux-2core-8gb 276 15 5:27
✅​ aws-windows-latest-4core-16gb 278 13 4:31
✅​ azure linux-2core-8gb 275 15 5:21
✅​ azure-windows-latest-4core-16gb 277 13 4:04
✅​ gcp linux-2core-8gb 276 15 5:24
✅​ gcp-windows-latest-4core-16gb 278 13 4:18
Top 6 slowest tests (at least 2 minutes):
duration env testname
4:25 aws-windows-latest-4core-16gb TestAccept
4:12 gcp-windows-latest-4core-16gb TestAccept
3:59 azure-windows-latest-4core-16gb TestAccept
3:57 azure linux-2core-8gb TestAccept
3:52 aws linux-2core-8gb TestAccept
3:48 gcp linux-2core-8gb TestAccept

@denik
denik added this pull request to the merge queue Oct 19, 2026
Merged via the queue into main with commit 1f33f2c Oct 19, 2026
26 checks passed
@denik
denik deleted the denik/acc-github-token branch October 19, 2026 15:24
ruslantalpa pushed a commit to ruslantalpa/cli that referenced this pull request Oct 21, 2026
…low (databricks#7119)

Follow-up to databricks#7105 (added the
regression-report workflow) and
databricks#7108 (authenticated the latest
release lookup).

The workflow used its own cache key, which is only written by runs on
main. It never runs on main, so every run started with a cold Go cache.
Restore the key that the Linux test job saves instead. A full run goes
from 505-587s to 292s, a run with no acceptance changes from 370s to
196s.

This pull request and its description were written by Isaac.

---------

Co-authored-by: Isaac <no-reply@databricks.com>
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.

3 participants