Skip to content

Conversation

Donien
Copy link
Member

@Donien Donien commented Oct 2, 2025

This PR adds a role for Icinga for Windows (ifw).

The role can install the Icinga PowerShell Framework, manage repositories and components, and configure Icinga 2 via the framework.

@Donien Donien requested a review from losten-git October 2, 2025 12:17
@Donien Donien self-assigned this Oct 2, 2025
Copy link
Member

@dgoetz dgoetz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I managed to run in a big problem by adjusting ifw_icinga2_cn from hostname to fqdn which resulted in an installation command which IfW did not like. When running it manually it dropped back to the wizard which is very likely same resulting in a hanging Ansible run. So this should be fixed and requires a time out to prevent such a situation in the future.

Then I would like a design change as current defaults of the role are based on current defaults of IfW which are based on minimum functionality. I would prefer it to use defaults based in best practices as this would give a better user experience and if need for some reason changing back to less would be possible for the user. So I would like to see JEA manageable and by default being enabled and the background daemon being installed and set up so usage of the API is possible resulting in better performance and check results. I am also not sure if we should allow setting the user with the initial release instead of having it as a Todo, especially when adding JEA.

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.

2 participants