)]}'
{"tools/setup_Debian.sh":[{"author":{"_account_id":19300,"name":"Eric Lemoine","email":"elemoine@mirantis.com","username":"elemoine"},"change_message_id":"524d00ea01aa31136f849f0ee19dd2cc8f2fae4a","unresolved":false,"context_lines":[{"line_number":14,"context_line":"        # hkp://pool.sks-keyservers.net intermittenly doesnt have the correct"},{"line_number":15,"context_line":"        # keyring. p80 is what the docker script pulls from and what we should"},{"line_number":16,"context_line":"        # use for reliability too"},{"line_number":17,"context_line":"        sudo apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 58118E89F3A912897C070ADBF76221572C52609D \u0026\u0026 break || :"},{"line_number":18,"context_line":"        sleep 5"},{"line_number":19,"context_line":"    done"},{"line_number":20,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5a5ae5dd_aef0edc9","line":17,"updated":"2016-02-08 07:05:41.000000000","message":"Not that I\u0027d disagree with adding it, but I guess \"|| :\" is not strictly necessary here, no?","commit_id":"74cd09a3fc5a970c5147a6dccb6aa93aa989e48d"},{"author":{"_account_id":19300,"name":"Eric Lemoine","email":"elemoine@mirantis.com","username":"elemoine"},"change_message_id":"11802cbb2458fd2d36bcb5dd4e0d8609382a726b","unresolved":false,"context_lines":[{"line_number":14,"context_line":"        # hkp://pool.sks-keyservers.net intermittenly doesnt have the correct"},{"line_number":15,"context_line":"        # keyring. p80 is what the docker script pulls from and what we should"},{"line_number":16,"context_line":"        # use for reliability too"},{"line_number":17,"context_line":"        sudo apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 58118E89F3A912897C070ADBF76221572C52609D \u0026\u0026 break || :"},{"line_number":18,"context_line":"        sleep 5"},{"line_number":19,"context_line":"    done"},{"line_number":20,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5a5ae5dd_b16b7284","line":17,"in_reply_to":"5a5ae5dd_11c3668a","updated":"2016-02-08 07:40:51.000000000","message":"Right.  I had missed the `set -o errexit` above in the file.","commit_id":"74cd09a3fc5a970c5147a6dccb6aa93aa989e48d"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"a383ab33e356f315f65cee5f212431d369e95ee7","unresolved":false,"context_lines":[{"line_number":14,"context_line":"        # hkp://pool.sks-keyservers.net intermittenly doesnt have the correct"},{"line_number":15,"context_line":"        # keyring. p80 is what the docker script pulls from and what we should"},{"line_number":16,"context_line":"        # use for reliability too"},{"line_number":17,"context_line":"        sudo apt-key adv --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 58118E89F3A912897C070ADBF76221572C52609D \u0026\u0026 break || :"},{"line_number":18,"context_line":"        sleep 5"},{"line_number":19,"context_line":"    done"},{"line_number":20,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5a5ae5dd_11c3668a","line":17,"in_reply_to":"5a5ae5dd_aef0edc9","updated":"2016-02-08 07:37:25.000000000","message":"In this case it _is_ needed. We set errexit which means if this command errors we wuold exit the script. \u0027|| :\u0027 means if it fails, succeed. Basically it is akin to python \u0027pass\u0027 for an exception","commit_id":"74cd09a3fc5a970c5147a6dccb6aa93aa989e48d"}]}