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

S3 Config Documentation Update #8579

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

S3 Config Documentation Update #8579

wants to merge 3 commits into from

Conversation

goabv
Copy link

@goabv goabv commented Mar 14, 2024

Issue #, if available:

Description of changes: Updating S3 configuration doc to encourage using CRT S3 client

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@goabv goabv changed the title Docpatch S3 Config Documentation Update Mar 14, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.08%. Comparing base (d0fb0a0) to head (977e3fa).
Report is 12 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff           @@
##           develop   #8579   +/-   ##
=======================================
  Coverage     0.08%   0.08%           
=======================================
  Files          208     208           
  Lines        16716   16716           
=======================================
  Hits            14      14           
  Misses       16702   16702           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@elysahall elysahall added the documentation This is a problem with documentation. label Mar 14, 2024
@@ -22,6 +22,7 @@ Configuration Values
These are the configuration values you can set specifically for the ``aws s3``
command set:

* ``preferred_transfer_client`` - This determines the underlying Amazon S3 transfer client to use for transferring files to and from S3. For most workloads, choosing ``crt`` will enable faster uploads and downloads.
* ``max_concurrent_requests`` - The maximum number of concurrent requests.
* ``max_queue_size`` - The maximum number of tasks in the task queue.
Copy link

Choose a reason for hiding this comment

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

Some of these config options only apply to classic, and some only apply to crt. If we're going to suggest crt we probably shouldn't show a bunch of settings that don't affect it.

This is currently detailed further down in the doc: https://awscli.amazonaws.com/v2/documentation/api/latest/topic/s3-config.html#preferred-transfer-client

Should we do a better job in this doc of breaking up which options affect both, which affect crt-only, and which are classic-only?

@nateprewitt
Copy link
Member

Thanks for the PR, @goabv. Most of this information is already in the CLI v2 docs here. We won't want to add this to the CLI v1 docs, as this PR does, because the CRT isn't available by default.

Was there something specifically insufficient with the current docs? Any changes should be made here on the v2 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a problem with documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants