Tracks the merge request created from vulnerability resolution

changed milestone to %17.6

added Category:Vulnerability Management GitLab Ultimate devopsgovern groupthreat insights missed:17.5 sectionsec typefeature workflowrefinement labels

assigned to @subashis

added pipelinetier-1 label

added 1 commit

be17d360 - Tracks the merge request created from vulnerability resolution

Compare with previous version

added analytics instrumentation analytics instrumentationreview pending backend labels

	1 Warning
	b9e84174: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.

1 Message

CHANGELOG missing:

If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog.

If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

Reviewer roulette

Category	Reviewer	Maintainer
analytics instrumentation	`@nisong` (UTC+1, 7 hours ahead of author)	Maintainer review is optional for analytics instrumentation
backend	`@micthomas` (UTC-4, 2 hours ahead of author)	`@rutgerwessels` (UTC+1, 7 hours ahead of author)

Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

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

Generated by Danger

added 1 commit

45df4ee6 - Tracks the merge request created from vulnerability resolution

Compare with previous version

added 1 commit

b5a55835 - Tracks the merge request created from vulnerability resolution

Compare with previous version

added 1 commit

65b20d6e - Tracks the merge request created from vulnerability resolution

Compare with previous version

@ankit.panchal Can you please do the review for analytics instrumentation ? I have a question here: will it be ok if we want to add another additional_property in the event later?

requested review from @ankit.panchal

marked this merge request as ready

@darbyfrey Can you please do the initial review for this MR while I am still waiting for the answer here?

requested review from @darbyfrey

changed the description

approved this merge request

added pipeline:mr-approved label

added pipelinetier-3 pipeline:run-e2e-omnibus-once labels and removed pipelinetier-1 label

Before you set this MR to auto-merge

This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

Before you set this MR to auto-merge, please check the following:

You are the last maintainer of this merge request
The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
This pipeline is recent enough (created in the last 8 hours)

If all the criteria above apply, please set auto-merge for this merge request.

See pipeline tiers and merging a merge request for more details.

E2E Test Result Summary

allure-report-publisher generated test report!

e2e-test-on-gdk: test report for b9e84174

expand test summary

+-------------------------------------------------------------+
|                       suites summary                        |
+--------+--------+--------+---------+-------+-------+--------+
|        | passed | failed | skipped | flaky | total | result |
+--------+--------+--------+---------+-------+-------+--------+
| Create | 2      | 0      | 0       | 0     | 2     | ✅     |
+--------+--------+--------+---------+-------+-------+--------+
| Total  | 2      | 0      | 0       | 0     | 2     | ✅     |
+--------+--------+--------+---------+-------+-------+--------+

e2e-test-on-cng: test report for b9e84174

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Verify      | 50     | 0      | 15      | 10    | 65    | ✅     |
| Plan        | 86     | 0      | 8       | 12    | 94    | ✅     |
| Fulfillment | 2      | 0      | 7       | 0     | 9     | ✅     |
| Package     | 24     | 0      | 14      | 0     | 38    | ✅     |
| Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
| Govern      | 82     | 0      | 11      | 10    | 93    | ✅     |
| Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
| Create      | 139    | 0      | 21      | 18    | 160   | ✅     |
| Data Stores | 33     | 0      | 10      | 1     | 43    | ✅     |
| Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
| Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
| Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
| Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
| Secure      | 1      | 0      | 6       | 0     | 7     | ✅     |
| ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
| Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 433    | 0      | 122     | 51    | 555   | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+

added 1 commit

2f75aaa2 - Tracks the merge request created from vulnerability resolution

Compare with previous version

reset approvals from @ankit.panchal by pushing to the branch

added 1 commit

6a2023c2 - Tracks the merge request created from vulnerability resolution

Compare with previous version

added 1 commit

b689be97 - Tracks the merge request created from vulnerability resolution

Compare with previous version

approved this merge request

added 1265 commits

b689be97...223998f2 - 1264 commits from branch master
b9e84174 - Tracks the merge request created from vulnerability resolution

Compare with previous version

unapproved this merge request

approved this merge request

changed the description

@avielle Can you please do the maintainer review for this MR ?

requested review from @avielle

approved this merge request

added analytics instrumentationapproved label and removed analytics instrumentationreview pending label

resolved all threads

approved this merge request

enabled automatic add to merge train when checks pass

started a merge train

merged

Hello @subashis

The Analytics Instrumentation team is actively working on improving the metrics implementation and event tracking processes. We would love to hear about your experience and any feedback you might have!

To provide your feedback, please use this Google Form.

Thanks for your help!

Looking for existing metrics data? Check out the Metrics Exploration Dashboard for Group:threat_insights.
Need further assistance? Have comments?
- Mention @gitlab-org/analytics-section/analytics-instrumentation/engineers
- Reach out in #g_monitor_analytics_instrumentation channel on Slack.

This message was generated automatically. You're welcome to improve it.