Skip to content

Conversation

@yawkat
Copy link
Member

@yawkat yawkat commented Oct 12, 2023

With local and oci implementation.

Local only lists configured buckets, not much else that makes sense.

OCI needs a namespace configuration unfortunately. It's also not clear what the prefix should be, micronaut.object-storage.oracle-cloud.* is already used so i used micronaut.object-storage.oracle-cloud-dynamic but that doesn't really make obvious what this is for.

I think other cloud providers should be easier to implement, with a single BucketOperations impl only.

WDYT @alvarosanchez

With local and oci implementation.

Local only lists configured buckets, not much else that makes sense.

OCI needs a namespace configuration unfortunately. It's also not clear what the prefix should be, micronaut.object-storage.oracle-cloud.* is already used so i used micronaut.object-storage.oracle-cloud-dynamic but that doesn't really make obvious what this is for.

I think other cloud providers should be easier to implement, with a single BucketOperations impl only.
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 7 Code Smells

80.9% 80.9% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@yawkat
Copy link
Member Author

yawkat commented Oct 20, 2023

@alvarosanchez this passes now. but the azure changes kind of suck, so i'll leave this with you for now.

@alvarosanchez
Copy link
Member

@yawkat I'm fine thanks, this will wait for you once you're available again 😉

@yawkat yawkat linked an issue Oct 26, 2023 that may be closed by this pull request
@CLAassistant
Copy link

CLAassistant commented Feb 7, 2024

CLA assistant check
All committers have signed the CLA.

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.

Implement a bucket/container management API

4 participants