Allow docs-sync sessions up to 6 hours - #74
Merged
Merged
Conversation
The initial full upload of html/ja died at the 1-hour mark with ExpiredToken: configure-aws-credentials issues a 1-hour session by default and the roles cap MaxSessionDuration at the same default. Site-wide regeneration days (template changes) re-upload most of the tree and need longer. Raise max_session_duration to 21600 (the GitHub-hosted job limit) on the docs-sync roles; the workflows opt in per-run with role-duration-seconds. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
2 participants
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.
Raise
max_session_durationon the docs-sync roles to 21600 seconds (6 hours, the GitHub-hosted job limit).The initial full upload of
html/jadied exactly at the 1-hour mark withExpiredToken(rurema/generated-documents run 32343869053):configure-aws-credentialsissues a 1-hour session by default, and the roles capMaxSessionDurationat the same default. Site-wide regeneration days (template changes) re-upload most of the ~270k objects and can run long.With the concurrency bump on the workflow side a full upload now fits in well under an hour, so this is a safety margin rather than a hard requirement; the workflows opt in per-run with
role-duration-seconds, and normal incremental runs keep using short sessions.Please apply when convenient — applying together with #75 would make it a single round.
(日本語) docs-sync ロールのセッション上限を 6 時間へ。初回フル sync が既定 1 時間のセッション期限ちょうどで ExpiredToken になった件の保険です。並列度引き上げ後はフルでも 1 時間内に収まるため必須ではありません。#75 とまとめて apply いただけると 1 回で済みます。
🤖 Generated with Claude Code