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

ReviewApp: Status の timeout の追加 #59

Open
ShotaKitazawa opened this issue Oct 31, 2021 · 0 comments
Open

ReviewApp: Status の timeout の追加 #59

ShotaKitazawa opened this issue Oct 31, 2021 · 0 comments
Labels
bug Something isn't working medium priority

Comments

@ShotaKitazawa
Copy link
Member

ShotaKitazawa commented Oct 31, 2021

やりたいこと

  • 特定の status に居続けた場合に、一定時間経過後に status を初期化するようにしたい

なぜ必要か

  • 「実クラスタの ArgoCD Application の更新を待つ」ことは、 commentToAppRepoPullRequest メソッド内で「ReviewApp.Status.Sync.AppRepoLatestCommitSha == Application.Metadata.Annotations["dreamkast.cloudnativedays.jp/app-commit-hash"] 」を判定することで判断しており、これが true とならない限り ra.Status.Sync.Status は更新されない (別状態に遷移しない)
  • ここで、 ReviewApp.Status.Sync.AppRepoLatestCommitSha が更新されるタイミングは confirmAppRepoIsUpdated メソッド内のみであるため、 ra.Status.Sync.Status == dreamkastv1alpha1.SyncStatusCodeUpdatedInfraRepo である限り ReviewApp.Status.Sync.AppRepoLatestCommitSha は更新されない
  • 上記2つよりデッドロックが起きうるので、「一定時間 ra.Status.Sync.Status == dreamkastv1alpha1.SyncStatusCodeUpdatedInfraRepo のままだった場合 ra.Status.Sync.Status を別の値に更新する」処理を入れてデッドロックを回避したい
@ShotaKitazawa ShotaKitazawa added bug Something isn't working medium priority labels Oct 31, 2021
@jacopen jacopen moved this to Todo in Dreamkast Nov 9, 2021
@jacopen jacopen added this to Dreamkast Nov 9, 2021
Repository owner moved this from Todo to Done in Dreamkast May 22, 2022
@ShotaKitazawa ShotaKitazawa reopened this May 22, 2022
Repository owner moved this from Done to In Progress in Dreamkast May 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working medium priority
Projects
Status: In Progress
Development

No branches or pull requests

1 participant