)]}'
{"id":"openstack%2Fpython-manilaclient~236055","triplet_id":"openstack%2Fpython-manilaclient~master~I7f31d6177c23438d113da3b2e6f77295f414b4ed","project":"openstack/python-manilaclient","branch":"master","topic":"functional_tests","hashtags":[],"change_id":"I7f31d6177c23438d113da3b2e6f77295f414b4ed","subject":"Enable Functional Tests to Run in NetApp CI","status":"ABANDONED","created":"2015-10-16 19:00:01.000000000","updated":"2015-11-08 21:17:32.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f2764b92bdd16e5278620b0db4e266989451e316","_number":236055,"virtual_id_number":236055,"owner":{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},{"value":1,"date":"2015-10-16 20:01:50.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2015-10-27 16:55:06.000000000","_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"value":-1,"date":"2015-10-16 21:18:18.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"date":"2015-10-27 18:11:36.000000000","_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-10-16 20:01:50.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-10-16 21:18:18.000000000","updated_by":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"reviewer":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"state":"REVIEWER"},{"updated":"2015-10-27 18:11:36.000000000","updated_by":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"reviewer":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"state":"REVIEWER"}],"messages":[{"id":"e0310a458fd892182e77e376cd6a1f3e4af31a1e","author":{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"},"date":"2015-10-16 19:00:01.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c37873fd8b7cc43649011c14febffb417592aee8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-10-16 20:01:50.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-manilaclient-python26 http://logs.openstack.org/55/236055/1/check/gate-python-manilaclient-python26/2c66646/ : SUCCESS in 1m 22s\n- gate-python-manilaclient-pep8 http://logs.openstack.org/55/236055/1/check/gate-python-manilaclient-pep8/07f1f9a/ : SUCCESS in 1m 02s\n- gate-python-manilaclient-docs http://docs-draft.openstack.org/55/236055/1/check/gate-python-manilaclient-docs/86a9b62//doc/build/html/ : SUCCESS in 1m 14s\n- gate-python-manilaclient-python27 http://logs.openstack.org/55/236055/1/check/gate-python-manilaclient-python27/5063350/ : SUCCESS in 1m 24s\n- gate-python-manilaclient-python34 http://logs.openstack.org/55/236055/1/check/gate-python-manilaclient-python34/3910ffc/ : SUCCESS in 2m 15s\n- gate-manilaclient-dsvm-neutron-functional http://logs.openstack.org/55/236055/1/check/gate-manilaclient-dsvm-neutron-functional/da655e3/ : SUCCESS in 54m 42s","accounts_in_message":[],"_revision_number":1},{"id":"85d0b675ca0658653ad97568d2baa0a165037550","author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"date":"2015-10-16 21:18:18.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nOS_AUTH_URL does not need fix anymore after merge of Keystone v3 API support.","accounts_in_message":[],"_revision_number":1},{"id":"072898d1e9b57590ed55762ab8e60c3d6a207dc6","author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"date":"2015-10-19 15:17:40.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"6cce830a3ac8b1fb6e571b81a3d5cf2a8e23f24a","author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"date":"2015-10-26 21:59:35.000000000","message":"Patch Set 1:\n\nI guess, this can be abandoned.","accounts_in_message":[],"_revision_number":1},{"id":"d719507c815097e8f1187792f4e1206361a5ac5b","author":{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"},"date":"2015-10-27 16:55:06.000000000","message":"Patch Set 1:\n\nNetApp CI wasn\u0027t working yesterday. Was able to run functional tests today, without this patch, and the test still failed. The failure appears to be due to setting the VOLUME_BACKING_FILE_SIZE too low.\n\n16:47:25 2015-10-27 16:47:25.987 | ERROR:   functional: commands failed\n16:47:25 2015-10-27 16:47:25.990 | ++ EXIT_CODE\u003d1\n16:47:25 2015-10-27 16:47:25.993 | ++ \u0027[\u0027 -d .testrepository \u0027]\u0027\n16:47:25 2015-10-27 16:47:25.995 | ++ \u0027[\u0027 -f .testrepository/0.2 \u0027]\u0027\n16:47:25 2015-10-27 16:47:25.998 | ++ \u0027[\u0027 -f .testrepository/0 \u0027]\u0027\n16:47:25 2015-10-27 16:47:26.000 | ++ .tox/functional/bin/subunit-1to2\n16:47:26 2015-10-27 16:47:26.292 | ++ /usr/os-testr-env/bin/subunit2html ./subunit_log.txt testr_results.html\n16:47:26 2015-10-27 16:47:26.294 | /opt/stack/new/python-manilaclient/contrib/ci/post_test_hook.sh: line 96: /usr/os-testr-env/bin/subunit2html: No such file or directory\n16:47:26 2015-10-27 16:47:26.296 | +++ du -k ./subunit_log.txt\n16:47:26 2015-10-27 16:47:26.298 | +++ awk \u0027{print $1}\u0027\n16:47:26 2015-10-27 16:47:26.299 | ++ SUBUNIT_SIZE\u003d156\n16:47:26 2015-10-27 16:47:26.301 | ++ gzip -9 ./subunit_log.txt\n16:47:26 2015-10-27 16:47:26.309 | ++ gzip -9 ./testr_results.html\n16:47:26 2015-10-27 16:47:26.311 | gzip: ./testr_results.html: No such file or directory\n16:47:26 2015-10-27 16:47:26.313 | ++ sudo mv testr_results.html.gz /home/jenkins/workspace/manilaclient-cDOT-functional-neutron-ss/logs\n16:47:26 2015-10-27 16:47:26.324 | mv: cannot stat ‘testr_results.html.gz’: No such file or directory\n16:47:26 2015-10-27 16:47:26.326 | ++ sudo mv subunit_log.txt.gz /home/jenkins/workspace/manilaclient-cDOT-functional-neutron-ss/logs","accounts_in_message":[],"_revision_number":1},{"id":"fd1dfaf4601fe060bd2306df1253f134acbfdccd","author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"date":"2015-10-27 17:17:11.000000000","message":"Patch Set 1:\n\nChuck,\n\nHow env var \"VOLUME_BACKING_FILE_SIZE\", that is used by Cinder, can make influence for you when you use Manila NetApp share driver?\n\nAlso, in latest master, functional tests do create only 17 shares. Default value for Cinder backing file size is 10G. Cinder has two backends, each has 10G. So, even if create all shares at once, we will not exceed Cinder quota of 20G. If we use Cinder, of course.","accounts_in_message":[],"_revision_number":1},{"id":"16355a66fe076cca1f75304d2eb5a4f09a235cb8","author":{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"},"date":"2015-10-27 18:14:44.000000000","message":"Patch Set 1:\n\nHi Valeriy,\n\nI am not absolutely sure how it influences it. Alex and I were looking at what appeared to be an insufficient size issue and he recommended that I add this setting. If I recall at all correctly he said that it influenced the size of the devstack instance that is created to run the functional test against. I\u0027ve Alex to the review.\n\nOne other thing is I notice that you added the same setting to the pre_test_hook for Manila here: https://github.com/openstack/manila/commit/738c7780b75350f48abcd938b79918c86963f92e. Was this added for an unrelated reason?\n\nThanks for looking into this.","accounts_in_message":[],"_revision_number":1},{"id":"45407ee470be0befacc1a11587ea945d9fdee69b","author":{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"},"date":"2015-10-27 18:16:36.000000000","message":"Patch Set 1:\n\nhttps://github.com/openstack/manila/commit/738c7780b75350f48abcd938b79918c86963f92e \u003c-- link without period at the end","accounts_in_message":[],"_revision_number":1},{"id":"17a3374c702b86b8e5e0e5b60ea68f225598f81d","author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"date":"2015-10-28 08:59:50.000000000","message":"Patch Set 1:\n\nChuck,\n\n\"VOLUME_BACKING_FILE_SIZE\" is used by Cinder LVM driver, that is used in CI. In Manila pre_test_hook.sh it is useful for Generic driver, that uses Cinder. It should not make any gain for other types of drivers, that do not use Cinder as back end.","accounts_in_message":[],"_revision_number":1},{"id":"07a7e09317b298ec592ccb0a4defe0e8416ead8f","author":{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"},"date":"2015-11-08 21:17:32.000000000","message":"Abandoned\n\nComments made by Valeriy are correct. Manila Client functional tests are now passing in the NetApp CI.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"2f7ff1201f7586bfe48c634efdf8618b8c232b54","revisions":{"2f7ff1201f7586bfe48c634efdf8618b8c232b54":{"kind":"REWORK","_number":1,"created":"2015-10-16 19:00:01.000000000","uploader":{"_account_id":16212,"name":"Chuck Fouts","email":"fchuck@netapp.com","username":"cFouts"},"ref":"refs/changes/55/236055/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-manilaclient","ref":"refs/changes/55/236055/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-manilaclient refs/changes/55/236055/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-manilaclient refs/changes/55/236055/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-manilaclient refs/changes/55/236055/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-manilaclient refs/changes/55/236055/1"}}},"commit":{"parents":[{"commit":"0a731dcafab03478eb8fe084b6027977bf78666e","subject":"Merge \"Fix args parsing for quota-class-update\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-manilaclient/commit/0a731dcafab03478eb8fe084b6027977bf78666e"}]}],"author":{"name":"cFouts","email":"fchuck@netapp.com","date":"2015-10-12 17:29:24.000000000","tz":-240},"committer":{"name":"Chuck Fouts","email":"fchuck@netapp.com","date":"2015-10-16 19:00:00.000000000","tz":0},"subject":"Enable Functional Tests to Run in NetApp CI","message":"Enable Functional Tests to Run in NetApp CI\n\nTo date functional tests haven\u0027t been able to run in the NetApp CI\nsystem. It was determined that the OS_AUTH_URL is not set properly\nduring setup. It was also determined that \"neutron\" has to be included\nin the Jenkins job name in order for the job to succeed.\n\nAlso, the value for VOLUME_BACKING_FILE_SIZE had to be increased. This\nshould be set somewhere other than pre_test_hook.sh though.\n\nFixes are provided in the post_test_hook.sh and pre_test_hook.sh script\nfiles to make functional tests succeed in NetApp CI.\n\nChange-Id: I7f31d6177c23438d113da3b2e6f77295f414b4ed\nCloses-Bug: #1506591\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-manilaclient/commit/2f7ff1201f7586bfe48c634efdf8618b8c232b54"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-manilaclient/commit/2f7ff1201f7586bfe48c634efdf8618b8c232b54"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}