Skip to content

Remove actors_aware_gitaly_calls flag

Quang-Minh Nguyen requested to merge qmnguyen0711/remote-actors-aware-gitaly-calls into master

What does this MR do and why?

In a prior MR (Fully support feature flag actors in Gitaly (!101218 - merged)), we supported feature flag actor propagation to Gitaly. The actors are collected before being passed to Gitaly via Feature::Gitaly.server_feature_flags helper method. Due to uncertain performance impact of the addition, we hid this propagation behind a feature flag. After turning this flag globally on production, we observed no impacts or relative errors. Therefore, it's safe to always propagate the actors.

More information about the rollout: [Feature flag] Rollout of `actors_aware_gitaly_... (#381516 - closed)

Screenshots or screen recordings

N/A

How to set up and validate locally

N/A

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports