search

cloudnativedaysjp / reviewapp-operator

MIT License
5 stars 0 forks source link

make preStopJob not run more than once #90

Open ShotaKitazawa opened 2 years ago

ShotaKitazawa commented 2 years ago

問題点

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

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

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

解決案

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