Skip to content

Commit

Permalink
more updates
Browse files Browse the repository at this point in the history
  • Loading branch information
danielvegamyhre committed Apr 5, 2024
1 parent 3922b91 commit 13f4e98
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions keps/498-GroupScheduling/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,8 @@ Since node pool provisioning takes a variable amount of time, users are running

For statically provisioned clusters, we have recommended users use Kueue to handle group scheduling of JobSets once sufficient resources are available.

However, for dynmaically provisioned clusters, we have no support for group scheduling in JobSet yet.
However, JobSets running independently (without Kueue) on dynmaically provisioned clusters, we need to support group
scheduling semantics in order to avoid these timeout issues.

### Goals

Expand Down Expand Up @@ -315,7 +316,7 @@ spec:
The testing plan will focus on:
- Unit and integration tests to validate the webhook injection works as intended
- E2E tests, since we need pods to be created to injectthe initContainer and ConfigMap volume mount.
- E2E tests, since we need pods to be created to inject the initContainer and ConfigMap volume mount.
- Scale tests, performance benchmarking
##### Prerequisite testing updates
Expand Down Expand Up @@ -347,7 +348,11 @@ extending the production code to implement this enhancement.
- `controllers`: `04/05/2024` - `25.4%`

#### Integration tests
- Group schedu
- JobSet webhook integration tests validating that for JobSets with group scheduling
config defined, the ConfigMap is created, and the correct child Jobs have the expected
initContainers and ConfigMap volume mounts injected.
- JobSet controller integration tests validating that for JobSets with group scheduling
config defined, the ConfigMap used for broadcasting the startup signal is created.

### Graduation Criteria

Expand Down

0 comments on commit 13f4e98

Please sign in to comment.