Skip to content

Send whippet's STDERR to both the journal and the console#858

Closed
arnaldo2792 wants to merge 1 commit intodevelopfrom
whippet-logging/core-kit
Closed

Send whippet's STDERR to both the journal and the console#858
arnaldo2792 wants to merge 1 commit intodevelopfrom
whippet-logging/core-kit

Conversation

@arnaldo2792
Copy link
Contributor

Description of changes:

Send whippet's STDERR to both the journal and the console

Testing done:

Details
[root@admin]# sheltie systemctl status whippet.service
● whippet.service - D-Bus System Message Bus
     Loaded: loaded (/x86_64-bottlerocket-linux-gnu/sys-root/usr/lib/systemd/system/whippet.service; enabled; preset: enabled)
    Drop-In: /x86_64-bottlerocket-linux-gnu/sys-root/usr/lib/systemd/system/service.d
             └─00-aws-config.conf
     Active: active (running) since Fri 2026-03-06 21:25:56 UTC; 2min 46s ago
 Invocation: 71e9e6417a0249ed8ab7275467592b4d
TriggeredBy: ● dbus.socket
   Main PID: 1518 (whippet)
      Tasks: 10 (limit: 37593)
     Memory: 6.2M (peak: 6.9M)
        CPU: 148ms
     CGroup: /system.slice/whippet.service
             ├─1518 /usr/bin/whippet
             └─1537 /usr/bin/dbus-broker --log 12 --controller 11 --machine-id ec294d8464c32b55954ee4625e3e7410 --max-bytes 536870912 --max-fds 4096 --max-matches 16384

Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] read_commands; n_commands=1
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] Using listener socket FD: 3
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] Setting broker listener socket and policy
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] socket reader;
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] read_socket;
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] read_socket;
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] Done setting broker listener socket and policy
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] Policy applied successfully
Mar 06 21:25:56 localhost whippet[1518]: 21:25:56 [INFO] Notified systemd that service is ready
Mar 06 21:25:56 localhost systemd[1]: Started D-Bus System Message Bus.
[root@admin]# sheltie systemctl cat whippet.service
# /x86_64-bottlerocket-linux-gnu/sys-root/usr/lib/systemd/system/whippet.service
[Unit]
Description=D-Bus System Message Bus
DefaultDependencies=false
# Ensure the dbus user is created before starting dbus service
After=dbus.socket systemd-sysusers.service
Wants=dbus.socket systemd-sysusers.service
Before=basic.target shutdown.target
Requires=dbus.socket
Conflicts=shutdown.target

[Service]
User=dbus
Type=notify
Sockets=dbus.socket
OOMScoreAdjust=-900
LimitNOFILE=16384
ProtectSystem=full
# Disable private /tmp to avoid dependency on systemd-tmpfiles and consequently
# local-fs.target, allowing dbus to start earlier in the boot
PrivateTmp=no
PrivateDevices=true
ExecStart=/usr/bin/whippet
StandardError=journal+console

[Install]
Alias=dbus.service
WantedBy=preconfigured.target

# /x86_64-bottlerocket-linux-gnu/sys-root/usr/lib/systemd/system/service.d/00-aws-config.conf
[Service]
# Set the AWS_SDK_LOAD_CONFIG system-wide instead of at the individual service
# level, to make sure new system services that use the AWS SDK for Go read the
# shared AWS config
Environment=AWS_SDK_LOAD_CONFIG=true

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
@bcressey
Copy link
Contributor

bcressey commented Mar 9, 2026

What do the console logs show in the event of an error?

@arnaldo2792
Copy link
Contributor Author

I'll be closing this, and provide the requested logs in a different PR.

@arnaldo2792 arnaldo2792 closed this Mar 9, 2026
@arnaldo2792 arnaldo2792 deleted the whippet-logging/core-kit branch March 9, 2026 18:21
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.

6 participants