)]}'
{".zuul.d/rally-tox-cli.yaml":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"change_message_id":"19c9d0c16864328bba4b0c369fdf889a133956f5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:\r"},{"line_number":2,"context_line":"    name: rally-tox-cli\r"},{"line_number":3,"context_line":"    parent: rally-tox\r"},{"line_number":4,"context_line":"    description:\r"},{"line_number":5,"context_line":"      Run cli test for rally-openstack project.\r"},{"line_number":6,"context_line":"      Uses tox with the ``cli`` environment.\r"},{"line_number":7,"context_line":"    vars:\r"},{"line_number":8,"context_line":"      rally_tox_env: \"cli\""}],"source_content_type":"text/x-yaml","patch_set":30,"id":"3f79a3b5_9b62af09","line":7,"range":{"start_line":1,"start_character":2,"end_line":7,"end_character":26},"updated":"2018-08-14 11:25:42.000000000","message":"Job rally-tox not defined","commit_id":"20aabd4b05ec252ca1517155afa038c16b54e2f3"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"2e6fdf5610e6a4bbf8456bbbd1f9c31c16537070","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:\r"},{"line_number":2,"context_line":"    name: rally-tox-cli\r"},{"line_number":3,"context_line":"    parent: rally-tox\r"},{"line_number":4,"context_line":"    description:\r"},{"line_number":5,"context_line":"      Run cli test for rally-openstack project.\r"},{"line_number":6,"context_line":"      Uses tox with the ``cli`` environment.\r"},{"line_number":7,"context_line":"    vars:\r"},{"line_number":8,"context_line":"      rally_tox_env: \"cli\""}],"source_content_type":"text/x-yaml","patch_set":31,"id":"3f79a3b5_ec573a57","line":8,"range":{"start_line":1,"start_character":0,"end_line":8,"end_character":26},"updated":"2018-08-16 10:13:46.000000000","message":"I\u0027m not sure that we need this cli job. It runs tests/cli tests which left from rally main repo and they test framework functionality (i.e. doesn\u0027t relate to openstack plugins itself).\n\nMaybe we need just remove the whole tests/cli dir? or you thing that we can add some test to check `rally env sheck` `rally deployment check` output for openstack platform. Thoughts?","commit_id":"cbcb1754015a40e40663bc4c902d658f678aaa91"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"908a076af2c7459414413b74f4d1ef167984d515","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:\r"},{"line_number":2,"context_line":"    name: rally-tox-cli\r"},{"line_number":3,"context_line":"    parent: rally-tox\r"},{"line_number":4,"context_line":"    description:\r"},{"line_number":5,"context_line":"      Run cli test for rally-openstack project.\r"},{"line_number":6,"context_line":"      Uses tox with the ``cli`` environment.\r"},{"line_number":7,"context_line":"    vars:\r"},{"line_number":8,"context_line":"      rally_tox_env: \"cli\""}],"source_content_type":"text/x-yaml","patch_set":31,"id":"3f79a3b5_38f488f0","line":8,"range":{"start_line":1,"start_character":0,"end_line":8,"end_character":26},"in_reply_to":"3f79a3b5_5f084e90","updated":"2018-08-16 13:02:46.000000000","message":"the main purpose of splitting CLI tests and checking samples in openstack/rally repo was an ability to run CLI tests without OpenStack. As for rally-openstack project we do not need to run those tests which do not require OpenStack, so I propose:\n1) merge checking samples back to functional tests\n2) remove all unrelated tests\n3) name the job \"functional\" :)","commit_id":"cbcb1754015a40e40663bc4c902d658f678aaa91"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"ced4f28966d47a69aa62c589508b768076bcfd79","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:\r"},{"line_number":2,"context_line":"    name: rally-tox-cli\r"},{"line_number":3,"context_line":"    parent: rally-tox\r"},{"line_number":4,"context_line":"    description:\r"},{"line_number":5,"context_line":"      Run cli test for rally-openstack project.\r"},{"line_number":6,"context_line":"      Uses tox with the ``cli`` environment.\r"},{"line_number":7,"context_line":"    vars:\r"},{"line_number":8,"context_line":"      rally_tox_env: \"cli\""}],"source_content_type":"text/x-yaml","patch_set":31,"id":"3f79a3b5_5f084e90","line":8,"range":{"start_line":1,"start_character":0,"end_line":8,"end_character":26},"in_reply_to":"3f79a3b5_ec573a57","updated":"2018-08-16 11:08:10.000000000","message":"Primarily use this job to check our samples which is in samples/tasks directory.\nAs you said,we could use `tox -e functional` to check in my thought, i will only save those test cases which is related with deployment.\nAlso, I agree you that we should rename them.","commit_id":"cbcb1754015a40e40663bc4c902d658f678aaa91"}]}