Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Model repository directory doesn't require port suffix #853

Closed
111pontes opened this issue Jan 14, 2019 · 1 comment
Closed

Model repository directory doesn't require port suffix #853

111pontes opened this issue Jan 14, 2019 · 1 comment
Assignees
@ygorelik
Labels
enhancement

Comments

@111pontes
Copy link
Collaborator

When using NetconfServiceProvider, a model repository is created with name hostname_port. As models are not a function of the port, the directory name should just have name hostname.
@ygorelik ygorelik self-assigned this Jan 30, 2019
ygorelik pushed a commit to ygorelik/ydk-gen that referenced this issue Jan 30, 2019
Enhancement CiscoDevNet#853
bebd554
@ygorelik
Copy link
Collaborator

Implemented in YDK-0.8.1 pre-release.

ygorelik pushed a commit to ygorelik/ydk-gen that referenced this issue Feb 1, 2019
Fixing travis test after enhancement CiscoDevNet#853
acfd2cb
@ygorelik ygorelik mentioned this issue Feb 1, 2019
Merged
ygorelik added a commit that referenced this issue Feb 1, 2019
@ygorelik
Merge pull request #868 from ygorelik/gnmi
529a7a8 
Addressed Codacy issues and #853, #864, #844
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ygorelik ygorelik

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@111pontes @ygorelik