get_results calls mdiag

[cadejager]

When the binary mdiag exists on the system (according to which) pavilion tries to call it to tell the user some information about the state of the system. Pavilion continues to work but prints an error if there is placeholder on the system for mdiag to tell a user to use slurm.

Currently the offending line is here

Pavilion/PAV/scripts/get_results:157

I see two options to fix this in the future:

1. Change it to be dependent on some slurm call. This would be vulnerable to a scheduler change in the same way in the future.
2. Remove mdiag from the system. This would be the same result as removing it and users would no longer get their error.

I intend to do option 2 the change is easy but I need to find the time to test it.