Skip to content

fix: Added meaningful error msg for consumergroups#442

Open
shivaygupta-dotcom wants to merge 5 commits intomainfrom
fix/decK-1916
Open

fix: Added meaningful error msg for consumergroups#442
shivaygupta-dotcom wants to merge 5 commits intomainfrom
fix/decK-1916

Conversation

@shivaygupta-dotcom
Copy link
Copy Markdown
Contributor

@shivaygupta-dotcom shivaygupta-dotcom commented Apr 1, 2026

Summary

When syncing consumers that reference consumer groups, if those groups aren’t present in the state, the error message isn’t very helpful. It doesn’t clearly indicate which entity is missing, as it does not provide clear details about the missing entity.
more details here : https://kongstrong.slack.com/archives/C04349E4KRC/p1772639980263509

Issues resolved

Fix for Kong/deck#1916

Documentation

Testing

  • Unit tests
  • E2E tests
  • Manual testing on Universal
  • Manual testing on Kubernetes

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 1, 2026

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 28.89%. Comparing base (afd6c37) to head (cf43850).

Files with missing lines Patch % Lines
pkg/file/builder.go 0.00% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
- Coverage   28.90%   28.89%   -0.02%     
==========================================
  Files         121      121              
  Lines       16524    16530       +6     
==========================================
  Hits         4777     4777              
- Misses      11129    11135       +6     
  Partials      618      618              

☔ 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.

@shivaygupta-dotcom shivaygupta-dotcom marked this pull request as ready for review April 8, 2026 08:55
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