Removes read_prometheus_alert permission in favor of admin_operations

changed milestone to %17.4

added Category:Metrics backend devopsmonitor groupanalytics instrumentation maintenanceremoval typemaintenance labels

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	`@Quintasan` (UTC+2, 6 hours ahead of author)	`@kerrizor` (UTC-8, 4 hours behind author)
groupauthorization	Reviewer review is optional for groupauthorization	`@mokhax` (UTC-6, 2 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-review job that generated this comment.

Generated by Danger

added 1 commit

ee80ed3b - Removes read_prometheus_alert permission in favor of admin_operations

Compare with previous version

@ck3g Could you do backend / grouprespond domain review for this removal?

@alexbuijs Could you do groupauthorization review?

requested review from @ck3g and @alexbuijs

approved this merge request

removed review request for @alexbuijs

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-publisher generated test report!

e2e-test-on-gdk: test report for ee80ed3b

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Govern      | 71     | 0      | 0       | 0     | 71    | ✅     |
| Create      | 128    | 0      | 15      | 0     | 143   | ✅     |
| Plan        | 73     | 0      | 0       | 0     | 73    | ✅     |
| Verify      | 44     | 0      | 2       | 0     | 46    | ✅     |
| Package     | 20     | 0      | 12      | 0     | 32    | ✅     |
| Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
| Data Stores | 31     | 0      | 1       | 0     | 32    | ✅     |
| Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
| Fulfillment | 2      | 0      | 0       | 0     | 2     | ✅     |
| Secure      | 3      | 0      | 0       | 0     | 3     | ✅     |
| Manage      | 1      | 0      | 1       | 0     | 2     | ✅     |
| Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 388    | 0      | 31      | 0     | 419   | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+