Skip to content

Support credential(s) as input param in aws local#1139

Closed
kurotych wants to merge 4 commits intomainfrom
kurotych/clean-awslocal-old
Closed

Support credential(s) as input param in aws local#1139
kurotych wants to merge 4 commits intomainfrom
kurotych/clean-awslocal-old

Conversation

@kurotych
Copy link
Copy Markdown
Contributor

@kurotych kurotych commented Feb 16, 2026

Rustfs requires credentials to be set. Added fn cred to the builder to make it possible to pass credential parameters.

Also, Remove leftover. The Aws local was moved to file file_store

@kurotych kurotych marked this pull request as ready for review February 16, 2026 16:59
@kurotych kurotych marked this pull request as draft February 17, 2026 13:18
@kurotych kurotych changed the title Remove leftover aws local artifact Support credential(s) as input param in aws local Feb 17, 2026
@kurotych kurotych marked this pull request as ready for review February 17, 2026 16:12
@michaeldjeffrey
Copy link
Copy Markdown
Contributor

Can we put some effort into finding a change that allows us to continue running tests with more than 1 thread?

@kurotych
Copy link
Copy Markdown
Contributor Author

kurotych commented Feb 17, 2026

Can we put some effort into finding a change that allows us to continue running tests with more than 1 thread?

I don't know why but fast workaround (I've tested it really works) is using nextest
found here: seanmonstar/reqwest#1148 (comment)

I will return to this problem after I complete my main ticket. Meanwhile, I will use nextest.

Converted to draft

@kurotych kurotych marked this pull request as draft February 17, 2026 18:04
@kurotych kurotych marked this pull request as ready for review February 19, 2026 14:47
@kurotych kurotych closed this Feb 25, 2026
@kurotych kurotych deleted the kurotych/clean-awslocal-old branch February 25, 2026 17:41
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