community.general icon indicating copy to clipboard operation
community.general copied to clipboard

icinga2_host - ip should not be required

Open MarcusCaepio opened this issue 1 year ago • 3 comments

Summary

Hi all, as one can see in https://icinga.com/docs/icinga-2/latest/doc/09-object-types/#host the address variable is not mandatory, so IP should be optional in the plugin, too.

Issue Type

Bug Report

Component Name

icinga2_host

Ansible Version

$ ansible --version
2.11.4

Community.general Version

$ ansible-galaxy collection list community.general
5.5.0

Configuration


OS / Environment

Ubuntu 22.04

Steps to Reproduce

Try to create a host without given an IP

Expected Results

Address is optionally

Actual Results

Address is mandatory

Code of Conduct

  • [X] I agree to follow the Ansible Code of Conduct

MarcusCaepio avatar Sep 06 '22 06:09 MarcusCaepio

Files identified in the description:

If these files are incorrect, please update the component name section of the description or use the !component bot command.

click here for bot help

ansibullbot avatar Sep 06 '22 07:09 ansibullbot

cc @t794104 click here for bot help

ansibullbot avatar Sep 06 '22 07:09 ansibullbot

Hi @MarcusCaepio thanks for submitting this issue. Would you be willing to submit a PR to fix that? It seems to be somewhat small a patch. We would be more than happy to help you through.

russoz avatar Sep 11 '22 04:09 russoz

Files identified in the description:

If these files are incorrect, please update the component name section of the description or use the !component bot command.

click here for bot help

ansibullbot avatar Nov 06 '22 05:11 ansibullbot