Skip to content

[fix][ci] Add missing permissions to ci-pulsarbot.yaml - #24918

Closed
lhotari wants to merge 1 commit into
apache:masterfrom
lhotari:lh-followup-pulsarbot-permissions
Closed

[fix][ci] Add missing permissions to ci-pulsarbot.yaml#24918
lhotari wants to merge 1 commit into
apache:masterfrom
lhotari:lh-followup-pulsarbot-permissions

Conversation

@lhotari

@lhotari lhotari commented Oct 29, 2025

Copy link
Copy Markdown
Member

Motivation

Modifications

  • Add more permissions so that pulsarbot could work

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
@lhotari lhotari changed the title [fix][ci] Add missing permission to ci-pulsarbot.yaml [fix][ci] Add missing permissions to ci-pulsarbot.yaml 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 commented Oct 29, 2025

Copy link
Copy Markdown
Member Author

Actually, it might be a bug in pulsarbot. URL encoding should be used for branch name at https://github.com/apache/pulsar-test-infra/blob/a0ef898276e191789153842618f854a39f972c61/pulsarbot/entrypoint.sh#L63 .
I'll test with another PR.

@lhotari

lhotari commented Oct 29, 2025

Copy link
Copy Markdown
Member Author

This PR isn't needed. It turns out to be a bug in Pulsar bot as mentioned in #24918 (comment) . Restarting worked fine in this case: https://github.com/apache/pulsar/actions/runs/18908512581/job/53972649999

@lhotari lhotari closed this Oct 29, 2025
@lhotari

lhotari commented Oct 29, 2025

Copy link
Copy Markdown
Member Author

Pulsarbot bug is fixed by apache/pulsar-test-infra@8ff059e . The problem came up with PRs submitted from a repository different from the PR creator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

1 participant