WIP: ADD roles/icingadb/tasks/install_on_suse.yml

[johanneskastl]

AFAIK adding the Suse installation requires the community.general.zypper module (even though the task says ansible.builtin.package).

Should I add a collections/requirements.yml or a line in the galaxy.yml to make this visible?

fix #231

[cla-bot]

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @johanneskastl

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please contact us if you think this is the case.

• If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.

[cla-bot]

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @johanneskastl

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please contact us if you think this is the case.

• If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.

[bobapple]

@cla-bot check

[johanneskastl]

Should I add a collections/requirements.yml or a line in the galaxy.yml to make this visible?

Any opinions?

[mocdaniel]

What do you think about this, @mkayontour? Since this is an OS-related dependency, it'd feel wrong for me to list it as a 'general' dependency, though I don't know of any other way of doing this...

[mkayontour]

Hey, the documentation on the galaxy.yml "metadata" is sparse. But as reading a few threads it seems, that the dependencies will be managed automatically if listed in galaxy.yml.

I would recommend state it in the documentation to avoid the direct dependency for every OS.

Create something like Known Issues or Install Requirements for Suse somewhere prominent, like Getting Started.

[mkayontour]

The WIP looks fine to me, if you want to prepare a PR please make sure to add a changelog fragment.

We described it here "https://github.com/Icinga/ansible-collection-icinga/blob/main/CONTRIBUTING.md"

Thanks for your work! Appreciate it!

[johanneskastl]

The WIP looks fine to me, if you want to prepare a PR please make sure to add a changelog fragment.

We described it here "https://github.com/Icinga/ansible-collection-icinga/blob/main/CONTRIBUTING.md"

Thanks for your work! Appreciate it!

Thanks for the hint. I'll add this to all of the open PRs on Wednesday (I hope...)

[mkayontour]

Work continues in PR #273.