Skip to content

Gardener Community Roles#715

Merged
timuthy merged 4 commits into
gardener:masterfrom
timuthy:community-roles
Sep 19, 2025
Merged

Gardener Community Roles#715
timuthy merged 4 commits into
gardener:masterfrom
timuthy:community-roles

Conversation

@timuthy

@timuthy timuthy commented Sep 3, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

What this PR does / why we need it:
This PR introduces the definition of Gardener community roles, outlining the various levels and member statuses relevant to code contribution and ownership.

Gardener sub-projects may adopt this framework with technical support from Gardener’s Prow instance, aiming to establish a unified process for all contributors interested in joining the Gardener community.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
/cc @gardener/gardener-maintainers @gardener/gardener-extensions-maintainers @timebertt @dimityrmirchev @vpnachev

PR remains in draft status until the gardener/org repository is prepared.

Release note:

NONE

@timuthy timuthy requested a review from a team as a code owner September 3, 2025 13:36
@ghost ghost added needs/review Needs review size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 3, 2025
dimityrmirchev
dimityrmirchev reviewed Sep 3, 2025
View reviewed changes

@dimityrmirchev dimityrmirchev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for driving this topic! I have left two suggestions

Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
vpnachev
vpnachev reviewed Sep 3, 2025
View reviewed changes

@vpnachev vpnachev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

I have two suggestions in-line

Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/assets/_issue_template.md Outdated
timebertt
timebertt reviewed Sep 4, 2025
View reviewed changes

@timebertt timebertt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Very nice, thank you for setting this up!
This definitely fills a gap in our current community setup – appreciated.

No concerns from my side, looks good in general :)

Comment thread website/documentation/contribute/code/roles/assets/_issue_template.md Outdated
Comment thread website/documentation/contribute/code/roles/assets/_issue_template.md Outdated
ScheererJ
ScheererJ reviewed Sep 4, 2025
View reviewed changes

@ScheererJ ScheererJ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for taking the time to formalise the process. It looks good overall.

Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
marc1404
marc1404 reviewed Sep 4, 2025
View reviewed changes

@marc1404 marc1404 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for writing up the result of our discussion!
I appreciate that we're staying aligned with Kubernetes' approach to community management while adjusting it to suit our size and needs. Having this written down and being able to refer to it is an improvement.

I've left some optional formatting comments.

Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
plkokanov
plkokanov requested changes Sep 4, 2025
View reviewed changes

@plkokanov plkokanov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks a lot for preparing this document!

Couple of suggestions on my side.

Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
Comment thread website/documentation/contribute/code/roles/assets/_issue_template.md Outdated
Comment thread website/documentation/contribute/code/roles/assets/_issue_template.md Outdated
Comment thread website/documentation/contribute/code/roles/assets/_issue_template.md Outdated
@ghost ghost added the needs/changes Needs (more) changes label Sep 4, 2025
tobschli
tobschli reviewed Sep 8, 2025
View reviewed changes

@tobschli tobschli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you very much for this!
I think this is really valuable in terms of community building 😊

Comment thread website/documentation/contribute/code/roles/_index.md Outdated
timuthy and others added 2 commits September 8, 2025 13:41
@timuthy @rfranzke @marc1404
Define community roles
cec7949 
Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>
Co-authored-by: Marc Vornetran <marc.vornetran@sap.com>
@timuthy
Address review feedback
827b363
@timuthy timuthy changed the title Gardener Community Roles [WIP] Gardener Community Roles Sep 8, 2025
@timuthy
Delete issue template
c42a5c8 
Template will be placed in the `gardener/org` repo.
@ghost ghost added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 8, 2025
@timuthy

timuthy commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

Thanks for your valuable feedback folks! I've addressed all comments - PTAL!

As agreed in this comment, we're currently preparing a new gardener/org repo and will place the issue template there.

@timuthy

timuthy commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

/hold
Until gardener/org is prepared.

@ghost ghost added the reviewed/do-not-merge Has no approval for merging as it may break things, be of poor quality or have (ext.) dependencies label Sep 8, 2025
tobschli
tobschli reviewed Sep 8, 2025
View reviewed changes

@tobschli tobschli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice 😊
/lgtm

@ghost ghost added reviewed/lgtm Has approval for merging needs/changes Needs (more) changes and removed needs/changes Needs (more) changes needs/review Needs review reviewed/lgtm Has approval for merging labels Sep 8, 2025
timebertt
timebertt reviewed Sep 8, 2025
View reviewed changes
Comment thread website/documentation/contribute/code/roles/_index.md Outdated
@timuthy timuthy mentioned this pull request Sep 9, 2025
Merged
@timuthy timuthy changed the title [WIP] Gardener Community Roles Gardener Community Roles Sep 10, 2025
@timuthy timuthy mentioned this pull request Sep 10, 2025
Open
3 tasks
@timuthy

timuthy commented Sep 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Member Author

The https://github.com/gardener/org has been prepared (incl. issue template)
/unhold

@ghost ghost removed the reviewed/do-not-merge Has no approval for merging as it may break things, be of poor quality or have (ext.) dependencies label Sep 10, 2025
marc1404
marc1404 approved these changes Sep 10, 2025
View reviewed changes

@marc1404 marc1404 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for working through all the suggestions and setting up the new gardener/org repository! 🎉

n-boshnakov
n-boshnakov approved these changes Sep 10, 2025
View reviewed changes

@n-boshnakov n-boshnakov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@ghost ghost added reviewed/lgtm Has approval for merging and removed needs/changes Needs (more) changes labels Sep 10, 2025
vpnachev
vpnachev reviewed Sep 10, 2025
View reviewed changes

@vpnachev vpnachev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

dimityrmirchev
dimityrmirchev approved these changes Sep 11, 2025
View reviewed changes

@dimityrmirchev dimityrmirchev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@timuthy timuthy added the area/ipcei IPCEI (Important Project of Common European Interest) label Sep 11, 2025
timebertt
timebertt approved these changes Sep 11, 2025
View reviewed changes

@timebertt timebertt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for your efforts!

LucaBernstein
LucaBernstein reviewed Sep 17, 2025
View reviewed changes

@LucaBernstein LucaBernstein left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you very much for formalizing the roles. Looking good from my side as well. 😃 👍

@timuthy timuthy requested a review from plkokanov September 18, 2025 14:37
plkokanov
plkokanov approved these changes Sep 19, 2025
View reviewed changes

@plkokanov plkokanov left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks again for the effort and sorry for the delay!

/lgtm

@timuthy timuthy merged commit c6edb41 into gardener:master Sep 19, 2025
5 checks passed
@ghost ghost added the status/closed Issue is closed (either delivered or triaged) label Sep 19, 2025
@timebertt timebertt mentioned this pull request Mar 13, 2026
Merged
@timebertt timebertt mentioned this pull request Apr 13, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vpnachev vpnachev vpnachev left review comments
@LucaBernstein LucaBernstein LucaBernstein left review comments
@ScheererJ ScheererJ ScheererJ left review comments
@oliver-goetz oliver-goetz oliver-goetz left review comments
@tobschli tobschli tobschli left review comments
@marc1404 marc1404 marc1404 approved these changes
@n-boshnakov n-boshnakov n-boshnakov approved these changes
@dimityrmirchev dimityrmirchev dimityrmirchev approved these changes
@plkokanov plkokanov plkokanov approved these changes
@timebertt timebertt timebertt approved these changes

Assignees

No one assigned

Labels

area/ipcei IPCEI (Important Project of Common European Interest) reviewed/lgtm Has approval for merging size/S Denotes a PR that changes 10-29 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.