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

Proposing a new Fluentd maintainer #4069

Merged
merged 1 commit into from
Mar 27, 2023
Merged

Proposing a new Fluentd maintainer #4069

merged 1 commit into from
Mar 27, 2023

Conversation

kenhys
Copy link
Contributor

@kenhys kenhys commented Feb 24, 2023

Which issue(s) this PR fixes:

N/A

What this PR does / why we need it:

According to our governance policy, we need to vote for changing maintainership.

As @daipom contributing to fluentd a lot, so I'll propose @daipom as a new maintainer.

Currently, there are following organizations:

And individual developers.

(We need to also update affilicated compaty information in MAINTAINERS.md later)

Thus, the total number of organization is 8 and satisfying the majority of 2/3 must be not less than 6.

So every organization need to show 👍 or 👎

Docs Changes:

N/A

Release Note:

N/A

Signed-off-by: Kentaro Hayashi <[email protected]>
@edsiper
Copy link
Member

edsiper commented Feb 25, 2023

Hi @kenhys

I think we need to put a plan in place to incorporate new maintainers, happy to support that and have a voting system. Would @daipom open a PR and document intentions and list of contributions that can help to attribute him a maintainer status ?

@daipom
Copy link
Contributor

daipom commented Feb 26, 2023

Thanks for proposing me.

I am interested in processing Ruby and log data. I want to learn these things through this product, and I want to leverage the benefits and new features of Ruby for this product.

Would @daipom open a PR and document intentions and list of contributions that can help to attribute him a maintainer status ?

Sure!
Would it be better to comment on this PR this time? Or is it better to open a new PR?

@cosmo0920
Copy link
Contributor

FYI: My GitHub account name is @cosmo0920 not @hatake...

@kenhys
Copy link
Contributor Author

kenhys commented Feb 26, 2023

FYI: My GitHub account name is @cosmo0920 not @hatake...

Oh, orz... fixed it.

@daipom
Copy link
Contributor

daipom commented Feb 27, 2023

Thanks for proposing me.

I describe myself below.
Please let me know if any explanations are missing.

Intentions

I deliver Fluentd enterprise services in ClearCode over the past year.
I want to make Fluentd a better software, not only within the framework of our service.
This is also the philosophy of ClearCode. Our goal is to develop and grow free softwares.

Personally, I am interested in processing Ruby and log data.
I want to learn these things through this product, and I want to leverage the benefits and new features of Ruby for this product.

I would be happy to contribute more to Fluentd, a great software that is used a lot in the world.

List of contributions

My contributions to this repository:

My contributions to fluentd-website:

My contributions to fluentd-docs-gitbook :

I participated in the development of some plugins:

In addition to these plugins, I also maintain the following plugin.

I want to be able to maintain other plugins etc. little by little.

@kenhys
Copy link
Contributor Author

kenhys commented Mar 10, 2023

I think we need to put a plan in place to incorporate new maintainers, happy to support that and have a voting system. Would @daipom open a PR and document intentions and list of contributions that can help to attribute him a maintainer status ?

It may be better to use a proper voting-system in the future, but anyway, daipom explains his contributions in this PR. it is good enough at the moment.

Current status:

And individual developers.

@ashie
Copy link
Member

ashie commented Mar 24, 2023

Current voting status

Companies:

Individual developers.

Result

New maintainers are proposed by an existing maintainer and are elected by a 2/3 majority organization vote.

Now we've got 6/8 votes, so we are able to have @daipom as a new maintainer.
I'll merge this and grant him write permission of fluentd related repositories.
If you have any objection, please let me know in a few days.

@ashie ashie merged commit bfa5032 into master Mar 27, 2023
@ashie ashie deleted the add-new-maintainer branch March 27, 2023 05:34
@daipom
Copy link
Contributor

daipom commented Mar 27, 2023

Thanks!
I am honored to join the maintainers.
I’ll keep on trying to make Fluentd and the community better!

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.

5 participants