We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
My eyes hurt from seeing so many `elif'. See
scylla-machine-image/common/scylla_cloud_io_setup
Line 41 in ad01b2a
The text was updated successfully, but these errors were encountered:
Part of this (I should open a separate issue for it!) is that we need to validate the numbers are still real. Ref https://github.com/scylladb/scylla-enterprise/issues/4713#issuecomment-2367619463 We do have a job that is testing a wide range of instances, it should probably compare the results and the hard-coded values (CC @roydahan )
Sorry, something went wrong.
I created a separate issue for the above comment in qa-tasks.
My eyes hurt from seeing so many `elif'. See scylla-machine-image/common/scylla_cloud_io_setup Line 41 in ad01b2a def generate(self): We should store this in a dictionary, JSON or whatnot. Single location preferably.
We should store this in a dictionary, JSON or whatnot. Single location preferably.
Hurt is understatement. @yaronkaikov a dictionary is the simplest and quickest solution.
syuu1228
No branches or pull requests
My eyes hurt from seeing so many `elif'. See
scylla-machine-image/common/scylla_cloud_io_setup
Line 41 in ad01b2a
We should store this in a dictionary, JSON or whatnot. Single location preferably.
The text was updated successfully, but these errors were encountered: