Skip to content

Conversation

@sunker
Copy link
Contributor

@sunker sunker commented Sep 23, 2025

Historically, when this repo depended on grafana/e2e, the CI workflow needed to determine which version of the Grafana packages to install before running e2e tests against a canary Grafana build. With the latest version of plugin-e2e, this is no longer necessary, as it works across all Grafana versions.

This PR removes that legacy step from the CI workflow and cleans up the process overall. It also simplifies the script used to retrieve the latest grafana-dev image tag. The script is copied from the plugin-actions repo. In the future, we may want to move this logic into a composite action for reuse, but for now this approach is sufficient.

@sunker sunker force-pushed the ci/fix-matrix-timeout branch from c7285e2 to 1bdfc4b Compare September 23, 2025 07:06
@tolzhabayev tolzhabayev moved this from 📬 Triage to 🧑‍💻 In development in Plugins Platform / Grafana Community Sep 23, 2025
@sunker sunker marked this pull request as ready for review September 25, 2025 10:19
@sunker sunker requested review from a team as code owners September 25, 2025 10:19
Copy link
Contributor

@jackw jackw left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@github-project-automation github-project-automation bot moved this from 🧑‍💻 In development to 🔬 In review in Plugins Platform / Grafana Community Sep 25, 2025
@sunker sunker merged commit b7705d4 into main Sep 25, 2025
14 checks passed
@sunker sunker deleted the ci/fix-matrix-timeout branch September 25, 2025 10:24
@github-project-automation github-project-automation bot moved this from 🔬 In review to 🚀 Shipped in Plugins Platform / Grafana Community Sep 25, 2025
@sunker sunker changed the title [wip]: Add retries for setup-matrix Add retries for setup-matrix Sep 25, 2025
@sunker sunker changed the title Add retries for setup-matrix Cleanup ci workflow Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants