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

Add configuration parameters to server and agent #976

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

githubgehrmann
Copy link

Pull Request (PR) description

We'd like to use this Puppet module to deploy our Zabbix infrastructure but to fully configer it for our needs there are missing some parameters. Im not able to fully test my additions, so a code review and/or testing is recommended.

Following parameters where added
  • Server:
    • StartHistoryPollers
    • TrendFunctionCacheSize
  • Agent:
    • StatusPort
    • PersistentBufferPeriod
    • ControlSocket

This Pull Request (PR) fixes the following issues

n.a.

@Valantin
Copy link
Contributor

Valantin commented Oct 2, 2024

Hi, can you add the Type to parameter added in this PR?
Can you squash the commits?

@githubgehrmann githubgehrmann force-pushed the add-configuration-parameters-to-server-and-agent branch 2 times, most recently from f4f8610 to a05e0cd Compare October 7, 2024 10:33
@bastelfreak bastelfreak added the enhancement New feature or request label Oct 7, 2024
@githubgehrmann githubgehrmann force-pushed the add-configuration-parameters-to-server-and-agent branch from a05e0cd to 2899ecb Compare October 7, 2024 11:50
- Server:
   - making StartHistoryPollers configurable
   - adding TrendFunctionCacheSize parameter to config file

- Agent:
   - adding controlsocket and persistantbuffer parameters
@githubgehrmann githubgehrmann force-pushed the add-configuration-parameters-to-server-and-agent branch from 2899ecb to 8b86dea Compare October 7, 2024 12:58
@githubgehrmann
Copy link
Author

All added parameters have a type specified.
The commits are squashed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants