ROX-24358: compliance last scan time sortable

[dashrews78]

Description

Added sort capability for the Last Scan Time on the /v2/compliance/scan/stats/profiles/{profile-name}/clusters endpoint. This required a slight modification to the query framework. See testing section from examples.

Checklist

• Investigated and inspected CI test results
• Unit test and regression tests added
  - [ ] Evaluated and added CHANGELOG entry if required
  - [ ] Determined and documented upgrade steps
  - [ ] Documented user facing changes (create PR based on [openshift/openshift-docs](https://github.com/openshift/openshift-docs) and merge into rhacs-docs)

If any of these don't apply, please comment below.

Testing Performed

Here I tell how I validated my change

curl --insecure "https://central-stackrox.apps.ds-05-29-profuse-help-implem.ocp.infra.rox.systems:443/v2/compliance/scan/stats/profiles/ocp4-cis-node/clusters" -u admin:XXX | json_pp > ~/Downloads/no-sort.json

no-sort.json

curl --insecure "https://central-stackrox.apps.ds-05-29-profuse-help-implem.ocp.infra.rox.systems:443/v2/compliance/scan/stats/profiles/ocp4-cis-node/clusters?query.pagination.sortOption.field=Compliance%20Scan%20Last%20Executed%20Time%20Max&query.pagination.sortOption.reversed=false" -u admin:XXX | json_pp > ~/Downloads/sort.json

sort.json

curl --insecure "https://central-stackrox.apps.ds-05-29-profuse-help-implem.ocp.infra.rox.systems:443/v2/compliance/scan/stats/profiles/ocp4-cis-node/clusters?query.pagination.sortOption.field=Compliance%20Scan%20Last%20Executed%20Time%20Max&query.pagination.sortOption.reversed=true" -u admin:XXX | json_pp > ~/Downloads/reverse-sort.json

reverse-sort.json

Reminder for reviewers

In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

[openshift-ci]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[rhacs-bot]

Images are ready for the commit at 5ad4290.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.4.x-814-g5ad429069e.

[codecov]

Codecov Report

Attention: Patch coverage is 52.94118% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 47.98%. Comparing base (6b22b10) to head (5ad4290).
Report is 6 commits behind head on master.

Files	Patch %	Lines
...rt/storagetov2/compliance_results_stats_service.go	0.00%	3 Missing ⚠️
pkg/search/postgres/pagination.go	62.50%	3 Missing ⚠️
pkg/search/postgres/query/common.go	0.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #11193   +/-   ##
=======================================
  Coverage   47.97%   47.98%           
=======================================
  Files        2330     2330           
  Lines      166730   166760   +30     
=======================================
+ Hits        79990    80019   +29     
- Misses      80387    80390    +3     
+ Partials     6353     6351    -2     

Flag	Coverage Δ
go-unit-tests	47.98% <52.94%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[SimonBaeumer]

Nice! 🚢 🐳

[dashrews78]

/retest