Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Joe Eltgroth as contributor #1037

Merged
merged 2 commits into from
Jan 10, 2025
Merged

Conversation

joeeltgroth
Copy link
Contributor

New team member of the team supporting credhub at Broadcom

Copy link

linux-foundation-easycla bot commented Dec 12, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@beyhan
Copy link
Member

beyhan commented Dec 16, 2024

Hi @joeeltgroth,

Please sign the CLA which is required to merge this PR and we need two thumps up or approvals from approver in the Credhub WG area.
FYI: @cloudfoundry/wg-foundational-infrastructure-credential-management-credhub-approvers

@beyhan beyhan added the wg-fi label Dec 16, 2024
@beyhan
Copy link
Member

beyhan commented Dec 16, 2024

See comment in #1036 (comment) for details about how to sing the CLA.

@joeeltgroth
Copy link
Contributor Author

I have made my membership public here: https://github.com/orgs/pivotal-cf/people?query=joeeltgroth
Do I now need to sign the CLA again? When the CLA dialog asks what company, what do I enter in to map it to pivotal-cf?

@beyhan
Copy link
Member

beyhan commented Dec 16, 2024

@jpalermo, @rkoster help need ^

@joeeltgroth
Copy link
Contributor Author

@beyhan @rkoster I have gotten the EasyCLA authorization. Could this now be approved?

@hsinn0
Copy link
Contributor

hsinn0 commented Jan 6, 2025

need two thumps up or approvals from approver in the Credhub WG area. FYI: @cloudfoundry/wg-foundational-infrastructure-credential-management-credhub-approvers

Approving.

@rkoster rkoster merged commit 21489bf into cloudfoundry:main Jan 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants