Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

make preStopJob not run more than once #90

Open
ShotaKitazawa opened this issue Mar 31, 2022 · 0 comments
Open

make preStopJob not run more than once #90

ShotaKitazawa opened this issue Mar 31, 2022 · 0 comments
Labels
bug Something isn't working high priority

Comments

@ShotaKitazawa
Copy link
Member

問題点

ReviewApp Controller の実装において、現状 reconcileDelete 内で以下の処理を行っている。

  1. preStopJob に指定された JobTemplate から Job リソースを作成 (Job の exit status は問わない)
  2. ReviewApp オブジェクトに紐づく infra repo 上のマニフェストを削除

上記は、 2 の処理に失敗した場合 (cf. infra repo に push するための secret が存在しない)、複数回 preStopJob が走ってしまう問題点がある。

解決案

ra.status 内に preStopJob を実行したかどうかを管理するパラメータを生やす

@ShotaKitazawa ShotaKitazawa changed the title preStopJob が複数回走らないようにする make preStopJob not run more than once Apr 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working high priority
Projects
None yet
Development

No branches or pull requests

1 participant