X Tutup
Skip to content

s3: add UCloud Object Storage provider#9230

Open
jinkeyuu wants to merge 1 commit intorclone:masterfrom
jinkeyuu:feat-add-US3
Open

s3: add UCloud Object Storage provider#9230
jinkeyuu wants to merge 1 commit intorclone:masterfrom
jinkeyuu:feat-add-US3

Conversation

@jinkeyuu
Copy link

@jinkeyuu jinkeyuu commented Mar 5, 2026

What is the purpose of this change?

Was the change discussed in an issue or in the forum before?

Checklist

  • I have read the contribution guidelines.
  • I have added tests for all changes in this PR if appropriate.
  • I have added documentation for the changes if appropriate.
  • All commit messages are in house style.
  • I'm done, this Pull Request is ready for review :-)

@ncw
Copy link
Member

ncw commented Mar 5, 2026

Hi @jinkeyuu thanks for doing this.

Do the integration tests pass properly with the quirks you have made?

Thanks

@jinkeyuu
Copy link
Author

jinkeyuu commented Mar 6, 2026

@ncw Thanks!
Only the mac_amd64 test failed. Is this due to the changes?

@ncw
Copy link
Member

ncw commented Mar 6, 2026

Sorry I wasn't clear enough.

Do the tests when you do

cd backend/s3
go test -v -remote YourProvider:

Work properly?

(the mac CI test is a flake)

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

X Tutup