-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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
Plugins: Add feat toggle to install managed plugins #75973
Conversation
17a7df6
to
c166104
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a comment/question
pkg/services/featuremgmt/registry.go
Outdated
Name: "managedPluginsInstall", | ||
Description: "Install managed plugins directly from plugins catalog", | ||
Stage: FeatureStageExperimental, | ||
RequiresDevMode: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit. Is this true?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to be safe and test it before going to production.
Will this make my installation tests more difficult to perform? (I don't know much about our environments)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed anyway XD
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would require app_mode=development I think which is not enabled in cloud. Merging this the feature toggle will be disabled unless configured to be enabled
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tks for pointing this out!
What is this feature?
Add feat toggle to install managed plugins
Why do we need this feature?
This is necessary to control the availability of new feature https://github.com/grafana/grafana-plugins-platform-team/issues/21
release.
Who is this feature for?
Grafana Developers
Which issue(s) does this PR fix?:
This is part of https://github.com/grafana/grafana-plugins-platform-team/issues/21
Special notes for your reviewer:
Please check that: