Add configuration parameters to server, proxy and agent #976
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hi, can you add the Type to parameter added in this PR? |
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
2 times, most recently
from
f4f8610 to
a05e0cd
Compare
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
2 times, most recently
from
2899ecb to
8b86dea
Compare
|
All added parameters have a type specified. |
|
Hello, |
|
Hello, |
Valantin
requested changes
Nov 23, 2024
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
2 times, most recently
from
8b2c580 to
ba8c193
Compare
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
from
ba8c193 to
72ce78e
Compare
githubgehrmann
changed the title
|
Can you fix the failed check? |
|
Trying to. I do not have the plattform for local testing and am not that familiar with puppet errors. |
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
2 times, most recently
from
db41b0a to
8ede922
Compare
Valantin
requested changes
May 15, 2025
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
10 times, most recently
from
8774123 to
db4f043
Compare
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
from
db4f043 to
0c2853d
Compare
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
3 times, most recently
from
0825093 to
7b01b70
Compare
All other checks fail, in no context to this code |
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
3 times, most recently
from
4375014 to
34be9e4
Compare
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
2 times, most recently
from
c4ab83d to
9c1f6bb
Compare
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
from
9c1f6bb to
d79ff41
Compare
- Server:
- making StartHistoryPollers configurable
- adding TrendFunctionCacheSize parameter to config file
- Agent:
- adding StatusPort parameter to config file
- adding PersistentBufferPeriod parameter to config file
- adding ControlSocket parameter to config file
- Proxy
- making ProxyBufferMode configurable
- making ProxyMemoryBufferSize configurable
- making ProxyMemoryBufferAge configurable
return parameters to a desired state
githubgehrmann
force-pushed
the
add-configuration-parameters-to-server-and-agent
branch
from
d79ff41 to
68cba90
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
This Pull Request (PR) fixes the following issues
n.a.