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

chore(ci): add test for aws s3 assume role #3139

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

everpcpc
Copy link
Contributor

@everpcpc everpcpc commented Sep 20, 2023

@everpcpc everpcpc marked this pull request as ready for review September 20, 2023 01:43
Copy link
Member

@PsiACE PsiACE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

almost lgtm,

cc @Xuanwo how to trigger it in this PR for testing

@Xuanwo Xuanwo merged commit e146703 into apache:main Sep 20, 2023
10 checks passed
@everpcpc everpcpc deleted the fix-assume-role branch September 20, 2023 06:56
@everpcpc
Copy link
Contributor Author

everpcpc commented Sep 20, 2023

It seems not working as expected: https://github.com/apache/incubator-opendal/actions/runs/6244545530/job/16951629630

test services_s3::test_writer_write ... FAILED

failures:

---- services_s3::test_writer_write ----
Unexpected (temporary) at Writer::write, context: { called: reqsign::LoadCredential, service: s3, path: 7d2ba7eb-47fd-4505-b125-09e70039bb25, write_buf: 5242880 } => loading credential to sign http request, source: source credential is required for AssumeRole, but not found, please check your configuration


failures:
    services_s3::test_writer_write

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 102 filtered out; finished in 15.43s

@Xuanwo
Copy link
Member

Xuanwo commented Sep 20, 2023

Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ci: Add integration test for s3 assume role
3 participants