Skip to content

[improve][ci] update pulsarbot to use github-script based implementation#24940

Merged
lhotari merged 12 commits intomasterfrom
pulsarbot
Jan 14, 2026
Merged

[improve][ci] update pulsarbot to use github-script based implementation#24940
lhotari merged 12 commits intomasterfrom
pulsarbot

Conversation

@liangyepianzhou
Copy link
Copy Markdown
Contributor

Motivation

The ci-pulsarbot.yaml is slightly out-dated, update pulsarbot with github-script

Verifying this change

test in liangyepianzhou#19

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@github-actions github-actions Bot added the doc-required Your PR changes impact docs and you will update later. label Nov 4, 2025
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Comment thread .github/workflows/ci-pulsarbot.yaml
xiangying added 2 commits November 4, 2025 17:08
fix
(cherry picked from commit 678c92b)
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Co-authored-by: Lari Hotari <lhotari@users.noreply.github.com>
lhotari
lhotari previously approved these changes Nov 4, 2025
Copy link
Copy Markdown
Member

@lhotari lhotari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lhotari
Copy link
Copy Markdown
Member

lhotari commented Nov 4, 2025

/pulsarbot rerun-failure-checks

@lhotari lhotari changed the title [improve][ci] update pulsarbot with github-script [improve][ci] update pulsarbot to use github-script based implementation Nov 4, 2025
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
@lhotari lhotari requested a review from Copilot November 4, 2025 15:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR replaces the external apache/pulsar-test-infra/pulsarbot@master action with an inline implementation using actions/github-script@v7. The change provides direct control over the pulsarbot functionality for managing PR workflow runs.

Key changes:

  • Migrated from external action to inline JavaScript implementation
  • Implements commands: /pulsarbot rerun, /pulsarbot rerun <jobname>, /pulsarbot stop/cancel, and /pulsarbot rerun-failure-checks
  • Adds comprehensive logging and error handling for workflow/job management operations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
@lhotari lhotari dismissed their stale review November 4, 2025 15:36

Check re-running of failed jobs only

Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.45%. Comparing base (1708933) to head (3c8a88a).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #24940       +/-   ##
=============================================
+ Coverage     36.42%   74.45%   +38.03%     
- Complexity    13094    34103    +21009     
=============================================
  Files          1706     1902      +196     
  Lines        140911   150051     +9140     
  Branches      16351    17467     +1116     
=============================================
+ Hits          51323   111718    +60395     
+ Misses        82571    29398    -53173     
- Partials       7017     8935     +1918     
Flag Coverage Δ
inttests 26.27% <ø> (-0.17%) ⬇️
systests 23.05% <ø> (?)
unittests 73.97% <ø> (+39.66%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1507 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@liangyepianzhou
Copy link
Copy Markdown
Contributor Author

liangyepianzhou commented Dec 30, 2025

@lhotari The above comments had been fixed. do you have time take a look?
I have test it in liangyepianzhou#19 (comment)

Comment thread .github/workflows/ci-pulsarbot.yaml
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Comment thread .github/workflows/ci-pulsarbot.yaml Outdated
Copy link
Copy Markdown
Member

@lhotari lhotari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lhotari lhotari merged commit 4346165 into master Jan 14, 2026
54 checks passed
@merlimat merlimat deleted the pulsarbot branch March 23, 2026 19:58
@lhotari lhotari added this to the 4.2.0 milestone Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-required Your PR changes impact docs and you will update later.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants