feat(ci): AWS S3 SigV4リージョンのE2Eテストを追加#41
Open
tishin-endou wants to merge 5 commits into
Open
Conversation
Co-Authored-By: An Qiuyu <qiuyu.an@hotmail.com>
This was referenced Jun 5, 2026
tishin-endou
changed the title
- テスト手順-管理者機能-S3-機関ストレージ.ipynb を新規作成 (S3CompatSigV4版をベースに、Amazon S3用にエンドポイントURL関連を除去) URL/entityIDは実環境の固有ホスト名を残さず、他ノートブックと同様に example.comプレースホルダ or None(.config.yaml/プロンプト入力で補完)とした - 取りまとめ-S3共通.ipynb: .config.yaml から未設定パラメータを補完する ローダーセルを追加（対話実行を容易にするため） - .gitignore: .claude/ (ローカルのAI開発ツール設定) を除外
user-aws-s3 マトリクス項目は実AWS認証情報のGitHub Secretsを必要とするが、 パブリックリポジトリではSecretを登録できず、有効なままだと認証情報未設定で CIが失敗する。そのため既定でコメントアウトし、ローカル/Secret設定済みforkで テストする場合のみ有効化する旨をコメントで明記した。 generate_ci_config.sh 側の --aws-s3 処理は温存（有効化時にそのまま使える）。
コンフリクト解消（いずれも双方の追加を共存させる形）: - .github/workflows/e2e-test.yml: user-minio に upstream の wiki_enabled: false を 取り込みつつ、コメントアウト済みの user-aws-s3 ブロックを維持 - .github/scripts/generate_ci_config.sh: usage 文字列に --aws-s3 と --wiki を併記 （引数パース・処理ブロックは自動マージ済みで AWS_S3/WIKI 両対応）
tishin-endou
marked this pull request as ready for review
July 21, 2026 06:01
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
1 participant
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.
概要
実際のAWS S3に対してSigV4認証でE2Eテストを実行するCI設定を追加し、S3アドオンのSignature V2からV4への移行を検証します。
変更内容
CIワークフロー (
.github/workflows/e2e-test.yml)user-aws-s3テストグループをマトリクスに追加AWS_S3_ENABLED環境変数と条件分岐ロジックを追加AWS_S3_ACCESS_KEY、AWS_S3_SECRET_KEYAWS_S3_LEGACY_REGION、AWS_S3_LEGACY_BUCKET_NAMEAWS_S3_V4_REGION、AWS_S3_V4_BUCKET_NAME設定生成スクリプト (
.github/scripts/generate_ci_config.sh)--aws-s3オプションを追加storages_s3設定を生成--minioと--aws-s3の排他チェックを追加セットアップ要件
リポジトリに以下のGitHub Secretsの設定が必要です:
AWS_S3_ACCESS_KEYAWS_S3_SECRET_KEYAWS_S3_LEGACY_REGIONAWS_S3_LEGACY_BUCKET_NAMEAWS_S3_V4_REGIONap-northeast-1）AWS_S3_V4_BUCKET_NAME関連PR
Based on anqiuy/RDM-e2e-test-nb@feature/s3-sigv4