scylladb / scylla-machine-image

Apache License 2.0
18 stars 25 forks source link

Azure: Standard_L8as_v3 not recognized by cloud setup #439

Closed tarzanek closed 4 months ago

tarzanek commented 1 year ago

instance: Standard_L8as_v3 Scylla 2022.2.3 + machine image 2022.2.3

root@gc-scylla-test001:/home/azureuser# sudo /opt/scylladb/scylla-machine-image/scylla_cloud_io_setup  

ERROR:root:This is not a recommended Azure Cloud instance setup for auto local disk tuning. 
Traceback (most recent call last): 
 File "/opt/scylladb/scylla-machine-image/libexec/scylla_cloud_io_setup", line 357, in <module> 
   io.generate() 
 File "/opt/scylladb/scylla-machine-image/libexec/scylla_cloud_io_setup", line 298, in generate 
   raise UnsupportedInstanceClassError() 
TypeError: UnsupportedInstanceClassError() missing 1 required positional argument: 'Exception' 

During handling of the above exception, another exception occurred: 

Traceback (most recent call last): 
 File "/opt/scylladb/scylla-machine-image/libexec/scylla_cloud_io_setup", line 359, in <module> 
   except UnsupportedInstanceClassError as e: 
TypeError: catching classes that do not inherit from BaseException is not allowed 
root@gc-scylla-test001:/home/azureuser#
tarzanek commented 1 year ago

@syuu1228 @yaronkaikov ^

tarzanek commented 1 year ago

ah, I see now https://github.com/scylladb/scylla-machine-image/commit/f6e759c89dadafac0e2ad687ac7627ce73c7da87 added support

tarzanek commented 1 year ago

can you backport above @syuu1228 ?

tarzanek commented 1 year ago

actually, even with above, the fallback to iotune should still work, is above an iotune bug?

yaronkaikov commented 1 year ago

@tarzanek is it possible to work with 5.2 or 2023.1? all Azure verification started officially from those releases

tarzanek commented 1 year ago

@gcarmin can you use 2023.1 RC ?

yaronkaikov commented 4 months ago

Any Official release starting from 2023.1.X and 2024.1.X should support this type. Closing this issue, if you facing this issue in an official release, please feel free to open it again