Update OBJ Group and Objects for MultiCluster Object Storage API Changes

[zliang-akamai]

📝 Description

This is to modify ObjectStorageGroup and object storage related SDK objects to adapt to the MultiCluster Object Storage API changes.

✔️ How to Test

Automated Testing

make testunit

make TEST_SUITE=object_storage testint

Manual Testing

import os

from linode_api4 import LinodeClient
from linode_api4.objects import ObjectStorageACL
from linode_api4.objects.object_storage import ObjectStorageKeyPermission

client = LinodeClient(os.getenv("LINODE_TOKEN"))

TEST_REGION="us-mia"
TEST_CLUSTER="us-mia-1"

bucket1 = client.object_storage.bucket_create(
    cluster_or_region=TEST_REGION,
    label="test-python-sdk-bucket",
    acl=ObjectStorageACL.PRIVATE,
    cors_enabled=False,
)

key1 = client.object_storage.keys_create(
    label="test-python-sdk-limited-key",
    bucket_access=client.object_storage.bucket_access(
        cluster_or_region=TEST_REGION,
        bucket_name=bucket1.label,
        permissions=ObjectStorageKeyPermission.READ_ONLY,
        use_region=True,
    ),
    regions=["us-mia"],
)

bucket1._api_get()
key1._api_get()

print("Created with a region:")
print(bucket1._raw_json)
print(key1._raw_json)

key1.delete()
bucket1.delete()

bucket2 = client.object_storage.bucket_create(
    cluster_or_region=TEST_CLUSTER,
    label="test-python-sdk-bucket",
    acl=ObjectStorageACL.PRIVATE,
    cors_enabled=False,
)

key2 = client.object_storage.keys_create(
    label="test-python-sdk-limited-key",
    bucket_access=client.object_storage.bucket_access(
        cluster_or_region=TEST_CLUSTER,
        bucket_name=bucket2.label,
        permissions=ObjectStorageKeyPermission.READ_ONLY,
        use_region=False,
    ),
)

print("Created with a cluster:")
print(bucket2._raw_json)
print(key2._raw_json)

key2.delete()
bucket2.delete()

[lgarber-akamai]

Tests are passing on my end and I haven't run into any more issues when testing against this from Ansible. Excellent work!

[yec-akamai]

Everything works locally, nice work!

[yec-akamai]

Should we add a deprecated note to the cluster property as well?

[zliang-akamai]

Thanks for the head up! I think there isn't a clear way to mark it as deprecated in Python since the property is in a dict... Let me think about it maybe we can modify Property class to accept a deprecation flag. I can do that in a future PR, I think.