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
Wrong when not configure a partition #15
Comments
Does your cluster not have any? Not even a default or normal one? |
I'm not clear how you are generating a file that doesn't have debug but your config does? Can you give me the exact commands to reproduce what you are doing? We need to figure out what is going wrong so the partitions show up. The partition information is how every node is defined in terms of its resources (memory, etc.) so if a cluster doesn't have partitions technically all the nodes would be under a faux partition called "normal." Also feel free to find the bug on your own and submit a PR to fix. |
Using the slurm config tool. https://slurm.schedmd.com/configurator.html |
ah gotcha. So maybe just try adding partitions, and test with the tool when you have a complete config filled out? |
I think it better can work for that case. Because the slurm work well with that slurm.conf. I think some body no need to configure a partition. |
okay, then would you like to take a shot at debugging the issue? Look at script.js - I can tell you what is happening. It assumes partitions are there and defaults to using index 0. So you would want to figure out how else you'd want to accomplish this. |
ok. I am not web programer, I am not familar with js. I know which the error happens, it at line 142 in script.js but I don't know how to fix it. if (self.partition_name == null) { |
okay so we can talk through this then - given that there are no partitions, what should be the default behavior to look up machines? Currently the machine attributes are known through the partitions, so I suspect we'd need to get that somewhere else? |
I think slurm2json.py generate wrong machines.json |
I think so too! |
okay please see #16 |
ok,thank you! |
You would need to:
You should not hit this error you are showing me because there should be a partition there. I've added the machines.json that I generated to the PR so you can check against the one you make. |
We need to parse this line: https://github.com/researchapps/job-maker/pull/16/files#diff-f795e9bdafd6d171ce2ad965cc735764621d0760d8605889156cadd885b87e0dR147 (last one in the files for the PR that says PartitionName) |
if I not config a partition, the progarm run wrong!
The text was updated successfully, but these errors were encountered: