CRAYSAT-1936: add ability to sort reports by multiple fields

[ethanholen-hpe]

Summary and Scope

Added ability to sort reports by multiple fields

Issues and Related PRs

List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

• Resolves CRAYSAT-1936

Testing

Tested on:

• drax
• Local development environment

Test description:

How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

• Were continuous integration tests run? If not, why? yes
• Was upgrade tested? If not, why? yes
• Was downgrade tested? If not, why? yes
• Were new tests (or test issues/Jiras) created for this change? yes

Pull Request Checklist

• Version number(s) incremented, if applicable
• Copyrights updated
• License file intact
• Target branch correct
• CHANGELOG.md updated
• Testing is appropriate and complete, if applicable
• HPC Product Announcement prepared, if applicable

[ethanholen-hpe]

Output from testing on drax: https://gist.github.com/ethanholen-hpe/a4bc6a67dc7076ebb164c8b8b14644a4

[haasken-hpe]

Looks good. My comments are minor.

I would suggest also testing behavior when given one or more unrecognized fields for --sort-by. This relates to one of my comments on how we handle that as well.

[ethanholen-hpe]

Issue happening when running sat showrev: https://gist.github.com/ethanholen-hpe/445b90d6cc9349c52a764b510b0be5ae

[ethanholen-hpe]

Testing and fix for issue: https://gist.github.com/ethanholen-hpe/c2017cb8402048ad9ff4a43dfb16e9fd

this both tests and fixes the issue that @haasken-hpe mentioned where sort-by provides multiple fields that are invalid, I added testing for this as well

[ethanholen-hpe]

/backport release/3.33

[github-actions]

Backporting into branch release/3.33 was successful. New PR: #298