Skip to content

IcingaServiceSet: Correctly load the service set#3077

Merged
lippserd merged 1 commit into
masterfrom
fix/add-service-to-service-set-postgres
Jul 1, 2026
Merged

IcingaServiceSet: Correctly load the service set#3077
lippserd merged 1 commit into
masterfrom
fix/add-service-to-service-set-postgres

Conversation

@raviks789

Copy link
Copy Markdown
Collaborator

To correctly load the services for the service set, the service set template where the services where added must be selected correctly.

Fixes #3076

@cla-bot cla-bot Bot added the cla/signed label May 19, 2026
@raviks789 raviks789 requested a review from lippserd May 19, 2026 13:52
@raviks789 raviks789 self-assigned this May 19, 2026
@raviks789 raviks789 added this to the v1.12.0 milestone Jul 1, 2026
@raviks789 raviks789 force-pushed the fix/add-service-to-service-set-postgres branch 4 times, most recently from 348f642 to 451ac87 Compare July 1, 2026 14:08
To correctly load the services for the service set, the service set template where the services
where added must be selected correctly. If the argument $id is the name of the service set,
to correctly retrieve the services linked to the service set, the object_type must be set to 'template'.
As the direct addition of service set to an host creates a row in the service set table in the database
with the object type 'object' and this row is not linked to the services under the service set.
@raviks789 raviks789 force-pushed the fix/add-service-to-service-set-postgres branch from 451ac87 to 19baac3 Compare July 1, 2026 14:13
@lippserd lippserd merged commit be3387b into master Jul 1, 2026
12 checks passed
@lippserd lippserd deleted the fix/add-service-to-service-set-postgres branch July 1, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Modifying directly applied Service Set fails on Postgres

2 participants