Skip to content

2024.2.0.0-b53

@mchiddy mchiddy tagged this 10 Oct 18:30
Summary:
Original commit: ac5ac5bdf553f162a953e6c153e29d7bd8ef359f / D38644
Original commit: 8920a4d7ec47142dd1b71b5779b462c4e395c6c8 / D38761
We were not passing the prometheus log file anywhere to actually write the logs but were showing it in the status output which was kind of confusing. This diff adds the StandardOutput and StandardError redirection fields directly to the service file.

NOTE: This only works with systemd >= 236 which should cover most modern linux distros. If the version is < 240 we will use file mode which will replace the file on each restart of the service, but at least it's logged to a file in this case.

Test Plan: install yba and make sure prometheus logs present

Reviewers: dshubin, sanketh

Reviewed By: dshubin

Subscribers: svc_phabricator, yugaware

Differential Revision: https://phorge.dev.yugabyte.com/D38867
Assets 2
Loading