Revert "Merge branch 'sc1-add-endpoint-id-to-web-txn' into 'master' "
Purpose of revert
Revert "Merge branch 'sc1-add-endpoint-id-to-web-txn' into 'master' "
Changes in Add endpoint_id label to web transaction metrics (!132095 - merged) cause consuming projects (such as CustomersDot to be unable to authenticate
The changes are also causing e2e specs to fail
How to set up and validate locally
- Verify MR is straight revert
- Can spin up CDot locally and verify that auth is now working again
- Can also spin up
browser_ui/1_manage/login/login_via_oauth_and_oidc_with_gitlab_as_idp_spec.rb | Manage OAuth behaves like Instance OAuth Application creates oauth application and uses it to login
e2e test to ensure it now passes
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.
Checklist
-
Create an issue to reinstate the merge request and assign it to the author of the reverted merge request. -
If the revert is to resolve a broken 'master' incident, please read through the Responsibilities of the Broken master
resolution DRI. -
If the revert involves a database migration, please read through Deleting existing migrations. -
Add the appropriate labels before the MR is created. We can skip CI/CD jobs only if the labels are added before the CI/CD pipeline is created.
Milestone info
-
I am reverting something in the current milestone. No changelog is needed, and I've added a ~"regression:*"
label. -
I am reverting something in a different milestone. A changelog is needed, and I've removed the ~"regression:*"
label.
Related issues and merge requests
- Failure in browser_ui/1_manage/login/login_via_... (#425841 - closed)
- Add endpoint_id label to web transaction metrics (!132095 - merged)
cc @schin1
Edited by Laura Callahan