Database Trainee Maintainer: Bogdan Denkovych
Reviewer of database changes
- Include namespaces in project searches when que... (gitlab-org/gitlab!108906 - merged)
- Step 1: Prepare `ci_build_needs` for bigint con... (gitlab-org/gitlab!110521 - merged)
- Include memberships from groups share in Preloader (gitlab-org/gitlab!111157 - merged)
- ScimToken and SamlProvider to be deleted for no... (gitlab-org/gitlab!115640 - merged)
- Add admin compliance framework custom permission (gitlab-org/gitlab!144183 - merged)
- Zoekt: Fix perform_at for tasks (gitlab-org/gitlab!150145 - merged)
- Remove a MR index with state_id, iid synchronously (gitlab-org/gitlab!150403 - merged)
- Add creator relationship to namespaces (gitlab-org/gitlab!150051 - merged)
- Increase Atlassian token constraint (gitlab-org/gitlab!152472 - merged)
- Setting for resource token expiry notifications... (gitlab-org/gitlab!168245 - merged)
- Disable password authentication for SSO users (gitlab-org/gitlab!168111 - merged)
- Backfill project bot user details bot namespace (gitlab-org/gitlab!171326 - merged)
- Optimise query hierarchy for filtered add on el... (gitlab-org/gitlab!173550 - merged)
- Add namespace_id column to redirect_routes (gitlab-org/gitlab!183002 - merged)
- Finalize BBM MigrateScimTokens (gitlab-org/gitlab!194247 - merged)
Author of database changes
- https://gitlab.com/gitlab-org/gitlab/-/merge_requests?scope=all&state=all&author_username=bdenkovych&label_name[]=database
- https://gitlab.com/gitlab-org/security/gitlab/-/merge_requests/3962+
- https://gitlab.com/gitlab-org/gitlab/-/issues/488502+
Working towards becoming a maintainer
This is not a checklist, but guidelines that will help you become a maintainer. Remember that there is no specific timeline on this, and that you should work together with your manager and current maintainers.
It is up to you to ensure that you are getting enough MRs to review. If you are not receiving enough MRs to advance in your training, be proactive and seek out opportunities for review. Maintainers are available to help guide you.
After each MR is merged or closed, add a discussion to this issue using this template:
### (Merge request title): (Merge request URL)
During review:
- (List anything of note, or a quick summary. "I suggested/identified/noted...")
Post-review:
- (List anything of note, or a quick summary. "I missed..." or "Merged as-is")
(@support-maintainer-username) please add feedback, and compare this review to
the average maintainer review.
Warning: For security MRs, no discussion should take place in the issue (even if using an internal note), as this might reveal security issue details. Instead, if you want to discuss the MR, you may create a discussion in the original MR, and link to it from a discussion in the trainee maintainer issue.
When you're ready to make it official
When reviews have accumulated, and recent reviews consistently fulfill maintainer responsibilities:
-
Create a merge request updating your team member entry using one of available merge request templates proposing yourself as a maintainer for the relevant application. -
Follow the instructions in the merge request template