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.

Sign up for GitHub

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

Introduce managedBy field and Remove managed-by label #487

Merged
merged 22 commits into from
Mar 29, 2024
Merged

Introduce managedBy field and Remove managed-by label #487

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
cd66b3e
Remove in const declaration and add field to JobSetSpec.
Mar 27, 2024
8c45f05
Remove references to LabelManagedBy in jobset_webhook.go.
Mar 27, 2024
7820c47
Add struct tag for ManagedBy field of JobSetSpec.
Mar 27, 2024
83ee86f
Add missing ` character in ManagedBy struct tag.
Mar 27, 2024
0713346
Update jobset_webhook_test.go to remove references to LabelManagedBy.
Mar 27, 2024
e866ed4
Update Reconcile to use ManagedBy field of JobSetSpec.
Mar 27, 2024
34c2b26
Update jobset_controller_test.go to use ManagedBy field of JobSetSpec.
Mar 27, 2024
ef199e7
Update jobset_webhook_test.go to use ManagedBy field of JobSetSpec.
Mar 27, 2024
55fb065
Fix bug where uninitialized value for js.Spec.ManagedBy was treated a…
Mar 28, 2024
6d3f0a0
Add an integration test that only tests job creation.
Mar 28, 2024
80643db
Ran non-test related make commands.
Mar 28, 2024
c3f9aef
Correct grammatical error.
Mar 28, 2024
0db3446
Regenerated files.
Mar 28, 2024
a10faa1
Remove TODO for jobset controller.
Mar 28, 2024
a82e440
Update comment for JobSetManager to no longer reference LabelManagedBy.
Mar 28, 2024
66b55c4
Change implementation of JobSet controller manager to better align wi…
Mar 29, 2024
f8a2bf5
Add comment providing context for checking JobSet manager.
Mar 29, 2024
dae1c28
Add validation for managedBy field.
Mar 29, 2024
7559f34
Create tests for jobset controller name validation.
Mar 29, 2024
98238ac
Create variable for overly long controller name error.
Mar 29, 2024
76d5546
Add tests for valid values of managedBy.
Mar 29, 2024
80ac00c
Correct test for unset jobset controller name.
Mar 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Regenerated files.
  • Loading branch information
Justin Edwins committed Mar 28, 2024
commit 0db3446aea29eec8e48e82a911c10cb25582c9bb
2 changes: 1 addition & 1 deletion api/jobset/v1alpha2/openapi_generated.go
View file
Open in desktop

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

2 changes: 1 addition & 1 deletion config/components/crd/bases/jobset.x-k8s.io_jobsets.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ spec:
rule: self == oldSelf
managedBy:
description: ManagedBy is used to indicate the controller or entity
that manages an JobSet
that manages a JobSet
type: string
network:
description: Network defines the networking options for the jobset.
Expand Down
2 changes: 1 addition & 1 deletion hack/python-sdk/swagger.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@
"$ref": "#/definitions/jobset.v1alpha2.FailurePolicy"
},
"managedBy": {
"description": "ManagedBy is used to indicate the controller or entity that manages an JobSet",
"description": "ManagedBy is used to indicate the controller or entity that manages a JobSet",
"type": "string"
},
"network": {
Expand Down
2 changes: 1 addition & 1 deletion sdk/python/docs/JobsetV1alpha2JobSetSpec.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ JobSetSpec defines the desired state of JobSet
Name | Type | Description | Notes
------------ | ------------- | ------------- | -------------
**failure_policy** | [**JobsetV1alpha2FailurePolicy**](JobsetV1alpha2FailurePolicy.md) | | [optional]
**managed_by** | **str** | ManagedBy is used to indicate the controller or entity that manages an JobSet | [optional]
**managed_by** | **str** | ManagedBy is used to indicate the controller or entity that manages a JobSet | [optional]
**network** | [**JobsetV1alpha2Network**](JobsetV1alpha2Network.md) | | [optional]
**replicated_jobs** | [**list[JobsetV1alpha2ReplicatedJob]**](JobsetV1alpha2ReplicatedJob.md) | ReplicatedJobs is the group of jobs that will form the set. | [optional]
**startup_policy** | [**JobsetV1alpha2StartupPolicy**](JobsetV1alpha2StartupPolicy.md) | | [optional]
Expand Down
4 changes: 2 additions & 2 deletions sdk/python/jobset/models/jobset_v1alpha2_job_set_spec.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ def failure_policy(self, failure_policy):
def managed_by(self):
"""Gets the managed_by of this JobsetV1alpha2JobSetSpec. # noqa: E501

ManagedBy is used to indicate the controller or entity that manages an JobSet # noqa: E501
ManagedBy is used to indicate the controller or entity that manages a JobSet # noqa: E501

:return: The managed_by of this JobsetV1alpha2JobSetSpec. # noqa: E501
:rtype: str
Expand All @@ -118,7 +118,7 @@ def managed_by(self):
def managed_by(self, managed_by):
"""Sets the managed_by of this JobsetV1alpha2JobSetSpec.

ManagedBy is used to indicate the controller or entity that manages an JobSet # noqa: E501
ManagedBy is used to indicate the controller or entity that manages a JobSet # noqa: E501

:param managed_by: The managed_by of this JobsetV1alpha2JobSetSpec. # noqa: E501
:type: str
Expand Down