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

Add missing i4g.4xlarge instnace type #476

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

yaronkaikov
Copy link
Collaborator

Adding pre-configure io_setup values
Also since i4g doesn't support enhanced networking adding it to use ENA

Adding pre-configure io_setup values
Also since `i4g` doesn't support enhanced networking adding it to use
ENA
@Annamikhlin
Copy link
Collaborator

Please check if you need to remove the i4g.large type, if it not should be supported

elif self.idata.instancetype == "i4g.large":
self.disk_properties["read_iops"] = 34035
self.disk_properties["read_bandwidth"] = 288471904
self.disk_properties["write_iops"] = 27943
self.disk_properties["write_bandwidth"] = 126763269

@yaronkaikov
Copy link
Collaborator Author

Please check if you need to remove the i4g.large type, if it not should be supported

elif self.idata.instancetype == "i4g.large":
self.disk_properties["read_iops"] = 34035
self.disk_properties["read_bandwidth"] = 288471904
self.disk_properties["write_iops"] = 27943
self.disk_properties["write_bandwidth"] = 126763269

We should support it as well. so keeping it

@Annamikhlin
Copy link
Collaborator

Please check if you need to remove the i4g.large type, if it not should be supported

elif self.idata.instancetype == "i4g.large":
self.disk_properties["read_iops"] = 34035
self.disk_properties["read_bandwidth"] = 288471904
self.disk_properties["write_iops"] = 27943
self.disk_properties["write_bandwidth"] = 126763269

We should support it as well. so keeping it

So we are supporting it but not testing it?

@Annamikhlin
Copy link
Collaborator

Please check if you need to remove the i4g.large type, if it not should be supported

elif self.idata.instancetype == "i4g.large":
self.disk_properties["read_iops"] = 34035
self.disk_properties["read_bandwidth"] = 288471904
self.disk_properties["write_iops"] = 27943
self.disk_properties["write_bandwidth"] = 126763269

We should support it as well. so keeping it

So we are supporting it but not testing it?

Ok, I see your answer in https://github.com/scylladb/scylla-pkg/pull/3556#discussion_r1309918566 :)

Copy link
Collaborator

@Annamikhlin Annamikhlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@yaronkaikov
Copy link
Collaborator Author

@yaronkaikov yaronkaikov merged commit a6f7f2e into scylladb:next Aug 30, 2023
1 check passed
@yaronkaikov yaronkaikov deleted the add-missing-i4g-instnace branch August 30, 2023 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants