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

Fix systemd restart policy #1618

Merged
merged 1 commit into from
Aug 8, 2023
Merged

Fix systemd restart policy #1618

merged 1 commit into from
Aug 8, 2023

Conversation

MrStonedOne
Copy link
Member

@MrStonedOne MrStonedOne commented Aug 8, 2023

SystemD enum values are case sensitive. Setting this to Always was doing nothing, it has to be always

[Release Notes]:

🆑 Host Watchdog
Fix capitalization issue in systemd unit file that was keeping systemd from restarting tgs in certain cases.
/:cl:

[Why]: This not being always killed campbell when systemd killed the tgs service for OOM.

These are case sensitive,
@MrStonedOne MrStonedOne changed the title Fix restart policy Fix systemd restart policy Aug 8, 2023
@Cyberboss Cyberboss added Fix Fixes incorrect functionality Host Watchdog Change A change to one of the Host./<Watchdog/Service/Console> projects which do not receive auto updates Posix Only Issue only applies to non-Windows deployments labels Aug 8, 2023
@Cyberboss Cyberboss added this to the v5.13.5 milestone Aug 8, 2023
@codecov
Copy link

codecov bot commented Aug 8, 2023

Codecov Report

Merging #1618 (83dbe1b) into master (1f8973c) will decrease coverage by 0.01%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1618      +/-   ##
==========================================
- Coverage   95.01%   95.01%   -0.01%     
==========================================
  Files         606      606              
  Lines      126346   126346              
==========================================
- Hits       120051   120046       -5     
- Misses       6295     6300       +5     

@Cyberboss Cyberboss merged commit a80e606 into master Aug 8, 2023
117 checks passed
@Cyberboss Cyberboss deleted the MrStonedOne-patch-1 branch August 8, 2023 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fixes incorrect functionality Host Watchdog Change A change to one of the Host./<Watchdog/Service/Console> projects which do not receive auto updates Posix Only Issue only applies to non-Windows deployments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants