Skip to content

misc: Sync reference charts to main #6579

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

Merged
merged 3 commits into from
May 16, 2025
Merged

misc: Sync reference charts to main #6579

merged 3 commits into from
May 16, 2025

Conversation

akshatsinha007
Copy link
Contributor

No description provided.

Copy link

gitguardian bot commented May 16, 2025

⚠️ GitGuardian has uncovered 3 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
9257608 Triggered Base64 AWS keys a92dcd5 scripts/devtron-reference-helm-charts/deployment-chart_4-21-0/test-values.json View secret
2763127 Triggered Generic High Entropy Secret a92dcd5 scripts/devtron-reference-helm-charts/deployment-chart_4-21-0/secrets-test-values.yaml View secret
141558 Triggered Generic High Entropy Secret a92dcd5 scripts/devtron-reference-helm-charts/deployment-chart_4-21-0/env-values.yaml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@akshatsinha007 akshatsinha007 changed the title misc: fixed statefulset version in Readme misc: Sync refernce charts to main May 16, 2025
@akshatsinha007 akshatsinha007 changed the title misc: Sync refernce charts to main misc: Sync reference charts to main May 16, 2025
{{- if $.Values.topologySpreadConstraint }}
topologySpreadConstraints:
{{ toYaml .Values.topologySpreadConstraint }}
{{- end }}

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium

Bind this Service Account to RBAC or disable "automountServiceAccountToken". See more on SonarQube Cloud
{{- end}}
{{- end}}
containers:
- name: {{ $.Chart.Name }}

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium

Specify a memory limit for this container. See more on SonarQube Cloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@pawan-mehta-dt pawan-mehta-dt merged commit 667bd63 into main May 16, 2025
10 of 11 checks passed
@pawan-mehta-dt pawan-mehta-dt deleted the sts-readme branch May 16, 2025 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants