Skip to content

install: restart guacd when it is running, not only rustguac - #239

Open
pletch wants to merge 1 commit into
sol1:mainfrom
pletch:fix/install-restart-guacd
Open

pletch wants to merge 1 commit into
sol1:mainfrom
pletch:fix/install-restart-guacd

Conversation

@pletch

@pletch pletch commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

make install replaces guacd's binary and libraries while the old guacd is still running, and the end of install.sh restarted rustguac only. Requires=rustguac-guacd.service starts a stopped guacd but never restarts a running one, so an upgrade rebuilt guacd and then kept serving the previous build until someone ran systemctl restart rustguac-guacd by hand. From the outside that looks exactly like a guacd change that did nothing.

This restarts rustguac-guacd whenever it is active, ahead of rustguac, and stops the install with a pointer to the journal if it does not come back. No sessions are cut off by it: they go through rustguac, which is already stopped for the upgrade or was never running. A first install still leaves both services stopped, as before, so an admin can be created first.

Tested with a full ./install.sh --no-deps --no-drive over a running install on Debian 13: both units restarted (guacd a second ahead of rustguac), and the running guacd's /proc/<pid>/exe points at the new binary rather than a deleted one.

make install replaces guacd's binary and libraries while the old guacd is
still running, and the end of the install restarted rustguac alone.
Requires= starts a stopped guacd but never restarts a running one, so an
upgrade rebuilt guacd and then went on serving the previous build until
someone restarted it by hand -- which looks exactly like a guacd change
that did nothing.

guacd is now restarted, ahead of rustguac, whenever it is active. Nothing
is cut off by it: sessions go through rustguac, which is already stopped
for the upgrade or was never running. A first install still leaves both
stopped.
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

Successfully merging this pull request may close these issues.

1 participant