Allow instance level runners to be restricted to certain plan types

added [Deprecated] Category:Runner devopsverify grouprunner sectionops labels

@erushton @pedropombeiro This is my PoC for the idea of allowing separation of runners for specific GitLab plans.

marked this merge request as draft

1 Warning

featureaddition and featureenhancement merge requests normally have a documentation change. Consider adding a documentation update or confirming the documentation plan with the Technical Writer counterpart.

For more information, see:

The Handbook page on merge request types.
The definition of done documentation.

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
backend	Jonathan Schafer (`@jschafer`) (UTC-5, 7 hours behind `@tmaczukin`)	Etienne Baqué (`@ebaque`) (UTC+3, 1 hour ahead of `@tmaczukin`)
database	Matt Kasa (`@mattkasa`) (UTC+0, 2 hours behind `@tmaczukin`)	Mayra Cabrera (`@mayra-cabrera`) (UTC-5, 7 hours behind `@tmaczukin`)
~migration	No reviewer available	No maintainer available

To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

Generated by Danger

added featureenhancement + 1 deleted label

added typefeature label

changed the description

Allure report

allure-report-publisher generated test report!

review-qa-reliable: test report
review-qa-smoke: test report
package-and-qa-ff-enabled: test report
package-and-qa-ff-disabled: test report

review-qa-blocking: test report for 436ba944

expand test summary

+-------------------------------------------------------------------+
|                          suites summary                           |
+----------------------+--------+--------+---------+-------+--------+
|                      | passed | failed | skipped | flaky | result |
+----------------------+--------+--------+---------+-------+--------+
| Plan                 | 41     | 0      | 1       | 41    | ❗     |
| Manage               | 31     | 0      | 2       | 29    | ❗     |
| Create               | 24     | 0      | 2       | 22    | ❗     |
| Verify               | 12     | 0      | 1       | 12    | ❗     |
| Package              | 0      | 0      | 1       | 0     | ➖     |
| Version sanity check | 0      | 0      | 1       | 0     | ➖     |
| Protect              | 2      | 0      | 0       | 2     | ❗     |
| Configure            | 0      | 0      | 1       | 0     | ➖     |
+----------------------+--------+--------+---------+-------+--------+
| Total                | 110    | 0      | 9       | 106   | ❗     |
+----------------------+--------+--------+---------+-------+--------+

**package-and-qa-ff-disabled**:

test report for 436ba944

expand test summary

+-------------------------------------------------------------------+
|                          suites summary                           |
+----------------------+--------+--------+---------+-------+--------+
|                      | passed | failed | skipped | flaky | result |
+----------------------+--------+--------+---------+-------+--------+
| Create               | 148    | 1      | 6       | 7     | ❌     |
| Verify               | 33     | 0      | 8       | 5     | ❗     |
| Plan                 | 53     | 0      | 0       | 1     | ❗     |
| Secure               | 16     | 0      | 3       | 2     | ❗     |
| Manage               | 91     | 0      | 6       | 2     | ❗     |
| Version sanity check | 0      | 0      | 1       | 0     | ➖     |
| Fulfillment          | 2      | 0      | 10      | 0     | ✅     |
| Configure            | 0      | 0      | 3       | 0     | ➖     |
| Release              | 6      | 0      | 0       | 3     | ❗     |
| Package              | 0      | 0      | 3       | 0     | ➖     |
| Non-devops           | 2      | 0      | 0       | 0     | ✅     |
| Protect              | 2      | 0      | 0       | 0     | ✅     |
+----------------------+--------+--------+---------+-------+--------+
| Total                | 353    | 1      | 40      | 20    | ❌     |
+----------------------+--------+--------+---------+-------+--------+

**package-and-qa-ff-enabled**:

test report for 436ba944

expand test summary

+----------------------------------------------------------+
|                      suites summary                      |
+-------------+--------+--------+---------+-------+--------+
|             | passed | failed | skipped | flaky | result |
+-------------+--------+--------+---------+-------+--------+
| Plan        | 48     | 0      | 0       | 1     | ❗     |
| Create      | 122    | 1      | 3       | 6     | ❌     |
| Verify      | 28     | 0      | 6       | 4     | ❗     |
| Manage      | 66     | 0      | 6       | 1     | ❗     |
| Release     | 6      | 0      | 0       | 3     | ❗     |
| Fulfillment | 2      | 0      | 8       | 0     | ✅     |
| Secure      | 11     | 0      | 3       | 2     | ❗     |
| Package     | 0      | 0      | 3       | 0     | ➖     |
| Non-devops  | 1      | 0      | 0       | 0     | ✅     |
| Configure   | 0      | 0      | 3       | 0     | ➖     |
| Protect     | 2      | 0      | 0       | 0     | ✅     |
+-------------+--------+--------+---------+-------+--------+
| Total       | 286    | 1      | 32      | 17    | ❌     |
+-------------+--------+--------+---------+-------+--------+

This MR is a work-in-progress. It will still require few updates (+description) before it'll be ready for the first review

added 1 commit

7450a4f4 - Add Runner separation by plan PoC

Compare with previous version

changed title from Draft: Add Runner separation by plan PoC to Draft: Allow instance level runners to be restricted to certain plan types (PoC)

changed the description

Allow instance level runners to be restricted to certain plan types

Database query plans

Activity

Reviewer roulette

Allure report

Allow instance level runners to be restricted to certain plan types

Database query plans

Merge request reports

Activity

Reviewer roulette

Allure report