Skip to content

Commit

Permalink
[stable/prometheus-operator] Re-sync prometheus rules and grafana das…
Browse files Browse the repository at this point in the history
…hboards (#21248)

* Re-sync prometheus rules

Signed-off-by: iuri aranda <[email protected]>

* Manually fix special notiation in prometheus rules

I see that this was previously manually fixed as well, so I kept it like before

Signed-off-by: iuri aranda <[email protected]>

* Bump chart version

Signed-off-by: iuri aranda <[email protected]>

* Bump patch version

Signed-off-by: iuri aranda <[email protected]>

* Fix Helm interpolation escaping in sync scripts

This should fix the issue of having a backtick (`) inside a {{ ... }} command block in prometheus syntax.

As per helm/charts#21248 (comment)

Signed-off-by: iuri aranda <[email protected]>

* Bump patch version

Signed-off-by: iuri aranda <[email protected]>

* Remove unnecessary whitespace

Signed-off-by: iuri aranda <[email protected]>

* Bump patch version

Signed-off-by: iuri aranda <[email protected]>

* Bump prometheus-operator chart patch version

Signed-off-by: iuri aranda <[email protected]>

* Update prometheus rules gain

Signed-off-by: iuri aranda <[email protected]>

* Sync grafana dashboards as well

Signed-off-by: iuri aranda <[email protected]>

* Bump patch version for prometheus-operator chart

Signed-off-by: iuri aranda <[email protected]>
  • Loading branch information
iuriaranda authored and Endre Czirbesz committed Aug 14, 2020
1 parent 3ad2c9f commit 739e0e3
Show file tree
Hide file tree
Showing 72 changed files with 3,289 additions and 1,399 deletions.
2 changes: 1 addition & 1 deletion charts/prometheus-operator/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ sources:
- https://github.com/coreos/kube-prometheus
- https://github.com/coreos/prometheus-operator
- https://coreos.com/operators/prometheus
version: 8.12.4
version: 8.12.5
appVersion: 0.37.0
tillerVersion: ">=2.12.0"
home: https://github.com/coreos/prometheus-operator
Expand Down
2 changes: 1 addition & 1 deletion charts/prometheus-operator/hack/sync_grafana_dashboards.py
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ def init_yaml_styles():


def escape(s):
return s.replace("{{", "{{`{{").replace("}}", "}}`}}")
return s.replace("{{", "{{`{{").replace("}}", "}}`}}").replace("{{`{{", "{{`{{`}}").replace("}}`}}", "{{`}}`}}")


def yaml_str_repr(struct, indent=2):
Expand Down
4 changes: 2 additions & 2 deletions charts/prometheus-operator/hack/sync_prometheus_rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ def new_representer(dumper, data):
}

# standard header
header = '''{{- /*
header = '''{{- /*
Generated from '%(name)s' group from %(url)s
Do not change in-place! In order to change this file first read following link:
https://github.com/helm/charts/tree/master/stable/prometheus-operator/hack
Expand Down Expand Up @@ -153,7 +153,7 @@ def init_yaml_styles():


def escape(s):
return s.replace("{{", "{{`{{").replace("}}", "}}`}}")
return s.replace("{{", "{{`{{").replace("}}", "}}`}}").replace("{{`{{", "{{`{{`}}").replace("}}`}}", "{{`}}`}}")


def fix_expr(rules):
Expand Down

Large diffs are not rendered by default.

Loading

0 comments on commit 739e0e3

Please sign in to comment.