)]}'
{"tools/fix-openstack-quotas.py":[{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"ec6ea972af9eda8e073d5ee396a875409a856a7e","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        body \u003d {\u0027floatingip\u0027: \u0027100\u0027,"},{"line_number":42,"context_line":"                \u0027network\u0027: 1000,"},{"line_number":43,"context_line":"                \u0027security_group\u0027: 200,"},{"line_number":44,"context_line":"                \u0027router\u0027: 1000, "},{"line_number":45,"context_line":"                \u0027port\u0027: 1000}"},{"line_number":46,"context_line":"        self.neutron.update_quota(\u0027admin\u0027, {\"quota\": body})"},{"line_number":47,"context_line":"        result \u003d self.neutron.show_quota(\u0027admin\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_353b293a","line":44,"range":{"start_line":44,"start_character":31,"end_line":44,"end_character":32},"updated":"2016-03-02 14:27:43.000000000","message":"Please remove white spaces","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":18903,"name":"Yuli","email":"stremovsky@gmail.com","username":"yuli"},"change_message_id":"f9d2716112c8619d7606f85499e7c8555ff4abd9","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        body \u003d {\u0027floatingip\u0027: \u0027100\u0027,"},{"line_number":42,"context_line":"                \u0027network\u0027: 1000,"},{"line_number":43,"context_line":"                \u0027security_group\u0027: 200,"},{"line_number":44,"context_line":"                \u0027router\u0027: 1000, "},{"line_number":45,"context_line":"                \u0027port\u0027: 1000}"},{"line_number":46,"context_line":"        self.neutron.update_quota(\u0027admin\u0027, {\"quota\": body})"},{"line_number":47,"context_line":"        result \u003d self.neutron.show_quota(\u0027admin\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_7073ff15","line":44,"range":{"start_line":44,"start_character":31,"end_line":44,"end_character":32},"in_reply_to":"7af24918_353b293a","updated":"2016-03-02 14:53:35.000000000","message":"Done","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"68d514d586edb0f41974225193bb95f219ae9f31","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    \"\"\"Retrieves credentials to run functional tests\"\"\""},{"line_number":29,"context_line":"    return get_cloud_config(cloud\u003dcloud).get_auth_args()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class FixQuota():"},{"line_number":32,"context_line":"    def __init__(self):"},{"line_number":33,"context_line":"        creds \u003d credentials()"},{"line_number":34,"context_line":"        tenant_name \u003d creds[\u0027project_name\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"7af24918_43d4eabb","line":31,"range":{"start_line":31,"start_character":6,"end_line":31,"end_character":14},"updated":"2016-03-03 14:30:16.000000000","message":"I am not sure this is the best Name \n\nI would call it UpdateQuota \n\nPlease consider allowing to be configure\n\nCurrently the  quota is hard coded","commit_id":"15648e0c15e9d71d1d255bb61a7dce8ff30e0be0"},{"author":{"_account_id":18903,"name":"Yuli","email":"stremovsky@gmail.com","username":"yuli"},"change_message_id":"2ba2aae70e8b63f8406c9205c766c2a439520953","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    \"\"\"Retrieves credentials to run functional tests\"\"\""},{"line_number":29,"context_line":"    return get_cloud_config(cloud\u003dcloud).get_auth_args()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class FixQuota():"},{"line_number":32,"context_line":"    def __init__(self):"},{"line_number":33,"context_line":"        creds \u003d credentials()"},{"line_number":34,"context_line":"        tenant_name \u003d creds[\u0027project_name\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"7af24918_d9a34d4e","line":31,"range":{"start_line":31,"start_character":6,"end_line":31,"end_character":14},"in_reply_to":"7af24918_43d4eabb","updated":"2016-03-07 07:15:29.000000000","message":"I chaged the name as you sugegsted.\n\nHow you suggest to make it configurable ?","commit_id":"15648e0c15e9d71d1d255bb61a7dce8ff30e0be0"},{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"68d514d586edb0f41974225193bb95f219ae9f31","unresolved":false,"context_lines":[{"line_number":38,"context_line":"             tenant_name\u003dtenant_name)"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def fix(self):"},{"line_number":41,"context_line":"        body \u003d {\u0027floatingip\u0027: \u0027100\u0027,"},{"line_number":42,"context_line":"                \u0027network\u0027: 1000,"},{"line_number":43,"context_line":"                \u0027security_group\u0027: 200,"},{"line_number":44,"context_line":"                \u0027router\u0027: 1000,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7af24918_43b34ae9","line":41,"range":{"start_line":41,"start_character":30,"end_line":41,"end_character":35},"updated":"2016-03-03 14:30:16.000000000","message":"why floating ip is a string and all the are numbers","commit_id":"15648e0c15e9d71d1d255bb61a7dce8ff30e0be0"},{"author":{"_account_id":18903,"name":"Yuli","email":"stremovsky@gmail.com","username":"yuli"},"change_message_id":"2ba2aae70e8b63f8406c9205c766c2a439520953","unresolved":false,"context_lines":[{"line_number":38,"context_line":"             tenant_name\u003dtenant_name)"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def fix(self):"},{"line_number":41,"context_line":"        body \u003d {\u0027floatingip\u0027: \u0027100\u0027,"},{"line_number":42,"context_line":"                \u0027network\u0027: 1000,"},{"line_number":43,"context_line":"                \u0027security_group\u0027: 200,"},{"line_number":44,"context_line":"                \u0027router\u0027: 1000,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7af24918_198655b9","line":41,"range":{"start_line":41,"start_character":30,"end_line":41,"end_character":35},"in_reply_to":"7af24918_43b34ae9","updated":"2016-03-07 07:15:29.000000000","message":"I chaged to int.\nWhen I will have a working envrironment I will test it.","commit_id":"15648e0c15e9d71d1d255bb61a7dce8ff30e0be0"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"f98d3d80ed0de6f1615ed15cbb618f680556bfb7","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    \"\"\"Retrieves credentials to run functional tests\"\"\""},{"line_number":29,"context_line":"    return get_cloud_config(cloud\u003dcloud).get_auth_args()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class UpdateQuota():"},{"line_number":32,"context_line":"    def __init__(self):"},{"line_number":33,"context_line":"        creds \u003d credentials()"},{"line_number":34,"context_line":"        tenant_name \u003d creds[\u0027project_name\u0027]"}],"source_content_type":"text/x-python","patch_set":6,"id":"5aef4532_bf900197","line":31,"updated":"2016-03-07 13:46:02.000000000","message":"To make the class configurable: The parameters can be class properties/members, passed to __init__(...), or passed to fix(...).","commit_id":"2b8dfc30b44265a9ca3005a909688488dfe16594"}],"tox.ini":[{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"ec6ea972af9eda8e073d5ee396a875409a856a7e","unresolved":false,"context_lines":[{"line_number":14,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":15,"context_line":"whitelist_externals \u003d sh"},{"line_number":16,"context_line":"commands \u003d"},{"line_number":17,"context_line":"  python tools/fix-openstack-quotas.py"},{"line_number":18,"context_line":"  sh tools/pretty_tox.sh \u0027{posargs}\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:fullstack]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7af24918_95079d1b","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":38},"updated":"2016-03-02 14:27:43.000000000","message":"Try to run this command only in the context of the fullstack","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":18903,"name":"Yuli","email":"stremovsky@gmail.com","username":"yuli"},"change_message_id":"00798a124232ce9a7152ec415fe013783d8c13c7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":15,"context_line":"whitelist_externals \u003d sh"},{"line_number":16,"context_line":"commands \u003d"},{"line_number":17,"context_line":"  python tools/fix-openstack-quotas.py"},{"line_number":18,"context_line":"  sh tools/pretty_tox.sh \u0027{posargs}\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:fullstack]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7af24918_7981598f","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":38},"in_reply_to":"7af24918_834a7224","updated":"2016-03-07 07:46:24.000000000","message":"In new revision I moved entries from the [testenv] to [testenv:fullstack]","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":18903,"name":"Yuli","email":"stremovsky@gmail.com","username":"yuli"},"change_message_id":"f9d2716112c8619d7606f85499e7c8555ff4abd9","unresolved":false,"context_lines":[{"line_number":14,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":15,"context_line":"whitelist_externals \u003d sh"},{"line_number":16,"context_line":"commands \u003d"},{"line_number":17,"context_line":"  python tools/fix-openstack-quotas.py"},{"line_number":18,"context_line":"  sh tools/pretty_tox.sh \u0027{posargs}\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:fullstack]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7af24918_9d75039a","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":38},"in_reply_to":"7af24918_95079d1b","updated":"2016-03-02 14:53:35.000000000","message":"Hello\n\nCurrently it is only executed only for fullstack.\nOther sections have the \"commands\" actions that prevents further command execution.","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"3c9e9985f477baaa302a37a581a16f377d234849","unresolved":false,"context_lines":[{"line_number":14,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":15,"context_line":"whitelist_externals \u003d sh"},{"line_number":16,"context_line":"commands \u003d"},{"line_number":17,"context_line":"  python tools/fix-openstack-quotas.py"},{"line_number":18,"context_line":"  sh tools/pretty_tox.sh \u0027{posargs}\u0027"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:fullstack]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7af24918_834a7224","line":17,"range":{"start_line":17,"start_character":2,"end_line":17,"end_character":38},"in_reply_to":"7af24918_9d75039a","updated":"2016-03-03 14:24:50.000000000","message":"This is not true look why the CI test is failing in py 2.7 \nI do not see why you want to run it implicitly and not explicitly please elaborate","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"3c9e9985f477baaa302a37a581a16f377d234849","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:fullstack]"},{"line_number":21,"context_line":"basepython \u003d python2.7"},{"line_number":22,"context_line":"setenv \u003d OS_TEST_PATH\u003d./dragonflow/tests/fullstack"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"[testenv:pep8]"},{"line_number":25,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7af24918_43f6aaf9","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":6},"updated":"2016-03-03 14:24:50.000000000","message":"Unless I am missing some thing  why not run it here \ncommands \u003d\n  python tools/fix-openstack-quotas.py","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":18903,"name":"Yuli","email":"stremovsky@gmail.com","username":"yuli"},"change_message_id":"00798a124232ce9a7152ec415fe013783d8c13c7","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[testenv:fullstack]"},{"line_number":21,"context_line":"basepython \u003d python2.7"},{"line_number":22,"context_line":"setenv \u003d OS_TEST_PATH\u003d./dragonflow/tests/fullstack"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"[testenv:pep8]"},{"line_number":25,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"7af24918_59557d1b","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":6},"in_reply_to":"7af24918_43f6aaf9","updated":"2016-03-07 07:46:24.000000000","message":"I had to add all other entries from [testenv] to [testenv:fullstack] because commands from different sections can are not executed. Only one command block is executed.","commit_id":"c3f7eb8fd9808498519737f48c391b6ad0ba3fa1"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"f98d3d80ed0de6f1615ed15cbb618f680556bfb7","unresolved":false,"context_lines":[{"line_number":21,"context_line":"setenv \u003d OS_TEST_PATH\u003d./dragonflow/tests/fullstack"},{"line_number":22,"context_line":"         VIRTUAL_ENV\u003d{envdir}"},{"line_number":23,"context_line":"         PYTHONHASHSEED\u003d0"},{"line_number":24,"context_line":"usedevelop \u003d True"},{"line_number":25,"context_line":"install_command \u003d {toxinidir}/tools/tox_install.sh {opts} {packages}"},{"line_number":26,"context_line":"deps \u003d -r{toxinidir}/requirements.txt"},{"line_number":27,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":28,"context_line":"whitelist_externals \u003d sh"},{"line_number":29,"context_line":"commands \u003d"},{"line_number":30,"context_line":"  python tools/fix-openstack-quotas.py"}],"source_content_type":"text/x-properties","patch_set":6,"id":"5aef4532_bf55a1c7","line":27,"range":{"start_line":24,"start_character":0,"end_line":27,"end_character":42},"updated":"2016-03-07 13:46:02.000000000","message":"Aren\u0027t these parameter inherited from [testenv]?","commit_id":"2b8dfc30b44265a9ca3005a909688488dfe16594"}]}