Open hanefi opened 2 years ago
Open question: Can we point arbitrary configs to HSC cluster such that we do not maintain multiple test suites.
Currently HSC schedule might have more coverage, but we are gradually making arbitrary configs more powerful
Open question: Can we point arbitrary configs to HSC cluster such that we do not maintain multiple test suites.
It may be possible after we solve some blockers/deal breakers:
citus_add_node
and similar commands may not work in HyperScale.Currently HSC schedule might have more coverage, but we are gradually making arbitrary configs more powerful
I agree on both statements here. I also would like to see arbitrary configs run on a HyperScale cluster in the future.
We need privileged user in most of our tests. Getting access for that is easy on development clusters but it slightly defeats the purposes of running tests on HyperScale if we do not run the tests using a role with limited privileges.
arbitrary config runs with a regular user, so shouldn't be a problem. Even if there is 1 schedule that runs superuser -- which I think there is -- we can easily skip that on on HSC.
citus_add_node and similar commands may not work in HyperScale.
Arbitrary config tests do not use those, as they run as non-superuser. There is an initialization step which does that, and that part should not run on HSC.
Connecting to some other worker nodes might not work on HyperScale. Though I am not so sure if we have such tests.
On Citus 11+, you can enable worker query access, even from portal.
We have 2 seperate test schedules for hyperscale:
multi_schedule_hyperscale
that contains the tests that can be run as unprivileged user andmulti_schedule_hyperscale_superuser
that contains tests that need a privileged user.Here are some ideas that can improve our test infrastructure for hyperscale schedules:
COPY
commands that access test data files do not work for running tests on remote clusters. Consider changing them into client-side psql\copy
commands as we did in https://github.com/citusdata/citus/pull/6138Related: