flux-framework / flux-k8s

Project to manage Flux tasks needed to standardize kubernetes HPC scheduling interfaces
Apache License 2.0
20 stars 10 forks source link

[WIP] fluence: refactor to use new PodGroup #63

Closed vsoch closed 5 months ago

vsoch commented 5 months ago

Problem: fluence should only be storing state of jobid and presence of a group name in a map to indicate node assignment. Soluion: update the code here. Note that this is not working yet, and I am pushing / opening the PR to not use the work (and will update accordingly, and using this PR to test).

Going for a run, will continue debugging when I get back. There is a bug somewhere where we aren't asking, and the map is empty - likely in the PreFilter function I missed something.

vsoch commented 5 months ago

yes!!

vsoch commented 5 months ago

The bug was needing a pointer, which is exactly what I thought it might be!

This is just going into the other PR branch so no review needed.