Skip to content

Conversation

@ghernandez345
Copy link
Contributor

@ghernandez345 ghernandez345 commented Jan 7, 2026

Related issue: Resolves #35645

This changes the error message for an invalid license when trying to apply certificate authorities via GitOps

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
  • Added/updated automated tests
  • QA'd all new/changed functionality manually

@ghernandez345 ghernandez345 requested a review from a team as a code owner January 7, 2026 14:12
@ghernandez345 ghernandez345 changed the title fix ca license error gitops Improved error message when trying to apply CAs via gitops without a premium license Jan 7, 2026
@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.14%. Comparing base (5b5a3e5) to head (a287482).
⚠️ Report is 26 commits behind head on main.

Files with missing lines Patch % Lines
server/service/client.go 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #37981      +/-   ##
==========================================
+ Coverage   62.75%   64.14%   +1.38%     
==========================================
  Files        2197     2385     +188     
  Lines      172921   190020   +17099     
  Branches     8429     8429              
==========================================
+ Hits       108523   121888   +13365     
- Misses      54013    56879    +2866     
- Partials    10385    11253     +868     
Flag Coverage Δ
backend 65.72% <60.00%> (+1.44%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ghernandez345
Copy link
Contributor Author

Ignoring failing test as this is a known unrelated test failure being fixed elsewhere

@ghernandez345 ghernandez345 merged commit c7c0ee3 into main Jan 8, 2026
65 of 71 checks passed
@ghernandez345 ghernandez345 deleted the fix-ca-license-error-gitops branch January 8, 2026 11:07
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.

Misleading error is displayed when trying to add CA via Gitops on Fleet free: "Error: applying certificate authorities: missing or invalid license"

3 participants