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

start service only if network is really up #12

Merged
merged 1 commit into from
Jul 24, 2021

Conversation

bastelfreak
Copy link
Member

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

@bastelfreak bastelfreak added the bug Something isn't working label Jul 24, 2021
@bastelfreak bastelfreak self-assigned this Jul 24, 2021
mosquitto is a bit picky about the network setup. It refuses to start
until an ip config is present:

```
-- Journal begins at Mon 2021-05-24 13:52:31 UTC, ends at Sat 2021-07-24 11:53:34 UTC. --
Jul 24 11:22:08 server systemd[1]: Started Mosquitto MQTT Broker daemon.
Jul 24 11:22:08 server mosquitto[318]: 1627125728: mosquitto version 2.0.11 starting
Jul 24 11:22:08 server mosquitto[318]: 1627125728: Config loaded from /etc/mosquitto/mosquitto.conf.
Jul 24 11:22:08 server mosquitto[318]: 1627125728: Opening ipv4 listen socket on port 1883.
Jul 24 11:22:08 server mosquitto[318]: 1627125728: Warning: Interface enp3s0 does not support IPv4 configuration.
Jul 24 11:22:08 server mosquitto[318]: 1627125728: Opening ipv6 listen socket on port 1883.
Jul 24 11:22:08 server mosquitto[318]: 1627125728: Warning: Interface enp3s0 does not support IPv6 configuration.
Jul 24 11:22:08 server systemd[1]: mosquitto.service: Main process exited, code=exited, status=1/FAILURE
Jul 24 11:22:08 server systemd[1]: mosquitto.service: Failed with result 'exit-code'.
```

This commit fixes this and tell mosquitto to wait until the network is
present.
@bastelfreak bastelfreak merged commit 18c444b into voxpupuli:master Jul 24, 2021
@bastelfreak bastelfreak deleted the network branch July 24, 2021 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants