Skip to content

Commit

Permalink
fix(charts/simple-app): Update istio-alerts (#293)
Browse files Browse the repository at this point in the history
This propagate the istio-alerts update. It must be done as a separate PR
and only merged after due to the use of http for referencing the charts
repo
  • Loading branch information
corydolphin authored Apr 18, 2024
1 parent 83b4e13 commit 337d9ba
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 4 deletions.
4 changes: 2 additions & 2 deletions charts/simple-app/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,14 @@ apiVersion: v2
name: simple-app
description: Default Microservice Helm Chart
type: application
version: 1.6.6
version: 1.7.0
appVersion: latest
maintainers:
- name: diranged
email: [email protected]
dependencies:
- name: istio-alerts
version: 0.2.0
version: 0.4.0
repository: https://k8s-charts.nextdoor.com
condition: istio-alerts.enabled
- name: nd-common
Expand Down
11 changes: 9 additions & 2 deletions charts/simple-app/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

Default Microservice Helm Chart

![Version: 1.6.6](https://img.shields.io/badge/Version-1.6.6-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: latest](https://img.shields.io/badge/AppVersion-latest-informational?style=flat-square)
![Version: 1.7.0](https://img.shields.io/badge/Version-1.7.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: latest](https://img.shields.io/badge/AppVersion-latest-informational?style=flat-square)

[deployments]: https://kubernetes.io/docs/concepts/workloads/controllers/deployment/
[hpa]: https://kubernetes.io/docs/tasks/run-application/horizontal-pod-autoscale/
Expand All @@ -13,6 +13,13 @@ defaults for you like the Kubernetes [Horizontal Pod Autoscaler][hpa].

## Upgrade Notes

### 1.2.x -> 1.7.x

**BREAKING: Istio Alerts have changed**

The Istio Alerts chart was updated to 4.0.0 which updates the alert on the 5XX
rate to only aggregate per service, rather than including the client source workload.

### 1.1.2 -> 1.2.x

**BREAKING: Istio Alerts have changed**
Expand Down Expand Up @@ -319,7 +326,7 @@ secretsEngine: sealed
| Repository | Name | Version |
|------------|------|---------|
| file://../nd-common | nd-common | 0.3.1 |
| https://k8s-charts.nextdoor.com | istio-alerts | 0.2.0 |
| https://k8s-charts.nextdoor.com | istio-alerts | 0.4.0 |

## Values

Expand Down
8 changes: 8 additions & 0 deletions charts/simple-app/README.md.gotmpl
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,14 @@ defaults for you like the Kubernetes [Horizontal Pod Autoscaler][hpa].

## Upgrade Notes


### 1.2.x -> 1.7.x

**BREAKING: Istio Alerts have changed**

The Istio Alerts chart was updated to 4.0.0 which updates the alert on the 5XX
rate to only aggregate per service, rather than including the client source workload.

### 1.1.2 -> 1.2.x

**BREAKING: Istio Alerts have changed**
Expand Down

0 comments on commit 337d9ba

Please sign in to comment.