Skip to content

docs: add experimental Bazel build commands - #2600

Merged
krishicks merged 1 commit into
NVIDIA:mainfrom
rhuss:docs/bazel-experimental-commands
Aug 4, 2026
Merged

docs: add experimental Bazel build commands#2600
krishicks merged 1 commit into
NVIDIA:mainfrom
rhuss:docs/bazel-experimental-commands

Conversation

@rhuss

@rhuss rhuss commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Document Bazel build targets alongside existing mise commands in CONTRIBUTING.md, marked as experimental with a link to RFC 0012.

No issue required: documentation addition for an already-merged feature (#2414).

Changes

  • Added Bazelisk install instructions under Prerequisites (optional)
  • Added .bazelignore guidance for Cargo/Bazel coexistence
  • Added Bazel targets table under Main Tasks with [!IMPORTANT] admonition marking it experimental
  • Links to RFC 0012 and feedback channels (GitHub issues, CNCF Slack #openshell-dev)

Testing

  • mise run pre-commit passes
  • Markdown lint passes (heading levels validated)
  • Verified all Bazel target labels match existing BUILD.bazel files

Checklist

@copy-pr-bot

copy-pr-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@rhuss
rhuss force-pushed the docs/bazel-experimental-commands branch from 93db24d to 0fdb18a Compare August 3, 2026 16:55
@krishicks

Copy link
Copy Markdown
Collaborator

/ok to test 0fdb18a

@krishicks

Copy link
Copy Markdown
Collaborator

This will need a rebase to get over a flaky test. Sorry, @rhuss.

@krishicks krishicks self-assigned this Aug 3, 2026
Document Bazel targets alongside existing mise commands with a prominent
experimental notice linking to RFC 0012. Includes Bazelisk install
instructions, .bazelignore guidance for Cargo coexistence, and a mapping
of available build and test targets.

Signed-off-by: Roland Huß <rhuss@redhat.com>
@rhuss
rhuss force-pushed the docs/bazel-experimental-commands branch from 0fdb18a to 789a939 Compare August 4, 2026 09:19
@SDAChess

SDAChess commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

/ok-to-test 789a939

@rhuss

rhuss commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

The ARM64 CI failure is unrelated to this docs change. It's the watch_producer_releases_request_span_when_client_disconnects test in openshell-server failing with a timing assertion (expected span count 1, got 0).

This test was supposed to be stabilized by #2582 (merged Aug 1), but it's still flaking on ARM64. The fix may not be sufficient for the different scheduling behavior on ARM runners.

Needs a maintainer to re-run the failed job.

@krishicks

Copy link
Copy Markdown
Collaborator

I merged #2604 as a maybe actual fix for the tracing test failures.

@krishicks
krishicks added this pull request to the merge queue Aug 4, 2026
Merged via the queue into NVIDIA:main with commit d063751 Aug 4, 2026
38 of 39 checks 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.

3 participants