Install two versions of Sentry Client SDK

assigned to @mrincon

	2 Warnings
	This merge request is quite big (904 lines changed), please consider splitting it into multiple merge requests.
	156e7854: 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.

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	Mario Celi (`@mcelicalderonG`) (UTC-5, 6 hours behind `@mrincon`)	Sean McGivern (`@smcgivern`) (UTC+0, 1 hour behind `@mrincon`)
frontend	Janis Altherr (`@janis`) (UTC+1, same timezone as `@mrincon`)	Kushal Pandya (`@kushalpandya`) (UTC+5.5, 4.5 hours ahead of `@mrincon`)
test for `spec/features/*`	Aleksandr Lyubenkov (`@alyubenkov`) (UTC+1, same timezone as `@mrincon`)	Maintainer review is optional for test for `spec/features/*`
Application Security	Reviewer review is optional for Application Security	Nikhil George (`@ngeorge1`) (UTC+0, 1 hour behind `@mrincon`)

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

Proper labels assigned to this merge request. Please ignore me.

@mrincon - please see the following guidance and update this merge request.

	1 Error
	Please add typebug typefeature, or typemaintenance label to this merge request.

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

Setting label grouprunner based on @mrincon's group.

added grouprunner label

added 1 commit

05b19c5b - Install two versions of Sentry

Compare with previous version

added devopsverify sectionops labels

added 1 commit

8f28076c - Install two versions of Sentry

Compare with previous version

added maintenanceworkflow typemaintenance labels

resolved all threads

changed milestone to %15.6

Allure report

allure-report-publisher generated test report!

e2e-review-qa: test report for 80e500fd

expand test summary

+-----------------------------------------------------------------------------------------+
|                                     suites summary                                      |
+------------------------------------+--------+--------+---------+-------+-------+--------+
|                                    | passed | failed | skipped | flaky | total | result |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Verify                             | 12     | 0      | 1       | 0     | 13    | ✅     |
| Manage                             | 39     | 0      | 4       | 3     | 43    | ❗     |
| Plan                               | 49     | 0      | 1       | 0     | 50    | ✅     |
| Create                             | 28     | 0      | 1       | 0     | 29    | ✅     |
| Package                            | 0      | 0      | 1       | 0     | 1     | ➖     |
| Feature flag handler sanity checks | 9      | 0      | 0       | 0     | 9     | ✅     |
| Govern                             | 10     | 0      | 5       | 1     | 15    | ❗     |
| Configure                          | 0      | 0      | 1       | 0     | 1     | ➖     |
| Version sanity check               | 0      | 0      | 1       | 0     | 1     | ➖     |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Total                              | 147    | 0      | 15      | 4     | 162   | ❗     |
+------------------------------------+--------+--------+---------+-------+-------+--------+

e2e-package-and-test: test report for 80e500fd

expand test summary

+---------------------------------------------------------------------------+
|                              suites summary                               |
+----------------------+--------+--------+---------+-------+-------+--------+
|                      | passed | failed | skipped | flaky | total | result |
+----------------------+--------+--------+---------+-------+-------+--------+
| Manage               | 351    | 0      | 27      | 3     | 378   | ❗     |
| Plan                 | 312    | 0      | 0       | 0     | 312   | ✅     |
| Create               | 818    | 0      | 27      | 2     | 845   | ❗     |
| Govern               | 209    | 0      | 0       | 0     | 209   | ✅     |
| Package              | 144    | 0      | 37      | 0     | 181   | ✅     |
| Verify               | 229    | 0      | 42      | 0     | 271   | ✅     |
| Fulfillment          | 10     | 0      | 75      | 0     | 85    | ✅     |
| Release              | 30     | 0      | 0       | 0     | 30    | ✅     |
| Monitor              | 5      | 0      | 0       | 0     | 5     | ✅     |
| Analytics            | 11     | 0      | 0       | 0     | 11    | ✅     |
| ModelOps             | 0      | 0      | 5       | 0     | 5     | ➖     |
| Secure               | 30     | 0      | 10      | 1     | 40    | ❗     |
| GitLab Metrics       | 2      | 0      | 1       | 1     | 3     | ❗     |
| Systems              | 3      | 0      | 0       | 0     | 3     | ✅     |
| Data Stores          | 13     | 0      | 1       | 0     | 14    | ✅     |
| Configure            | 1      | 0      | 15      | 0     | 16    | ✅     |
| Version sanity check | 0      | 0      | 7       | 4     | 7     | ➖     |
+----------------------+--------+--------+---------+-------+-------+--------+
| Total                | 2168   | 0      | 247     | 11    | 2415  | ❗     |
+----------------------+--------+--------+---------+-------+-------+--------+

added 1 commit

d44709f7 - Install two versions of Sentry

Compare with previous version

mentioned in merge request !102650 (merged)

added 3 commits

3b6db16c - 1 commit from branch mrincon-enable_new_sentry_clientside_integration
66c96c27 - Move sentry constants to separate file
09d96366 - Install two versions of Sentry

Compare with previous version

changed title from Install two versions of Sentry to Install two versions of Sentry Client SDK

changed the description

deleted the mrincon-enable_new_sentry_clientside_integration branch. This merge request now targets the master branch

changed the description

added 1 commit

b28863f5 - Install two versions of Sentry

Compare with previous version

added feature flag label

Bundle size analysis [beta]

This compares changes in bundle size for entry points between the commits 725e7136 and 80e500fd

Special assets

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
average	3.55 MB	3.44 MB	-121.42 KB	-3.3 %
mainChunk	1.96 MB	1.84 MB	-118.67 KB	-5.9 %

Significant Growth: 1

Expand

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
sentry	171.13 KB	262.23 KB	+91.1 KB	53.2 %

New entry points: 1

Expand

Entrypoint / Name	Size before	Size after	Diff	Diff in percent
legacy_sentry	0 Bytes	293.41 KB	+293.41 KB	100.0 %

Your MR has at least one entrypoint growing significantly (more > 1 KB or 2%). If you write new or extend existing features, this is expected and there is nothing to worry about.

Please consider pinging someone from the FE Foundations (@leipert, @markrian, @mikegreiling, @ohoral or @pgascouvaillancourt) for review, if you are unsure about the size increase.

Note: We do not have exact data for 725e7136. So we have used data from: 4a957259.
The target commit was too new, so we used the latest commit from master we have info on.
It might help to rerun the bundle-size-review job
This might mean that you have a few false positives in this report. If something unrelated to your code changes is reported, you can check this comparison in order to see if they caused this change.

Please look at the full report for more details


Remove unusable Sentry feature flag (!104231 - merged)
Move and rename Sentry constants (!104247 - merged)
Install two versions of Sentry Client SDK (!102790 - merged)	you are here

Install two versions of Sentry Client SDK

What does this MR do and why?

Note

Use case

Screenshots or screen recordings

How to set up and validate locally

Legacy Integration

New integration

Simulate triggering an error

MR acceptance checklist

Activity

Reviewer roulette

Allure report

Bundle size analysis [beta]

Special assets

Significant Growth: 1

New entry points: 1

Install two versions of Sentry Client SDK

What does this MR do and why?

Note

Use case

Screenshots or screen recordings

How to set up and validate locally

Legacy Integration

New integration

Simulate triggering an error

MR acceptance checklist

Merge request reports

Activity

Reviewer roulette

Allure report

Bundle size analysis [beta]

Special assets

Significant Growth: 1

New entry points: 1