populationgenomics / hail

Scalable genomic data analysis.
https://hail.is
MIT License
1 stars 1 forks source link

Add pool label to "resource requests ... are unsatisfiable" message #342

Closed jmarshall closed 2 months ago

jmarshall commented 2 months ago

Pool labels are a CPG-local addition (added in PR #197) that are considered in select_inst_coll() so should be displayed in this error message — unfortunately we neglected to report them in the error message at the time.

I've verified that this is the only new select_inst_coll() parameter not represented in the error message (worker_type is also not directly printed, but it is derived within _create_jobs() from other fields rather than being a config field), and that there are no other similarly bereft error messages elsewhere.

Context: https://centrepopgen.slack.com/archives/C030X7WGFCL/p1718060676098609?thread_ts=1717563218.399159&cid=C030X7WGFCL