失敗したジョブ: Jobs::Jobs:Module の未定義メソッド `cancel_scheduled_job`

merefield · 2022 年 5 月 26 日午前 9:07

Jobs::ProcessPost の実行中

コマンドラインで再現できます。

一方、以下で正常に到達できます。

これに関連している可能性があります。

github.com/discourse/discourse

FEATURE: Pull hotlinked images immediately after posting

committed 01:28PM - 23 May 22 UTC

davidtaylorhq

+193 -199

Previously, with the default `editing_grace_period`, hotlinked images were pulle…d 5 minutes after a post is created. This delay was added to reduce the chance of automated edits clashing with user edits. This commit refactors things so that we can pull hotlinked images immediately. URLs are immediately updated in the post's `cooked` HTML. The post's raw markdown is updated later, after the `editing_grace_period`. This involves a number of behind-the-scenes changes including: - Schedule Jobs::PullHotlinkedImages immediately after Jobs::ProcessPost. Move scheduling to after the `update_column` call to avoid race conditions - Move raw changes into a separate job, which is delayed until after the ninja-edit window - Move disable_if_low_on_disk_space logic into the `pull_hotlinked_images` job - Move raw-parsing/replacing logic into `InlineUpload` so it can be easily be shared between `UpdateHotlinkedRaw` and `PullUserProfileHotlinkedImages`

含まれるもの：

github.com/discourse/discourse

app/jobs/regular/process_post.rb

bf6f8299a


      
                end
              end
          
              # onebox may have added some links, so extract them now
              def extract_links(post)
                TopicLink.extract_from(post)
                QuotedPost.extract_from(post)
              end
          
              def enqueue_pull_hotlinked_images(post)
                Jobs.cancel_scheduled_job(:pull_hotlinked_images, post_id: post.id)
                Jobs.enqueue(:pull_hotlinked_images, post_id: post.id)
              end
            end
          
          end

このコンテキストでは、先頭に :: が必要でしょうか？

david · 2022 年 5 月 26 日午前 11:35

うーん、先頭に :: を追加すれば確かに解決するだろうけど、必須ではないはずだ

呼び出し元は ::Jobs::ProcessPost の内部なので、Ruby はツリーを遡って探すはずだ。まず ::Jobs::ProcessPost::Jobs を探し、次に ::Jobs::Jobs、そして最終的には ::Jobs モジュールを探すことになる。

君が見ているエラーは、何かが ::Jobs::Jobs を定義していることを示唆している… それは奇妙だ！私の開発インスタンスで見てみよう：

[1] pry(main)> Jobs::Jobs
=> Jobs::Jobs
[2] pry(main)> Jobs::Jobs.constants
=> [:RemapOldBotImages, :GrantBadges]

どうやら、discourse-narrative-bot のこれらの行が問題を引き起こしているようだ。コメントアウトすると、問題は解決する。

github.com/discourse/discourse

plugins/discourse-narrative-bot/plugin.rb

aabbc9e63


      
          if Rails.env == "development"
            # workaround, teach reloader to reload jobs
            # if we do not do this then
            #
            # 1. on reload rails goes and undefines Jobs::Base
            # 2. as a side effect this undefines Jobs::BotInput
            # 3. we have a post_edited hook that queues a job for bot input
            # 4. if you are not running sidekiq in dev every time you save a post it will trigger it
            # 5. but the constant can not be autoloaded
            Rails.configuration.autoload_paths << File.expand_path('../autoload', __FILE__)
          end

良いニュースは、これが開発専用のものであることだ。ジョブのファイルパスとモジュールの名前が一致しないことに関連している。これを整理するための PR がこちらだ：

https://github.com/discourse/discourse/pull/16924

報告ありがとう @merefield

merefield · 2022 年 5 月 26 日午前 11:41

ああ、そうなんですね、奇妙なことですが、プロダクションが爆発しなかった理由がわかりました！

迅速なご対応ありがとうございます!!

david · 2022 年 5 月 27 日午前 7:00

このトピックは17時間後に自動的に閉じられました。新しい返信はもう許可されていません。

トピック		返信	表示
Wrapped NoMethodError: undefined method `enqueue'' for Jobs::Jobs:Module Dev	2	880	2019 年 10 月 2 日
Job exception: undefined method `name' for Support	6	1161	2020 年 6 月 15 日
Error Log: "Job exception: undefined method `closed' for nil:NilClass Did you mean? clone" Bug	2	774	2017 年 4 月 3 日
"Job exception: undefined method `scheduled?' for nil" error when mentioning AI persona in post Bug ai	8	167	2025 年 12 月 2 日
Job Delete Replies error Bug	3	671	2021 年 2 月 8 日

失敗したジョブ: Jobs::Jobs:Module の未定義メソッド `cancel_scheduled_job`

関連トピック