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

add docker hub auth to CircleCI #430

Merged
merged 1 commit into from Oct 9, 2020
Merged

add docker hub auth to CircleCI #430

merged 1 commit into from Oct 9, 2020

Conversation

sot528
Copy link
Contributor

@sot528 sot528 commented Oct 9, 2020

概要

  • なぜこの変更をするのか、
    • DockerHubが11月からpull数に制限を課すので、それを回避するため
  • 課題は何か、
    • 制限のせいでCIが落ちるかも
  • これによってどう解決されるのか、
    • ひとまず 200pulls/6hまでは制限が回避される

環境変数(SSMパラメータ)

  • CircleCIのContextに該当情報を追加済み

影響範囲(ユーザ)

  • 影響を与えるユーザは誰か
    • なし

影響範囲(システム)

  • 影響を与えるシステムはどこか
  • CI/CD

影響範囲(開発者)

  • 開発チームに共有すべきことはあるか
    • 本ページを共有

技術的変更点概要

  • なにをどう変更したか
    • DockerをDocker Hubからpullする処理に認証を追加

使い方

  • 使い方の説明
    • いつもと変わらず

テスト結果とテスト項目

  • ビルドが問題なく動作することを確認

レビュワーに依頼したいこと

  • 内容を確認し、問題なければマージしてください。
  • ステージング環境のビルドが問題なく完了することを確認してください
    • 基本的に本番環境もステージング環境も処理は変わらないはずなので、ステージング環境のみの確認でOKです。

@sot528 sot528 force-pushed the ALIS-5857 branch 4 times, most recently from 40f26e9 to 6a78cc8 Compare October 9, 2020 06:01
@sot528 sot528 requested a review from a team October 9, 2020 06:39
@ghost ghost requested review from keillera and removed request for a team October 9, 2020 06:40
@keillera
Copy link
Member

keillera commented Oct 9, 2020

LGTMです!

@keillera keillera merged commit 2b8fcb1 into master Oct 9, 2020
@keillera keillera deleted the ALIS-5857 branch October 9, 2020 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants