Support multiple UPS instances and per-UPS systemd services (Debian 12+)#12
Open
tdy91 wants to merge 3 commits intosaltstack-formulas:masterfrom
Open
Support multiple UPS instances and per-UPS systemd services (Debian 12+)#12tdy91 wants to merge 3 commits intosaltstack-formulas:masterfrom
tdy91 wants to merge 3 commits intosaltstack-formulas:masterfrom
Conversation
javierbertoli
approved these changes
Jun 23, 2025
Member
javierbertoli
left a comment
javierbertoli
left a comment
There was a problem hiding this comment.
Hey, @tdy91 !
Thanks for this, it looks like a great addition!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds support for systems with multiple UPS units configured via pillars.
It adjusts the
map.jinjalogic to generate one systemdnut-driver@<ups>service per UPS instance on Debian 12+, where the packaging now uses per-unit services.For compatibility, systems running Debian <=11 continue to use the legacy
nut-driverservice.The
service.runninglogic innut/server/service/running.slshas been updated to loop over all generated services for multi-UPS setups.Tested on:
nut-driver)nut-driver@<ups>)This change ensures backward compatibility with existing setups while aligning with systemd service changes in newer distributions.