Removed feature flag extended_expiry_webhook_execution_setting
What does this MR do and why?
Removed feature flag extended_expiry_webhook_execution_setting
This flag is enabled for .com version since last 3 weeks or so. There had been no related incidents or functionality issues. The functionality is also verified
Changelog: removed
Solves - #513684 (closed)
Merge request reports
Activity
changed milestone to %17.10
assigned to @sgarg_gitlab
added pipelinetier-1 label
added documentation frontend labels
2 Warnings ⚠️ 41736524: The commit body should not contain more than 72 characters per line. For more information, take a look at our Commit message guidelines. ⚠️ 5f111fa2: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines. 1 Message 📖 This merge request adds or changes documentation files and requires Technical Writing review. The review should happen before merge, but can be post-merge if the merge request is time sensitive. Documentation review
The following files require a review from a technical writer:
-
doc/user/project/settings/_index.md(Link to current live version)
The review does not need to block merging this merge request. See the:
-
Metadata for the
*.mdfiles that you've changed. The first few lines of each*.mdfile identify the stage and group most closely associated with your docs change. - The Technical Writer assigned for that stage and group.
- Documentation workflows for information on when to assign a merge request for review.
Reviewer roulette
Category Reviewer Maintainer backend @hmerscher
(UTC+1, 4.5 hours behind author)@alipniagov (UTC+1, 4.5 hours behind author)frontend @ms.mondrian (UTC-4, 9.5 hours behind author)@jivanvl (UTC-6, 11.5 hours behind author)groupauthentication Reviewer review is optional for groupauthentication @fernando-c (UTC-5, 10.5 hours behind 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-reviewjob that generated this comment.Generated by
🚫 DangerEdited by ****-
requested review from @hmerscher
requested review from @fernando-c
requested review from @tachyons-gitlab and removed review request for @fernando-c
removed review request for @hmerscher
requested review from @anastasia.khom
requested review from @mlunoe and removed review request for @anastasia.khom
Resolved by Aleksei Lipniagov@tachyons-gitlab Could you please take backend and groupauthentication review for this MR ?
@mlunoe Could you please take frontend review for this MR? This is just a feature flag removal, hence I thought we can straight away for a maintainer review
- Last reply by Smriti Garg
added pipeline:mr-approved label
added pipelinetier-2 label 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-publishergenerated test report!e2e-test-on-gdk:
✅ test report for 5f111fa2expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Plan | 82 | 0 | 8 | 2 | 90 | ✅ | | Govern | 79 | 0 | 13 | 0 | 92 | ✅ | | Create | 140 | 0 | 22 | 3 | 162 | ✅ | | Data Stores | 31 | 0 | 10 | 0 | 41 | ✅ | | Verify | 48 | 0 | 24 | 1 | 72 | ✅ | | Manage | 1 | 0 | 9 | 0 | 10 | ✅ | | Monitor | 8 | 0 | 12 | 0 | 20 | ✅ | | Fulfillment | 2 | 0 | 7 | 0 | 9 | ✅ | | Package | 24 | 0 | 14 | 2 | 38 | ✅ | | Release | 5 | 0 | 1 | 0 | 6 | ✅ | | Configure | 0 | 0 | 3 | 0 | 3 | ➖ | | Growth | 0 | 0 | 2 | 0 | 2 | ➖ | | ModelOps | 0 | 0 | 1 | 0 | 1 | ➖ | | Secure | 4 | 0 | 4 | 0 | 8 | ✅ | | Ai-powered | 0 | 0 | 2 | 0 | 2 | ➖ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 426 | 0 | 132 | 8 | 558 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+e2e-test-on-cng:
✅ test report for 5f111fa2expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Plan | 86 | 0 | 8 | 0 | 94 | ✅ | | Govern | 82 | 0 | 12 | 0 | 94 | ✅ | | Create | 144 | 0 | 22 | 0 | 166 | ✅ | | Verify | 49 | 0 | 23 | 0 | 72 | ✅ | | Data Stores | 31 | 0 | 10 | 0 | 41 | ✅ | | Package | 29 | 0 | 15 | 0 | 44 | ✅ | | Manage | 1 | 0 | 9 | 0 | 10 | ✅ | | Monitor | 8 | 0 | 12 | 0 | 20 | ✅ | | Ai-powered | 0 | 0 | 2 | 0 | 2 | ➖ | | Secure | 4 | 0 | 4 | 0 | 8 | ✅ | | Release | 5 | 0 | 1 | 0 | 6 | ✅ | | Growth | 0 | 0 | 2 | 0 | 2 | ➖ | | Fulfillment | 2 | 0 | 7 | 0 | 9 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | | Configure | 0 | 0 | 3 | 0 | 3 | ➖ | | ModelOps | 0 | 0 | 1 | 0 | 1 | ➖ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 443 | 0 | 131 | 0 | 574 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+Edited by ****
