ansible-collection-icinga icon indicating copy to clipboard operation
ansible-collection-icinga copied to clipboard

Refactor task names for clarity and consistency across Icinga2 role

Open lutin-malin opened this issue 9 months ago • 7 comments

Hi icinga team, I've been trying to troubleshoot a deployment issue (only on my side) and ended up looking at your collection. I found some improvments to be performed, so i took the freedom to give it a try. Here you find my proposal.

In a nutshell, i have :

  • moved all modules call to fqdn
  • prefixed internal variables with __icinga2 to reduce its exposure to extern variable or any unwanted superseed
  • implemented the argument_specs for role variable validation
  • done some linting
  • rearranged the defaults/vars where applicable.

I carefully bear attention to not touch the business logic being executed.

Also, this would be the 1st stone to build automated documentation for antsibull and ansible-doc. (cf #321 )

So far, i've done it only on the icinga2 role. If this suits you, i could do further on the remaining roles.

Let me know your thoughts.

lutin-malin avatar Mar 12 '25 15:03 lutin-malin

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar Mar 12 '25 15:03 cla-bot[bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar Mar 12 '25 15:03 cla-bot[bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar Mar 12 '25 16:03 cla-bot[bot]

@bobapple you can retrigger the cla-bot here as well

lutin-malin avatar Mar 13 '25 10:03 lutin-malin

@cla-bot check

bobapple avatar Mar 20 '25 14:03 bobapple

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar Mar 20 '25 14:03 cla-bot[bot]

Seems like there's a misconfiguration in your git client. If you follow the steps from cla-bot you should be able to fix it.

bobapple avatar Mar 20 '25 14:03 bobapple

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar May 14 '25 16:05 cla-bot[bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar May 14 '25 16:05 cla-bot[bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL, lutin_malin. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar May 14 '25 16:05 cla-bot[bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL, lutin_malin. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar May 14 '25 16:05 cla-bot[bot]

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL, lutin_malin. This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

cla-bot[bot] avatar May 14 '25 16:05 cla-bot[bot]