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

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
Add an integration test that only tests job creation.
  • Loading branch information
Justin Edwins committed Mar 28, 2024
commit 6d3f0a0bf7f3446dad6457ffcf087a2c6d20b2d9
3 changes: 3 additions & 0 deletions test/integration/controller/jobset_controller_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -198,6 +198,9 @@ var _ = ginkgo.Describe("JobSet controller", func() {
}
}
},
ginkgo.Entry("jobset should successfully create jobs", &testCase{
makeJobSet: testJobSet,
}),
ginkgo.Entry("jobset should succeed after all jobs succeed", &testCase{
makeJobSet: testJobSet,
updates: []*update{
Expand Down