Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build plugin #57

Merged
merged 12 commits into from
Jul 3, 2024
Merged

Build plugin #57

merged 12 commits into from
Jul 3, 2024

Conversation

SandersAaronD
Copy link
Collaborator

This is currently waiting on this pr:

grafana/plugin-actions#28

And the github action here should be changed to reference the grafana-maintained github action instead of referencing the branch from my PR. (In the past I have not done this and this has created technical debt over in grafana-llm-app).

Provided that is done, the release workflow from this action would be:

  1. Push the plugin version tag to the desired commit, which will trigger the workflow
  2. Go to the draft PR created by github actions and publish it
  3. Manually publish the plugin on grafana.com if we want it to be downloadable from grafana.com

Steps 2. and 3. can be rolled into automation if necessary, but we don't actually need to do this to run the plugin in dev environments; we can simply use the github release.

@SandersAaronD SandersAaronD marked this pull request as ready for review July 3, 2024 09:51
@SandersAaronD
Copy link
Collaborator Author

This should be good now; there is unfortunately no way to test the publish to gcom step without merging it, so we may need to do a few small patches before it becomes fully end to end.

Copy link
Contributor

@annanay25 annanay25 left a comment

Choose a reason for hiding this comment

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

LGTM (mod the pending PR in plugin-actions repo)

@SandersAaronD SandersAaronD merged commit 32d507c into grafana:main Jul 3, 2024
1 check passed
@SandersAaronD
Copy link
Collaborator Author

@annanay25 Wasn't sure they'd ever merge that PR, decided to copy all the code over here, should be just fine (and a lot easier to maintain over time).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants