Skip to content

Commit

Permalink
remove comments about defaults
Browse files Browse the repository at this point in the history
  • Loading branch information
danielvegamyhre committed Jul 26, 2024
1 parent 95fc11b commit 8a1b2ee
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 16 deletions.
2 changes: 0 additions & 2 deletions api/jobset/v1alpha2/jobset_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -344,11 +344,9 @@ type Coordinator struct {

// JobIndex is the index of Job which contains the coordinator pod
// (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1).
// Defaults to 0 if unset.
JobIndex int `json:"jobIndex,omitempty"`

// PodIndex is the Job completion index of the coordinator pod.
// Defaults to 0 if unset.
PodIndex int `json:"podIndex,omitempty"`
}

Expand Down
4 changes: 2 additions & 2 deletions api/jobset/v1alpha2/openapi_generated.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 2 additions & 4 deletions config/components/crd/bases/jobset.x-k8s.io_jobsets.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,12 +71,10 @@ spec:
description: |-
JobIndex is the index of Job which contains the coordinator pod
(i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1).
Defaults to 0 if unset.
type: integer
podIndex:
description: |-
PodIndex is the Job completion index of the coordinator pod.
Defaults to 0 if unset.
description: PodIndex is the Job completion index of the coordinator
pod.
type: integer
replicatedJob:
description: |-
Expand Down
4 changes: 2 additions & 2 deletions hack/python-sdk/swagger.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@
],
"properties": {
"jobIndex": {
"description": "JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1). Defaults to 0 if unset.",
"description": "JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1).",
"type": "integer",
"format": "int32"
},
"podIndex": {
"description": "PodIndex is the Job completion index of the coordinator pod. Defaults to 0 if unset.",
"description": "PodIndex is the Job completion index of the coordinator pod.",
"type": "integer",
"format": "int32"
},
Expand Down
4 changes: 2 additions & 2 deletions sdk/python/docs/JobsetV1alpha2Coordinator.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ Coordinator defines which pod can be marked as the coordinator for the JobSet wo
## Properties
Name | Type | Description | Notes
------------ | ------------- | ------------- | -------------
**job_index** | **int** | JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1). Defaults to 0 if unset. | [optional]
**pod_index** | **int** | PodIndex is the Job completion index of the coordinator pod. Defaults to 0 if unset. | [optional]
**job_index** | **int** | JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1). | [optional]
**pod_index** | **int** | PodIndex is the Job completion index of the coordinator pod. | [optional]
**replicated_job** | **str** | ReplicatedJob is the name of the ReplicatedJob which contains the coordinator pod. | [default to '']

[[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)
Expand Down
8 changes: 4 additions & 4 deletions sdk/python/jobset/models/jobset_v1alpha2_coordinator.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ def __init__(self, job_index=None, pod_index=None, replicated_job='', local_vars
def job_index(self):
"""Gets the job_index of this JobsetV1alpha2Coordinator. # noqa: E501
JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1). Defaults to 0 if unset. # noqa: E501
JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1). # noqa: E501
:return: The job_index of this JobsetV1alpha2Coordinator. # noqa: E501
:rtype: int
Expand All @@ -76,7 +76,7 @@ def job_index(self):
def job_index(self, job_index):
"""Sets the job_index of this JobsetV1alpha2Coordinator.
JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1). Defaults to 0 if unset. # noqa: E501
JobIndex is the index of Job which contains the coordinator pod (i.e., for a ReplicatedJob with N replicas, there are Job indexes 0 to N-1). # noqa: E501
:param job_index: The job_index of this JobsetV1alpha2Coordinator. # noqa: E501
:type: int
Expand All @@ -88,7 +88,7 @@ def job_index(self, job_index):
def pod_index(self):
"""Gets the pod_index of this JobsetV1alpha2Coordinator. # noqa: E501
PodIndex is the Job completion index of the coordinator pod. Defaults to 0 if unset. # noqa: E501
PodIndex is the Job completion index of the coordinator pod. # noqa: E501
:return: The pod_index of this JobsetV1alpha2Coordinator. # noqa: E501
:rtype: int
Expand All @@ -99,7 +99,7 @@ def pod_index(self):
def pod_index(self, pod_index):
"""Sets the pod_index of this JobsetV1alpha2Coordinator.
PodIndex is the Job completion index of the coordinator pod. Defaults to 0 if unset. # noqa: E501
PodIndex is the Job completion index of the coordinator pod. # noqa: E501
:param pod_index: The pod_index of this JobsetV1alpha2Coordinator. # noqa: E501
:type: int
Expand Down

0 comments on commit 8a1b2ee

Please sign in to comment.