Skip to content

Conversation

@hackeramitkumar
Copy link
Contributor

Description

Fixes: #1057

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@hackeramitkumar hackeramitkumar requested a review from a team as a code owner January 17, 2026 13:38
@amitami2 amitami2 force-pushed the feat/sub-unsub-ack-handling branch from c37e7a7 to 80f07b7 Compare January 17, 2026 13:40
@codecov
Copy link

codecov bot commented Jan 17, 2026

Codecov Report

❌ Patch coverage is 54.11255% with 106 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
data-plane/core/controller/src/service.rs 2.43% 80 Missing ⚠️
data-plane/core/service/src/app.rs 72.85% 19 Missing ⚠️
data-plane/core/datapath/src/message_processing.rs 91.13% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

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.

Add Acknowledgements for Subscription/Unsubscription Operations

2 participants