Skip to content

Conversation

@RoshaniN
Copy link
Collaborator

@RoshaniN RoshaniN commented Mar 11, 2025

  • Defines the PathwaysJob CRD.
  • Controller code to create PathwaysJob object based on JobSet.
  • Supports two deployment modes - colocate and default. Also supports headless mode for the default deployment. (constructs the pods, containers, success policies, etc for the same).
  • Validates user-provided input for type and topology.
  • Computes instance type, number of workers for parallelisms/completions, and other attributes needed for Pathways and JobSet.
  • RBACs and owner logic updates so that the controller has access to create and monitor JobSet. Deleting PathwaysJob cascade deletes underlying JobSet.
  • Updated Makefile to also install JobSet, providing users a one-step installation.
  • Sample configs to use the PathwaysJob CRD.

Note: all the initial features may be squashed into one commit.

RoshaniN added 28 commits March 11, 2025 01:30
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link
Collaborator

@shauryagup shauryagup left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Collaborator

@SujeethJinesh SujeethJinesh left a comment

Choose a reason for hiding this comment

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

Thanks Roshani! Looks great! Had a couple of questions.

Copy link
Collaborator

@SujeethJinesh SujeethJinesh left a comment

Choose a reason for hiding this comment

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

Thanks Roshani! Excited to use this :)

@RoshaniN RoshaniN merged commit 2906810 into main Mar 18, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants