Skip to content
New issue

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

scylla_cloud_io_setup: move aws io preset values to external YAML file #562

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

syuu1228
Copy link
Contributor

@syuu1228 syuu1228 commented Dec 9, 2024

Since we support many instance types on AWS EC2, handling all instance types by "if - elif" conditions is difficult to read and hard to maintain.

We should move out these preset values to the external file.

Note that, in the original parameter definition, " * nr_disks" was applied to some instance types, but the new code applies it to all instance types.

This is because we know that instance types to which does not applied " * nr_disks" have only one disk.

Fixes #544

Since we support many instance types on AWS EC2, handling all instance
types by "if - elif" conditions is difficult to read and hard to maintain.

We should move out these preset values to the external file.

Note that, in the original parameter definition, " * nr_disks" was applied to
some instance types, but the new code applies it to all instance types.

This is because we know that instance types to which does not applied
" * nr_disks" have only one disk.

Fixes scylladb#544
@syuu1228 syuu1228 requested a review from yaronkaikov December 9, 2024 12:40
@syuu1228
Copy link
Contributor Author

syuu1228 commented Dec 9, 2024

Note that this is not included #561.
I will rebase it if this one merge first.

@yaronkaikov
Copy link
Collaborator

@syuu1228 did you verify this change? can you provide the details ?

@yaronkaikov
Copy link
Collaborator

@yaronkaikov
Copy link
Collaborator

Verified with https://jenkins.scylladb.com/job/releng-testing/job/next-machine-image/7/

Based on the AMI created, i have started an i4i.4xlarge instance and the io_properties are working:

scyllaadm@ip-10-0-1-171:~$ cat /etc/scylla.d/io_properties.yaml 
disks:
- mountpoint: /var/lib/scylla
  read_bandwidth: 3087631104
  read_iops: 385400
  write_bandwidth: 2289281280
  write_iops: 240628

@yaronkaikov yaronkaikov merged commit 828c2a2 into scylladb:next Dec 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor aws_io_setup class - specifically, the IO values per instance
3 participants