Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

みんなのブログが更新されない。 #6929

Closed
machida opened this issue Sep 27, 2023 · 9 comments
Closed

みんなのブログが更新されない。 #6929

machida opened this issue Sep 27, 2023 · 9 comments
Assignees
Labels

Comments

@machida
Copy link
Member

machida commented Sep 27, 2023

リリース時のまま止まっている。

@komagata
Copy link
Member

@siso25 こちら確認いただければありがたいです〜

@siso25
Copy link
Contributor

siso25 commented Sep 27, 2023

@komagata
承知しました。
以下のissueが原因かと思いますので、こちらと一緒に対応いたします。

#6901

@ogawa-tomo
Copy link
Contributor

@siso25
横から失礼します:bow:
こちら、下記コントローラの定期処理が関係していますでしょうか?

class Scheduler::Daily::FetchExternalEntryController < ApplicationController

本番で動作していないのは、これがSchedulerControllerを継承していないからかもしれません。ベストアンサーなし通知で同じことが起きていました。
#6970
その場合、こちらのイシューの対応も必要となります。(一部は上記PRでも対応しています)
#6961
ご確認お願いします:pray:

@siso25
Copy link
Contributor

siso25 commented Oct 15, 2023

@ogawa-tomo
こちらに関しては以下のエラーで処理が止まっていることが原因のようですので、直接は関係ないかもしれません。
ただ、SchedulerControllerを継承するよう変更する必要はあるとおもいますので、対応いたします。
#6901

@JunichiIto
Copy link
Contributor

こちらに関しては以下のエラーで処理が止まっていることが原因のようですので、直接は関係ないかもしれません。

なんでこっちはユーザ認証をすり抜けてfetch_and_save_rss_feedsがキックされたんでしょうか?
#6901 で報告されたエラーはスケジューラの呼び出しによるものですか?
(誰かログイン済みのユーザーが該当URLにアクセスしたのではなく?)

この点を明確にしないとちょっと気持ち悪いですね。

@siso25
Copy link
Contributor

siso25 commented Oct 18, 2023

@JunichiIto

こちらに関しては以下のエラーで処理が止まっていることが原因のようですので、直接は関係ないかもしれません。

上記のように記載したのですが、こちら勘違いをしておりました。申し訳ありません。
#6901 のエラーについては、本番での確認の際に私が誤って該当URLにアクセスした際に発生したものと思われます。

みんなのブログリリース後、スケジューラ上は正常系のHTTPステータスが返ってきているのに登録内容は更新されていなかったことから、本issueに関してはSchedulerControllerを継承していないためにfetch_and_save_rss_feedsが実行されていないことが原因だと思います。

@JunichiIto
Copy link
Contributor

ですよね!!

@machida
Copy link
Member Author

machida commented Nov 23, 2023

本番で更新されているのが確認できたのでクローズします。

@siso25
Copy link
Contributor

siso25 commented Dec 27, 2023

本番で正常に更新されていることが確認できました。また、#6901 のバグも発生していないとのことなので、本issueをクローズさせていただきます。

@siso25 siso25 closed this as completed Dec 27, 2023
@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

5 participants