Skip to content

Add hjoshi123 as maintainer#67

Merged
cert-manager-prow[bot] merged 1 commit intomainfrom
add_hjoshi123_as_maintainer
Mar 10, 2026
Merged

Add hjoshi123 as maintainer#67
cert-manager-prow[bot] merged 1 commit intomainfrom
add_hjoshi123_as_maintainer

Conversation

@erikgb
Copy link
Member

@erikgb erikgb commented Feb 22, 2026

Hemant (@hjoshi123 ) has been an important contributor to cert-manager.

After joining the project, Hemant has worked on several important features, in particular, gateway-api ListenerSet support and renewal policy, and assisted with reviewing PRs from community contributors. As Hemant is located in the Mountain West area of the USA, he cannot join our stand-ups due to time zone differences. But he usually joins our (now) weekly development meetings and engages in discussions. Adding Hemant as a maintainer will diversify our maintainer team and increase the capacity to review and merge from contributors. All current maintainers are located in Europe, which means that by adding Hemant to the maintainer team, we will extend our capabilities across time zones and KubeCon NA participation. I have worked together with Hemant on several tasks, and I find him very knowledgeable and trustworthy.

I (@erikgb) think @hjoshi123 is an important part of our community and, for that reason, must receive the privileges required to contribute optimally. All CNCF maintainers are granted access to free GitHub Copilot Enterprise, which I am using more and more, not only to improve my own PRs, but also in reviewing PRs from contributors. And by promoting Hemant to maintainer, he will also get access to this feature.

Dear maintainers, I'd like us to reach an agreement on the following matter using lazy consensus:
Add hjoshi123 as maintainer: #67
🧑‍💻 Participants: @cert-manager-maintainers
📢 Deadline: March 6th, 2026 23:59 UTC
🚨 Note: to speed up the process, you may answer on the PR with a 👍 or a comment stating that you are lazy to help reach consensus before the deadline.

Signed-off-by: Erik Godding Boye <egboye@gmail.com>
@cert-manager-prow cert-manager-prow bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 22, 2026
@ThatsMrTalbot
Copy link
Contributor

Deadline is today, adding my approval with a hold we can remove once deadline has passed

/lgtm
/approve
/hold

@cert-manager-prow cert-manager-prow bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged. labels Mar 6, 2026
@cert-manager-prow
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ThatsMrTalbot

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cert-manager-prow cert-manager-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 6, 2026
@maelvls
Copy link
Member

maelvls commented Mar 10, 2026

The deadline has passed, it's all green, let's proceed.

@maelvls
Copy link
Member

maelvls commented Mar 10, 2026

/unhold

@cert-manager-prow cert-manager-prow bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 10, 2026
@cert-manager-prow cert-manager-prow bot merged commit 6b667b6 into main Mar 10, 2026
2 checks passed
@maelvls
Copy link
Member

maelvls commented Mar 10, 2026

Pre-Onboarding Checklist – info to be requested to the new maintainer:

  • New maintainer's email: mail at hjoshi.me
  • New maintainer's GitHub ID: @hjoshi123
  • New maintainer's Kubernetes Slack ID (see here for how to get it): U08EEJK95Q9

Onboarding Checklist – tasks to be performed by an existing maintainer:

Onboarding Maintainer: @maelvls

Don't forget to add the links to the various PRs or IT tickets as you complete the tasks, and add any missing item.

Initial Privileges:

Self-access Privileges:

  • Go over the list of self-service privileges the new maintainer can get access to by showing them the infrastructure README. Let them know that they can admin access to any of these services by opening a ticket at servicedesk.cncf.io.

Privileges added outside of the CNCF Service Desk:

These are the list of privileges that weren't asked through a ticket at servicedesk.cncf.io because this service isn't owned by the CNCF yet.

none yet

maelvls added a commit to maelvls/foundation that referenced this pull request Mar 10, 2026
The vote took place in cert-manager/community#67.

Signed-off-by: Maël Valais <mael@vls.dev>
maelvls added a commit to cert-manager/makefile-modules that referenced this pull request Mar 10, 2026
Hemant was added as a maintainer in cert-manager/community#67

Signed-off-by: Maël Valais <mael@vls.dev>
@maelvls maelvls deleted the add_hjoshi123_as_maintainer branch March 12, 2026 15:46
maelvls added a commit to maelvls/kubernetes-community that referenced this pull request Mar 12, 2026
Hemant Joshi (@hjoshi123) was recently voted as a new maintainer for the
cert-manager project as documented in cert-manager/community#67.

This change adds him to the cert-manager-maintainers Slack usergroup to
enable efficient communication and decision-making within the maintainer team.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
maelvls added a commit to maelvls/kubernetes-community that referenced this pull request Mar 12, 2026
Hemant Joshi (@hjoshi123) was recently voted as a new maintainer for the
cert-manager project as documented in cert-manager/community#67.

This change adds him to the cert-manager-maintainers Slack usergroup to
enable efficient communication and decision-making within the maintainer team.

Signed-off-by: Maël Valais <mael@vls.dev>
maelvls added a commit to maelvls/foundation that referenced this pull request Mar 12, 2026
The vote took place in cert-manager/community#67.

Signed-off-by: Maël Valais <mael@vls.dev>
maelvls added a commit to maelvls/foundation that referenced this pull request Mar 12, 2026
The vote took place in cert-manager/community#67.

Signed-off-by: Maël Valais <mael@vls.dev>
maelvls added a commit to maelvls/foundation that referenced this pull request Mar 12, 2026
The vote took place in cert-manager/community#67.

Signed-off-by: Maël Valais <mael@vls.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants