Skip to content

⚗️ feat(oos): experimental s3 wrapper for OOS#1044

Merged
jfbus merged 1 commit intov3from
oos
Mar 3, 2026
Merged

⚗️ feat(oos): experimental s3 wrapper for OOS#1044
jfbus merged 1 commit intov3from
oos

Conversation

@jfbus
Copy link
Contributor

@jfbus jfbus commented Feb 25, 2026

Description

This PR adds an experimental wrapper above the S3 API, with all the OOS supported calls.

An alternative implementation would be to use an interface with all supported methods and return the standard S3 client.

Returning a wrapping struct allows easier integration in octl and allows (potentially) to redefine inputs with only the supported fields or do additional validation.

The PR also updates the template in order to add a //sdk:group comment to methods to allow grouping of commands in octl.

Note: //tool: comments are directive comments and do not appear in the godoc documentation.

Type of Change

Please check the relevant option(s):

  • 🐛 Bug fix
  • ✨ New feature
  • 🧹 Code cleanup or refactor
  • 📝 Documentation update
  • 🔧 Build or CI-related change
  • 🔒 Security fix
  • Other (specify):

How Has This Been Tested?

Please describe the test strategy:

  • Manual testing
  • Unit tests
  • Integration tests
  • Not tested yet

Checklist

  • I have followed the Contributing Guidelines
  • I have added tests or explained why they are not needed
  • I have updated relevant documentation (README, examples, etc.)
  • My changes follow the Conventional Commits specification
  • My commits include appropriate Gitmoji

Additional Context

@jfbus jfbus force-pushed the oos branch 2 times, most recently from 162e1f4 to 08fabc7 Compare February 26, 2026 08:17
@jfbus jfbus marked this pull request as ready for review February 26, 2026 10:20
@jfbus jfbus force-pushed the oos branch 5 times, most recently from 324d54b to 319e32d Compare March 2, 2026 19:19
@jfbus jfbus merged commit ff874bd into v3 Mar 3, 2026
5 checks passed
@jfbus jfbus deleted the oos branch March 3, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants