Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

plugin prevents successful weave launch on reboot #1869

Closed
rade opened this issue Jan 10, 2016 · 4 comments
Closed

plugin prevents successful weave launch on reboot #1869

rade opened this issue Jan 10, 2016 · 4 comments
Assignees
Milestone

Comments

@rade
Copy link
Member

rade commented Jan 10, 2016

The plugin is started with a restart policy of always. Consequently, if you have a startup script that runs weave launch, such as our published systemd, it will fail the 2nd time round, since weave launch will report weaveplugin is already running; you can stop it with 'weave stop-plugin'. and exit with status 1.

@rade rade added this to the 1.4.2 milestone Jan 10, 2016
@rade rade changed the title plugin prevents successful weave launch on restart plugin prevents successful weave launch on reboot Jan 10, 2016
@bboreham
Copy link
Contributor

If there was a $WEAVEPLUGIN_DOCKER_ARGS, then folks who didn't want the restart policy could put --restart=none there.

@bboreham
Copy link
Contributor

Although that may not work work if you actually use the plugin; I've seen Docker get stuck trying to talk to the plugin before starting to accept commands.

@DanielDent
Copy link

In a configuration I've been building, I've gone with 4 systemd unit files, plus an environment file, which I've attached.

These files might be helpful for #1836 too.

The only problem with this arrangement is that the weave-plugin service doesn't start cleanly on a reboot because launch-plugin exits uncleanly. Perhaps a "--ifnotlaunched" or similar flag could be made available so that the weave script returns 0 if the plugin is already running.

weave-config.txt
weave-proxy.service.txt
weave-expose.service.txt
weave-plugin.service.txt
weave-router.service.txt

@bboreham bboreham self-assigned this Jan 11, 2016
awh added a commit that referenced this issue Jan 13, 2016
Allow plugin to keep running on 'weave launch'
@awh
Copy link
Contributor

awh commented Jan 13, 2016

Fixed by b3a29be

@awh awh closed this as completed Jan 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants