Remove secure product metrics from usage_data.rb

changed milestone to %15.4

added [deprecated] Accepting merge requests backend devopsanalyze groupanalytics instrumentation maintenancerefactor sectionanalytics typemaintenance workflowin dev labels

assigned to @nbelokolodov

added 1 commit

43127b3b - Move ci_ related metrics to instrumentation classes

Compare with previous version

Suggested Reviewers (beta)

The individuals below may be good candidates to participate in the review based on various factors.

You can use slash commands in comments to quickly assign /assign_reviewer @user1.

Suggested Reviewers
`@rspeicher`, `@psimyn`, `@jivanvl`, `@nick.thomas`, `@rymai`

If you do not believe these suggestions are useful, please apply the label Bad Suggested Reviewer. You can also provide feedback for this feature on this issue: https://gitlab.com/gitlab-org/gitlab/-/issues/357923.

Automatically generated by Suggested Reviewers Bot - an experimental ML-based recommendation engine created by ~"group::applied ml".

added database databasereview pending product intelligence product intelligencereview pending labels

1 Warning

For the following files, a review from the Data team and Product Intelligence team is recommended
Please check the product intelligence Service Ping guide or the Snowplow guide.

For MR review guidelines, see the Service Ping review guidelines or the Snowplow review guidelines.

ee/lib/ee/gitlab/usage_data.rb
ee/spec/lib/ee/gitlab/usage_data_spec.rb

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
product intelligence	Tarun Vellishetty (`@tvellishetty`) (UTC+5.5, 2.5 hours behind `@nbelokolodov`)	Maintainer review is optional for product intelligence

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 [deprecated] Accepting merge requests label

changed title from Remove ci related metrics from usage_activity_by_stage_secure from usage_data.rb to Remove ci related metrics from usage_data.rb

added 1 commit

890cc3a6 - Move ci_ related metrics to instrumentation classes

Compare with previous version

Allure report

allure-report-publisher generated test report!

review-qa-blocking: test report for ecbb7a25

expand test summary

+-----------------------------------------------------------------------------------------+
|                                     suites summary                                      |
+------------------------------------+--------+--------+---------+-------+-------+--------+
|                                    | passed | failed | skipped | flaky | total | result |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Create                             | 28     | 0      | 1       | 11    | 29    | ❗     |
| Plan                               | 47     | 0      | 1       | 11    | 48    | ❗     |
| Verify                             | 12     | 0      | 1       | 10    | 13    | ❗     |
| Manage                             | 46     | 0      | 3       | 19    | 49    | ❗     |
| Feature flag handler sanity checks | 9      | 0      | 0       | 0     | 9     | ✅     |
| Version sanity check               | 0      | 0      | 1       | 0     | 1     | ➖     |
| Secure                             | 2      | 0      | 0       | 2     | 2     | ❗     |
| Protect                            | 2      | 0      | 0       | 1     | 2     | ❗     |
| Configure                          | 0      | 0      | 1       | 0     | 1     | ➖     |
| Package                            | 0      | 0      | 1       | 0     | 1     | ➖     |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Total                              | 146    | 0      | 9       | 54    | 155   | ❗     |
+------------------------------------+--------+--------+---------+-------+-------+--------+

e2e-review-qa-blocking: test report for ecbb7a25

expand test summary

+-----------------------------------------------------------------------------------------+
|                                     suites summary                                      |
+------------------------------------+--------+--------+---------+-------+-------+--------+
|                                    | passed | failed | skipped | flaky | total | result |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Plan                               | 47     | 0      | 1       | 41    | 48    | ❗     |
| Create                             | 28     | 0      | 1       | 25    | 29    | ❗     |
| Manage                             | 46     | 0      | 3       | 35    | 49    | ❗     |
| Feature flag handler sanity checks | 9      | 0      | 0       | 0     | 9     | ✅     |
| Verify                             | 12     | 0      | 1       | 10    | 13    | ❗     |
| Package                            | 0      | 0      | 1       | 0     | 1     | ➖     |
| Version sanity check               | 0      | 0      | 1       | 0     | 1     | ➖     |
| Protect                            | 2      | 0      | 0       | 2     | 2     | ❗     |
| Secure                             | 2      | 0      | 0       | 2     | 2     | ❗     |
| Configure                          | 0      | 0      | 1       | 0     | 1     | ➖     |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Total                              | 146    | 0      | 9       | 115   | 155   | ❗     |
+------------------------------------+--------+--------+---------+-------+-------+--------+