Skip to content

ci: add CODEOWNERS file - #6453

Merged
mergify[bot] merged 1 commit into
ceph:develfrom
Rakshith-R:add-owners-file
Aug 5, 2026
Merged

ci: add CODEOWNERS file#6453
mergify[bot] merged 1 commit into
ceph:develfrom
Rakshith-R:add-owners-file

Conversation

@Rakshith-R

@Rakshith-R Rakshith-R commented Aug 4, 2026 •
edited by nixpanic
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Add .github/CODEOWNERS to automatically request reviews on PRs
  • Assigns @ceph/ceph-csi-maintainers and @ceph/ceph-csi-contributors as default owners for all files
  • Follows GitHub CODEOWNERS format

Maintainers

  • Madhu-1
  • nixpanic
  • Rakshith-R

Contributors

  • black-dragon74
  • gadididi
  • iPraveenParihar
  • Nikhil-Ladha

Test plan

  • Verify review requests are automatically assigned on new PRs

Depends-on: #6444

@Rakshith-R Rakshith-R added ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures Repo activity Process/activites on ceph-csi repo labels Aug 4, 2026
@mergify mergify Bot added the component/testing Additional test cases or CI work label Aug 4, 2026
@Rakshith-R
Rakshith-R requested review from a team August 4, 2026 12:28
nixpanic
nixpanic reviewed Aug 4, 2026
View reviewed changes
Comment thread CODEOWNERS Outdated
@@ -0,0 +1,14 @@
approvers:

@nixpanic nixpanic Aug 4, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the format Kubernetes uses. Ceph has .github/CODEOWNERS which is probably nicer to use a we're part of that GitHub organization.

@Rakshith-R Rakshith-R Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.
expands to include members in comments

@Rakshith-R
Rakshith-R force-pushed the add-owners-file branch 2 times, most recently from 6766f3b to 7cc32a3 Compare August 4, 2026 14:41
@Rakshith-R Rakshith-R changed the title ci: add CODE-OWNERS file ci: add CODEOWNERS file Aug 4, 2026
@Rakshith-R
Rakshith-R requested review from a team and nixpanic August 4, 2026 14:42
nixpanic
nixpanic reviewed Aug 4, 2026
View reviewed changes
Comment thread .github/CODEOWNERS Outdated
# Maintainers: nixpanic, Madhu-1, Rakshith-R
# Contributors: gman0, ShyamsundarR, pkalever, black-dragon74,
# agarwal-mudit, Nikhil-Ladha, yati1998,
# iPraveenParihar, gadididi

@nixpanic nixpanic Aug 4, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gman0 , @ShyamsundarR , @pkalever , @agarwal-mudit , @yati1998 , you have not been active recently in code reviewing or other contributions. Do you plan to review PRs on regular basis? If not, it is probably time to drop you from the @ceph/ceph-csi-contributors team. Please let us know your opinion, thanks!

@yati1998 yati1998 Aug 5, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah its fine for me to drop, I am currently working on ceph-monitoring side so not sure of active reviews here. thanks

@agarwal-mudit agarwal-mudit Aug 5, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls remove me. I have not contributed anything for ages.

@pkalever pkalever Aug 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, thanks Niels. It makes sense to drop me from the team at-least for now.
As I'm currently focused on RBD development and haven't been able to contribute to ceph-csi reviews regularly. Thanks for reaching out :-)

@Rakshith-R
Rakshith-R force-pushed the add-owners-file branch 2 times, most recently from ed67b6a to 1ba4e0a Compare August 5, 2026 12:05
@Rakshith-R
Rakshith-R requested review from a team and nixpanic August 5, 2026 12:05
Madhu-1
Madhu-1 reviewed Aug 5, 2026
View reviewed changes
Comment thread .github/CODEOWNERS Outdated
Comment on lines +11 to +13
# - Nikhil-Ladha
# - iPraveenParihar
# - gadididi

@Madhu-1 Madhu-1 Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

arrange it in alphabet order?

@Rakshith-R
ci: add CODEOWNERS file
2d5e794 
Add a CODEOWNERS file to automatically assign reviewers.

Maintainers:
- Madhu-1
- nixpanic
- Rakshith-R

Contributors:
- black-dragon74
- gadididi
- iPraveenParihar
- Nikhil-Ladha

Assisted-by: Claude <noreply@anthropic.com>
Signed-off-by: Rakshith R <rar@redhat.com>
@Rakshith-R
Rakshith-R requested a review from Madhu-1 August 5, 2026 12:13
@mergify

mergify Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Deprecation notice: This pull request comes from a fork and was queued with update_method=rebase and update_bot_account impersonation. This capability will be removed on July 1, 2026. After this date, the merge queue will no longer be able to rebase fork pull requests with this configuration. To avoid disruption, switch to update_method=merge in your queue rule.

@mergify

mergify Bot commented Aug 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-08-05 16:06 UTC · Rule: default · triggered by merge protections
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-08-05 16:07 UTC · at 2d5e79429c0e42b35e4dcb3c90beb98bce3bd653 · rebase

This pull request spent 50 seconds in the queue, including 9 seconds running CI.

Required conditions to merge

@mergify
mergify Bot merged commit 9259fb5 into ceph:devel Aug 5, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nixpanic nixpanic nixpanic approved these changes
@Madhu-1 Madhu-1 Madhu-1 approved these changes
+3 more reviewers
@pkalever pkalever pkalever left review comments
@agarwal-mudit agarwal-mudit agarwal-mudit left review comments
@yati1998 yati1998 yati1998 left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/testing Additional test cases or CI work Repo activity Process/activites on ceph-csi repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Rakshith-R @nixpanic @pkalever @Madhu-1 @agarwal-mudit @yati1998