Add desired sharding keys for pipeline security
Ref:
What does this MR do and why?
This MR adds the desired sharding_key
for the tables owned by grouppipeline security.
Merge request reports
Activity
changed milestone to %Next 1-3 releases
assigned to @morefice
added pipelinetier-1 label
- A deleted user
added database databasereview pending labels
2 Warnings 605652c2: 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. 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.
1 Message CHANGELOG missing: If this merge request needs a changelog entry, add the
Changelog
trailer to the commit message you want to add to the changelog.If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.
Reviewer roulette
Category Reviewer Maintainer backend @bauerdominic
(UTC+2, same timezone as author)
@drew
(UTC+0, 2 hours behind author)
database @bala.kumar
(UTC+5.5, 3.5 hours ahead of author)
@pshutsin
(UTC+2, same timezone as 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 1 commit
- 60963327 - Add desired sharding keys for pipeline security
added 1 commit
- b4b807b7 - Add desired sharding keys for pipeline security
- A deleted user
added backend label
- Resolved by Max Orefice
- Resolved by Max Orefice
@manojmj can you review this MR please?
requested review from @manojmj
mentioned in issue #463243 (closed)
- Resolved by Max Orefice
- Resolved by Manoj M J
added 497 commits
-
b4b807b7...17630dfb - 495 commits from branch
master
- 007cf457 - Add desired sharding keys for pipeline security
- 61b3260b - apply code review feedback
-
b4b807b7...17630dfb - 495 commits from branch
requested review from @manojmj
added 195 commits
-
6b79aae3...10235fc0 - 191 commits from branch
master
- a3e00d58 - Add desired sharding keys for pipeline security
- 24a3a5f8 - apply code review feedback
- 1a94e22c - fix specs
- d97650fa - apply code review feedback
Toggle commit list-
6b79aae3...10235fc0 - 191 commits from branch
- Resolved by Manoj M J
- Resolved by Max Orefice
- Resolved by Max Orefice
- Resolved by Max Orefice
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.
- Resolved by Max Orefice
@OmarQunsulGitlab could you please review database?
requested review from @OmarQunsulGitlab
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for 605652c2expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Plan | 73 | 0 | 0 | 0 | 73 | ✅ | | Verify | 44 | 0 | 2 | 0 | 46 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Create | 128 | 0 | 15 | 0 | 143 | ✅ | | Govern | 71 | 0 | 0 | 0 | 71 | ✅ | | Data Stores | 31 | 0 | 1 | 0 | 32 | ✅ | | Secure | 3 | 0 | 0 | 0 | 3 | ✅ | | Package | 20 | 0 | 12 | 0 | 32 | ✅ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Manage | 1 | 0 | 1 | 0 | 2 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | | Fulfillment | 2 | 0 | 0 | 0 | 2 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 388 | 0 | 31 | 0 | 419 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for 11ff5296expand test summary
+-------------------------------------------------------------+ | suites summary | +--------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +--------+--------+--------+---------+-------+-------+--------+ | Verify | 51 | 0 | 15 | 0 | 66 | ✅ | | Create | 270 | 0 | 34 | 0 | 304 | ✅ | +--------+--------+--------+---------+-------+-------+--------+ | Total | 321 | 0 | 49 | 0 | 370 | ✅ | +--------+--------+--------+---------+-------+-------+--------+
added 2 commits
reset approvals from @manojmj by pushing to the branch
added pipelinetier-3 pipeline:run-e2e-omnibus-once labels and removed pipelinetier-2 label
added databaseapproved label and removed databasereview pending label
aborted automatic add to merge train because the source branch was updated. Learn more.
reset approvals from @OmarQunsulGitlab by pushing to the branch
removed pipeline:run-e2e-omnibus-once label
aborted automatic add to merge train because the source branch was updated. Learn more.