Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: add Suse support for the monitoring_plugins role #236

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

johanneskastl
Copy link
Contributor

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?

And: Someone more experienced than me should maybe go over the list of available plugins. There are a lot more available. And I have not yet tested whether all that pointed to the tcp plugin in other vars files are also valid on Suse.

Copy link

cla-bot bot commented Dec 7, 2023

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.

@johanneskastl johanneskastl force-pushed the 20231206_add_Suse_support_monitoring_plugins_role branch from 473ffa1 to b435684 Compare December 17, 2023 16:42
Copy link

cla-bot bot commented Dec 17, 2023

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
Copy link
Member

@cla-bot check

@cla-bot cla-bot bot added the cla/signed label Dec 18, 2023
@johanneskastl
Copy link
Contributor Author

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

Any opinions?

@mkayontour mkayontour mentioned this pull request Jan 17, 2024
8 tasks
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.

2 participants