prometheus-alertmanager: merge_yaml not doing recursive copy of properties
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
kolla-ansible |
Fix Released
|
Medium
|
Jorge Niedbalski |
Bug Description
[Environment]
Kolla-ansible master
[Description]
Change https:/
introduced the usage of merge_yaml for prometheus-
However the merge_yaml module doesn't do a deep copy of >= first level properties, so it doesn't works for most configurations.
Example config:
global:
resolve_timeout: 5m
smtp_require_tls: true
pagerduty_url: https:/
route:
receiver: default-receiver
group_wait: 10s
group_interval: 5m
repeat_interval: 3h
routes:
- match:
severity: 'page'
receiver: xxx-dc-
receivers:
- name: default-receiver
- name: xxx-dc-
pagerduty_
- service_key: xxxx
templates: []
Default template:
global:
resolve_timeout: 5m
smtp_require_tls: true
route:
receiver: default-receiver
group_wait: 10s
group_interval: 5m
repeat_interval: 3h
receivers:
- name: default-receiver
templates: []
Changed in kolla-ansible: | |
status: | New → In Progress |
assignee: | nobody → Jorge Niedbalski (niedbalski) |
Fix proposed to branch: master /review. openstack. org/589966
Review: https:/