プロジェクト

全般

プロフィール

Defect #630

未完了

コードレビューしようとしてレスポンスが返ってこない状況になることがある

Yuichi Ishii さんが13年以上前に追加. 約11年前に更新.

ステータス:
担当(Assigned)
優先度:
通常(Normal)
担当者:
対象バージョン:
開始日:
2010/11/19
期日:
進捗率:

0%

予定工数:

説明

いつもCodeReview Pluginを活用させていただいています。
ありがとうございます。

CodeReview Pluginを使用していて、不具合?と思われる現象に遭遇したので
この場を借りてご報告させていただきます。

参考にしていただければ嬉しく思います。
また、何かご協力できることがあれば喜んでやらせていただきます。

現象

「リポジトリ」から特定のファイルに対してコードレビューしようとしてレスポンスが返ってこない状況になる。
また、Redmineシステム全体でリクエストを処理できない状態になってしまう。

→ コードレビュー用のウィンドウを表示させて、適当な値を入れて「送信」ボタンを押しても
そのまましばらく応答が返ってこない(送信ボタンが無効化されて待機状態のままとなっている)。
しばらく経つとHTTP 500エラーがブラウザーに表示される。

Redmineのログとして以下のものが記録される

Mongrel::TimeoutError (Mongrel timed out this thread: shutdown):
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue.rb:421:in `all_dependent_issues'
  app/models/issue.rb:419:in `all_dependent_issues'
  app/models/issue_relation.rb:50:in `validate'
  vendor/plugins/redmine_code_review/app/controllers/code_review_controller.rb:95:in `new'
  vendor/plugins/redmine_code_review/app/controllers/code_review_controller.rb:89:in `new'
  vendor/plugins/redmine_code_review/app/controllers/code_review_controller.rb:65:in `new'

Rendering d:/Redmine/redmine-1.0.3/public/500.html (500 Internal Server Error)

環境

サーバー

  • code_review_plugin 0.3.1
  • Redmine 1.0.3
  • Ruby 1.8.6
  • mongrel (1.1.5)
  • mongrel_service (0.3.4)
  • MySQL 5.1
  • Windows Server 2008 SP2
  • IIS

クライアント

  • Windows 7 Enterprise x64
  • Google Chrome

備考

リポジトリーのすべてのファイルで発生しているわけではなく、特定のファイルで発生している。
ただ、そのパターンはつかめていない。

Haru Iida さんが13年以上前に更新

  • ステータス新規(New) から 担当(Assigned) に変更
  • 担当者Haru Iida にセット
  • 対象バージョン0.4.0 にセット

ありがとうございます。調べてみます。

Yuichi Ishii さんが13年以上前に更新

ありがとうございます。
一点、ポイントになりえる情報の記述が漏れていたので追記します。
問題が発生している環境では、 app/model/issue.rbvalidate メソッドの以下の箇所をコメントアウトしています。
(チケットの開始日の設定ルールを緩めたい意図)

#    if start_date && soonest_start && start_date < soonest_start
#      errors.add :start_date, :invalid
#    end

あまり関係は無いかと思ってはいますが...

Haru Iida さんが13年以上前に更新

チケットの関連チケットを再帰的に取得しようとして無限ループしてるようですね。
関連チケットの取得に時間がかかっているためにタイムアウトしているようです。
なんでこんなことになるんだろう。

もしくは本当に大量のチケットが関連づいている・・・なんてことは無いですよね。

Yuichi Ishii さんが13年以上前に更新

調査のほど、ありがとうございます。今の全体のチケット数ですが、16000を超えています。
問題が発生するプロジェクトで10000以上あります。

また、ひとつの(ファイルの)ソースコードのひとつのリビジョンに対して10-30くらいのレビューをつけることがあります。
そのレビューを多いときは3回(リビジョン3つ)くらいする場合もあります。
これがすべて関連付けされるのだとすると、多いほうなのでしょうか。

Yuichi Ishii さんが13年以上前に更新

ご無沙汰しています。

我々のRedmine環境を1.1.0にアップグレードしました。
この状態でまたこの問題が発生するかどうかをウォッチしていきたいと思います。

Haru Iida さんが13年以上前に更新

  • プロジェクトr-labs から Code Review に変更

ご連絡ありがとうございます。
状況が進展してなくてすみません。

Haru Iida さんが約13年前に更新

  • 対象バージョン0.4.0 から 0.4.1 に変更

Yuichi Ishii さんがほぼ13年前に更新

現在、Redmine1.2にアップグレードしています。
1.2でFIXしている問題で、ログが似ているものがあったのでリンクを記述します。

Redmine - circular dependency >2 issues
http://www.redmine.org/issues/7320

最近、発生していないように思います。
1.2でこの問題が再発するかどうか、引き続きウォッチします。

Haru Iida さんがほぼ13年前に更新

Yuichi Ishii は書きました:

現在、Redmine1.2にアップグレードしています。
1.2でFIXしている問題で、ログが似ているものがあったのでリンクを記述します。

Redmine - circular dependency >2 issues
http://www.redmine.org/issues/7320

最近、発生していないように思います。
1.2でこの問題が再発するかどうか、引き続きウォッチします。

情報ありがとうございます。問題が発生したらお知らせください。

Haru Iida さんがほぼ13年前に更新

  • 対象バージョン0.4.1 から いつかやる(Unplanned) に変更

Yuichi Ishii さんが約11年前に更新

お久しぶりです。
変わらず活用させていただいております。

Code Reviewプラグインとは関係なく、Redmineにはチケットの循環参照チェックに関する問題があるように思います。
別の状況で似たような問題が発生したのでRedmine Usersメーリングリストにフィードバックさせていただきました。

https://groups.google.com/forum/?fromgroups=#!topic/redmine-users-ja/qtgz7kinbbQ

他の形式にエクスポート: Atom PDF