Skip to content
This repository was archived by the owner on Nov 8, 2025. It is now read-only.

Conversation

@pjiang-dev
Copy link
Contributor

@pjiang-dev pjiang-dev commented Oct 3, 2025

Need to make this same change argoproj/argo-cd#24785
in order to cherry-pick for argo-cd v3.2

Signed-off-by: Peter Jiang <peterjiang823@gmail.com>
@pjiang-dev pjiang-dev marked this pull request as ready for review October 3, 2025 18:50
@pjiang-dev pjiang-dev requested a review from a team as a code owner October 3, 2025 18:50
labels:
app: test-app
spec:
containers:

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
app: test-app
spec:
containers:
- name: nginx

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
name: https
- containerPort: 8080
name: metrics
- name: sidecar

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
labels:
app: test-app
spec:
containers:

Check warning

Code scanning / SonarCloud

Service account permissions should be restricted Medium test

Bind this resource's automounted service account to RBAC or disable automounting. See more on SonarQube Cloud
app: test-app
spec:
containers:
- name: nginx

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
terminationMessagePath: /dev/termination-log
terminationMessagePolicy: File
imagePullPolicy: IfNotPresent
- name: sidecar

Check warning

Code scanning / SonarCloud

Memory limits should be enforced Medium test

Specify a memory limit for this container. See more on SonarQube Cloud
Signed-off-by: Peter Jiang <peterjiang823@gmail.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 3, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
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

@codecov
Copy link

codecov bot commented Oct 3, 2025

Codecov Report

❌ Patch coverage is 90.62500% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 47.63%. Comparing base (8849c3f) to head (90e8700).
⚠️ Report is 64 commits behind head on master.

Files with missing lines Patch % Lines
pkg/diff/diff.go 90.62% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #791      +/-   ##
==========================================
- Coverage   54.26%   47.63%   -6.63%     
==========================================
  Files          64       64              
  Lines        6164     6604     +440     
==========================================
- Hits         3345     3146     -199     
- Misses       2549     3201     +652     
+ Partials      270      257      -13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pjiang-dev
Copy link
Contributor Author

Qualiy failures are from test failes

Copy link
Contributor

@leoluz leoluz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@crenshaw-dev crenshaw-dev merged commit cf8d42f into argoproj:master Oct 3, 2025
3 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants