Skip to content

During functional testing against 11.5.4, test/functional/tm/ltm/test_virtual.py test fails#592

Merged
caphrim007 merged 2 commits intoF5Networks:developmentfrom
pjbreaux:bugfix.virt_11_5_4
Aug 2, 2016
Merged

During functional testing against 11.5.4, test/functional/tm/ltm/test_virtual.py test fails#592
caphrim007 merged 2 commits intoF5Networks:developmentfrom
pjbreaux:bugfix.virt_11_5_4

Conversation

@pjbreaux
Copy link
Copy Markdown
Contributor

@pjbreaux pjbreaux commented Aug 2, 2016

@zancas

Issues:
Fixes #589

Problem:
A profile cannot be created or loaded on 11.5.4 under a virtual without
specifying the partition.

Analysis:
Added a tmos version check in the init of the Profiles resource
under Virtual. If the tmos version is less than 11.6.0, the partition is
required for creation and load.

Tests:
Virtual tests pass against 11.5.4 and 11.6.0

@pjbreaux
Copy link
Copy Markdown
Contributor Author

pjbreaux commented Aug 2, 2016

@caphrim007: I made zancas assignee by sheer habit. Let me know if you have time to look at this.

@pjbreaux pjbreaux mentioned this pull request Aug 2, 2016
@caphrim007
Copy link
Copy Markdown
Contributor

@pjbreaux this works for me. Rebase and I'll merge

Paul Breaux added 2 commits August 2, 2016 10:52
test/functional/tm/ltm/test_virtual.py test fails

Issues:
Fixes F5Networks#589

Problem:
A profile cannot be created or loaded on 11.5.4 under a virtual without
specifying the partition.

Analysis:
Added a tmos version check in the __init__ of the Profiles resource
under Virtual. If the tmos version is less than 11.6.0, the partition is
required for creation and load.

Tests:
Virtual tests pass against 11.5.4 and 11.6.0
@pjbreaux pjbreaux force-pushed the bugfix.virt_11_5_4 branch from 0451411 to ab98333 Compare August 2, 2016 16:52
@caphrim007 caphrim007 merged commit abab879 into F5Networks:development Aug 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants