Skip to content

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

Merged
Xuanwo merged 1 commit intoapache:mainfrom
everpcpc:fix-assume-role
Sep 20, 2023
Merged

chore(ci): add test for aws s3 assume role#3139
Xuanwo merged 1 commit intoapache:mainfrom
everpcpc:fix-assume-role

Conversation

@everpcpc
Copy link
Copy Markdown
Contributor

@everpcpc everpcpc commented Sep 20, 2023

@github-actions github-actions Bot added the releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell label Sep 20, 2023
@everpcpc everpcpc marked this pull request as ready for review September 20, 2023 01:43
Copy link
Copy Markdown
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
@everpcpc everpcpc deleted the fix-assume-role branch September 20, 2023 06:56
@everpcpc
Copy link
Copy Markdown
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
Copy Markdown
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

Labels

releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: Add integration test for s3 assume role

3 participants