{"push_id":1357243324,"size":57,"distinct_size":1,"ref":"refs/heads/release_1","head":"7197dfa87f3c12c43ddba0bbb6c5bb600264cfaf","before":"62bbdc9d7f0fa5835953e8e6d1df45023414f5ed","commits":[{"sha":"6141f5d6771e669dedbbbc75f7d13724a103abeb","author":{"name":"wypior","email":"21d6394f3caceecee5d40e383cc6fc18472ab918@f5.com"},"message":"Addresses v12.1 issues with ltm traffic class tests\n\nFixes Issues: #726\n\nUpdated Files:\n\n    test/functional/ltm/test_traffic_class.py\n\nUpdates:\n\nChanged the tests as selfLink tests in 12.1.0 were failing, due to \"~Common~\" being inserted into self link uri, when loading the resource. Basically create and load selfLinks differed in the presence of absence of ~Common~ in the resource uri","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/6141f5d6771e669dedbbbc75f7d13724a103abeb"},{"sha":"25fe8344e186eb3b11acfad3db9219ba26cd6af3","author":{"name":"Tim Rupp","email":"7b9800a52a5264fedd3b1a58b25f4ec268d063bc@gmail.com"},"message":"Merge pull request #728 from wojtek0806/ltm_traf_class_fix\n\nAddresses v12.1 issues with ltm traffic class tests","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/25fe8344e186eb3b11acfad3db9219ba26cd6af3"},{"sha":"d90ed7c7aac508f95a7b89cc93f044795a5e8cbb","author":{"name":"wypior","email":"21d6394f3caceecee5d40e383cc6fc18472ab918@f5.com"},"message":"Addresses Profile IPother test issues, corrected json kind for Webcurity as it was invalid. Changed the test code to\n\nFixes Issues: #622\n\nUpdated Files:\n\n    test/functional/ltm/test_profile.py\n    f5/bigip/tm/ltm/profile.py\n\nUpdates:\n\nProfile IPOther does not have 'description' attribute in 11.5.4, so the test was refactored to allow multiversion compatibility', corrected json kind for Webcurity as it was invalid. Changed the test code for profiles to use mgmt_root fixture rather than bigip","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/d90ed7c7aac508f95a7b89cc93f044795a5e8cbb"},{"sha":"f85d2f29953be81b92d47ee9cd15535ccebb3fa1","author":{"name":"Paul Breaux","email":"df749123cee0934ec89955f3c705295de8905728@f5.com"},"message":"Adding init commit for policy drafts.","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/f85d2f29953be81b92d47ee9cd15535ccebb3fa1"},{"sha":"5aa70a0fa4c05f1279dea495455af56e81752905","author":{"name":"wypior","email":"21d6394f3caceecee5d40e383cc6fc18472ab918@f5.com"},"message":"Addresses v12.1 changes and incompatibilities in GTM sub-module\n\nFixes Issues: #719,  #679\n\nUpdated Files:\n\n    test/functional/gtm/test_pool.py\n    test/functional/test_server.py\n    f5/bigip/tm/gtm/pool.py\n    f5/bigip/tm/gtm/server.py\n\nUpdates:\n\nAs we started adding Partition to the selflinks in 12.1.0, some tests needed to be amended. There is a serious bug in 12.x branch with members, as described in #719, I have had to create custom create methods for 2 member classes in pools.py.\nCorrected subcollection resource names to be singular not plural as per discussion with Tim, amended tests accordingly","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/5aa70a0fa4c05f1279dea495455af56e81752905"},{"sha":"e012fc714042f6c66ce287c38f5aa5d72cd748df","author":{"name":"rahm","email":"77f6e233c8fa5adea4314de2b1cd0f8c36ff33e1@f5.com"},"message":"Issue #730 - Fixes snat functional test for 12.1\n\nUpdated Files\n  - test/functional/tm/ltm/test_snat.py\n\n Updates\n  - Updated bigip fixture to mgmt_root fixture in all test cases\n  - Changed test assertion from exact match to make sure string was present as the response text has changed slightly in 12.1","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/e012fc714042f6c66ce287c38f5aa5d72cd748df"},{"sha":"ac33c435549d3d97409dfb256fbae0e5cd986473","author":{"name":"Tim Rupp","email":"7b9800a52a5264fedd3b1a58b25f4ec268d063bc@gmail.com"},"message":"Merge pull request #736 from jasonrahm/bugfix.test_snat\n\nIssue #730 - Fixes snat functional test for 12.1","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/ac33c435549d3d97409dfb256fbae0e5cd986473"},{"sha":"c79edfa120859250b9b4defcdc7cc97c35673fc6","author":{"name":"wypior","email":"21d6394f3caceecee5d40e383cc6fc18472ab918@f5.com"},"message":"Addresses NTP and FDB test issues in 12.1\n\nFixes Issues: #733, #734\n\nUpdated Files:\n\n    test/functional/net/test_fdb.py\n     test/functional/sys/test_ntp.py\n\nUpdates:\n\nThere are some changes again in some resource names not prepended by /Common/, so some conditionals required to be added to ntp tests. FDB tests were failing due to a change in required parameters inside 'records', in 12.1 apart from name, also endpoint parameter was required for the record to be created. This corresponds to TMSH command changes.\n\nTests:\n\nFlake8\nFunctional Test","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/c79edfa120859250b9b4defcdc7cc97c35673fc6"},{"sha":"8b02d3fa8d207abae62d8694210b48357771cb22","author":{"name":"Tim Rupp","email":"7b9800a52a5264fedd3b1a58b25f4ec268d063bc@gmail.com"},"message":"Merge pull request #737 from wojtek0806/fdb_ntp_fix_tests\n\nAddresses NTP and FDB test issues in 12.1","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/8b02d3fa8d207abae62d8694210b48357771cb22"},{"sha":"8730e3c4dc756706d1df9f96c55ff251f2ca5ff6","author":{"name":"Tim Rupp","email":"7b9800a52a5264fedd3b1a58b25f4ec268d063bc@gmail.com"},"message":"Adds requests-mock package for testing\n\nIssues:\nFixes #738\n\nProblem:\nThis change supplies the python requests-mock package in the\ntest pip and docker files so that it can be used in tests\n\nAnalysis:\nContributors asked that this package be included for writing\nunit and functional tests. The change updates the dockerfiles\nand the pip requirements file to support version 1.1.0 of\nthe requests-mock package\n\nTests:\nnone","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/8730e3c4dc756706d1df9f96c55ff251f2ca5ff6"},{"sha":"e89d78a0df0de06890f977a53975167c5620a0b2","author":{"name":"Wojciech Wypior","email":"21d6394f3caceecee5d40e383cc6fc18472ab918@f5.com"},"message":"Merge pull request #739 from caphrim007/feature.add-requests-mock-to-test-tools\n\nAdds requests-mock package for testing","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/e89d78a0df0de06890f977a53975167c5620a0b2"},{"sha":"896ba29c99f1fd805da98243b084432d714d190a","author":{"name":"rahm","email":"77f6e233c8fa5adea4314de2b1cd0f8c36ff33e1@f5.com"},"message":"Issue #731 - Fixes virtual server functional tests for 12.1\n\nUpdated Files\n  - test/functional/tm/ltm/test_virtual.py\n\n Updates\n  - Removed stats tests; not directly relevant to testing the virtual server code\n  - split the problematic assertion into two assertions to match different parts of the string for virtual/profile","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/896ba29c99f1fd805da98243b084432d714d190a"},{"sha":"d0a0b0e367cecee77e16aadc9d899fcfcb57c077","author":{"name":"rahm","email":"77f6e233c8fa5adea4314de2b1cd0f8c36ff33e1@f5.com"},"message":"Issue #731 - flake8 update - removing unnecessary import","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/d0a0b0e367cecee77e16aadc9d899fcfcb57c077"},{"sha":"94b2ce67c9ae6d9f75f77d2797d2492dca6c8019","author":{"name":"Paul Breaux","email":"df749123cee0934ec89955f3c705295de8905728@f5.com"},"message":"ltm policy functional rewrite in tmos version 12.1 needs attention in\nsdk\n\nIssues:\nFixes #575 #668\n\nProblem:\nIn 12.1, policies cannot just be created and attached to virtual\nservers. There is a workflow to create a draft, then publish, then\nattach. For existing policies, you need to create a draft of the policy,\npatch the draft, then publish it.\n\nAnalysis:\nImplemented some defaults to allow the legacy method to execute by\ndefault, that is, a policy is always published with defaults. If the\nuser is on 12.1.0 and wishes to create a draft then publish. A new\npublic method exists on the Policy resource.\n\nTests:\nAll functional tests pass. These all were used to get coverage in lieu\nof unit tests.","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/94b2ce67c9ae6d9f75f77d2797d2492dca6c8019"},{"sha":"c26db4734675fb2148a6f4cad74e92b7d01d8ec9","author":{"name":"Paul Breaux","email":"df749123cee0934ec89955f3c705295de8905728@f5.com"},"message":"Updating setup.py with new version of f5-icontrol-rest-python.","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/c26db4734675fb2148a6f4cad74e92b7d01d8ec9"},{"sha":"448a38f045ebfb6bb515e6fb992ad74220d9ac8d","author":{"name":"Wojciech Wypior","email":"21d6394f3caceecee5d40e383cc6fc18472ab918@f5.com"},"message":"Merge pull request #740 from jasonrahm/bugfix.test_virtual\n\nIssue #731 - Fixes virtual server functional tests for 12.1","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/448a38f045ebfb6bb515e6fb992ad74220d9ac8d"},{"sha":"333af3e1bc57b2d5e5fd9eb613b3f57f89770383","author":{"name":"wypior","email":"21d6394f3caceecee5d40e383cc6fc18472ab918@f5.com"},"message":"Addresses SNMP  issues in 12.1\n\nFixes Issues: #735\n\nUpdated Files:\n\n    test/functional/net/test_snmp.py\n    f5/bigip/tm/sys/test/test_snmp.py\n    f5/bigip/tm/sys/snmp.py\n\nUpdates:\n\nIn 12.1.0 there are problems with decryption of the hashed password when using update() method. Given the number of ways this can break, the update() method has been disabled for Users and Traps. The error message returns advisory to use modify() method which functions correctly.\n\nTests:\n\nFlake8\nFunctional Test\nUnit Tests","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/333af3e1bc57b2d5e5fd9eb613b3f57f89770383"},{"sha":"9e99fd5732e4b3cbe2495bf9d47f55f00c3fe7bb","author":{"name":"Tim Rupp","email":"7b9800a52a5264fedd3b1a58b25f4ec268d063bc@gmail.com"},"message":"Make dockerfiles respect the requirements.txt pip files\n\nIssues:\nFixes #744\n\nProblem:\nThe dockerfiles maintained a separate list of dependencies and this\nwould cause issues when those dependencies were updated because they\nneeded to be updated in two places at once\n\nAnalysis:\nThis patch makes the dockerfiles use the requirements file and behind\nthe scenes we also adjust the CI build to accomodate this.\n\nTests:\nnone","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/9e99fd5732e4b3cbe2495bf9d47f55f00c3fe7bb"},{"sha":"85e51616fbd168b54a2c5ba33f4d08ab2d6dc059","author":{"name":"pjbreaux","email":"df749123cee0934ec89955f3c705295de8905728@f5.com"},"message":"Merge pull request #745 from caphrim007/bugfix.rebuild-docker-container-as-needed\n\nMake dockerfiles respect the requirements.txt pip files","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/85e51616fbd168b54a2c5ba33f4d08ab2d6dc059"},{"sha":"dafb479a602b96ff99a310905981c309d50ef9d3","author":{"name":"Tim Rupp","email":"7b9800a52a5264fedd3b1a58b25f4ec268d063bc@gmail.com"},"message":"Merge pull request #742 from wojtek0806/fix_snmp_tests\n\nAddresses SNMP  issues in 12.1","distinct":false,"url":"https://api.github.com/repos/F5Networks/f5-common-python/commits/dafb479a602b96ff99a310905981c309d50ef9d3"}]}
{"actor":{"display_login":"pjbreaux"}}