-
Notifications
You must be signed in to change notification settings - Fork 159
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
Update edge-view to work with the latest pillar #4139
Conversation
Signed-off-by: Milan Lenco <milan@zededa.com>
I have no idea why Yetus keeps failing. The output has much less files than usually. |
pkg/edgeview/src/network.go
Outdated
dstNet := route.DstNetwork.String() | ||
if route.GatewayApp != emptyUUID { | ||
fmt.Printf(" - route dst: %s, gw app: %s\n", | ||
route.DstNetwork, route.GatewayApp) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we also add the 'Gateway' in the display of the routes in both if and else cases?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
After introducing support for Local NI with multiple ports, few fields in the NetworkInstance status and metrics have changed. Also, VolumeRefConfig no longer has the RefCount field. Edge-view needs to be updated to reflect all these changes. Signed-off-by: Milan Lenco <milan@zededa.com>
654c5ed
to
a24bd64
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4139 +/- ##
==========================================
+ Coverage 17.51% 19.69% +2.17%
==========================================
Files 3 8 +5
Lines 805 2600 +1795
==========================================
+ Hits 141 512 +371
- Misses 629 1985 +1356
- Partials 35 103 +68 ☔ View full report in Codecov by Sentry. |
After introducing support for Local NI with multiple ports, few fields in the
NetworkInstanceStatus
andNetworkInstanceMetrics
have changed.Also,
VolumeRefConfig
no longer has theRefCount
field.Edge-view needs to be updated to reflect all these changes.