-
Notifications
You must be signed in to change notification settings - Fork 73
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
add keybase authentication option in netconf provider #619
Conversation
ylil93
commented
Nov 6, 2017
- add keybase authentication option in netconf provider
- update testing
- update documentation
- add keybase authentication option in netconf provider - update testing - update documentation
Codecov Report
@@ Coverage Diff @@
## master #619 +/- ##
==========================================
+ Coverage 73.27% 73.35% +0.08%
==========================================
Files 104 105 +1
Lines 10351 10386 +35
Branches 1449 1450 +1
==========================================
+ Hits 7585 7619 +34
Misses 2488 2488
- Partials 278 279 +1
Continue to review full report at Codecov.
|
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.
Thanks for this change, Lily! Just have a few minor comments.
Also, can you please address the codacy review: https://www.codacy.com/app/ydk/ydk-gen/pullRequest?prid=1050462&/login
:param repo: Reference to an instance of :cpp:class:`Repository<ydk::path::Repository>` | ||
:param address: IP address of the device supporting a netconf interface. | ||
:param username: Username to log in to the device. | ||
:param private_key_path: Path to private key file. |
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 you indicate if this is the absolute path or relative path?
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.
I've tried both and it takes either, but yeah I can specify that.
address, port, timeout); | ||
} | ||
|
||
// NetconfSession::NetconfSession( |
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 the commented lines be removed?
@@ -49,3 +49,22 @@ TEST_CASE("get_capabilities") | |||
NetconfServiceProvider provider{ "127.0.0.1", "admin", "admin", 12022}; | |||
CHECK_NOTHROW(provider.get_capabilities()); | |||
} | |||
|
|||
// TEST_CASE("CreateKeybaseAuthRepo_Domingo") |
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 the commented lines be removed?
public_key_path = public, | ||
port = self.port) | ||
|
||
# private = "%s/domingo_sshkey" % dir_path |
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 the commented lines be removed?