search

Tecnativa / doodba

Base image for making the creation of customized Odoo environments a piece of cake
Apache License 2.0
436 stars 305 forks source link

Inotify for functional hot code reloading #308

Closed theangryangel closed 4 years ago

theangryangel commented 4 years ago

Hiya,

Out of the box Doodba's hot code reloading flat out doesn't work because watchdog doesn't follow symlinks.

However Odoo does support inotify, which does follow symlinks correctly.

I've tested it in a a v12 and v13 test project and it seems to be working fine, afaik.

Would you accept a PR adding inotify to the Dockerfiles for the relevant versions of Odoo that will use it? Or was there a reason why it was excluded (too heavy, etc)?

yajo commented 4 years ago

I was too lazy to dig deep on that, and I just reboot odoo when needed. :grin:

Yes, you can add it, but please include docs on the feature. Thanks!