| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Introduces a new 'yaml' strategy using Ansible's `from_yaml` to read
target files. This allows users to group multiple targets with
distinct label sets (e.g., severity or job) within a single file,
optimizing Prometheus file-based service discovery.
- Refactored task structure into `strategy_*.yml` for consistency.
- Extracted shared exporter variable computation.
- Implemented label merging where exporter labels override defaults.
- Included support for groups and hosts without labels to prevent crashes.
PR: (#1) https://github.com/kliwniloc/ansible-role-prometheus-target/pull/1
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
| |
|
|
| |
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before you could do multiple hosts per exporter like this:
- id: node_exporter
hosts:
- host1
- host2
This was removed since it simplifies the code quite a bit and you can
still archive the same behavior like this:
- { id: node_exporter, host: host1 }
- { id: node_exporter, host: host2 }
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
|
|
Signed-off-by: Colin Wilk <colin.wilk@tum.de>
|
