Merge pull request #7 from grafana/20220608_dependecies-ring-memberlist

Ensure dependecies for ring, memberlist are respected
grafana · Jun 8, 2022 · 1d66667 · 1d66667
2 parents efbae74 + ab201ca
commit 1d66667
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/pkg/fire/fire.go b/pkg/fire/fire.go
@@ -131,11 +131,12 @@ func (f *Fire) setupModuleManager() error {
 
 	// Add dependencies
 	deps := map[string][]string{
-		All:         {Agent, Ingester, Distributor},
-		Distributor: {Ring, Server},
-		Agent:       {Server},
-		Ingester:    {Server, MemberlistKV},
-		Ring:        {MemberlistKV},
+		All:          {Agent, Ingester, Distributor},
+		Distributor:  {Ring, Server},
+		Agent:        {Server},
+		Ingester:     {Server, MemberlistKV},
+		Ring:         {Server, MemberlistKV},
+		MemberlistKV: {Server},
 
 		// Store:                    {Overrides, IndexGatewayRing},
 		// Querier:                  {Store, Ring, Server, IngesterQuerier, TenantConfigs, UsageReport},