Remove ignore rules from Component
What does this MR do and why?
A milestone has passed and now we can remove the ignore rules on inputs
and path
from Ci::Catalog::Resources::Component
.
Issue: #455431 (closed)
Merge request reports
Activity
changed milestone to %17.1
assigned to @avielle
1 Warning There were no new or modified feature flag YAML files detected in this MR. If the changes here are already controlled under an existing feature flag, please add
the feature flagexists. Otherwise, if you think the changes here don't need
to be under a feature flag, please add the label feature flagskipped, and
add a short comment about why we skipped the feature flag.For guidance on when to use a feature flag, please see the documentation.
Reviewer roulette
Category Reviewer Maintainer backend @tyleramos
(UTC-4, 6 hours behind author)
@pshutsin
(UTC+2, same timezone as author)
~"Verify" Reviewer review is optional for ~"Verify" @hfyngvason
(UTC-4, 6 hours behind author)
Please check reviewer's status!
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
DangerEdited by Ghost User- Resolved by Allison Browne
@tyleramos can you please do the first review?
requested review from @tyleramos
added pipelinetier-1 label
mentioned in epic &7462
added pipeline:mr-approved label
- Resolved by Pavel Shutsin
@tyleramos
, thanks for approving this merge request.This is the first time the merge request has been approved. To ensure we don't only run predictive pipelines, and we don't break
master
, a new pipeline will be started shortly.Please wait for the pipeline to start before resolving this discussion and set auto-merge for the new pipeline. See merging a merge request for more details.
added pipelinetier-2 label and removed pipelinetier-1 label
@pshutsin Do you mind running the maintainer review for this MR when you have time? Thanks in advance!
requested review from @pshutsin and removed review request for @tyleramos
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for 9c22966fexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Verify | 30 | 0 | 2 | 29 | 32 | ✅ | | Create | 121 | 0 | 10 | 93 | 131 | ✅ | | Govern | 64 | 0 | 1 | 41 | 65 | ✅ | | Plan | 54 | 0 | 2 | 47 | 56 | ✅ | | Analytics | 1 | 0 | 1 | 0 | 2 | ✅ | | Data Stores | 31 | 0 | 0 | 22 | 31 | ✅ | | Package | 19 | 0 | 12 | 19 | 31 | ✅ | | Release | 5 | 0 | 0 | 5 | 5 | ✅ | | Manage | 0 | 0 | 1 | 0 | 1 | ➖ | | Monitor | 8 | 0 | 0 | 7 | 8 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 333 | 0 | 29 | 263 | 362 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for 9c22966fexpand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Verify | 57 | 0 | 13 | 0 | 70 | ✅ | | Create | 318 | 0 | 36 | 6 | 354 | ✅ | | Data Stores | 22 | 0 | 0 | 0 | 22 | ✅ | | Plan | 44 | 0 | 4 | 0 | 48 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Govern | 26 | 0 | 0 | 0 | 26 | ✅ | | Release | 2 | 0 | 0 | 0 | 2 | ✅ | | Package | 6 | 0 | 8 | 0 | 14 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 483 | 0 | 61 | 6 | 544 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
Edited by Ghost Userremoved review request for @pshutsin
requested review from @allison.browne