Repository navigation
Conversation
Assisted-by: reasoning/desy-assistant
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Suggested documentation update for 158529f ("Merge branch 's3-for-master'"). AI-generated — please review for accuracy before merging.
Why this commit needs docs. The merge introduces the new S3 door (the whole of
modules/dcache-s3andmodules/s3proxy-vendored), a new user-facing service that site administrators must set up. It also updatesconfig.mdandindex.mdto link a new chapterconfig-s3.mdfrom the book's tables of contents, and adds thes3principal to the principal table inconfig-gplazma.md— butdocs/TheBook/src/main/markdown/config-s3.mdis not present in the commit, so the book is built with dangling links to the new service.What this PR does. Adds the missing
config-s3.md, covering:s3.authn.keystore.file) and mapping access keys to dCache users with amapplugin (s3:<access-key>) and asessionplugin;s3.root, the keys dCache cannot hold, reserved names and the user's root directory;s3.*extended attributes;s3 uploads ls|rm|sweep);GLACIER) and RestoreObject;Please note while reviewing. The chapter is written to match the tables of contents already added by this commit, so its section headings are fixed by those anchors (
#introduction,#setting-up,#buckets-and-objects,#writing-objects,#files-on-tape,#clients,#operation,#limitations).Also: while preparing this, master has since gained
c485d1b("[skip-ci] docs: add S3 door chapter to the dCache Book"), which adds an equivalent chapter for the same feature. This PR was opened from the commit under evaluation (158529f), where the file is absent; ifc485d1balready covers it, this PR should simply be closed in favour of that commit. Any details worth keeping from this draft can be merged into the existing chapter.