feat: add helm chart release workflow #2436
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
license-eye has checked 2291 files.
| Valid | Invalid | Ignored | Fixed |
|---|---|---|---|
| 2021 | 19 | 251 | 0 |
Click to see the invalid file list
- charts/zac/Chart.lock
- charts/zac/templates/_helpers.tpl
- charts/zac/templates/config.yaml
- charts/zac/templates/configmap-nginx.yaml
- charts/zac/templates/deployment.yaml
- charts/zac/templates/extra-deploy.yaml
- charts/zac/templates/hpa.yaml
- charts/zac/templates/ingress.yaml
- charts/zac/templates/nginx-deployment.yaml
- charts/zac/templates/nginx-service.yaml
- charts/zac/templates/office-converter-deployment.yaml
- charts/zac/templates/office-converter-service.yaml
- charts/zac/templates/opa-deployment.yaml
- charts/zac/templates/opa-service.yaml
- charts/zac/templates/secret.yaml
- charts/zac/templates/service.yaml
- charts/zac/templates/serviceaccount.yaml
- charts/zac/templates/signaleren-cron-job.yaml
- charts/zac/templates/signaleren-delete-cron-job.yaml
Use this command to fix any missing license headers
```bash
docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix
</details>
There was a problem hiding this comment.
license-eye has checked 2288 files.
| Valid | Invalid | Ignored | Fixed |
|---|---|---|---|
| 2018 | 19 | 251 | 0 |
Click to see the invalid file list
- charts/zac/Chart.lock
- charts/zac/templates/_helpers.tpl
- charts/zac/templates/config.yaml
- charts/zac/templates/configmap-nginx.yaml
- charts/zac/templates/deployment.yaml
- charts/zac/templates/extra-deploy.yaml
- charts/zac/templates/hpa.yaml
- charts/zac/templates/ingress.yaml
- charts/zac/templates/nginx-deployment.yaml
- charts/zac/templates/nginx-service.yaml
- charts/zac/templates/office-converter-deployment.yaml
- charts/zac/templates/office-converter-service.yaml
- charts/zac/templates/opa-deployment.yaml
- charts/zac/templates/opa-service.yaml
- charts/zac/templates/secret.yaml
- charts/zac/templates/service.yaml
- charts/zac/templates/serviceaccount.yaml
- charts/zac/templates/signaleren-cron-job.yaml
- charts/zac/templates/signaleren-delete-cron-job.yaml
Use this command to fix any missing license headers
```bash
docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix
</details>
unit-test-results 78 files 105 suites 2m 3s ⏱️ Results for commit 8b0e7be. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
license-eye has checked 2288 files.
| Valid | Invalid | Ignored | Fixed |
|---|---|---|---|
| 2018 | 19 | 251 | 0 |
Click to see the invalid file list
- charts/zac/Chart.lock
- charts/zac/templates/_helpers.tpl
- charts/zac/templates/config.yaml
- charts/zac/templates/configmap-nginx.yaml
- charts/zac/templates/deployment.yaml
- charts/zac/templates/extra-deploy.yaml
- charts/zac/templates/hpa.yaml
- charts/zac/templates/ingress.yaml
- charts/zac/templates/nginx-deployment.yaml
- charts/zac/templates/nginx-service.yaml
- charts/zac/templates/office-converter-deployment.yaml
- charts/zac/templates/office-converter-service.yaml
- charts/zac/templates/opa-deployment.yaml
- charts/zac/templates/opa-service.yaml
- charts/zac/templates/secret.yaml
- charts/zac/templates/service.yaml
- charts/zac/templates/serviceaccount.yaml
- charts/zac/templates/signaleren-cron-job.yaml
- charts/zac/templates/signaleren-delete-cron-job.yaml
Use this command to fix any missing license headers
```bash
docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix
</details>
There was a problem hiding this comment.
license-eye has checked 2288 files.
| Valid | Invalid | Ignored | Fixed |
|---|---|---|---|
| 2018 | 19 | 251 | 0 |
Click to see the invalid file list
- charts/zac/Chart.lock
- charts/zac/templates/_helpers.tpl
- charts/zac/templates/config.yaml
- charts/zac/templates/configmap-nginx.yaml
- charts/zac/templates/deployment.yaml
- charts/zac/templates/extra-deploy.yaml
- charts/zac/templates/hpa.yaml
- charts/zac/templates/ingress.yaml
- charts/zac/templates/nginx-deployment.yaml
- charts/zac/templates/nginx-service.yaml
- charts/zac/templates/office-converter-deployment.yaml
- charts/zac/templates/office-converter-service.yaml
- charts/zac/templates/opa-deployment.yaml
- charts/zac/templates/opa-service.yaml
- charts/zac/templates/secret.yaml
- charts/zac/templates/service.yaml
- charts/zac/templates/serviceaccount.yaml
- charts/zac/templates/signaleren-cron-job.yaml
- charts/zac/templates/signaleren-delete-cron-job.yaml
Use this command to fix any missing license headers
```bash
docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix
</details>
integration-test-results 27 files 27 suites 35s ⏱️ Results for commit 8b0e7be. ♻️ This comment has been updated with latest results. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2436 +/- ##
=========================================
Coverage 52.76% 52.76%
Complexity 3262 3262
=========================================
Files 991 991
Lines 22519 22519
Branches 1718 1718
=========================================
Hits 11883 11883
Misses 9862 9862
Partials 774 774
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
niorg
force-pushed
the
feature/helm-chart-release-workflow
branch
from
a891cc7 to
8b0e7be
Compare
|
when we merge we have to change some other pipelines in provisioning... |
|
yes, we need to be careful with merging until all is aligned. Also renaming the gh-pages branch and modify branch protection rules needs to be done prior to merging this. |
Add helm chart release flow to gh-pages
Also moved helm charts to
chartsdirectory as it is expected there.Fixed some linting and deployment errors along too
Solves PZ-4965