)]}'
{"requirements.txt":[{"author":{"_account_id":17386,"name":"Steve Noyes","email":"steve.noyes@oracle.com","username":"ssn1ssn"},"change_message_id":"32236ea9fa52e9b036fa9c37fd8d010846c57eac","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ansible\u003e\u003d2.2.0"},{"line_number":2,"context_line":"Babel\u003e\u003d0.9.6"},{"line_number":3,"context_line":"cliff\u003e\u003d1.13.0  # Apache-2.0"},{"line_number":4,"context_line":"cliff-tablib\u003c\u003d1.1"}],"source_content_type":"text/plain","patch_set":1,"id":"5f7c97a3_a96c539d","line":1,"updated":"2018-06-20 13:37:01.000000000","message":"I hadn\u0027t noticed this before. This seems odd. I don\u0027t see ansible at all in kolla-ansible/requirements.txt, so it seems this should probably be in one place, and that place is in kolla-ansible requirements? Right now, the only place the ansible requirement is handled by kollla-ansible in the setup_gate.sh CI/CD script.\n\nAnd if we do that, we should move the ansible requirement (with the new version) from here to test_requirements.txt as ansible is still needed for the functional tests.\n\nPaul, what do you think?","commit_id":"5cf3a4a5791fe10f639edfbfada79a0e4a1aed7b"},{"author":{"_account_id":16520,"name":"Borne Mace","email":"borne.mace@oracle.com","username":"bmace"},"change_message_id":"a15d1bdf7b242248ebc7143570acefff43035d92","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ansible\u003e\u003d2.2.0"},{"line_number":2,"context_line":"Babel\u003e\u003d0.9.6"},{"line_number":3,"context_line":"cliff\u003e\u003d1.13.0  # Apache-2.0"},{"line_number":4,"context_line":"cliff-tablib\u003c\u003d1.1"}],"source_content_type":"text/plain","patch_set":1,"id":"5f7c97a3_2e192421","line":1,"in_reply_to":"5f7c97a3_a96c539d","updated":"2018-06-20 16:17:38.000000000","message":"I agree.  I only ever added this because kolla-ansible didn\u0027t properly do the ansible dependency itself, and it really feels like that is where it should be.","commit_id":"5cf3a4a5791fe10f639edfbfada79a0e4a1aed7b"}]}