Default :roadmap_graphql and :unfiltered_epic_aggregates FF to on

Default feature flags to on for epic roadmap weights feature

doc/api/graphql/reference/gitlab_schema.graphql

@@ -1948,8 +1948,7 @@ type Epic implements Noteable {
   descendantCounts: EpicDescendantCount
 
   """
-  Total weight of open and closed issues in the epic and its descendants.
-  Available only when feature flag `unfiltered_epic_aggregates` is enabled.
+  Total weight of open and closed issues in the epic and its descendants
   """
   descendantWeightSum: EpicDescendantWeights
 

doc/api/graphql/reference/gitlab_schema.json

@@ -5731,7 +5731,7 @@
             },
             {
               "name": "descendantWeightSum",
-              "description": "Total weight of open and closed issues in the epic and its descendants. Available only when feature flag `unfiltered_epic_aggregates` is enabled.",
+              "description": "Total weight of open and closed issues in the epic and its descendants",
               "args": [
 
               ],

doc/api/graphql/reference/index.md

@@ -317,7 +317,7 @@ Represents an epic.
 | `closedAt` | Time | Timestamp of the epic's closure |
 | `createdAt` | Time | Timestamp of the epic's creation |
 | `descendantCounts` | EpicDescendantCount | Number of open and closed descendant epics and issues |
-| `descendantWeightSum` | EpicDescendantWeights | Total weight of open and closed issues in the epic and its descendants. Available only when feature flag `unfiltered_epic_aggregates` is enabled. |
+| `descendantWeightSum` | EpicDescendantWeights | Total weight of open and closed issues in the epic and its descendants |
 | `description` | String | Description of the epic |
 | `downvotes` | Int! | Number of downvotes the epic has received |
 | `dueDate` | Time | Due date of the epic |

ee/app/controllers/groups/epics_controller.rb

@@ -17,8 +17,8 @@ class Groups::EpicsController < Groups::ApplicationController
   before_action :verify_group_bulk_edit_enabled!, only: [:bulk_update]
 
   before_action do
-    push_frontend_feature_flag(:roadmap_graphql, @group)
-    push_frontend_feature_flag(:unfiltered_epic_aggregates, @group)
+    push_frontend_feature_flag(:roadmap_graphql, @group, default_enabled: true)
+    push_frontend_feature_flag(:unfiltered_epic_aggregates, @group, default_enabled: true)
     push_frontend_feature_flag(:vue_issuable_epic_sidebar, @group)
   end
 

ee/app/controllers/groups/roadmap_controller.rb

@@ -10,8 +10,8 @@ module Groups
     before_action :check_epics_available!
     before_action :persist_roadmap_layout, only: [:show]
     before_action do
-      push_frontend_feature_flag(:roadmap_graphql, @group)
-      push_frontend_feature_flag(:unfiltered_epic_aggregates, @group)
+      push_frontend_feature_flag(:roadmap_graphql, @group, default_enabled: true)
+      push_frontend_feature_flag(:unfiltered_epic_aggregates, @group, default_enabled: true)
       push_frontend_feature_flag(:roadmap_buffered_rendering, @group)
       push_frontend_feature_flag(:milestones_in_roadmap, @group)
     end

ee/app/graphql/types/epic_type.rb

@@ -125,7 +125,7 @@ module Types
     field :descendant_counts, Types::EpicDescendantCountType, null: true, complexity: 10,
       description: 'Number of open and closed descendant epics and issues',
       resolve: -> (epic, args, ctx) do
-        if Feature.enabled?(:unfiltered_epic_aggregates)
+        if Feature.enabled?(:unfiltered_epic_aggregates, epic.group, default_enabled: true)
           Gitlab::Graphql::Aggregations::Epics::LazyEpicAggregate.new(ctx, epic.id, COUNT)
         else
           Epics::DescendantCountService.new(epic, ctx[:current_user])
@@ -134,7 +134,6 @@ module Types
 
     field :descendant_weight_sum, Types::EpicDescendantWeightSumType, null: true, complexity: 10,
       description: "Total weight of open and closed issues in the epic and its descendants",
-      feature_flag: :unfiltered_epic_aggregates,
       resolve: -> (epic, args, ctx) do
         Gitlab::Graphql::Aggregations::Epics::LazyEpicAggregate.new(ctx, epic.id, WEIGHT_SUM)
       end

ee/spec/requests/api/graphql/group/epic/epic_aggregate_query_spec.rb

@@ -108,53 +108,5 @@ describe 'Epic aggregates (count and weight)' do
         )
       end
     end
-
-    context 'with feature flag disabled' do
-      before do
-        stub_feature_flags(unfiltered_epic_aggregates: false)
-      end
-
-      context 'when requesting counts' do
-        let(:epic_aggregates_query) do
-          <<~QUERY
-          nodes {
-            descendantCounts {
-              openedEpics
-              closedEpics
-              openedIssues
-              closedIssues
-            }
-          }
-          QUERY
-        end
-
-        it 'uses the DescendantCountService' do
-          expect(Epics::DescendantCountService).to receive(:new)
-
-          post_graphql(query, current_user: current_user)
-        end
-
-        it_behaves_like 'counts properly'
-      end
-
-      context 'when requesting weights' do
-        let(:epic_aggregates_query) do
-          <<~QUERY
-          nodes {
-            descendantWeightSum {
-              openedIssues
-              closedIssues
-            }
-          }
-          QUERY
-        end
-
-        it 'returns an error' do
-          post_graphql(query, current_user: current_user)
-
-          expect_graphql_errors_to_include /Field 'descendantWeightSum' doesn't exist on type 'Epic/
-        end
-      end
-    end
   end
 end