Remove dependency list alerts
What does this MR do and why?
The "dependency list incomplete" alert is dead code because we never return this status. The "dependecy list job failed" alert should not be shown, because we still ingest reports from failed pipelines.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
Merge request reports
Activity
assigned to @bwill
added pipelinetier-3 label
removed pipelinetier-3 label
- Resolved by 🤖 GitLab Bot 🤖
Proper labels assigned to this merge request. Please ignore me.
@bwill
- please see the following guidance and update this merge request.1 Error Please add typebug typefeature, or typemaintenance label to this merge request. Edited by 🤖 GitLab Bot 🤖
- A deleted user
added frontend maintenanceremoval typemaintenance labels
2 Warnings This merge request is quite big (517 lines changed), please consider splitting it into multiple merge requests. 35ef1009: 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
Category Reviewer Maintainer frontend @fernanda.toledo
(UTC-7, 2 hours behind author)
@psjakubowska
(UTC+2, 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
Dangeradded 801 commits
-
7b3e9820...620da422 - 800 commits from branch
master
- c1976aa8 - Remove dependency list alerts
-
7b3e9820...620da422 - 800 commits from branch
changed milestone to %17.3
- Resolved by Vitaly Slobodin
@lorenzvanherwaarden Could you please do initial frontend review for this MR?
requested review from @lorenzvanherwaarden
added pipeline:mr-approved label
added pipelinetier-3 label
requested review from @vitallium and removed review request for @lorenzvanherwaarden
mentioned in commit 679583fc
mentioned in merge request !161944 (merged)
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for 35ef1009expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Manage | 0 | 0 | 1 | 0 | 1 | ➖ | | Create | 127 | 0 | 12 | 0 | 139 | ✅ | | Verify | 44 | 0 | 2 | 0 | 46 | ✅ | | Plan | 70 | 0 | 0 | 0 | 70 | ✅ | | Package | 16 | 0 | 15 | 0 | 31 | ✅ | | Data Stores | 31 | 0 | 1 | 0 | 32 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Govern | 71 | 0 | 0 | 0 | 71 | ✅ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | | Fulfillment | 2 | 0 | 0 | 0 | 2 | ✅ | | Secure | 2 | 0 | 0 | 0 | 2 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 378 | 0 | 31 | 0 | 409 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for 35ef1009expand test summary
+-------------------------------------------------------------+ | suites summary | +--------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +--------+--------+--------+---------+-------+-------+--------+ | Govern | 105 | 0 | 8 | 1 | 113 | ✅ | | Create | 270 | 0 | 34 | 0 | 304 | ✅ | +--------+--------+--------+---------+-------+-------+--------+ | Total | 375 | 0 | 42 | 1 | 417 | ✅ | +--------+--------+--------+---------+-------+-------+--------+
added this merge request to the merge train at position 2
mentioned in commit 399ab57b
added workflowstaging-canary label
added workflowcanary label and removed workflowstaging-canary label
mentioned in commit 42f8a532
mentioned in commit f73813fe
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
added workflowpost-deploy-db-production label and removed workflowproduction label
added releasedcandidate label
mentioned in commit 6fdfd67d
added releasedpublished label and removed releasedcandidate label
mentioned in merge request kubitus-project/kubitus-installer!3241 (merged)