Skip to content

Add cwperks as a ccr maintainer#1687

Open
ankitkala wants to merge 3 commits into
opensearch-project:mainfrom
ankitkala:add-cwperks-maintainer
Open

Add cwperks as a ccr maintainer#1687
ankitkala wants to merge 3 commits into
opensearch-project:mainfrom
ankitkala:add-cwperks-maintainer

Conversation

@ankitkala
Copy link
Copy Markdown
Member

@ankitkala ankitkala commented May 12, 2026

Description

Adding cwperks (@cwperks) as a maintainer.

cwperks has been actively contributing to the cross cluster replication plugin and has been involved in maintaining and improving the project. https://github.com/opensearch-project/cross-cluster-replication/pulls?q=is%3Apr+author%3Acwperks

Related Issues

1686

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Ankit Kala <ankikala@amazon.com>
Signed-off-by: Ankit Kala <ankikala@amazon.com>
soosinha
soosinha previously approved these changes May 12, 2026
gbbafna
gbbafna previously approved these changes May 12, 2026
monusingh-1
monusingh-1 previously approved these changes May 12, 2026
@ankitkala ankitkala enabled auto-merge (squash) May 12, 2026 17:36
@ankitkala ankitkala closed this May 14, 2026
auto-merge was automatically disabled May 14, 2026 07:30

Pull request was closed

@ankitkala ankitkala reopened this May 14, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 14, 2026

PR Reviewer Guide 🔍

(Review updated until commit 3742dab)

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ Recommended focus areas for review

Missing @ prefix

Line 7 is missing the '@' prefix for the GitHub username 'mohit10011999'. All other entries in the CODEOWNERS file use '@' before usernames (e.g., '@krishna-ggk', '@cwperks'). Without the '@' prefix, GitHub will not recognize this as a valid user reference, and code ownership notifications will fail for this maintainer.

* mohit10011999

@ankitkala ankitkala enabled auto-merge (squash) May 15, 2026 04:04
Signed-off-by: Ankit Kala <ankikala@amazon.com>
@ankitkala ankitkala dismissed stale reviews from monusingh-1, gbbafna, and soosinha via 3742dab May 15, 2026 04:06
@github-actions
Copy link
Copy Markdown

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit 3742dab.

PathLineSeverityDescription
.github/CODEOWNERS5mediumMaintainer 'saikaranam-amazon' was removed from CODEOWNERS without a corresponding removal from MAINTAINERS.md, and a new maintainer '@cwperks' was added. Removal of an existing maintainer combined with addition of a new one warrants verification that these changes are authorized and reflect actual personnel decisions.

The table above displays the top 10 most important findings.

Total: 1 | Critical: 0 | High: 0 | Medium: 1 | Low: 0


Pull Requests Author(s): Please update your Pull Request according to the report above.

Repository Maintainer(s): You can bypass diff analyzer by adding label skip-diff-analyzer after reviewing the changes carefully, then re-run failed actions. To re-enable the analyzer, remove the label, then re-run all actions.


⚠️ Note: The Code-Diff-Analyzer helps protect against potentially harmful code patterns. Please ensure you have thoroughly reviewed the changes beforehand.

Thanks.

@github-actions
Copy link
Copy Markdown

Persistent review updated to latest commit 3742dab

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
Add missing '@' prefix

The GitHub handle mohit10011999 is missing the '@' prefix, which is required for
CODEOWNERS file syntax. Without it, GitHub won't recognize this as a valid code
owner and won't request reviews from this user.

.github/CODEOWNERS [7]

-* mohit10011999
+* @mohit10011999
Suggestion importance[1-10]: 9

__

Why: The GitHub handle mohit10011999 is missing the '@' prefix on line 7, which is required for proper CODEOWNERS syntax. Without it, GitHub won't recognize this user as a valid code owner, which is a critical functional issue.

High

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants