Extending pbs_diag wouldn't be meaningful to collect node information for ptl. This should be part of PTL, since
mom nodes contains config information, logs and core files if generated.
Why the option name is --post-analysis-data, as there is no analysis involve by PTL on the data, also this sounds more like a flag , rather than an option which is requires a directory to store the data. I feel it should be --post-failure-data-dir.
Another question, does ptl collect this data even in case where test case failed due to bug in PTL and does it include any data from PTL side for such failure.