-
Notifications
You must be signed in to change notification settings - Fork 51
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
Moved mnedc logic to discoverymgr, which fixes #178 #180
Conversation
Signed-off-by: ayush.kumar <[email protected]>
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.
@ayush-kr @suresh-lc Could we refer to any testing logs like that we refered to from #146 (comment)?
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.
Will implement discoverymgr to call StartMNEDCServer() and StartMNEDCClient(), not in main.go.
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
MNEDC Server logs
MNEDC client logs
Service request logs on Main NAT device (Virtual IP - 10.0.0.1)
Service execution on Sub NAT device (Virtual IP - 10.0.0.2)
|
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
@ayush-kr Thank you for the kind information (and your passionate efforts to validate the PR). |
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.
According to the log, it looks good to me.
Signed-off-by: ayush.kumar [email protected]
Description
Since MNEDC responsibility is similar to the Discovery Manager, the mnedcmgr package has now been moved inside discoverymgr package and renamed to mnedc.
Fixes #178
Type of change
How Has This Been Tested?
Similar to testing steps in #146
Test Configuration:
Checklist: