Eric Franz

@sleonard617 instead of using: ``` script: shell_path: "nil" ``` You want ``` script: shell_path: null ``` For YAML, [null is how you end up with nil](https://yaml.org/type/null.html): ``` irb(main):003:0> YAML.load "---\nfoo:...

Which is a great example of how difficult ERB rendered YAML can be!

On the first, https://github.com/OSC/ondemand/blob/26a265a3ebd5c63d376ccd0679602c64d9929a0f/apps/dashboard/lib/tasks/test.rake#L18 is the offending code: ```ruby output_path = WORKDIR.join("output_#{cluster.id}_#{Time.now.iso8601}.log") ``` Are dashes okay? ``` irb(main):002:0> cluster_id = "owens" => "owens" irb(main):011:0> "output_#{cluster_id}_#{Time.now.iso8601}.log" => "output_owens_2019-12-10T14:11:02-05:00.log" irb(main):008:0> "output_#{cluster_id}_#{Time.now.iso8601}.log".parameterize =>...

@viktoriaas, the issue with the rake task will be fixed in https://github.com/OSC/ondemand/pull/355, so included in a future 1.7.X release. For adding ability to specify the queue to use in the...

Using the slashes in the job name creates these nice job names - if the scheduler handles slashes for the job names:  At...

>By failed jobs I mean both kinds of jobs - I think it could be useful for every user to see whether his job has actually failed. Some users might...

@viktoriaas I opened a separate issue https://github.com/OSC/ood_core/issues/172 to track fixing the problem with sanitizing the job name of jobs submitted in OnDemand. If the suggested approach is acceptable to you...

`Script#merge(other_script)` could be added, and then this custom `merge` method could do a deep merge on the hashes like before but then set `native` to appending the original two together...

I thought they were grouped like quota warnings but could be wrong. What we do here should be done to quota warning too

Keyboard doesn't display because divs are displayed not text boxes, so this is the problem.