Skip to content

Update OpenCost maintainers#1284

Merged
riaankleinhans merged 3 commits into
cncf:mainfrom
ameijer:atm/update-maintainers-oc
May 18, 2026
Merged

Update OpenCost maintainers#1284
riaankleinhans merged 3 commits into
cncf:mainfrom
ameijer:atm/update-maintainers-oc

Conversation

@ameijer
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

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

@krook krook marked this pull request as draft February 27, 2026 20:35
@riaankleinhans riaankleinhans changed the title update maintainers Update OpenCost maintainers Apr 8, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 15, 2026

Deploy Preview for cncf-license-exceptions ready!

Name Link
🔨 Latest commit c3d0a23
🔍 Latest deploy log https://app.netlify.com/projects/cncf-license-exceptions/deploys/6a0751ce9bff030008f9e613
😎 Deploy Preview https://deploy-preview-1284--cncf-license-exceptions.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ameijer
Copy link
Copy Markdown
Contributor Author

ameijer commented May 15, 2026

@riaankleinhans we are ready for re-review here, have addressed the concerns

@ameijer ameijer marked this pull request as ready for review May 15, 2026 15:05
@riaankleinhans
Copy link
Copy Markdown
Contributor

Thank you @ameijer . Approved. Please sign your commit for me to be able to merge.

ameijer added 3 commits May 15, 2026 13:02
Signed-off-by: Alex Meijer <alexander.meijer@ibm.com>
Signed-off-by: Alex Meijer <alexander.meijer@ibm.com>
Signed-off-by: Alex Meijer <alexander.meijer@ibm.com>
@ameijer ameijer force-pushed the atm/update-maintainers-oc branch from aa62490 to c3d0a23 Compare May 15, 2026 17:03
@ameijer
Copy link
Copy Markdown
Contributor Author

ameijer commented May 15, 2026

@riaankleinhans fixed!

@riaankleinhans riaankleinhans merged commit 0ea09dc into cncf:main May 18, 2026
6 checks passed
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