Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Formation of WG Naming #4884
Formation of WG Naming #4884
Changes from all commits
b0010e6
53e7d0a
3b35f49
3ee9e82
0bfd6f7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would this also involve renaming the master branch for all repos and would renaming the branch be a criteria for dissolution? It's heavily tied to how quickly and widely GitHub responds so I just want us to be cognizant of that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At the moment, CNCF is planning to wait for GitHub to roll out changes regarding master branch naming. We're hoping that waiting on GitHub to make changes centrally results in less tooling issues down the line.
Ultimately the choice lies with the community, but I would strongly recommend Kubernetes take the same tactic. Personally I would not include this discussion in the WG's domain – the WG should focus on Kubernetes and related projects only, not the wording in the tooling, code language or libraries Kubernetes uses.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we're waiting for GitHub's lead here, and they should have a response soon.
(Discussed during the 6/25 GitHub Administration meeting.)
I'm taking point on this from the WG Naming side and will open an issue for this later in the week.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Added as a post-formation discussion point.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will help find a good home for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Added as a post-formation discussion point.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@justaugustus As this was added to the post-formation discussion point, should this line still exist in the goals?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cblecker -- intentionally duplicated, so we'd know where to edit the language later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not very clear on what this point means.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neither am I. @justaugustus @celestehorgan @zacharysarah Could you elaborate on this point?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@deads2k @cblecker The goal is for inclusive naming in software architecture, not just in docs. For example, replacing "master/slave" terminology in docs but not in functions, classes, variables, etc. is an empty gesture.
What's a more accurate way to refer to code in this specific context?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may need the full @kubernetes/code-of-conduct-committee to weigh in on this. One could argue that code is language as much as an issue or Slack conversation. That would cover future cases, but remediation of existing code would have to be handled by individuals in the OWNERS files IMO, much like if someone trashes a GitHub issue, we (the CoC Committee) ask the Kubernetes GitHub admin team to remove the offending comments. I still believe the existing CoC covers this, as we collectively define the edges of what is acceptable or not. From the CoC specifically:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I don't think the CoC needs to be modified here. Perhaps this goal can be changed to "Review with CoCC guidelines and polices around use of language/terminology in code" or something similar?
The idea being:
But bringing it back to how to we approve this WG, I'd suggest changing this to "review" as I've suggested above will allow the freedom to explore this question throughly with the CoCC to answer if the CoC is or is not the right tool to address this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now, I suggest we remove any mention of the COCC from this WG, and put it as an agenda item for the WG's work.
That said, per later discussion in this thread: @jdumars is joining us in this initiative, so involvement from that quarter will be there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Added as a post-formation discussion point.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems like sustainability after this working group disbands would require that the process create above and the future owners of that process, be able to handle identification, implementation, and timeline. Could we use that process and those owners to drive the first set of changes as well? If not, can we have confidence that this effort will be self-sustaining over time?
See the comment from Celeste in this PR about avoiding the need to reform this group for the next cultural moment: https://github.com/kubernetes/community/pull/4884/files#r445043710
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does "on which to replace" mean for such a timeline in an OpenSource project? Who drives it? Who contributes the resources to implement changes? I could imagine that the wg will output a plan with parties committed to implement it. I guess that's what this means, and not an order to be finished by date X.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working with the stakeholders and code owners will be key. Yes, this effort will need to be resourced. We've had lots of interest and volunteers to help with this, but ensuring we do this in a way that doesn't introduce bugs/regressions will need effort from code owners.
What that timeline is and how it will be resourced is an open question for the working group (and one they can answer after formation IMO)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I offer to program-manage this effort to the extent of bringing the stakeholders and parties together to discuss a strategy for getting the work moving, tracking the work (including visualisations and self-reporting documents), and updating on progress.
One of my first steps would be to bring folks together to discuss implementation strategy — how and where does the work start from a technical perspective.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Added as a post-formation discussion point.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few questions here:
Would this WG focus only on k/k? My understanding was that it would look at naming practices across all our repos (even k-sigs, k-csi, k-client), docs, etc.
Maybe it would be better to rename "component names" to something slightly more...generic? Just to ensure that we don't tie the dissolution criteria to only component names... 🤔
Would the WG disband after a timeline has been agreed upon or would it disband after all the changes have been made as per the timeline?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW, I personally would find it helpful if the group stuck around until changes have been made. The model I have is WG-Apply, which existed until server side apply was implemented.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't conferred with @justaugustus on this, so he may disagree, but here's my thoughts:
Per an earlier comment, we're changing the wording in this WG to address the Kubernetes project – so presumably.. everything? 😅
+1! "feature names?" "method names"? I'm unsure. I'll brainstorm on this for a bit.
Personally I'd like to stick around until work is executed for accountability's sake. This isn't high priority work in relation to new releases, for example, but I think it's high importance and I think having the WG around to see the work through would be good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Added as a post-formation discussion point.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed that the goal is to serve across the project, not just kubernetes/kubernetes.
Yep. We can workshop the language post-formation.
Agreed. I also intend to action on changes ahead of WG dissolution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
non-blocking nit: whitespace
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cblecker -- intentional. see the next line.