Skip to content

CNTRLPLANE-3945: fix(docs): set writable uv cache path for arc-runner-set - #9281

Merged
celebdor merged 1 commit into
mainfrom
fix-uv-cache-docs-build
Aug 11, 2026
Merged

CNTRLPLANE-3945: fix(docs): set writable uv cache path for arc-runner-set#9281
celebdor merged 1 commit into
mainfrom
fix-uv-cache-docs-build

Conversation

@celebdor

@celebdor celebdor commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • build / Build Docs CI check passes on this PR

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated documentation build configuration to use a temporary local cache directory for tooling, improving build environment isolation.

On arc-runner-set runners, HOME=/ so setup-uv resolves its cache
directory to /.cache/uv which is not writable, causing the docs build
to fail with "Permission denied". Use runner.temp for the cache path.

Fixes: CNTRLPLANE-3945

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com>
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: LGTM mode

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Aug 11, 2026
@openshift-ci-robot

openshift-ci-robot commented Aug 11, 2026

Copy link
Copy Markdown

@celebdor: This pull request references CNTRLPLANE-3945 which is a valid jira issue.

Details

In response to this:

Summary

Test plan

  • build / Build Docs CI check passes on this PR

🤖 Generated with Claude Code

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: f86b155f-13d1-45ff-9a29-f29a4cc3f608

📥 Commits

Reviewing files that changed from the base of the PR and between 6c3f984 and d2412ad.

📒 Files selected for processing (1)
  • .github/workflows/docs-build-reusable.yaml

📝 Walkthrough

Walkthrough

The reusable documentation workflow configures setup-uv to store its local cache at ${{ runner.temp }}/.uv-cache. No exported or public entities change.

Suggested reviewers: mgencur

🚥 Pre-merge checks | ✅ 11
✅ Passed checks (11 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the documentation fix and the writable UV cache path for arc-runner-set runners.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed The PR changes only a workflow cache path and adds no Ginkgo test declarations or test titles; the stability check is not applicable.
Test Structure And Quality ✅ Passed The PR changes only .github/workflows/docs-build-reusable.yaml; no Ginkgo test code or It blocks were added or modified, so this check is not applicable.
Topology-Aware Scheduling Compatibility ✅ Passed The change only updates setup-uv cache path in a GitHub Actions workflow; it adds no deployment manifest, operator code, controller, or scheduling constraint.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The PR changes only the docs workflow and adds no Ginkgo e2e tests, network assumptions, or external connectivity requirements.
No-Weak-Crypto ✅ Passed The PR changes only a workflow cache path; the commit diff and changed-file scan show no weak crypto, custom crypto, or secret/token comparison.
Container-Privileges ✅ Passed The PR only adds a writable setup-uv cache path in a GitHub Actions workflow; it changes no container or Kubernetes manifest and adds no listed privilege settings.
No-Sensitive-Data-In-Logs ✅ Passed The committed change adds only the runner-temporary uv cache path; it adds no logging and exposes no passwords, tokens, API keys, PII, hostnames, or customer data.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-uv-cache-docs-build

Comment @coderabbitai help to get the list of available commands.

@celebdor celebdor added the area/ci-tooling Indicates the PR includes changes for CI or tooling label Aug 11, 2026
@openshift-ci
openshift-ci Bot requested a review from Nirshal August 11, 2026 17:32
@openshift-ci

openshift-ci Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: celebdor
Once this PR has been reviewed and has the lgtm label, please assign csrwng 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

@openshift-ci
openshift-ci Bot requested a review from sdminonne August 11, 2026 17:32
@openshift-ci

openshift-ci Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

@celebdor: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/images d2412ad link true /test images

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@celebdor
celebdor merged commit 52557a1 into main Aug 11, 2026
18 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci-tooling Indicates the PR includes changes for CI or tooling jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants