Project

General

Profile

Defect #630

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

Added by Yuichi Ishii almost 9 years ago. Updated over 6 years ago.

Status:
担当(Assigned)
Priority:
通常(Normal)
Assignee:
Start date:
11/19/2010
Due date:
% Done:

0%

Estimated time:

Description

いつも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

備考

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

History

#1

Updated by Haru Iida almost 9 years ago

  • Status changed from 新規(New) to 担当(Assigned)
  • Assignee set to Haru Iida
  • Target version set to 0.4.0

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

#2

Updated by Yuichi Ishii almost 9 years ago

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

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

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

#3

Updated by Haru Iida almost 9 years ago

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

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

#4

Updated by Yuichi Ishii almost 9 years ago

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

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

#5

Updated by Yuichi Ishii over 8 years ago

ご無沙汰しています。

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

#6

Updated by Haru Iida over 8 years ago

  • Project changed from r-labs to Code Review

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

#7

Updated by Haru Iida over 8 years ago

  • Target version changed from 0.4.0 to 0.4.1
#8

Updated by Yuichi Ishii over 8 years ago

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

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

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

#9

Updated by Haru Iida over 8 years ago

Yuichi Ishii は書きました:

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

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

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

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

#10

Updated by Haru Iida over 8 years ago

  • Target version changed from 0.4.1 to いつかやる(Unplanned)
#11

Updated by Yuichi Ishii over 6 years ago

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

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

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

Also available in: Atom PDF