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

Restart openvpn-server fails #114

Open
gregewing opened this issue Nov 13, 2024 · 1 comment
Open

Restart openvpn-server fails #114

gregewing opened this issue Nov 13, 2024 · 1 comment
Assignees
Labels
bug Something isn't working fixed

Comments

@gregewing
Copy link

gregewing commented Nov 13, 2024

HI, I'm raising this incase anyone else has a similar problem.

I am using the two containers named as follows:

openvpn-server
openvpn-ui

I have a problem whereby if I don't set the containers to automatically restart when they fail using a docker restart policy, then the UI Restart server button just kills the container. I noticed after writing this that soe of the other events that trigger a conditional restart do seem to work. That said I'm manually rstarting containers as I work stuff out, so I might be masking some unexpected behaviors.

I don't quite know how to fix this, but looking in the restart.sh script I see that it is looking for a container names openvpn and not openvpn-server. It would seem that for this to work, then the naming of containers is very prescriptive.

It would be lovely if there were a way for the container to be restarted regardless of it's name, but I dont know how that might be accomplished.

Perhaps a suitable interim measure would be for the naming convention to be consistently applied to container naming as to image naming, and so hae the script look for openvpn-server.

Thanks

d3vilh added a commit that referenced this issue Dec 27, 2024
@d3vilh d3vilh self-assigned this Dec 27, 2024
@d3vilh d3vilh added bug Something isn't working fixed labels Dec 27, 2024
@d3vilh
Copy link
Owner

d3vilh commented Dec 27, 2024

Hi @gregewing, thank you for the reporting!
It was a bug in new future functionality, now fixed with the putty.

Will be included in next UI release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed
Projects
None yet
Development

No branches or pull requests

2 participants