mirror of
https://github.com/grafana/grafana.git
synced 2024-11-29 20:24:18 -06:00
08f2791b22
assign the last owner
309 lines
8.7 KiB
Go
309 lines
8.7 KiB
Go
package featuremgmt
|
|
|
|
import (
|
|
"bytes"
|
|
"fmt"
|
|
"html/template"
|
|
"os"
|
|
"path/filepath"
|
|
"strings"
|
|
"testing"
|
|
"unicode"
|
|
|
|
"github.com/google/go-cmp/cmp"
|
|
"github.com/olekukonko/tablewriter"
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/grafana/grafana/pkg/services/featuremgmt/strcase"
|
|
)
|
|
|
|
func TestFeatureToggleFiles(t *testing.T) {
|
|
legacyNames := map[string]bool{
|
|
"httpclientprovider_azure_auth": true,
|
|
"service-accounts": true,
|
|
"database_metrics": true,
|
|
"live-pipeline": true,
|
|
"live-service-web-worker": true,
|
|
"k8s": true, // Camel case does not like this one
|
|
}
|
|
|
|
t.Run("check registry constraints", func(t *testing.T) {
|
|
for _, flag := range standardFeatureFlags {
|
|
if flag.Expression == "true" && flag.State != FeatureStateStable {
|
|
t.Errorf("only stable features can be enabled by default. See: %s", flag.Name)
|
|
}
|
|
if flag.RequiresDevMode && flag.State != FeatureStateAlpha {
|
|
t.Errorf("only alpha features can require dev mode. See: %s", flag.Name)
|
|
}
|
|
if flag.State == FeatureStateUnknown {
|
|
t.Errorf("standard toggles should not have an unknown state. See: %s", flag.Name)
|
|
}
|
|
}
|
|
})
|
|
|
|
t.Run("all new features should have an owner", func(t *testing.T) {
|
|
for _, flag := range standardFeatureFlags {
|
|
if flag.Owner == "" {
|
|
t.Errorf("feature %s does not have an owner. please fill the FeatureFlag.Owner property", flag.Name)
|
|
}
|
|
}
|
|
})
|
|
|
|
t.Run("verify files", func(t *testing.T) {
|
|
// Typescript files
|
|
verifyAndGenerateFile(t,
|
|
"../../../packages/grafana-data/src/types/featureToggles.gen.ts",
|
|
generateTypeScript(),
|
|
)
|
|
|
|
// Golang files
|
|
verifyAndGenerateFile(t,
|
|
"toggles_gen.go",
|
|
generateRegistry(t),
|
|
)
|
|
|
|
// Docs files
|
|
verifyAndGenerateFile(t,
|
|
"../../../docs/sources/setup-grafana/configure-grafana/feature-toggles/index.md",
|
|
generateDocsMD(),
|
|
)
|
|
})
|
|
|
|
t.Run("check feature naming convention", func(t *testing.T) {
|
|
invalidNames := make([]string, 0)
|
|
for _, f := range standardFeatureFlags {
|
|
if legacyNames[f.Name] {
|
|
continue
|
|
}
|
|
|
|
if f.Name != strcase.ToLowerCamel(f.Name) {
|
|
invalidNames = append(invalidNames, f.Name)
|
|
}
|
|
}
|
|
|
|
require.Empty(t, invalidNames, "%s feature names should be camel cased", invalidNames)
|
|
// acronyms can be configured as needed via `ConfigureAcronym` function from `./strcase/camel.go`
|
|
})
|
|
}
|
|
|
|
func verifyAndGenerateFile(t *testing.T, fpath string, gen string) {
|
|
// nolint:gosec
|
|
// We can ignore the gosec G304 warning since this is a test and the function is only called explicitly above
|
|
body, err := os.ReadFile(fpath)
|
|
if err == nil {
|
|
if diff := cmp.Diff(gen, string(body)); diff != "" {
|
|
str := fmt.Sprintf("body mismatch (-want +got):\n%s\n", diff)
|
|
err = fmt.Errorf(str)
|
|
}
|
|
}
|
|
|
|
if err != nil {
|
|
e2 := os.WriteFile(fpath, []byte(gen), 0644)
|
|
if e2 != nil {
|
|
t.Errorf("error writing file: %s", e2.Error())
|
|
}
|
|
abs, _ := filepath.Abs(fpath)
|
|
t.Errorf("feature toggle do not match: %s (%s)", err.Error(), abs)
|
|
t.Fail()
|
|
}
|
|
}
|
|
|
|
func generateTypeScript() string {
|
|
buf := `// NOTE: This file was auto generated. DO NOT EDIT DIRECTLY!
|
|
// To change feature flags, edit:
|
|
// pkg/services/featuremgmt/registry.go
|
|
// Then run tests in:
|
|
// pkg/services/featuremgmt/toggles_gen_test.go
|
|
|
|
/**
|
|
* Describes available feature toggles in Grafana. These can be configured via
|
|
* conf/custom.ini to enable features under development or not yet available in
|
|
* stable version.
|
|
*
|
|
* Only enabled values will be returned in this interface
|
|
*
|
|
* @public
|
|
*/
|
|
export interface FeatureToggles {
|
|
[name: string]: boolean | undefined; // support any string value
|
|
|
|
`
|
|
for _, flag := range standardFeatureFlags {
|
|
buf += " " + getTypeScriptKey(flag.Name) + "?: boolean;\n"
|
|
}
|
|
|
|
buf += "}\n"
|
|
return buf
|
|
}
|
|
|
|
func getTypeScriptKey(key string) string {
|
|
if strings.Contains(key, "-") || strings.Contains(key, ".") {
|
|
return "['" + key + "']"
|
|
}
|
|
return key
|
|
}
|
|
|
|
func isLetterOrNumber(c rune) bool {
|
|
return !unicode.IsLetter(c) && !unicode.IsNumber(c)
|
|
}
|
|
|
|
func asCamelCase(key string) string {
|
|
parts := strings.FieldsFunc(key, isLetterOrNumber)
|
|
for idx, part := range parts {
|
|
parts[idx] = strings.Title(part)
|
|
}
|
|
return strings.Join(parts, "")
|
|
}
|
|
|
|
func generateRegistry(t *testing.T) string {
|
|
tmpl, err := template.New("fn").Parse(`
|
|
{{"\t"}}// Flag{{.CamelCase}}{{.Ext}}
|
|
{{"\t"}}Flag{{.CamelCase}} = "{{.Flag.Name}}"
|
|
`)
|
|
if err != nil {
|
|
t.Fatal("error reading template", "error", err.Error())
|
|
return ""
|
|
}
|
|
|
|
data := struct {
|
|
CamelCase string
|
|
Flag FeatureFlag
|
|
Ext string
|
|
}{
|
|
CamelCase: "?",
|
|
}
|
|
|
|
var buff bytes.Buffer
|
|
|
|
buff.WriteString(`// NOTE: This file was auto generated. DO NOT EDIT DIRECTLY!
|
|
// To change feature flags, edit:
|
|
// pkg/services/featuremgmt/registry.go
|
|
// Then run tests in:
|
|
// pkg/services/featuremgmt/toggles_gen_test.go
|
|
|
|
package featuremgmt
|
|
|
|
const (`)
|
|
|
|
for _, flag := range standardFeatureFlags {
|
|
data.CamelCase = asCamelCase(flag.Name)
|
|
data.Flag = flag
|
|
data.Ext = ""
|
|
|
|
if flag.Description != "" {
|
|
data.Ext += "\n\t// " + flag.Description
|
|
}
|
|
|
|
_ = tmpl.Execute(&buff, data)
|
|
}
|
|
buff.WriteString(")\n")
|
|
|
|
return buff.String()
|
|
}
|
|
|
|
func generateDocsMD() string {
|
|
hasDeprecatedFlags := false
|
|
|
|
buf := `---
|
|
aliases:
|
|
- /docs/grafana/latest/setup-grafana/configure-grafana/feature-toggles/
|
|
description: Learn about toggles for experimental and beta features, which you can enable or disable.
|
|
title: Configure feature toggles
|
|
weight: 150
|
|
---
|
|
|
|
<!-- DO NOT EDIT THIS PAGE, it is machine generated by running the test in -->
|
|
<!-- https://github.com/grafana/grafana/blob/main/pkg/services/featuremgmt/toggles_gen_test.go#L19 -->
|
|
|
|
# Configure feature toggles
|
|
|
|
You use feature toggles, also known as feature flags, to turn experimental or beta features on and off in Grafana. Although we do not recommend using these features in production, you can turn on feature toggles to try out new functionality in development or test environments.
|
|
|
|
This page contains a list of available feature toggles. To learn how to turn on feature toggles, refer to our [Configure Grafana documentation]({{< relref "../_index.md/#feature_toggles" >}}). Feature toggles are also available to Grafana Cloud Advanced customers. If you use Grafana Cloud Advanced, you can open a support ticket and specify the feature toggles and stack for which you want them enabled.
|
|
|
|
## Stable feature toggles
|
|
|
|
Some stable features are enabled by default. You can disable a stable feature by setting the feature flag to "false" in the configuration.
|
|
|
|
` + writeToggleDocsTable(func(flag FeatureFlag) bool {
|
|
return flag.State == FeatureStateStable
|
|
}, true)
|
|
|
|
buf += `
|
|
## Beta feature toggles
|
|
|
|
` + writeToggleDocsTable(func(flag FeatureFlag) bool {
|
|
return flag.State == FeatureStateBeta
|
|
}, false)
|
|
|
|
if hasDeprecatedFlags {
|
|
buf += `
|
|
## Deprecated feature toggles
|
|
|
|
When stable or beta features are slated for removal, they will be marked as Deprecated first.
|
|
|
|
` + writeToggleDocsTable(func(flag FeatureFlag) bool {
|
|
return flag.State == FeatureStateDeprecated
|
|
}, false)
|
|
}
|
|
|
|
buf += `
|
|
## Alpha feature toggles
|
|
|
|
These features are early in their development lifecycle and so are not yet supported in Grafana Cloud.
|
|
Alpha features might be changed or removed without prior notice.
|
|
|
|
` + writeToggleDocsTable(func(flag FeatureFlag) bool {
|
|
return flag.State == FeatureStateAlpha && !flag.RequiresDevMode
|
|
}, false)
|
|
|
|
buf += `
|
|
## Development feature toggles
|
|
|
|
The following toggles require explicitly setting Grafana's [app mode]({{< relref "../_index.md/#app_mode" >}}) to 'development' before you can enable this feature toggle. These features tend to be experimental.
|
|
|
|
` + writeToggleDocsTable(func(flag FeatureFlag) bool {
|
|
return flag.RequiresDevMode
|
|
}, false)
|
|
return buf
|
|
}
|
|
|
|
func writeToggleDocsTable(include func(FeatureFlag) bool, showEnableByDefault bool) string {
|
|
data := [][]string{}
|
|
|
|
for _, flag := range standardFeatureFlags {
|
|
if include(flag) {
|
|
row := []string{"`" + flag.Name + "`", flag.Description}
|
|
if showEnableByDefault {
|
|
on := ""
|
|
if flag.Expression == "true" {
|
|
on = "Yes"
|
|
}
|
|
row = append(row, on)
|
|
}
|
|
data = append(data, row)
|
|
}
|
|
}
|
|
|
|
header := []string{"Feature toggle name", "Description"}
|
|
if showEnableByDefault {
|
|
header = append(header, "Enabled by default")
|
|
}
|
|
|
|
sb := &strings.Builder{}
|
|
table := tablewriter.NewWriter(sb)
|
|
table.SetHeader(header)
|
|
table.SetBorders(tablewriter.Border{Left: true, Top: false, Right: true, Bottom: false})
|
|
table.SetCenterSeparator("|")
|
|
table.SetAutoFormatHeaders(false)
|
|
table.SetHeaderAlignment(tablewriter.ALIGN_LEFT)
|
|
table.SetAutoWrapText(false)
|
|
table.SetAlignment(tablewriter.ALIGN_LEFT)
|
|
table.AppendBulk(data) // Add Bulk Data
|
|
table.Render()
|
|
|
|
// Markdown table formatting (from prittier)
|
|
v := strings.ReplaceAll(sb.String(), "|--", "| -")
|
|
return strings.ReplaceAll(v, "--|", "- |")
|
|
}
|