Request: Role change for @praveenrewar

[praveenrewar]

Summary

This is a formal request for consideration of a role change in the Carvel project by a member of the community.

Each role is defined in our contribution model document. If you would like to request mutliple roles / multiple subprojects,
please open a new issue for each request.

GitHub Username

@praveenrewar

Role Requested

• Reviewer
• Approver
• Lead

Applicable Subproject

• kapp-controller
• ytt
• kapp
• kbld
• imgpkg
• vendir
• secretgen-controller
• kctrl

General Requirements

• I have reviewed the code of conduct (https://github.com/carvel-dev/carvel/blob/develop/CODE_OF_CONDUCT.md).
• I have enabled 2FA on my GitHub account (https://github.com/settings/security).
• I have active participation and provide support (such as GitHub, meetings, Slack, Stack Overflow) in the community for long enough to have demonstrated knowledge and competency (such as 3 months).
• I have at least one sponsor (see current maintainers list).
• I have spoken to my sponsors ahead of this application, and they have agreed to sponsor my application.

Role Specific Requirements

Reviewer

• I have reviewed / authored of at least 5 substantial* pull requests to the subproject(s).

Sponsors

• @neil-hickey
• @joaopapereira

List of contributions to the Carvel subprojects(s)

PRs reviewed / authored

All the PRs authored by me: https://github.com/carvel-dev/kapp-controller/pulls?q=is%3Apr+author%3Apraveenrewar+is%3Amerged+-label%3Acli+
Some of the most relevant PRs are:

• Unblock app deletion when ns is terminating and resources are in the same ns only kapp-controller#1208
• Add defaultNamespace in pkgi/app crs kapp-controller#1317
• Do not overwrite kapp deploy status during delete kapp-controller#1460
• Do not expose development values to kc package bundle kapp-controller#1111
• Expose values in kapp-controller package kapp-controller#1120

Issues responded to

• app cr downward api fails when referencing annotations with a dot kapp-controller#1328
• Cannot set nested value path via downward API kapp-controller#1520
• kapp-controller does not show kapp associatedResources after second deploy (noop) kapp-controller#863
• Is there a way to configure the merge strategy for the kapp-controller when reconciling the service resources?  kapp-controller#825
• Integration with Open Service Mesh kapp-controller#823

Projects I am involved with

• I am currently working on the proposal for dependency management.

Next Steps

• Once your sponsors have responded with a +1 for your nomination, your request will be reviewed by the project leads. Any missing information will be requested.

---

*Substantial is subject to the lead's discretion (e.g. refactors, enhancements rather than grammar correction or trivial pull requests)

[netlify]

✅ Deploy Preview for carvel ready!

Name	Link
🔨 Latest commit	c6da369
🔍 Latest deploy log	https://app.netlify.com/sites/carvel/deploys/6615c771ec6465000845f4a2
😎 Deploy Preview	https://deploy-preview-747--carvel.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[joaopapereira]

With all the work that @praveenrewar has been doing for years in kapp-controller, I believe he is more than ready to become a reviewer. 😄
Thanks for all the hard work you devoted to this project.

[neil-hickey]

+1 to @praveenrewar being a reviewer (or more ;) )! Goes above and beyond and has a deep understanding of kapp-controller. Happy to see!