Skip to content

Propose Bhanu Pulluri as a maintainer#8230

Merged
matthew1001 merged 7 commits intomainfrom
bhanu-pulluri-nomination
Feb 27, 2025
Merged

Propose Bhanu Pulluri as a maintainer#8230
matthew1001 merged 7 commits intomainfrom
bhanu-pulluri-nomination

Conversation

@matthew1001
Copy link
Contributor

@matthew1001 matthew1001 commented Feb 3, 2025

PR description

Propose Bhanu Pulluri as a Besu maintainer

He has 12 merged PRs and 3 WIP.

Sample of PRs

Bhanu has been actively contributing to a number of areas of Besu. He has a detailed knowledge of the QBFT protocol and has fixed bugs in QBFT and added enhancements that meet the QBFT spec:

He has done various work on JSON/RPC support:

He's added an enhancement to graphql:

He's also resolved high sev CVEs:

Other PRs he's working on include migration of Quorum chains using IBFT "legacy" to QBFT, a complex migrate path that will enable existing enterprise Quorum users to easily move to Besu:

Signed-off-by: Matt Whitehead <matthew.whitehead@kaleido.io>
Copy link
Contributor

@pinges pinges left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@pinges pinges left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

MAINTAINERS.md Outdated
| Name | Github | LFID |
| ---------------- | ---------------- | ---------------- |
| Ameziane Hamlat | ahamlat | ahamlat |
| Bhanu Pulluri | pullurib | bpulluri |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit - should be alphabetical by github

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh OK, no probs, I'll tweak that now

@matthew1001 matthew1001 changed the title Update MAINTAINERS.md Propose Bhanu Pulluri as a maintainer Feb 19, 2025
Signed-off-by: Matt Whitehead <matthew.whitehead@kaleido.io>
@matthew1001
Copy link
Contributor Author

I forgot that the approval process requires an email to the mailing list to inform others about the proposal. I've just sent that email out now.

Copy link
Contributor

@garyschulte garyschulte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@matthew1001
Copy link
Contributor Author

matthew1001 commented Feb 27, 2025

As per the maintainer approval process (3+ approvals and no vetos in 2 weeks) this is approved (see also discord message https://discord.com/channels/905194001349627914/905205502940696607/1344463925042745466) so I'll merge it. Congratulations @pullurib 🎉

@matthew1001 matthew1001 enabled auto-merge (squash) February 27, 2025 09:01
@matthew1001 matthew1001 merged commit 3d41ccb into main Feb 27, 2025
43 checks passed
@matthew1001 matthew1001 deleted the bhanu-pulluri-nomination branch February 27, 2025 09:26
matthew1001 added a commit to besu-eth/governance that referenced this pull request Mar 3, 2025
Now that Bhanu has been approved as a Besu maintainer (see PR hyperledger/besu#8230) he needs adding to the access control list.

Signed-off-by: Matt Whitehead <matthew.whitehead@kaleido.io>
matthew1001 added a commit to matthew1001/hyperledger-governance that referenced this pull request Mar 3, 2025
Now that Bhanu has been approved as a Besu maintainer (see PR hyperledger/besu#8230) he needs adding to the 

Signed-off-by: Matt Whitehead <matthew.whitehead@kaleido.io>
ryjones pushed a commit to hyperledger/governance that referenced this pull request Mar 3, 2025
Now that Bhanu has been approved as a Besu maintainer (see PR hyperledger/besu#8230) he needs adding to the 

Signed-off-by: Matt Whitehead <matthew.whitehead@kaleido.io>
matthew1001 added a commit to besu-eth/governance that referenced this pull request Mar 3, 2025
Now that Bhanu has been approved as a Besu maintainer (see PR hyperledger/besu#8230) he needs adding to the access control list.

Signed-off-by: Matt Whitehead <matthew.whitehead@kaleido.io>
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.

8 participants