[feat] Extend cray_cdt_version() utility function to recognise CPE versions too

[rsarm]

For this I guess with should change the name of the function since it has cdt. Maybe cray_pe_version and deprecate cray_cdt_version ?

[codecov-commenter]

Codecov Report

Base: 86.26% // Head: 86.27% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (1901ac7) compared to base (e90be6d).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2644   +/-   ##
=======================================
  Coverage   86.26%   86.27%           
=======================================
  Files          60       60           
  Lines       10990    10992    +2     
=======================================
+ Hits         9481     9483    +2     
  Misses       1509     1509           

Impacted Files	Coverage Δ
reframe/utility/osext.py	84.67% <100.00%> (+0.12%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ekouts]

Looks fine, besides a small typo. I am also not sure if we should rename the function. If we do we will also have to update all the reframe tests that use it.

[vkarak]

The PR lgtm, but generally, I think that the use of this function, in general should be deprecated in favor of the new syntax of valid_prog_environs that can match environments based on features and extras. That function and, probably similar ones in the future should be called from a feature autodetection mechanism from the framework. I think also that CSCS' config and tests should move towards that direction abandoning the use of explicit system names and environments in the valid_systems and valid_prog_environs.