Skip to content

Commit

Permalink
šŸ› fix: (helm/v1alpha1): install the prometheus-operator CRDs before iā€¦
Browse files Browse the repository at this point in the history
ā€¦nstalling a chart which configures a ServiceMonitor
  • Loading branch information
monteiro-renato committed Nov 22, 2024
1 parent ab96f9a commit aba0330
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions .github/workflows/test-helm-samples.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@ name: Helm Testdata Sample
on:
push:
paths:
- 'testdata/project-v4-with-plugins/**'
- '.github/workflows/test-helm-samples.yml'
- "testdata/project-v4-with-plugins/**"
- ".github/workflows/test-helm-samples.yml"
pull_request:
paths:
- 'testdata/project-v4-with-plugins/**'
- '.github/workflows/test-helm-samples.yml'
- "testdata/project-v4-with-plugins/**"
- ".github/workflows/test-helm-samples.yml"

jobs:
helm-test-project-v4-with-plugins:
Expand All @@ -23,7 +23,7 @@ jobs:
- name: Setup Go
uses: actions/setup-go@v5
with:
go-version: '~1.22'
go-version: "~1.22"

- name: Install the latest version of kind
run: |
Expand Down Expand Up @@ -55,6 +55,12 @@ jobs:
run: |
helm lint testdata/project-v4-with-plugins/dist/chart
- name: Install Prometheus Operator CRDs
run: |
helm repo add prometheus-community https://prometheus-community.github.io/helm-charts
helm repo update
helm install prometheus-crds prometheus-community/prometheus-operator-crds -n monitoring --create-namespace
- name: Install cert-manager via Helm
run: |
helm repo add jetstack https://charts.jetstack.io
Expand All @@ -69,7 +75,7 @@ jobs:
- name: Install Helm chart for project-v4-with-plugins
run: |
helm install my-release testdata/project-v4-with-plugins/dist/chart --create-namespace --namespace project-v4-with-plugins-system
helm install my-release testdata/project-v4-with-plugins/dist/chart --create-namespace --namespace project-v4-with-plugins-system --set prometheus.enable=true
- name: Check Helm release status
run: |
Expand Down

0 comments on commit aba0330

Please sign in to comment.