Skip to content

Comments

update maintainers#1284

Open
ameijer wants to merge 2 commits intocncf:mainfrom
ameijer:atm/update-maintainers-oc
Open

update maintainers#1284
ameijer wants to merge 2 commits intocncf:mainfrom
ameijer:atm/update-maintainers-oc

Conversation

@ameijer
Copy link
Contributor

@ameijer ameijer commented Jan 26, 2026

Checklist for maintainer updates

Note

Delete this template if you're not changing the CSV file

  • You've provided a link to documentation where the project has approved the maintainer changes.
  • The maintainer(s) also created or updated their LFX Individual Dashboard profile.
  • You've sent an email with the list of email address(es) to cncf-maintainer-changes@cncf.io for invitations to Service Desk and mailing lists. You can just mark this complete if you are only removing people.
  • Optional: You've also sent a PR with affiliation updates to cncf/gitdm.

@ameijer
Copy link
Contributor Author

ameijer commented Jan 26, 2026

@rajith77 @Manas23601 @peatey ping me if any of these details do not look accurate

@riaankleinhans riaankleinhans self-assigned this Jan 26, 2026
@riaankleinhans
Copy link
Contributor

@ameijer I looked a the PR.
The default requirement is that maintainers must be listed in the projects maintainer.md file.
https://github.com/opencost/opencost/blob/develop/MAINTAINERS.md

The second URL in the PR would break some of the backend automation. Can you please remove the URL and add Manas Sivakumar to the maintainers.md.

I also do not see Rajith Attapattu listed anywhere in the repo.

To clarify, the purpose of this file is to identify maintainer of project and use the information to give them access to CNCF project resource.

I also notice that the DCO failed as the commits must be signed.

Please reach out if you need help or clarify.

@riaankleinhans
Copy link
Contributor

@ameijer following up on my last comments. Thank you.

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.

2 participants