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

Use DefaultCredentialsChain AWS authentication in remote_storage #8440

Merged
merged 4 commits into from
Jul 19, 2024

Conversation

arpad-m
Copy link
Member

@arpad-m arpad-m commented Jul 19, 2024

PR #8299 has switched the storage scrubber to use DefaultCredentialsChain. Now we do this for remote_storage, as it allows us to use remote_storage from inside kubernetes. Most of the diff is due to GenericRemoteStorage::from_config becoming async fn.

@arpad-m arpad-m requested review from a team as code owners July 19, 2024 16:58
@arpad-m arpad-m requested review from arssher and conradludgate July 19, 2024 16:58
Copy link

github-actions bot commented Jul 19, 2024

3144 tests run: 3023 passed, 0 failed, 121 skipped (full report)


Flaky tests (1)

Postgres 15

  • test_wal_restore_initdb: debug

Code coverage* (full report)

  • functions: 32.7% (7001 of 21417 functions)
  • lines: 50.0% (55259 of 110458 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
be946eb at 2024-07-19T19:03:52.889Z :recycle:

@arpad-m arpad-m merged commit 4e547e6 into main Jul 19, 2024
65 checks passed
@arpad-m arpad-m deleted the arpad/default_remote_storage_config branch July 19, 2024 19:19
problame pushed a commit that referenced this pull request Jul 22, 2024
PR #8299 has switched the storage scrubber to use
`DefaultCredentialsChain`. Now we do this for `remote_storage`, as it
allows us to use `remote_storage` from inside kubernetes. Most of the
diff is due to `GenericRemoteStorage::from_config` becoming `async fn`.
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.

5 participants