[msarahan]

Retire telemetry collection from CI so builds no longer schedule telemetry jobs, upload telemetry artifacts or depend on the shared telemetry actions. Keep build output in the normal job logs and remove dependencies on the retired setup.

https://github.com/rapidsai/build-infra/issues/416

[coderabbitai]

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration

• Configuration used: Repository: rapidsai/nvforest/.coderabbit.yaml
• Review profile: CHILL
• Plan: Enterprise
• Run ID: b9631e48-aa3f-4557-b5b9-f708c9b8cea5

📥 Commits

Reviewing files that changed from the base of the PR and between 2f095bd and 9ed7adc.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: rapidsai/nvforest/.coderabbit.yaml
• Review profile: CHILL
• Plan: Enterprise
• Run ID: b3af83dc-1e5d-44d7-9d7a-da59ab9e8d97

📥 Commits

Reviewing files that changed from the base of the PR and between 6cac0a9 and 2f095bd.

📒 Files selected for processing: 1

• .github/workflows/pr.yaml

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

📝 Summary

Summary by CodeRabbit

• Chores
  • CI workflows no longer run telemetry collection or summarization steps. Build and check jobs no longer depend on those steps, and build output is no longer saved to telemetry artifacts.
  • No changes to end-user functionality.

The PR workflow removes telemetry jobs and their dependencies. Devcontainer build and sccache-stat commands now write directly to standard output instead of telemetry artifact files.

Changes: PR workflow

Layer / File(s)	Summary
Remove telemetry jobs and wiring .github/workflows/pr.yaml	The workflow removes telemetry job configuration and its dependencies from pr-builder, changed-files, checks, and devcontainer. The devcontainer build and sccache-stat commands write directly to standard output.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 2f095

The workflow retires telemetry collection while keeping build output in job logs. No actionable merge risk is evident from the supplied changes.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title clearly and concisely identifies the main change: removing CI telemetry setup and reporting.
Description check		The description directly explains the telemetry removal, affected CI behavior, retained build logs, and related issue.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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