)]}'
{"README.md":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"bba378d35552a2a396f8cb6bf5a62a1302e86512","unresolved":false,"context_lines":[{"line_number":25,"context_line":"OpenStack Infrastructure is deploying two jobs per supported Operating System"},{"line_number":26,"context_line":"(Ubuntu and CentOS): scenario001 and scenario002."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"OpenStack services are balanced between two scenarios because OpenStack"},{"line_number":29,"context_line":"Infastructure Jenkins slaves can not afford the load of running all on the"},{"line_number":30,"context_line":"same node."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":8,"id":"fa80f949_9459cf5c","line":28,"updated":"2015-11-05 20:08:20.000000000","message":"Looks like there are three scenarios now","commit_id":"3127e1223a6c9f2a69207d503ba88eab007be6f3"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"e515605a331156e5b40a6f621ced519f7b5c32b8","unresolved":false,"context_lines":[{"line_number":25,"context_line":"OpenStack Infrastructure is deploying two jobs per supported Operating System"},{"line_number":26,"context_line":"(Ubuntu and CentOS): scenario001 and scenario002."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"OpenStack services are balanced between two scenarios because OpenStack"},{"line_number":29,"context_line":"Infastructure Jenkins slaves can not afford the load of running all on the"},{"line_number":30,"context_line":"same node."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":8,"id":"fa80f949_0f0aa821","line":28,"in_reply_to":"fa80f949_9459cf5c","updated":"2015-11-05 20:10:17.000000000","message":"oops, nice catch !","commit_id":"3127e1223a6c9f2a69207d503ba88eab007be6f3"}],"functions":[{"author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"change_message_id":"6738ac98ad6e0cdeeb46b1646d600006369f80f8","unresolved":false,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"is_fedora() {"},{"line_number":73,"context_line":"    # note we consider CentOS 7 as fedora for now"},{"line_number":74,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"fedora\" || \\"},{"line_number":75,"context_line":"        lsb_release -i 2\u003e/dev/null | grep -iq \"CentOS\""},{"line_number":76,"context_line":"}"},{"line_number":77,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"fa80f949_00def53e","line":74,"updated":"2015-11-05 21:59:13.000000000","message":"using /etc/os-release might be cleaner:\nsource /etc/os-release\ntest \"$ID\" \u003d \"fedora\" -o \"$ID\" \u003d \"centos\"","commit_id":"2ff1c3104b2cdba3bba092fe8a690de9c9c0c8dd"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"f7043c13452fc1e40a82ea8dad0e066a0a795626","unresolved":false,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"is_fedora() {"},{"line_number":73,"context_line":"    # note we consider CentOS 7 as fedora for now"},{"line_number":74,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"fedora\" || \\"},{"line_number":75,"context_line":"        lsb_release -i 2\u003e/dev/null | grep -iq \"CentOS\""},{"line_number":76,"context_line":"}"},{"line_number":77,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"fa80f949_f3493fb8","line":74,"in_reply_to":"fa80f949_00def53e","updated":"2015-11-06 08:38:40.000000000","message":"It make sense. Also because Centos-7 may doesn\u0027t have lsb_release command:\n\n  lsb_release -i\n  -bash: lsb_release: command not found\n\nBut /etc/os-release works.","commit_id":"2ff1c3104b2cdba3bba092fe8a690de9c9c0c8dd"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"280e920fa85cb17f1aa1bdddd6649afbd9f169b9","unresolved":false,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"is_fedora() {"},{"line_number":73,"context_line":"    # note we consider CentOS 7 as fedora for now"},{"line_number":74,"context_line":"    lsb_release -i 2\u003e/dev/null | grep -iq \"fedora\" || \\"},{"line_number":75,"context_line":"        lsb_release -i 2\u003e/dev/null | grep -iq \"CentOS\""},{"line_number":76,"context_line":"}"},{"line_number":77,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":10,"id":"fa80f949_f03e078b","line":74,"in_reply_to":"fa80f949_f3493fb8","updated":"2015-11-06 13:25:53.000000000","message":"this is a move from run_tests.sh but I can change it, ok.","commit_id":"2ff1c3104b2cdba3bba092fe8a690de9c9c0c8dd"}]}