Skip to content

fix: server side apply status with strict validation#1186

Draft
zachaller wants to merge 9 commits intoargoproj-labs:mainfrom
zachaller:ssa-status-r3-strict-validation
Draft

fix: server side apply status with strict validation#1186
zachaller wants to merge 9 commits intoargoproj-labs:mainfrom
zachaller:ssa-status-r3-strict-validation

Conversation

@zachaller
Copy link
Contributor

No description provided.

Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
@codecov-commenter
Copy link

codecov-commenter commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 90.24390% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.02%. Comparing base (ff88239) to head (82419d7).

Files with missing lines Patch % Lines
...l/utils/statusapply/common_apply_configurations.go 79.64% 18 Missing and 5 partials ⚠️
...nternal/controller/promotionstrategy_controller.go 85.18% 3 Missing and 1 partial ⚠️
internal/utils/utils.go 92.10% 2 Missing and 1 partial ⚠️
...nternal/controller/timedcommitstatus_controller.go 90.47% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1186      +/-   ##
==========================================
+ Coverage   54.03%   55.02%   +0.99%     
==========================================
  Files          55       56       +1     
  Lines        6344     6633     +289     
==========================================
+ Hits         3428     3650     +222     
- Misses       2546     2592      +46     
- Partials      370      391      +21     

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

@codecov-commenter
Copy link

Bundle Report

Bundle size has no change ✅

Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
Signed-off-by: Zach Aller <zachaller@users.noreply.github.com>
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.

2 participants