Apply `without_count: true` to the `group/{id}/audit_events` API

What does this MR do and why?

In #337757 (closed), we added a query to return all of the project audit events under a group in addition to the group-level audit events.

The query appears to work ok, however the COUNT that happens before the query ends up timing out. The problem is the count query attempts to count 10001 rows. That reads 100x more data than the pagination query where we return only 100 rows.

For large datasets, of which the audit_events table qualifies, we usually skip the default behavior of doing a count query

The paginate() method accepts a without_count: true|false flag. We should add a toggle of that flag with a FF.

We want to be able to make sure that whenever audit_event_group_rollup FF is enabled, this is enabled. However we also want to be able to toggle the paginate flag independently

EE: true

Changelog: changed

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before X-Total-* included	After X-Total-* removed

How to set up and validate locally

1. make a verbose (to see headers) curl request

    curl -v \
         --header "PRIVATE-TOKEN: $TOKEN" \
         -H "Content-Type: application/json" \
         -H "Accept: application/json, text/plain, */*" \
         --url "localhost:3000/api/v4/groups/33/audit_events"

1. observe the calculated X-Total and X-Total-Pages headers are in the response
2. 1. In rails console enable the experiment fully

   Feature.enable(:audit_event_group_rollup)

3. make same curl
4. Observe the calculated X-Total and X-Total-Pages headers are not in the response

MR acceptance checklist

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

• I have evaluated the MR acceptance checklist for this MR.

Related to #414578 (closed)

changed milestone to %16.1

added Category:Audit Events feature feature flag featureenhancement groupcompliance workflowin dev labels

assigned to @wandering_person

added 1 commit

• eff4ccd1 - Apply `without_count: true` to the `group/{id}/audit_events` API

Compare with previous version

marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed

added backend label

	1 Warning
	featureaddition and featureenhancement merge requests normally have a documentation change. Consider adding a documentation update or confirming the documentation plan with the Technical Writer counterpart. For more information, see: The Handbook page on merge request types. The definition of done documentation.

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
backend	Serena Fang (@serenafang) (UTC-5, 12 hours behind @wandering_person)	Vasilii Iakliushin (@vyaklushin) (UTC+2, 5 hours behind @wandering_person)

To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

removed feature label

added typefeature label

mentioned in commit gitlab-org-sandbox/gitlab-jh-validation@249cbe73

Allure report

allure-report-publisher generated test report!

e2e-test-on-gdk: test report for 451806da

expand test summary

+-----------------------------------------------------------------------+
|                            suites summary                             |
+------------------+--------+--------+---------+-------+-------+--------+
|                  | passed | failed | skipped | flaky | total | result |
+------------------+--------+--------+---------+-------+-------+--------+
| Create           | 8      | 0      | 1       | 0     | 9     | ✅     |
| Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
| Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
| Plan             | 4      | 0      | 0       | 0     | 4     | ✅     |
| Govern           | 2      | 0      | 0       | 0     | 2     | ✅     |
| Data Stores      | 2      | 0      | 0       | 1     | 2     | ❗     |
| Manage           | 1      | 0      | 0       | 0     | 1     | ✅     |
+------------------+--------+--------+---------+-------+-------+--------+
| Total            | 21     | 0      | 2       | 1     | 23    | ❗     |
+------------------+--------+--------+---------+-------+-------+--------+

e2e-package-and-test: test report for 451806da

expand test summary

+-------------------------------------------------------------+
|                       suites summary                        |
+--------+--------+--------+---------+-------+-------+--------+
|        | passed | failed | skipped | flaky | total | result |
+--------+--------+--------+---------+-------+-------+--------+
| Govern | 96     | 2      | 2       | 4     | 100   | ❌     |
+--------+--------+--------+---------+-------+-------+--------+
| Total  | 96     | 2      | 2       | 4     | 100   | ❌     |
+--------+--------+--------+---------+-------+-------+--------+

Setting label(s) devopsgovern sectionsec based on groupcompliance.

added devopsgovern sectionsec labels

added 1 commit

• 3c1d1958 - Apply `without_count: true` to the `group/{id}/audit_events` API

Compare with previous version

mentioned in commit gitlab-org-sandbox/gitlab-jh-validation@c953c739

added 1 commit

• b8e5413e - Apply `without_count: true` to the `group/{id}/audit_events` API

Compare with previous version

marked this merge request as ready

changed the description

mentioned in commit gitlab-org-sandbox/gitlab-jh-validation@7bf0510f

changed title from Resolve "Apply without_count: true to the group/{id}/audit_events API endpoint" to Apply without_count: true to the group/{id}/audit_events API endpoint