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

[BUG] Supervised systemd option not working with keydb 6.2 #352

Closed
tchernomax opened this issue Aug 24, 2021 · 3 comments
Closed

[BUG] Supervised systemd option not working with keydb 6.2 #352

tchernomax opened this issue Aug 24, 2021 · 3 comments
Assignees

Comments

@tchernomax
Copy link
Contributor

Describe the bug

It the same bug as #226

basicaly --supervised systemd doesn't work.

To reproduce

# echo "deb https://download.keydb.dev/open-source-dist $(lsb_release -sc) main" | tee /etc/apt/sources.list.d/keydb.list
# wget -O /etc/apt/trusted.gpg.d/keydb.gpg https://download.keydb.dev/open-source-dist/keyring.gpg
# apt update
# apt install keydb
# systemctl stop keydb-server.service
# mkdir -p /etc/systemd/system/keydb-server.service.d/
# cat <<END > /etc/systemd/system/keydb-server.service.d/override.conf
[Service]
Type=notify
ExecStart=
ExecStart=/usr/bin/keydb-server /etc/keydb/keydb.conf --supervised systemd --daemonize no
ExecStop=
PIDFile=
TimeoutStartSec=30
TimeoutStopSec=30
Restart=no
END
# systemctl daemon-reload
# systemctl start keydb-server.service
Job for keydb-server.service failed because a timeout was exceeded.
See "systemctl status keydb-server.service" and "journalctl -xe" for details.

Expected behavior

--supervised systemd should be working.

Additional information

# dpkg -l keydb\*
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name           Version           Architecture Description
+++-==============-=================-============-==================================================================
ii  keydb          6:6.2.0-1+deb10u1 all          Persistent key-value database with network interface (metapackage)
ii  keydb-server   6:6.2.0-1+deb10u1 amd64        Persistent key-value database with network interface
ii  keydb-tools    6:6.2.0-1+deb10u1 amd64        Persistent key-value database with network interface (client)

# lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 10 (buster)
Release:        10
Codename:       buster
@tchernomax
Copy link
Contributor Author

@VivekSainiEQ #296 (which is include in 6.2.0 release) doesn't fix #226
(but I preferred to create a new one instead of re-opening #226 )

@VivekSainiEQ
Copy link
Contributor

Hi @tchernomax,

It looks like the KeyDB builds weren't compiled with systemd support. This will be fixed in the next release, which should be a smaller release coming in the next few weeks. In the interim, if you require systemd support, you can compile KeyDB from source with the flag USE_SYSTEMD=yes.

@VivekSainiEQ VivekSainiEQ self-assigned this Aug 26, 2021
@benschermel
Copy link
Collaborator

All KeyDB builds are now built with the flag USE_SYSTEMD=yes as of current 6.2.1 release. You can enable with --supervised systemd in keydb.conf and using Type=notify in service file. Closing issue.

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

3 participants