| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Cover lineinfile and YAML behavior for default exporters, skipped
defaults, exporters without IDs, duplicate definitions, path and host
overrides, path-prefix precedence, and inherited label overrides.
Verify YAML handling of null documents, multiple files, labeled and
unlabeled groups, and duplicate targets.
Add test coverage for command and shell handlers using run-once,
per-host, disabled, and non-become configurations.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The YAML strategy performed a separate read-modify-write operation for
every managed host. Because these tasks were delegated to the same
Prometheus host, parallel Ansible forks could read the same original
file before any fork wrote its update.
Each fork then generated YAML containing only its own target. Although
the copy module atomically replaced the file, it did not make the full
read-modify-write sequence atomic. The last fork to write therefore
overwrote targets added by earlier forks.
Collect exporter updates from all hosts through hostvars, process them
in one run_once operation, and write each distinct target file once.
This preserves every target without requiring users to set serial.
Add parallel coverage for multiple target files and label groups, and
reset the fixtures on every convergence so stale files cannot hide the
race.
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds the run_once option for the two handler types (shell and
command). The variables `prometheus_target_handler_command_run_once`
and `prometheus_target_handler_shell_run_once` are introduced to
control this behavior.
For this change a switch from the Ansible Handlers to regular tasks was
necessary to enforce run_once behavior with delegated hosts. If you
don't rely on handler specific features such as deferring, nothing
changes when using the role.
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The id property of the exporter is used for checking the default values
specified in `prometheus_target_exporter_defaults` variable.
However when both the `host` and the `path` fields are specified
manually in the `prometheus_target_exporter` list this id is never
looked up.
The way the templating was handled however, still resulted in an error
when leaving out the id field.
The templating code was now rewritten to allow for the following
exporter configuraion to run:
prometheus_target_exporter:
- { host: exporter_without_id, path: /opt/simple_target4.yml }
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
| |
|
|
|
|
|
|
| |
This will decrease the amount of redundant configuration for users by
allowing them to specify a general prefix on global / exporter / play
level
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
| |
|
|
| |
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
| |
|
|
|
|
| |
Add more distros to CI/CD and include them in the galaxy meta file.
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will change indentation in target files like:
- labels:
exporter: node
targets:
- host:9100
to
- labels:
exporter: node
targets:
- host:9100
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
| |
|
|
| |
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
|
|
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
