Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitaly.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Steinhardt <psteinhardt@gitlab.com>2022-06-30 14:59:47 +0300
committerPatrick Steinhardt <psteinhardt@gitlab.com>2022-07-06 09:13:23 +0300
commitd3701bb9efdd2cbfdf21b2560ea22f273ee1120b (patch)
treec1c83e72f5aedcbaeffe82bc8daeeb2618aaeda0 /internal/git2go/executor.go
parent8a7bee35e279a75a4e861ccff4d1eb222df15adc (diff)
git2go: Convert injection of feature flags to not use raw flags
Convert the logic to inject and extract feature flags in `gitaly-git2go` to not use raw feature flags anymore. Instead, use `FromMetadataKey()` and `FromContext()` to work on typed `FeatureFlag`s. This prepares for the removal of `featureflag.Raw`.
Diffstat (limited to 'internal/git2go/executor.go')
-rw-r--r--internal/git2go/executor.go12
1 files changed, 5 insertions, 7 deletions
diff --git a/internal/git2go/executor.go b/internal/git2go/executor.go
index c1e220f43..5007b6c6f 100644
--- a/internal/git2go/executor.go
+++ b/internal/git2go/executor.go
@@ -59,14 +59,12 @@ func (b *Executor) run(ctx context.Context, repo repository.GitRepo, stdin io.Re
var enabledFeatureFlags, disabledFeatureFlags []string
- for ff, value := range featureflag.RawFromContext(ctx) {
+ for flag, value := range featureflag.FromContext(ctx) {
switch value {
- case "true":
- enabledFeatureFlags = append(enabledFeatureFlags, ff)
- case "false":
- disabledFeatureFlags = append(disabledFeatureFlags, ff)
- default:
- return nil, fmt.Errorf("invalid value for feature flag %q: %q", ff, value)
+ case true:
+ enabledFeatureFlags = append(enabledFeatureFlags, flag.MetadataKey())
+ case false:
+ disabledFeatureFlags = append(disabledFeatureFlags, flag.MetadataKey())
}
}