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

Use consistent case for Job #12603

Merged
merged 1 commit into from
Feb 18, 2019
Merged

Conversation

sftim
Copy link
Contributor

@sftim sftim commented Feb 12, 2019

Issue #10086 highlights some confusing capitalization on https://kubernetes.io/docs/concepts/workloads/controllers/jobs-run-to-completion/

These changes are aimed at making the Job controller documentation more readable and less likely to confuse a reader. In deciding what changes to make, I've assumed that the reader is likely to have encountered the pod and container concepts already. Where it helps with readability I've opted to use “pod” as a normal English word, saving “Pod” for when this specifically describes an object of kind Pod.

There's a bit more detail on my thinking here in the commit message for 8006296.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. language/en Issues or PRs related to English language labels Feb 12, 2019
@netlify
Copy link

netlify bot commented Feb 12, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit d0e9121

https://deploy-preview-12603--kubernetes-io-master-staging.netlify.com

1. Parallel Jobs with a *fixed completion count*:
- specify a non-zero positive value for `.spec.completions`.
- the job is complete when there is one successful pod for each value in the range 1 to `.spec.completions`.
- **not implemented yet:** Each pod passed a different index in the range 1 to `.spec.completions`.
- the overall task is complete when there is one successful Pod for each value in the range 1 to `.spec.completions`.
Copy link
Contributor

Choose a reason for hiding this comment

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

job

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm keen for the reader to get the idea that there's an overall piece of work represented by the Job. Usings might be coming from other systems where “job” could mean a batch from a larger pool, or a even a single element inside a batch.

I'll see if I can make some wording that to reinforce that, without losing the link to the Job concept in a Kubernetes sense.

@sftim sftim changed the title Job consistent case Use consistent case for Job Feb 12, 2019
@sftim
Copy link
Contributor Author

sftim commented Feb 13, 2019

How about this revised approach?

Copy link
Contributor

@soltysh soltysh left a comment

Choose a reason for hiding this comment

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

Please squash your changes into a single commit.
From a tech pov this lgtm.

of successful completions is reached, the job itself is complete. Deleting a Job will cleanup the
pods it created.
A Job creates one or more pods and ensures that a specified number of them successfully terminate.
As pods successfully complete, the Job tracks the successful completions. When a specified number
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you want to fix Pod casing throughout this doc too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

To me this reads better using pod in lowercase where using the term more loosely. It comes down to a matter of style and taste, I think.

- Use “Job” to mean the Kubernetes object
- Use “job” where capitalization hinders readability
- Use “task”, or other synonymous concepts, to avoid confusion
- Use “Pod” to mean the Kubernetes object
- Use “pod” where capitalization hinders readability
- Use “container” in lowercase everywhere

Also:
- Clarify work queue example for Job controller
- Fix advice on running Job example
- Tweak emphasis
@sftim sftim force-pushed the job_consistent_case branch from 7bab99e to d0e9121 Compare February 14, 2019 07:48
@sftim
Copy link
Contributor Author

sftim commented Feb 14, 2019

Squashed & force-pushed

@zparnold
Copy link
Member

/approve
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 18, 2019
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: zparnold

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 18, 2019
@k8s-ci-robot k8s-ci-robot merged commit 5494391 into kubernetes:master Feb 18, 2019
@sftim sftim deleted the job_consistent_case branch February 18, 2019 12:17
kwiesmueller pushed a commit to kwiesmueller/website that referenced this pull request Feb 28, 2019
- Use “Job” to mean the Kubernetes object
- Use “job” where capitalization hinders readability
- Use “task”, or other synonymous concepts, to avoid confusion
- Use “Pod” to mean the Kubernetes object
- Use “pod” where capitalization hinders readability
- Use “container” in lowercase everywhere

Also:
- Clarify work queue example for Job controller
- Fix advice on running Job example
- Tweak emphasis
@elizabethharris
Copy link

Thanks, @sftim !

krmayankk pushed a commit to krmayankk/kubernetes.github.io that referenced this pull request Mar 11, 2019
- Use “Job” to mean the Kubernetes object
- Use “job” where capitalization hinders readability
- Use “task”, or other synonymous concepts, to avoid confusion
- Use “Pod” to mean the Kubernetes object
- Use “pod” where capitalization hinders readability
- Use “container” in lowercase everywhere

Also:
- Clarify work queue example for Job controller
- Fix advice on running Job example
- Tweak emphasis
yagonobre pushed a commit to yagonobre/website that referenced this pull request Mar 14, 2019
- Use “Job” to mean the Kubernetes object
- Use “job” where capitalization hinders readability
- Use “task”, or other synonymous concepts, to avoid confusion
- Use “Pod” to mean the Kubernetes object
- Use “pod” where capitalization hinders readability
- Use “container” in lowercase everywhere

Also:
- Clarify work queue example for Job controller
- Fix advice on running Job example
- Tweak emphasis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants