Bug 274711

Summary: www/garage: boot stuck due to garage not detaching from terminal
Product: Ports & Packages Reporter: vedad
Component: Individual Port(s)Assignee: Ashish SHUKLA <ashish>
Status: Closed FIXED    
Severity: Affects Only Me CC: vedad
Priority: --- Flags: bugzilla: maintainer-feedback? (ashish)
Version: Latest   
Hardware: Any   
OS: Any   

Description vedad 2023-10-25 10:47:44 UTC

    
Comment 1 vedad 2023-10-25 10:54:05 UTC
Hi,

(I've mistakenly submitted empty report, providing details here).

Garage doesn't detach from the terminal, resulting in /etc/rc boot process being stuck.

The garage team does not intend to change this behavior:

https://git.deuxfleurs.fr/Deuxfleurs/garage/issues/661

Another issue is that the rc.d script REQUIREs NETWORKING, which doesn't allow services such as VPN/tunnels (eg. wireguard) to run and up the interfaces garage might be using.

Thanks,
Kind regards,
Comment 2 commit-hook freebsd_committer freebsd_triage 2024-05-16 22:18:35 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=f923df1176eb6d8a38bb9ef465bb1b6388ca817f

commit f923df1176eb6d8a38bb9ef465bb1b6388ca817f
Author:     Ashish SHUKLA <ashish@FreeBSD.org>
AuthorDate: 2024-05-16 14:51:31 +0000
Commit:     Ashish SHUKLA <ashish@FreeBSD.org>
CommitDate: 2024-05-16 22:18:21 +0000

    www/garage: Update to 1.0.0

    - Update rc.d script to use daemon(8) to daemonize[0]
    - Add garage_log_file rc.conf parameter for log file

    PR:             274711 [0]
    Reported by:    vedad@kajtaz.net [0]

 www/garage/Makefile        |   3 +-
 www/garage/Makefile.crates | 489 ++++++++++++----------
 www/garage/distinfo        | 984 +++++++++++++++++++++++++--------------------
 www/garage/files/garage.in |  11 +-
 4 files changed, 814 insertions(+), 673 deletions(-)
Comment 3 Ashish SHUKLA freebsd_committer freebsd_triage 2024-05-16 22:34:22 UTC
Sorry for late attention to this problem. Unfortunately, I didn't get the notification for this PR.

I think the first part is taken care of by the commit.

I don't know what to do about the second part though, since this does require NETWORKING to be available as you notice.
Comment 4 vedad 2024-09-25 13:10:11 UTC
(In reply to Ashish SHUKLA from comment #3)

Hi,

regarding REQUIRE in rc.d script, both apache and nginx use:

REQUIRE: LOGIN (...)

So perhaps that would be appropriate. That allows VPN/tunnel services such as wireguard to be rcorder'd before garage.