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.

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

fluxion: add service to provide scheduler #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vsoch
Copy link
Member

@vsoch vsoch commented Jan 3, 2025

This build is a bit tricky, because we want to share packages for jgf/jobspecs along with protos but do not want to require flux-sched libs locally to build charts, etc. The trick is to store them alongside the fluxion build with underscore directories (so they are ignored on the local machine) and then move those directories to be found and used in the Dockerfile. It is imperfect, but an OK solution for now.

This PR is a wip.

This build is a bit tricky, because we want to share
packages for jgf/jobspecs along with protos but do not
want to require flux-sched libs locally to build charts,
etc. The trick is to store them alongside the fluxion build
with underscore directories (so they are ignored on the local
machine) and then move those directories to be found and used
in the Dockerfile. It is imperfect, but an OK solution for now.

Signed-off-by: vsoch <[email protected]>
@vsoch
Copy link
Member Author

vsoch commented Jan 3, 2025

TODO: I don't like this design - I want to instead try using https://github.com/converged-computing/fluxion. I'll make a branch off of this one to test in case it doesn't work. It will simplify things hugely.

@vsoch vsoch added the wontfix This will not be worked on label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant