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_conifgure.py: skip construct data volume on dev_instance_type #552

Conversation

syuu1228
Copy link
Contributor

Since dev_instance_type does not have local SSD, we should skip running scylla_create_devices.

Fixes #551

@syuu1228
Copy link
Contributor Author

@syuu1228 syuu1228 requested a review from yaronkaikov November 21, 2024 10:57
@fruch
Copy link
Collaborator

fruch commented Nov 21, 2024

@syuu1228 @yaronkaikov

there is a specific test for t3.micro

and it seems to be running just fine, on master and on 2024.2:
https://argus.scylladb.com/tests/scylla-cluster-tests/8885cca7-19dd-478b-88fd-4a0f3347ddf4

I would recommend checking out why it's not catching the issue encounter, and adapt the test to catch it.

Since dev_instance_type does not have local SSD, we should skip running
scylla_create_devices.

Fixes scylladb#551
@syuu1228 syuu1228 force-pushed the scylla_configure_skip_create_devices_for_dev_instance_type branch from 39213f9 to 25e13c4 Compare November 21, 2024 12:25
@syuu1228
Copy link
Contributor Author

I realized that extra disks may attached to the dev instance (at least SCT test does it), changed the code to handle it: https://github.com/scylladb/scylla-machine-image/compare/39213f94e52694846b218ff0ba6cf6c629bf60f9..25e13c4f217535ee9b60e7345aafb64137506c5a
And testing it now.

@fruch
Copy link
Collaborator

fruch commented Nov 24, 2024

Maybe there should be a test that shows scylla working with adding an extra disk ?

As in the original failure in issue this PR is fixing ?

@roydahan
Copy link
Contributor

@fruch IIUC the the artifact test we have in SCT, it must have an extra disk:

data_volume_disk_num: 1  # t3.micro doesn't include any local volumes (outside of root), so an additional disk is required for scylla

Which I assumed was a requirement for it to work.

@syuu1228
What is the current claimed problem? that it doesn't work without extra disk?

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.

scylla-image-setup.service fails on dev_instance_type
5 participants