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

hangs at "Generate mailcow.conf file" with < 2.5G of memory #11

Open
ckoehler opened this issue Oct 9, 2020 · 5 comments
Open

hangs at "Generate mailcow.conf file" with < 2.5G of memory #11

ckoehler opened this issue Oct 9, 2020 · 5 comments

Comments

@ckoehler
Copy link

ckoehler commented Oct 9, 2020

The ./generate_config.sh script blocks on the clamd check when memory is < 2.5G and prompts the user to decide whether to skip clamd. With Ansible, this makes the task hang.

Instead, it should probably respect my mailcow__config_skip_clamd setting.

@ckoehler
Copy link
Author

ckoehler commented Oct 9, 2020

As to how that may be done, I'd propose skipping the generate_config.sh script altogether, making a Jinja template from the default string in the script (starting here) instead, and populating that with the settings from "Template settings for mailcow.conf". That'd simplify the Ansible code for that part, too, but would make it necessary to maintain two copies of the config generation.

@ntimo
Copy link
Member

ntimo commented Jan 8, 2021

I can truly understand the problem. But I never wanted to have to create a jinja template for the mailcow conf, since this just adds more complexity for the maintenance of the role. I think we should adjust the generate_config.sh to be able to understand a kind of non interactive mode where there is no blocking manual input.

@ckoehler
Copy link
Author

ckoehler commented Jan 8, 2021

I think you're right about that.

@stefanux
Copy link

@ntimo if you really want to keep the script: add a non-interactive mode or detect non-interactive terminals.
its silly to debug since the task hangs a while and than you have to run it interactively yourself.

@Ayowel
Copy link
Contributor

Ayowel commented Mar 31, 2024

Opened a PR on the base repository to try to fix this in the problematic script mailcow/mailcow-dockerized#5804 .

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

No branches or pull requests

4 participants