Skip to content

Use repo variable for Anthropic base URL - #6672

Merged
eleftherias merged 1 commit into
mainfrom
use-anthropic-base-url-variable
Sep 16, 2026
Merged

eleftherias merged 1 commit into
mainfrom
use-anthropic-base-url-variable

Conversation

@eleftherias

@eleftherias eleftherias commented Sep 16, 2026 •

Copy link
Copy Markdown
Member

Summary

The Anthropic llm-gateway base URL is hardcoded in the Claude action workflows, so repointing the gateway means editing every workflow file. This moves it into a repo-level ANTHROPIC_BASE_URL variable so the URL can be changed in one place.

  • Replace the literal https://llm-gateway.stacklok.dev/anthropic with ${{ vars.ANTHROPIC_BASE_URL }} in claude.yml, issue-triage.yml and release-notes.yml (3 occurrences).
  • ⚠️ The repo-level ANTHROPIC_BASE_URL variable must be set before this merges — if unset, the env var resolves to an empty string and the action falls back to the public Anthropic API.

Type of change

  • Other (describe): internal CI/workflow configuration

Test plan

  • Manual testing (describe below)

Workflow-only change with no code surface:

  • yq -e '.' parses cleanly on all 3 changed files, and the parsed value of the env var is byte-identical to the already-merged docs-website reference that is running in production.
  • git diff reviewed line by line: 3 single-line replacements, nothing else touched.

API Compatibility

  • This PR does not break the v1beta1 API, OR the api-break-allowed label is applied and the migration guidance is described above.

Changes

File Change
.github/workflows/claude.yml ANTHROPIC_BASE_URL now reads from vars.ANTHROPIC_BASE_URL
.github/workflows/issue-triage.yml same
.github/workflows/release-notes.yml same

Does this introduce a user-facing change?

No.

Move the llm-gateway base URL out of the Claude action workflows and
into the ANTHROPIC_BASE_URL repo variable.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@eleftherias
eleftherias requested a review from JAORMX as a code owner September 16, 2026 15:30
@github-actions github-actions Bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Sep 16, 2026
@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.11%. Comparing base (84e3bea) to head (024c131).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6672      +/-   ##
==========================================
- Coverage   79.11%   79.11%   -0.01%     
==========================================
  Files         785      785              
  Lines       78460    78458       -2     
==========================================
- Hits        62076    62070       -6     
- Misses      16379    16383       +4     
  Partials        5        5              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions Bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels Sep 16, 2026
@eleftherias
eleftherias merged commit 9208d07 into main Sep 16, 2026
46 checks passed
@eleftherias
eleftherias deleted the use-anthropic-base-url-variable branch September 16, 2026 20:05
@github-actions github-actions Bot mentioned this pull request Sep 18, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants