Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Helm-Chart and improves release processes #89

Merged
merged 3 commits into from
May 25, 2024
Merged

Conversation

chr-fritz
Copy link
Owner

This PR adds a helm-chart to deploy it in an kubernetes environment and generally improves the release process.

chr-fritz added 3 commits May 25, 2024 17:42
Fixes using the wrong access token for releases.
The following tag formats are only available through multi arch
manifests but no longer as arch specific image tags:
* latest-<ARCH>
* v<MAJOR>.<MINOR>-<ARCH>
* v<MAJOR>-<ARCH>

Additionally, these tags are only published if it is not a prerelease.
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
containers:
- name: {{ .Chart.Name }}

Check warning

Code scanning / SonarCloud

CPU limits should be enforced

<!--SONAR_ISSUE_KEY:AY-wLxIdIVQu87BqApk3-->Specify a CPU limit for this container. <p>See more on <a href="https://sonarcloud.io/project/issues?id=chr-fritz_knx-exporter&issues=AY-wLxIdIVQu87BqApk3&open=AY-wLxIdIVQu87BqApk3&branch=feature%2Fhelm-chart">SonarCloud</a></p>
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
containers:
- name: {{ .Chart.Name }}

Check warning

Code scanning / SonarCloud

Memory limits should be enforced

<!--SONAR_ISSUE_KEY:AY-wLxIdIVQu87BqApk4-->Specify a memory limit for this container. <p>See more on <a href="https://sonarcloud.io/project/issues?id=chr-fritz_knx-exporter&issues=AY-wLxIdIVQu87BqApk4&open=AY-wLxIdIVQu87BqApk4&branch=feature%2Fhelm-chart">SonarCloud</a></p>
{{- with .Values.podLabels }}
{{- toYaml . | nindent 8 }}
{{- end }}
spec:

Check warning

Code scanning / SonarCloud

Service account tokens should not be mounted in pods

<!--SONAR_ISSUE_KEY:AY-wLxIdIVQu87BqApk5-->Set automountServiceAccountToken to false for this specification of kind Deployment. <p>See more on <a href="https://sonarcloud.io/project/issues?id=chr-fritz_knx-exporter&issues=AY-wLxIdIVQu87BqApk5&open=AY-wLxIdIVQu87BqApk5&branch=feature%2Fhelm-chart">SonarCloud</a></p>
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
containers:
- name: {{ .Chart.Name }}

Check warning

Code scanning / SonarCloud

Storage limits should be enforced

<!--SONAR_ISSUE_KEY:AY-wLxIdIVQu87BqApk6-->Specify a storage limit for this container. <p>See more on <a href="https://sonarcloud.io/project/issues?id=chr-fritz_knx-exporter&issues=AY-wLxIdIVQu87BqApk6&open=AY-wLxIdIVQu87BqApk6&branch=feature%2Fhelm-chart">SonarCloud</a></p>
@chr-fritz chr-fritz merged commit 8c6d6e0 into main May 25, 2024
5 of 7 checks passed
@chr-fritz chr-fritz deleted the feature/helm-chart branch May 25, 2024 15:49
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant