scylla_cloud_io_setup: fix NameError

scylladb / scylla-machine-image

Apache License 2.0

18 stars 25 forks source link

scylla_cloud_io_setup: fix NameError #474

Closed syuu1228 closed 10 months ago

syuu1228 commented 10 months ago

Since ecdbd44b5661cd8e97fec18c1d74a311ec4c2fa7 was bring from scylla_io_setup patch, there are incompatible variable names. We need to rename them in following rules:

idata and disk_properties should be member of self
rename instance_class() to instancetype

yaronkaikov commented 10 months ago

@syuu1228 Running https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/next-machine-image/169/ to make sure it will work. once it's completed i will merge

yaronkaikov commented 10 months ago

@syuu1228 Running https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/next-machine-image/169/ to make sure it will work. once it's completed i will merge

Passed