Fjord Choiceのワークフローが失敗した際の実行ステータスを「失敗」にする

[SuzukaHori]

Issue

• Fjord Choiceでjunohm410さんのポイントが食い違っている問題を修正する #7357

概要

junohm410さんの実際のpointは19ptですが、Fjord Choice上では17ptとなっており、ズレが発生しています。
本番環境のDBを確認したところ、#6480の2ptが反映されていませんでした。

このPRで、Fjord Choiceのワークフローが失敗した際の実行ステータスを「失敗」にすることで、失敗時のデータの修正や原因究明がやりやすくなります。

補足

Fjord Choiceの仕組みについて

bootcampのリポジトリが操作されると、GitHub Actionsが動き、以下のワークフローのコードが状況に応じて実行されます。

• Issueの割り振り時やポイントの操作時：.github/workflows/post-issue-event.yml
• PRでレビューがリクエストされた時　：.github/workflows/post-pr-event.yml
• PRでレビュアーがApproveした時　　：.github/workflows/update_only_when_reviewer_approved.yml

ワークフローでcurlコマンドが実行され、Fjord ChoiceにPOSTにリクエストが飛ぶ仕組みです。
詳しくはsakiさんのブログにも記載があります。

調査の経緯

Fjord Choice上の問題・bootcampのGitHub Actionsの問題双方を疑いました。

• Fjord Choice側
  検証用Fjord Choiceと検証用bootcampリポジトリを作り、複数アカウントを使って#6480に近い状況を作りましたが、バグが再現しませんでした。
• bootcamp側
  GitHub Actionsのログで以下を確認しました。該当のログ
  • ログにエラーは出ていません。
  • 実行時間がやや長めです（10秒）。
    他のリクエストの多くは1~3秒程度で終わっており、同じく10秒かかっているログの中には500エラーが出ているものもありました。

本PRの内容と変更理由

内容

GitHub Actionsが動いたときに実行される、curlコマンドに-fオプションをつけました。

理由

• 現状の実装では、検証用Fjord Choiceを停止してアクセスできなくした状態でも、GitHub Actionsは「成功」と判定されてしまいます。ログ

• -f（もしくは--fail）オプションをつけることにより、curlコマンド実行時のエラーでFjord ChoiceへのPOSTリクエストが失敗した場合、「失敗」のステータスになります。（失敗時には通知もくるようになります）ログ
  

変更確認方法

• 今まで通り（Issue・レビュー依頼・Approveの）GitHub Actionが動くか確認する。
  このPRから修正後のコードでGitHub Actionsが実行されるので、レビュー依頼→Approveの動作確認はログの確認をお願いします。
  Issueの割り振りの動作確認はマージ後に行います。
• GitHub Action失敗時に、GitHub Actionが「失敗」のステータスになるか確認する。
  本物のbootcampリポジトリでは、マージ後に「失敗」状態のアクションが見つかるまで確認できないため、検証用リポジトリのログの確認をお願いします。
  • 検証用Fjord Choiceを停止して、-fオプションありで実行した場合、「失敗」のステータスになっている。
    https://github.com/SuzukaHori/bootcamp_private/actions/runs/8137610247/job/22236371677

Screenshot

画面上の変更はありません。

[SuzukaHori]

よっちゃんさんにお手伝いいただき、-fオプションをつけた状態でもFjord Choiceに反映されることを確認しました。

レビュー依頼時

• ログ：https://github.com/fjordllc/bootcamp/actions/runs/8164638173/job/22320287427
  

Approve後

• ログ：https://github.com/fjordllc/bootcamp/actions/runs/81646
  

[SuzukaHori]

@komagata
お疲れさまです。こちらのPRのレビューをお願いします🙏

• こちらは先週の質問タイムで、メンバーではなく直接komagataさんのレビューと確認したPRです。
• レビュー依頼→Approveの成功時の動作確認のみ↑でしております。失敗時及び、Issueの割り振りの動作は、マージ後に確認する予定です。

[komagata]

確認させて頂きました。OKです〜🙆‍♂️