Skip to content

ci(maintain-deploy): add SKIP_UPSTREAM_BRANCHES; skip mirrored sgl-project/sglang#23003#11

Merged
DavidBellamy merged 1 commit intollm360-mainfrom
ci/skip-upstream-prs-already-mirrored
Apr 19, 2026
Merged

ci(maintain-deploy): add SKIP_UPSTREAM_BRANCHES; skip mirrored sgl-project/sglang#23003#11
DavidBellamy merged 1 commit intollm360-mainfrom
ci/skip-upstream-prs-already-mirrored

Conversation

@DavidBellamy
Copy link
Copy Markdown
Collaborator

Port harbor's `SKIP_UPSTREAM_BRANCHES` pattern to sglang. Removes the repeat merge-conflict on the upstream branch `fix/json-ib-device-passthrough` now that its content was mirrored into deploy via #9.

Why

With PR #10 (decouple) in effect, the octopus now discovers open PRs on `sgl-project/sglang:main` with LLM360-owned heads. The very first such PR — sgl-project#23003 — has branch `fix/json-ib-device-passthrough` based on upstream `main`, so it line-drift-conflicts when merged onto `llm360-main`. That branch was already cherry-picked cleanly as #9 (`fix/json-ib-device-passthrough-llm360-main`), so re-merging the upstream branch is redundant.

What this does

  • Adds env var `SKIP_UPSTREAM_BRANCHES` with inline doc string.
  • Computes a jq `NOT_SKIPPED` filter and applies it to both:
    • `UPSTREAM_PR_STATE`/`UPSTREAM_BRANCHES` (cross-fork PRs into `LLM360/sglang:llm360-main`)
    • `REAL_UPSTREAM_PR_STATE`/`REAL_UPSTREAM_BRANCHES` (upstream `sgl-project/sglang:main` PRs with LLM360 heads)
  • Skipped branches drop out of the state fingerprint too, so the skip doesn't re-trigger churn.

Maintenance

The upstream PR sgl-project#23003 stays open to track landing in `sgl-project/sglang`. When it merges upstream (or gets closed), remove the entry from `SKIP_UPSTREAM_BRANCHES`.

…project#23003

Upstream PR sgl-project#23003 (fix/json-ib-device-passthrough) is
superseded by #9 (fix/json-ib-device-passthrough-llm360-main),
which mirrors the same 7-line patch on top of llm360-main. The upstream
branch was based on sgl-project/sglang:main and line-drifts when merged
into llm360-main — was triggering a merge conflict in every octopus run
after the PR #10 decouple landed.

Port harbor's SKIP_UPSTREAM_BRANCHES pattern: env var + jq `NOT_SKIPPED`
filter applied to both UPSTREAM_PR_STATE/BRANCHES (cross-fork PRs on
LLM360/sglang:llm360-main) and REAL_UPSTREAM_PR_STATE/BRANCHES (upstream
sgl-project/sglang PRs). Removes the skipped branch cleanly from the
state fingerprint too, so re-running doesn't see it as a pending change.
@DavidBellamy DavidBellamy merged commit e244a45 into llm360-main Apr 19, 2026
1 check passed
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