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 pjfitzgibbons to MAINTAINERS.md #1270

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

pjfitzgibbons
Copy link
Contributor

Email evidence of approval :

From: "Gurudatt, Shenoy Pratik" [email protected]
Subject: Re: Github Observability Membership Nomination Date: November 3, 2022 at 2:38:04 PM PDT
To: "Mohan, Kavitha" [email protected], "Jadhav, Anirudha" [email protected], "Fitzgibbons, Peter" [email protected], "Li, Joshua" [email protected], "Wei, Eric" [email protected], "Cui, David" [email protected]

Hi all,

I nominate Peter to be added as a maintainer to Observability repository.

Thanks,
Shenoy Pratik

On 11/3/22, 2:36 PM, "Mohan, Kavitha" [email protected] wrote:

Hi everyone,

I nominate Peter to be added as a maintainer to Observability repository.

Cheers,
Kavitha

On 11/2/22, 1:45 PM, "Jadhav, Anirudha" [email protected] wrote:

   +1 

   On 11/2/22, 1:28 PM, "Fitzgibbons, Peter" <[email protected]> wrote:

       Hello All,

       I believe i need a nomination (reply-all) and 3 up-votes (reply-all).  And hopefully no downvotes (!).

       I believe the instructions declare that someone (not-me?) needs to create the PR for nomination in 1 week.

       Thanks so much for your support.

       Kindest Regards,
       Peter Fitzgibbons - AWS OpenSearch Observability

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.


Email evidence of approval :

From: "Gurudatt, Shenoy Pratik" <[email protected]>
Subject: Re: Github Observability Membership Nomination
Date: November 3, 2022 at 2:38:04 PM PDT
To: "Mohan, Kavitha" <[email protected]>, "Jadhav, Anirudha" <[email protected]>, "Fitzgibbons, Peter" <[email protected]>, "Li, Joshua" <[email protected]>, "Wei, Eric" <[email protected]>, "Cui, David" <[email protected]>

Hi all,

I nominate Peter to be added as a maintainer to Observability repository.

Thanks,
Shenoy Pratik 

On 11/3/22, 2:36 PM, "Mohan, Kavitha" <[email protected]> wrote:

   Hi everyone,

   I nominate Peter to be added as a maintainer to Observability repository.

   Cheers,
   Kavitha

   On 11/2/22, 1:45 PM, "Jadhav, Anirudha" <[email protected]> wrote:

       +1 

       On 11/2/22, 1:28 PM, "Fitzgibbons, Peter" <[email protected]> wrote:

           Hello All,

           I believe i need a nomination (reply-all) and 3 up-votes (reply-all).  And hopefully no downvotes (!).

           I believe the instructions declare that someone (not-me?) needs to create the PR for nomination in 1 week.

           Thanks so much for your support.

           Kindest Regards,
           Peter Fitzgibbons - AWS OpenSearch Observability
@pjfitzgibbons pjfitzgibbons requested a review from a team as a code owner November 9, 2022 00:13
@pjfitzgibbons
Copy link
Contributor Author

@mengweieric @derek-ho Could you please merge this PR and update the team settings to make me maintainer status?

@mengweieric mengweieric merged commit e24b858 into opensearch-project:main Nov 14, 2022
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.

3 participants