Skip to content

[guide] docs: document HIVE_SESSION multi-session contributor label (contributor-relay.md, env-vars.md) - #5608

Closed
kubestellar-hive[bot] wants to merge 1 commit into
v4from
guide/docs-hive-session
Closed

[guide] docs: document HIVE_SESSION multi-session contributor label (contributor-relay.md, env-vars.md)#5608
kubestellar-hive[bot] wants to merge 1 commit into
v4from
guide/docs-hive-session

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Documentation Fix

Documents the HIVE_SESSION multi-session contributor label added in b34254e ("contribute: allow multiple concurrent sessions per GitHub account"), which was previously discoverable only by reading bin/contributor-relay.sh.

Files/cluster claimed (docs only — no source code touched):

  • src/docs/contributor-relay.md — new HIVE_SESSION row in the "Important environment variables" table + new section "Running multiple backends under one GitHub account" (default = backend name, same-backend labeling, empty-string opt-out, per-account limits shared across sessions).
  • src/docs/env-vars.md — new HIVE_SESSION row in the "Contributor relay and top-level helper scripts" table, linking to the new section.

Disjoint from open hold-gated PRs #5559 (UPGRADE.md) and #4032 (proxy source).

Fixes #5605


Filed by guide agent (ACMM L5 — hold-gated mode). Hold-gated: human review required.

— hive: agent=guide backend=copilot model=claude-sonnet-4-6

Commit b34254e added an optional session label so one GitHub account can
run multiple concurrent contributor relays (independent task leases,
cooldowns, and failure streaks per ContributorID#session) while auth,
trust tier, model admission, and rate limits stay per-account. The
feature was only discoverable by reading bin/contributor-relay.sh.

Add a HIVE_SESSION row to the contributor-relay.md env table and the
env-vars.md contributor-relay section, plus a short 'Running multiple
backends under one GitHub account' section covering the default (backend
name), same-backend labeling, the empty-string opt-out, and shared
per-account limits.

Fixes #5605

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: sec-check <sec-check@hive.kubestellar.io>
@kubestellar-hive kubestellar-hive Bot added the hold label Sep 2, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign danathar for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Sep 2, 2026
@kubestellar-prow kubestellar-prow Bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 2, 2026
@kubestellar-hive kubestellar-hive Bot added documentation Improvements or additions to documentation agent/guide Approved by a Hive merger/owner for auto-merge on green CI hive/hosted-available-oke-11-placeholder-r05x Approved by a Hive merger/owner for auto-merge on green CI labels Sep 2, 2026
clubanderson added a commit that referenced this pull request Sep 2, 2026
Ported from the duplicate PR #5608: make explicit that two same-backend
relays with identical labels share one session identity and one
active-task slot — distinct labels are the whole point.

Signed-off-by: Andy Anderson <andy@clubanderson.com>
@clubanderson

Copy link
Copy Markdown
Member

Closing as a duplicate: this and #5613 both document the HIVE_SESSION multi-session label for #5605, and #5613 is the one moving forward. It carries the same table rows and multi-backend section, plus two things this PR misses:

  1. The container path. Current v4 forwards HIVE_SESSION in neither the just contribute-hive docker run nor src/compose-contributor.yaml, so an explicit label set on the host is silently dropped in the default docker mode — docs: document HIVE_SESSION multi-session label for contributors #5613 adds the forwarding (empty-preserving ${VAR+...} form, since the empty string is the opt-out) so the documented examples are actually true.
  2. The launch form. The AGENT_BACKEND=claude just contribute-hive example here does not select the backend: the recipe sources contributor.env with set -a before the ${AGENT_BACKEND:-claude} fallback, so the stored setup value clobbers the shell variable. The positional just contribute-hive claude form is the reliable one.

Your note that identical labels still collide on one task slot was a genuinely useful clarification and has been ported into #5613. Thanks — the gap was real, the surviving PR just closes it end to end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent/guide Approved by a Hive merger/owner for auto-merge on green CI copilot dco-signoff: yes Indicates the PR's author has signed the DCO. documentation Improvements or additions to documentation hive/hosted-available-oke-11-placeholder-r05x Approved by a Hive merger/owner for auto-merge on green CI hold size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[guide] contributor docs: HIVE_SESSION multi-session label (b34254e) missing from contributor-relay.md and env-vars.md

1 participant