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

Pin watchdog to 4.0.2 #5335

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Pin watchdog to 4.0.2 #5335

merged 1 commit into from
Sep 3, 2024

Conversation

novucs
Copy link
Contributor

@novucs novucs commented Aug 30, 2024

Watchdog 5.0.0 was recently released. This update causes Werkzeug to reload the Django server much more frequently than desired (causing the local docker development build to barely load).

Description

Checklist:

  • I've made sure that tests are updated accordingly (especially if adding or updating a template option)
  • I've updated the documentation or confirm that my change doesn't require any updates

Rationale

Without this change, playing around with the local docker development build consistently fails with ECONNRESET due to page loads triggering inotify events and therefore django server reboots

I've not tried to reproduce this issue on other environments, but this is the case on my M3 Mac.

Watchdog 5.0.0 was recently released. This update causes Werkzeug to reload the Django server much more frequently than desired (causing the local docker development build to barely load).
@browniebroke
Copy link
Member

browniebroke commented Aug 30, 2024

Have you reported the issue upstream? Would it be worth pinning it in werkzeug instead?

Copy link
Member

@browniebroke browniebroke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on a project, I had the same issue and this fixed it

@browniebroke browniebroke merged commit b7fc197 into cookiecutter:master Sep 3, 2024
14 checks passed
@browniebroke
Copy link
Member

I think this has now been fixed in werkzeug 3.0.5 so I'll remove this pin when I merge the upgrade: pallets/werkzeug#2945

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants