Skip to content

Create Plugin: Don't persist credentials#2498

Open
6nv wants to merge 1 commit intografana:mainfrom
6nv:main
Open

Create Plugin: Don't persist credentials#2498
6nv wants to merge 1 commit intografana:mainfrom
6nv:main

Conversation

@6nv
Copy link
Copy Markdown
Contributor

@6nv 6nv commented Feb 27, 2026

What this PR does / why we need it: When this setting is set to true, the CI workflow fails with an error message stating that the requested URL (being your repository URL) returned an error code 400.

Which issue(s) this PR fixes: grafana/plugin-actions#200

Fixes # (N/A)

Special notes for your reviewer: N/A

@tolzhabayev
Copy link
Copy Markdown
Collaborator

@sunker can you take a look if we need to persist credentials in this step by default for publish pages to work?

Copy link
Copy Markdown
Contributor

@sunker sunker left a comment

Choose a reason for hiding this comment

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

Could confirm that the publish-report job fails with persist-credentials: true and that it works with persist-credentials: false. Thanks for reporting this and coming up with a fix @6nv!

@sunker sunker added patch Increment the patch version when merged release Create a release when this pr is merged labels Mar 27, 2026
@sunker
Copy link
Copy Markdown
Contributor

sunker commented Mar 30, 2026

Have raised another PR to fix CI errors seen in this PR.

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

Labels

patch Increment the patch version when merged release Create a release when this pr is merged

Projects

Status: 🔬 In review

Development

Successfully merging this pull request may close these issues.

3 participants