diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-06-30 14:59:47 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-07-06 09:13:23 +0300 |
commit | d3701bb9efdd2cbfdf21b2560ea22f273ee1120b (patch) | |
tree | c1c83e72f5aedcbaeffe82bc8daeeb2618aaeda0 /internal/git2go/executor.go | |
parent | 8a7bee35e279a75a4e861ccff4d1eb222df15adc (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.go | 12 |
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()) } } |