Skip to content

misc: Add manual perf workflow title metadata#799

Merged
jensen-yan merged 1 commit intoxs-devfrom
mannual-ci-title
Mar 20, 2026
Merged

misc: Add manual perf workflow title metadata#799
jensen-yan merged 1 commit intoxs-devfrom
mannual-ci-title

Conversation

@happy-lx
Copy link
Copy Markdown
Contributor

@happy-lx happy-lx commented Mar 18, 2026

Change-Id: I7230fe9557206ac809f557d6c1201023c8c6a3dc

Summary by CodeRabbit

  • Chores
    • Enhanced manual performance testing workflow with improved run identification and optional notes support for better test organization and tracking.

Change-Id: I7230fe9557206ac809f557d6c1201023c8c6a3dc
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9e917886-51df-4612-b62b-fe194010e541

📥 Commits

Reviewing files that changed from the base of the PR and between 6b87b99 and 9297ee0.

📒 Files selected for processing (1)
  • .github/workflows/manual-perf.yml

📝 Walkthrough

Walkthrough

Updated the GitHub workflow file for manual performance testing by introducing a dynamic run-name that includes test parameters and adding a new optional "note" input to allow users to add descriptive text to workflow runs.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/manual-perf.yml
Added dynamic run-name combining note, configuration, benchmark_type, specific_benchmarks, branch/ref, and vector_type parameters. Introduced optional "note" input with default value 'Manual Performance Test' for workflow_dispatch trigger.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Yakkhini
  • CJ362ff

Poem

🐰 A note to remember, a name to be seen,
Workflow runs sparkle with details pristine,
Performance tests dance through parameters clear,
Dynamic run-names make logs bright and dear! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'misc: Add manual perf workflow title metadata' directly relates to the changeset, which adds a dynamic run-name and optional note input to the manual performance workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 mannual-ci-title
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Tip

CodeRabbit can suggest fixes for GitHub Check annotations.

Configure the reviews.tools.github-checks setting to adjust the time to wait for GitHub Checks to complete.

@happy-lx happy-lx requested a review from jensen-yan March 18, 2026 03:14
@github-actions
Copy link
Copy Markdown

🚀 Coremark Smoke Test Results

Branch IPC Change
Base (xs-dev) 2.2665 -
This PR 2.2665 ➡️ 0.0000 (0.00%)

✅ Difftest smoke test passed!

@jensen-yan jensen-yan merged commit a36a878 into xs-dev Mar 20, 2026
4 of 6 checks passed
@jensen-yan jensen-yan deleted the mannual-ci-title branch March 20, 2026 06:29
happy-lx added a commit that referenced this pull request Apr 1, 2026
Change-Id: I7230fe9557206ac809f557d6c1201023c8c6a3dc
happy-lx added a commit that referenced this pull request Apr 1, 2026
Change-Id: I7230fe9557206ac809f557d6c1201023c8c6a3dc
happy-lx added a commit that referenced this pull request Apr 1, 2026
Change-Id: I7230fe9557206ac809f557d6c1201023c8c6a3dc
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.

2 participants