Release group custom service account email

What does this MR do and why?

Removes the group_service_account_custom_email feature flag and releases the feature.

References

#519622 (closed)

Screenshots or screen recordings

Before	After

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

changed milestone to %17.11

assigned to @dblessing

added Category:System Access Deliverable backend devopssoftware supply chain security documentation feature flag featureenhancement groupauthentication missed-deliverable missed:17.9 pipelinetier-1 sectionsec typefeature labels

removed missed:17.9 label

removed missed-deliverable label

added 1 commit

• 4982f941 - Release group custom service account email

Compare with previous version

	1 Message
	This merge request adds or changes documentation files and requires Technical Writing review. The review should happen before merge, but can be post-merge if the merge request is time sensitive.

Documentation review

The following files require a review from a technical writer:

• doc/api/group_service_accounts.md (Link to current live version)

The review does not need to block merging this merge request. See the:

• Metadata for the *.md files that you've changed. The first few lines of each *.md file identify the stage and group most closely associated with your docs change.
• The Technical Writer assigned for that stage and group.
• Documentation workflows for information on when to assign a merge request for review.

Reviewer roulette

Category	Reviewer	Maintainer
backend	@hacks4oats (UTC-4, 1 hour ahead of author)	@rkadam3 (UTC+5.5, 10.5 hours ahead of author)
groupauthentication	Reviewer review is optional for groupauthentication	@sgarg_gitlab (UTC+5.5, 10.5 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 Danger

added 1 commit

• 5443ba96 - Release group custom service account email

Compare with previous version

requested review from @davidhampgonsalves

approved this merge request

LGTM, would you mind doing the backend maintainer review @rkadam3

requested review from @rkadam3

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 5443ba96

expand test summary

+-------------------------------------------------------------------+
|                          suites summary                           |
+--------------+--------+--------+---------+-------+-------+--------+
|              | passed | failed | skipped | flaky | total | result |
+--------------+--------+--------+---------+-------+-------+--------+
| Verify       | 48     | 0      | 24      | 0     | 72    | ✅     |
| Create       | 124    | 0      | 19      | 0     | 143   | ✅     |
| Govern       | 80     | 0      | 12      | 0     | 92    | ✅     |
| Plan         | 81     | 0      | 8       | 0     | 89    | ✅     |
| Fulfillment  | 2      | 0      | 7       | 0     | 9     | ✅     |
| Monitor      | 8      | 0      | 12      | 0     | 20    | ✅     |
| Tenant Scale | 31     | 0      | 10      | 0     | 41    | ✅     |
| Package      | 24     | 0      | 14      | 0     | 38    | ✅     |
| Secure       | 3      | 0      | 3       | 0     | 6     | ✅     |
| Ai-powered   | 0      | 0      | 2       | 0     | 2     | ➖     |
| Analytics    | 2      | 0      | 0       | 0     | 2     | ✅     |
| Configure    | 0      | 0      | 3       | 0     | 3     | ➖     |
| Release      | 5      | 0      | 1       | 0     | 6     | ✅     |
| Manage       | 1      | 0      | 9       | 0     | 10    | ✅     |
| ModelOps     | 0      | 0      | 1       | 0     | 1     | ➖     |
| Growth       | 0      | 0      | 2       | 0     | 2     | ➖     |
+--------------+--------+--------+---------+-------+-------+--------+
| Total        | 409    | 0      | 127     | 0     | 536   | ✅     |
+--------------+--------+--------+---------+-------+-------+--------+

e2e-test-on-cng: test report for 5443ba96

expand test summary

+-------------------------------------------------------------------+
|                          suites summary                           |
+--------------+--------+--------+---------+-------+-------+--------+
|              | passed | failed | skipped | flaky | total | result |
+--------------+--------+--------+---------+-------+-------+--------+
| Fulfillment  | 2      | 0      | 7       | 0     | 9     | ✅     |
| Manage       | 1      | 0      | 9       | 0     | 10    | ✅     |
| Verify       | 49     | 0      | 23      | 0     | 72    | ✅     |
| Create       | 128    | 0      | 19      | 0     | 147   | ✅     |
| Plan         | 84     | 0      | 8       | 0     | 92    | ✅     |
| Govern       | 82     | 0      | 12      | 0     | 94    | ✅     |
| Package      | 29     | 0      | 15      | 0     | 44    | ✅     |
| Tenant Scale | 31     | 0      | 10      | 0     | 41    | ✅     |
| Secure       | 3      | 0      | 3       | 0     | 6     | ✅     |
| Monitor      | 8      | 0      | 12      | 0     | 20    | ✅     |
| Analytics    | 2      | 0      | 0       | 0     | 2     | ✅     |
| Release      | 5      | 0      | 1       | 0     | 6     | ✅     |
| Configure    | 0      | 0      | 3       | 0     | 3     | ➖     |
| ModelOps     | 0      | 0      | 1       | 0     | 1     | ➖     |
| Ai-powered   | 0      | 0      | 2       | 0     | 2     | ➖     |
| Growth       | 0      | 0      | 2       | 0     | 2     | ➖     |
+--------------+--------+--------+---------+-------+-------+--------+
| Total        | 424    | 0      | 127     | 0     | 551   | ✅     |
+--------------+--------+--------+---------+-------+-------+--------+

approved this merge request

enabled automatic add to merge train when checks pass

I think we need groupauthentication review here.

@eduardosanz - Can you please help?

Thanks!

requested review from @eduardosanz