Skip to content

Fixes the sshd api#458

Merged
zancas merged 1 commit intoF5Networks:developmentfrom
caphrim007:fix-sshd-class
Jun 1, 2016
Merged

Fixes the sshd api#458
zancas merged 1 commit intoF5Networks:developmentfrom
caphrim007:fix-sshd-class

Conversation

@caphrim007
Copy link
Copy Markdown
Contributor

@caphrim007 caphrim007 commented May 26, 2016

Issues:
Fixes #438

Problem:
The sshd api was inheriting from Resource instead of ResourceBase and
this was causing a uri KeyError

Analysis:
This patch fixes the inheritence to use the ResourceBase class and
also adds functional tests for the sshd api to prevent this from
happening in the future.

Tests:
test/functional/tm/sys/test_sshd.py

@caphrim007
Copy link
Copy Markdown
Contributor Author

@wojtek0806

@caphrim007
Copy link
Copy Markdown
Contributor Author

@zancas @wojtek0806

@wojtek0806
Copy link
Copy Markdown
Contributor

SSHD tests are failing on 11.6.0 vanilla, also I think you need to rebase again to incorporate the latest 2 prs

@zancas zancas self-assigned this May 27, 2016
@zancas zancas added this to the v0.1.7 milestone Jun 1, 2016
@caphrim007 caphrim007 closed this Jun 1, 2016
@caphrim007 caphrim007 reopened this Jun 1, 2016
Issues:
Fixes F5Networks#438

Problem:
The sshd api was inheriting from Resource instead of ResourceBase and
this was causing a uri KeyError

Analysis:
This patch fixes the inheritence to use the ResourceBase class and
also adds functional tests for the sshd api to prevent this from
happening in the future.

Tests:
test/functional/tm/sys/test_sshd.py
@caphrim007
Copy link
Copy Markdown
Contributor Author

@zancas this is now ready for merge

@zancas zancas merged commit 3369e0d into F5Networks:development Jun 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sshd api is broken

3 participants