Skip to content

Commit

Permalink
Feat/install upgrade chart (#394)
Browse files Browse the repository at this point in the history
* feat(cicd): test install and upgrade of charts

* feat(chart): update version and images

* fix(chart): extra space
  • Loading branch information
paologallinaharbur authored Apr 11, 2022
1 parent 7f4c3b3 commit 7ed79fe
Show file tree
Hide file tree
Showing 7 changed files with 88 additions and 9 deletions.
27 changes: 27 additions & 0 deletions .github/workflows/push_pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,9 @@ jobs:
name: Helm chart Lint
runs-on: ubuntu-latest
timeout-minutes: 10
strategy:
matrix:
kubernetes-version: [ "v1.16.15", "v1.22.0" ]
steps:
- uses: actions/checkout@v2
with:
Expand Down Expand Up @@ -65,6 +68,30 @@ jobs:
echo "No unit tests found for $chart"
fi
done
- name: Setup Minikube
uses: manusa/[email protected]
if: steps.list-changed.outputs.changed == 'true'
with:
minikube version: v1.20.0
kubernetes version: ${{ matrix.kubernetes-version }}
github token: ${{ secrets.GITHUB_TOKEN }}
- uses: actions/setup-go@v2
if: steps.list-changed.outputs.changed == 'true'
with:
go-version: '1.17'
- name: Create image for chart testing
if: steps.list-changed.outputs.changed == 'true'
run: |
GOOS=linux GOARCH=amd64 make compile
DOCKER_BUILDKIT=1 docker build -t e2e/nri-kubernetes:test .
minikube image load e2e/nri-kubernetes:test
kubectl create ns ct
- name: Test install charts
if: steps.list-changed.outputs.changed == 'true'
run: ct install --namespace ct --config .github/ct.yaml --debug
- name: Test upgrade charts
if: steps.list-changed.outputs.changed == 'true'
run: ct install --namespace ct --config .github/ct.yaml --debug --upgrade

test:
name: Run tests
Expand Down
42 changes: 42 additions & 0 deletions charts/internal/e2e-resources/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
# e2e-resources

![Version: 1.2.0-devel](https://img.shields.io/badge/Version-1.2.0--devel-informational?style=flat-square)

This chart creates e2e resources for nri-kubernetes.

## Maintainers

| Name | Email | Url |
| ---- | ------ | --- |
| alvarocabanas | | https://github.com/alvarocabanas |
| carlossscastro | | https://github.com/carlossscastro |
| gsanchezgavier | | https://github.com/gsanchezgavier |
| kang-makes | | https://github.com/kang-makes |
| paologallinaharbur | | https://github.com/paologallinaharbur |
| roobre | | https://github.com/roobre |

## Requirements

| Repository | Name | Version |
|------------|------|---------|
| https://kubernetes.github.io/kube-state-metrics | kube-state-metrics | 2.13.2 |

## Values

| Key | Type | Default | Description |
|-----|------|---------|-------------|
| daemonSet.enabled | bool | `true` | |
| deployment.enabled | bool | `true` | |
| hpa.enabled | bool | `true` | |
| loadBalancerService.annotations | object | `{}` | |
| loadBalancerService.enabled | bool | `true` | |
| loadBalancerService.fakeIP | string | `""` | |
| pending.enabled | bool | `true` | |
| persistentVolume.enabled | bool | `false` | |
| persistentVolumeClaim.enabled | bool | `false` | |
| persistentVolumeClaim.storageClassName | string | `"standard"` | |
| scraper.enabled | bool | `false` | |
| statefulSet.enabled | bool | `true` | |

----------------------------------------------
Autogenerated from chart metadata using [helm-docs v1.5.0](https://github.com/norwoodj/helm-docs/releases/v1.5.0)
4 changes: 2 additions & 2 deletions charts/newrelic-infrastructure/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
apiVersion: v2
name: newrelic-infrastructure
description: A Helm chart to deploy the New Relic Kubernetes monitoring solution
version: 3.1.1
appVersion: 3.1.1
version: 3.2.0
appVersion: 3.2.0
kubeVersion: ">=1.16.0-0"
home: https://docs.newrelic.com/docs/kubernetes-pixie/kubernetes-integration/get-started/introduction-kubernetes-integration/
sources:
Expand Down
8 changes: 4 additions & 4 deletions charts/newrelic-infrastructure/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

# newrelic-infrastructure

![Version: 3.1.0](https://img.shields.io/badge/Version-3.1.0-informational?style=flat-square) ![AppVersion: 3.1.0](https://img.shields.io/badge/AppVersion-3.1.0-informational?style=flat-square)
![Version: 3.2.0](https://img.shields.io/badge/Version-3.2.0-informational?style=flat-square) ![AppVersion: 3.2.0](https://img.shields.io/badge/AppVersion-3.2.0-informational?style=flat-square)

A Helm chart to deploy the New Relic Kubernetes monitoring solution

Expand Down Expand Up @@ -42,11 +42,11 @@ Kubernetes: `>=1.16.0-0`
| customAttributes | object | `{}` | Custom attributes to be added to the data reported by all integrations reporting in the cluster. |
| images | object | See `values.yaml` | Images used by the chart for the integration and agents. |
| images.agent.repository | string | `"newrelic/infrastructure-bundle"` | Image for the agent and integrations bundle. |
| images.agent.tag | string | `"2.8.2"` | Tag for the agent and integrations bundle. |
| images.agent.tag | string | `"2.8.8"` | Tag for the agent and integrations bundle. |
| images.forwarder.repository | string | `"newrelic/k8s-events-forwarder"` | Image for the agent sidecar. |
| images.forwarder.tag | string | `"1.23.0"` | Tag for the agent sidecar. |
| images.forwarder.tag | string | `"1.24.1"` | Tag for the agent sidecar. |
| images.integration.repository | string | `"newrelic/nri-kubernetes"` | Image for the kubernetes integration. |
| images.integration.tag | string | `"3.0.0"` | Tag for the kubernetes integration. |
| images.integration.tag | string | `"3.1.0"` | Tag for the kubernetes integration. |
| integrations | object | `{}` | Config files for other New Relic integrations that should run in this cluster. |
| ksm | object | See `values.yaml` | Configuration for the Deployment that collects state metrics from KSM (kube-state-metrics). |
| ksm.config.retries | int | `3` | Number of retries after timeout expired |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -126,3 +126,8 @@ controlPlane:
mtls:
secretName: secret-name4
- url: http://localhost:8080

images:
integration:
tag: test
repository: e2e/nri-kubernetes
5 changes: 5 additions & 0 deletions charts/newrelic-infrastructure/ci/test-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -125,3 +125,8 @@ controlPlane:
mtls:
secretName: secret-name4
- url: http://localhost:8080

images:
integration:
tag: test
repository: e2e/nri-kubernetes
6 changes: 3 additions & 3 deletions charts/newrelic-infrastructure/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,23 +9,23 @@ images:
forwarder:
pullPolicy: IfNotPresent
# -- Tag for the agent sidecar.
tag: 1.23.0
tag: 1.24.1
# -- Image for the agent sidecar.
repository: newrelic/k8s-events-forwarder
registry: docker.io
# Image for the New Relic Infrastructure Agent plus integrations.
agent:
pullPolicy: IfNotPresent
# -- Tag for the agent and integrations bundle.
tag: 2.8.2
tag: 2.8.8
# -- Image for the agent and integrations bundle.
repository: newrelic/infrastructure-bundle
registry: docker.io
# Image for the New Relic Kubernetes integration.
integration:
pullPolicy: IfNotPresent
# -- Tag for the kubernetes integration.
tag: 3.0.0
tag: 3.1.0
# -- Image for the kubernetes integration.
repository: newrelic/nri-kubernetes
registry: docker.io
Expand Down

0 comments on commit 7ed79fe

Please sign in to comment.