Skip to content

Remove tools/ and ci/ from copybara pin#39

Merged
ddelnano merged 1 commit into
mainfrom
ddelnano/remove-tools-ci-pins-from-copybara
May 16, 2026
Merged

Remove tools/ and ci/ from copybara pin#39
ddelnano merged 1 commit into
mainfrom
ddelnano/remove-tools-ci-pins-from-copybara

Conversation

@ddelnano
Copy link
Copy Markdown

Summary: Remove tools/ and ci/ from copybara pin

Relevant Issues: N/A

Type of change: /kind cleanup

Test Plan: Re-run copybara action and verify it passes (currently failing)

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 16, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: e2d39b11-fc13-4551-a265-1f151c6562da

📥 Commits

Reviewing files that changed from the base of the PR and between 57e7026 and 8174141.

📒 Files selected for processing (3)
  • .github/workflows/copybara_pixie_oss.yaml
  • ci/private/run_copybara.sh
  • tools/private/copybara/copy.bara.sky

📝 Walkthrough

Walkthrough

The PR relocates Copybara infrastructure to private directories: the GitHub Actions workflow now invokes ci/private/run_copybara.sh with tools/private/copybara/copy.bara.sky. The Copybara config removes tools/** from ignored_dirs and updates fork_only_files to include ci/private/** and tools/private/**.

Changes

Copybara infrastructure reorganization to private directories

Layer / File(s) Summary
Workflow invocation update
.github/workflows/copybara_pixie_oss.yaml
GitHub Actions workflow is updated to invoke the Copybara script from ci/private/ and reference the config file from tools/private/copybara/ instead of the prior public locations.
Copybara path filtering configuration
tools/private/copybara/copy.bara.sky
Copybara configuration filters are updated: tools/** removed from ignored_dirs so tools/ is included in origin files; fork_only_files replaces ci/run_copybara.sh with ci/private/** and adds tools/private/**, keeping the other fork-only entries.

🎯 2 (Simple) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: removing tools/ and ci/ paths from the Copybara configuration pinning.
Description check ✅ Passed The description is clearly related to the changeset, explaining the removal of tools/ and ci/ from Copybara pin with context about testing.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ddelnano/remove-tools-ci-pins-from-copybara

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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

@ddelnano ddelnano temporarily deployed to pr-actions-approval May 16, 2026 20:44 — with GitHub Actions Inactive
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano ddelnano force-pushed the ddelnano/remove-tools-ci-pins-from-copybara branch from 57e7026 to 8174141 Compare May 16, 2026 20:46
@ddelnano ddelnano temporarily deployed to pr-actions-approval May 16, 2026 20:46 — with GitHub Actions Inactive
@ddelnano ddelnano changed the title Remove tools/ and ci/ from copybara pin Remove tools/ and ci/ from copybara pin May 16, 2026
@ddelnano ddelnano merged commit 3951370 into main May 16, 2026
11 of 15 checks passed
entlein pushed a commit that referenced this pull request Jun 2, 2026
…ins-from-copybara

Remove `tools/` and `ci/` from copybara pin
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