-
Notifications
You must be signed in to change notification settings - Fork 234
Connectivity API: get information about links and transports through Session::info() #2301
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2301 +/- ##
==========================================
- Coverage 72.06% 71.98% -0.09%
==========================================
Files 387 390 +3
Lines 62006 62729 +723
==========================================
+ Hits 44685 45155 +470
- Misses 17321 17574 +253 ☔ View full report in Codecov by Sentry. |
|
PR missing one of the required labels: {'breaking-change', 'internal', 'bug', 'new feature', 'documentation', 'ci', 'api-sync', 'enhancement', 'dependencies'} |
|
PR missing one of the required labels: {'new feature', 'internal', 'api-sync', 'enhancement', 'documentation', 'breaking-change', 'ci', 'dependencies', 'bug'} |
|
PR missing one of the required labels: {'internal', 'bug', 'new feature', 'breaking-change', 'dependencies', 'documentation', 'ci', 'enhancement', 'api-sync'} |
|
PR missing one of the required labels: {'enhancement', 'new feature', 'breaking-change', 'internal', 'api-sync', 'bug', 'ci', 'dependencies', 'documentation'} |
This reverts commit 76f4007.
implementation for #692
@/<local_zid>/session/transport/(multicast|unicast)/<remote_zid>and@/<local_zid>/session/transport/(multicast|unicast)/<remote_zid>/link/<link_id>works as before, but uses the new API. The keeping the same adminspace behavior is ensured by test Connectivity adminspace test #2334Linkare none if transport doesn't support qos🏷️ Label-Based Checklist
Based on the labels applied to this PR, please complete these additional requirements:
Labels:
new feature🆕 New Feature Requirements
Since this PR adds a new feature:
Consider: Can this feature be split into smaller, incremental PRs?
Instructions:
- [ ]to- [x])This checklist updates automatically when labels change, but preserves your checked boxes.