-
Notifications
You must be signed in to change notification settings - Fork 35
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
Feature/icingaweb module graphite #199
base: main
Are you sure you want to change the base?
Conversation
Hi @mkayontour, there's already a release date for this feature? Thanks |
Sorry, I don't have a fixed release date - as I'm working on the collection only at spare times. I try to prepare a release before the summit. I can't promise you anything right now. |
I did a bit of cleanup and checked if the code is still compatible. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
converge
needs the fix, the doc suggestion is...well, a suggestion.
LGTM 👍🏻
@@ -3,6 +3,54 @@ | |||
- name: Converge | |||
hosts: all | |||
vars: | |||
icingaweb2_modules: | |||
graphite: | |||
enabled: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
source
missing
enabled: true | |
enabled: true | |
source: package |
| insecure | number | "1" | | ||
|
||
#### Section ui | ||
|
||
| Variable | Type | Example | | ||
|-------------------------|--------|---------| | ||
| default_time_range | number | "1" | | ||
| defautl_time_range_unit | string | "hours" | | ||
| disable_no_graphs_found | bool | "0"/"1" | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
insecure
and disable_no_graphs_found
should both either be number
(or int
) or bool
, for consistency since both are essentially toggles.
This adds the feature to install and configure Icinga Web 2 Module Graphite.