Skip to content

[improve][ci] Replace PULSARBOT_TOKEN with GITHUB_TOKEN configured with permissions - #24916

Merged
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-replace-github-actions-PAT-with-permissions
Oct 29, 2025
Merged

[improve][ci] Replace PULSARBOT_TOKEN with GITHUB_TOKEN configured with permissions#24916
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-replace-github-actions-PAT-with-permissions

Conversation

@lhotari

@lhotari lhotari commented Oct 29, 2025

Copy link
Copy Markdown
Member

Motivation

The use of PULSARBOT_TOKEN secret is unnecessary and should be replaced with GITHUB_TOKEN that is configured with sufficient permissions in the workflow file.

Modifications

  • add sufficient permissions to GITHUB_TOKEN
  • pass GITHUB_TOKEN to pulsarbot action

Documentation

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

@lhotari lhotari added this to the 4.2.0 milestone Oct 29, 2025
@lhotari lhotari self-assigned this Oct 29, 2025
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Oct 29, 2025
@lhotari
lhotari requested a review from codelipenghui October 29, 2025 10:49

@liangyepianzhou liangyepianzhou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter

codecov-commenter commented Oct 29, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.28%. Comparing base (7a243f1) to head (585a35c).
⚠️ Report is 36 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #24916      +/-   ##
============================================
+ Coverage     74.21%   74.28%   +0.06%     
+ Complexity    33820    33480     -340     
============================================
  Files          1913     1913              
  Lines        149328   149328              
  Branches      17334    17334              
============================================
+ Hits         110825   110926     +101     
+ Misses        29636    29562      -74     
+ Partials       8867     8840      -27     
Flag Coverage Δ
inttests 26.63% <ø> (+0.23%) ⬆️
systests 22.85% <ø> (-0.05%) ⬇️
unittests 73.81% <ø> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 106 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.

@lhotari
lhotari merged commit d20eb2e into apache:master Oct 29, 2025
100 of 102 checks passed
lhotari added a commit to lhotari/pulsar that referenced this pull request Oct 29, 2025
@lhotari

lhotari commented Oct 29, 2025

Copy link
Copy Markdown
Member Author

There wasn't sufficient permissions. I made a follow-up in #24918, please review.

@lhotari

lhotari commented Oct 29, 2025

Copy link
Copy Markdown
Member Author

There wasn't sufficient permissions. I made a follow-up in #24918, please review.

It wasn't a problem with the permissions, but a problem in pulsarbot that came up in testing. No need for the follow-up.

guptas6est pushed a commit to Nordix/pulsar that referenced this pull request Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants