uyuni icon indicating copy to clipboard operation
uyuni copied to clipboard

Systems list queries optimization

Open cbosdo opened this issue 2 years ago • 3 comments

What does this PR change?

This PR is changing the way the systems are fetched in the systems lists by storing all the needed data in an overview table that is regularly updated using a pg/sql function.

This PR has to be used with the ReactJS port of the systems list, which will come in a separate PR.

GUI diff

No difference.

  • [X] DONE

Documentation

  • No documentation needed: only internal and user invisible changes

  • [X] DONE

Test coverage

  • Unit tests were added

  • [X] DONE

Links

Fixes https://github.com/SUSE/spacewalk/issues/16050

  • [ ] DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • [ ] No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Re-run a test

If you need to re-run a test, please mark the related checkbox, it will be unchecked automatically once it has re-run:

  • [ ] Re-run test "changelog_test"
  • [ ] Re-run test "backend_unittests_pgsql"
  • [ ] Re-run test "java_pgsql_tests"
  • [ ] Re-run test "schema_migration_test_pgsql"
  • [ ] Re-run test "susemanager_unittests"
  • [ ] Re-run test "javascript_lint"
  • [ ] Re-run test "spacecmd_unittests"

cbosdo avatar Sep 02 '22 13:09 cbosdo

Note this is still work in progress, I just want to start seeing test results

cbosdo avatar Sep 02 '22 13:09 cbosdo

@cbosdo JFYI: you can set the PR as draft. Test still run on drafts

cbbayburt avatar Sep 02 '22 13:09 cbbayburt

@cbosdo JFYI: you can set the PR as draft. Test still run on drafts

Really? that should have changed without me knowing

cbosdo avatar Sep 02 '22 13:09 cbosdo

Closing to only keep https://github.com/uyuni-project/uyuni/pull/5937 which contains everything

cbosdo avatar Sep 27 '22 12:09 cbosdo