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

CreatePlugin: Change default docker image to grafana-enterprise #197

Merged
merged 2 commits into from
Feb 2, 2023

Conversation

mckn
Copy link
Collaborator

@mckn mckn commented Feb 2, 2023

What this PR does / why we need it:
We got a question about why the GF_ENTERPRISE_* environment variables wasn't picked up properly after migrating to the create-plugin configuration files. The reason for this is that we are using the grafana/grafana docker image which doesn't contain the enerprise version of Grafana.

This PR will change so that all the docker related configuration that is scaffolded by create-plugin will use grafana/grafana-enterprise as the default docker image. It also adds a way to configure this without adding any changes to the ./config folder which might cause issues while updating.

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @grafana/create-plugin@0.10.0-canary.197.f95c2c8.0
# or 
yarn add @grafana/create-plugin@0.10.0-canary.197.f95c2c8.0

…ty to configure this value from outside the .config folder.
@mckn mckn added create-plugin related to the create-plugin tool minor Increment the minor version when merged type/bug Something isn't working labels Feb 2, 2023
@mckn mckn requested review from jackw and leventebalogh February 2, 2023 09:33
@mckn mckn self-assigned this Feb 2, 2023
@yesoreyeram
Copy link

On a related note, grafana's default image is enterprise.

Copy link
Collaborator

@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! 🚀

Left a couple of nits.

@jackw jackw added release Create a release when this pr is merged and removed type/bug Something isn't working labels Feb 2, 2023
@jackw
Copy link
Collaborator

jackw commented Feb 2, 2023

:shipit:

@mckn mckn merged commit cb240de into main Feb 2, 2023
@mckn mckn deleted the mckn/grafana-version-docker branch February 2, 2023 13:46
@grafanabot
Copy link
Contributor

🚀 PR was released in @grafana/create-plugin@0.10.0 🚀

@grafanabot grafanabot added the released This issue/pull request has been released. label Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
create-plugin related to the create-plugin tool minor Increment the minor version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
Development

Successfully merging this pull request may close these issues.

4 participants