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

feat: improving obj storage #7

Merged
merged 16 commits into from
Aug 2, 2024
Merged

feat: improving obj storage #7

merged 16 commits into from
Aug 2, 2024

Conversation

kkiani
Copy link
Contributor

@kkiani kkiani commented Aug 1, 2024

Description

Object storages are very basic requirements in many complex application. The previous implementation was just focusing on providing a few methods without any tests to POC the framework. This PR added all the basic functionality that is required.

  • added a method to read objects
  • added a method to write objects
  • added a method to check if object exists
  • added a method to list objects
  • added a method to deletes objects
  • added automated test to validate the above functionalities.

src/damavand/cloud/aws/bucket.py Show resolved Hide resolved
src/damavand/cloud/aws/bucket.py Outdated Show resolved Hide resolved
src/damavand/core.py Outdated Show resolved Hide resolved
@kkiani kkiani force-pushed the feature/improving-obj-storage branch from c9bae77 to 4e8e61a Compare August 1, 2024 15:36
@kkiani kkiani force-pushed the feature/improving-obj-storage branch from 4e8e61a to 17c918d Compare August 2, 2024 07:18
kkiani added 4 commits August 2, 2024 11:12
All the exceptions wrapping boto3 ClientError with set the ClientError
as their root cause
providers arguments were not directly available due to design of pulumi.
This solution creates defautl wapper around pulumi providers to keep
contexts.
@kkiani kkiani merged commit 2c8f9fc into main Aug 2, 2024
2 checks passed
@kkiani kkiani deleted the feature/improving-obj-storage branch August 2, 2024 13:27
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.

2 participants