)]}'
{"scripts/bootstrap-ansible.sh":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"be671135a6bc5b62dbc3e96f22b14bd93fd2f2b7","unresolved":false,"context_lines":[{"line_number":72,"context_line":"          python-minimal python-dev \\"},{"line_number":73,"context_line":"          python3 python3-dev \\"},{"line_number":74,"context_line":"          libssl-dev libffi-dev \\"},{"line_number":75,"context_line":"          python-apt python3-apt"},{"line_number":76,"context_line":"        ;;"},{"line_number":77,"context_line":"esac"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"df3967d1_4dcf567f","line":75,"range":{"start_line":75,"start_character":10,"end_line":75,"end_character":21},"updated":"2017-07-31 07:12:13.000000000","message":"why adding this in the same commit?","commit_id":"a2c275ae274d71aca5056453d7534bfd8c9b6f17"},{"author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"change_message_id":"675889055c63ded5c06548963237fa904c3d14fc","unresolved":false,"context_lines":[{"line_number":72,"context_line":"          python-minimal python-dev \\"},{"line_number":73,"context_line":"          python3 python3-dev \\"},{"line_number":74,"context_line":"          libssl-dev libffi-dev \\"},{"line_number":75,"context_line":"          python-apt python3-apt"},{"line_number":76,"context_line":"        ;;"},{"line_number":77,"context_line":"esac"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"df3967d1_87f6368f","line":75,"range":{"start_line":75,"start_character":10,"end_line":75,"end_character":21},"in_reply_to":"df3967d1_4dcf567f","updated":"2017-07-31 11:10:25.000000000","message":"Because it was removed in I06cf934b3af1005d4b2adec76e22f249b72a69ec when the bootstrapping switched to python3. We\u0027re not \"adding\" anything, just ensuring we install the python2 version of the same dep as well.\n\nWe should be doing that even without this commit because one of our fallback options in the python detection mechanism below is python2.\n\nI can drop a line in the commit message if you\u0027d like?","commit_id":"a2c275ae274d71aca5056453d7534bfd8c9b6f17"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"2d5f014738f9a222540dc6ece1210a640c1b1942","unresolved":false,"context_lines":[{"line_number":72,"context_line":"          python-minimal python-dev \\"},{"line_number":73,"context_line":"          python3 python3-dev \\"},{"line_number":74,"context_line":"          libssl-dev libffi-dev \\"},{"line_number":75,"context_line":"          python-apt python3-apt"},{"line_number":76,"context_line":"        ;;"},{"line_number":77,"context_line":"esac"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"df3967d1_b90dde91","line":75,"range":{"start_line":75,"start_character":10,"end_line":75,"end_character":21},"in_reply_to":"df3967d1_87f6368f","updated":"2017-07-31 12:50:44.000000000","message":"I do agree that we should re-add this. It will be required when doing apt actions on the local host. It\u0027s probably best to at least mention it in the commit message so that this change doesn\u0027t seem out of place to the reviewer.","commit_id":"a2c275ae274d71aca5056453d7534bfd8c9b6f17"},{"author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"change_message_id":"8a4b69b2d1b08812c1ff4fb2ab28dba81c115b4c","unresolved":false,"context_lines":[{"line_number":72,"context_line":"          python-minimal python-dev \\"},{"line_number":73,"context_line":"          python3 python3-dev \\"},{"line_number":74,"context_line":"          libssl-dev libffi-dev \\"},{"line_number":75,"context_line":"          python-apt python3-apt"},{"line_number":76,"context_line":"        ;;"},{"line_number":77,"context_line":"esac"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"df3967d1_8c9d365b","line":75,"range":{"start_line":75,"start_character":10,"end_line":75,"end_character":21},"in_reply_to":"df3967d1_b90dde91","updated":"2017-07-31 13:20:58.000000000","message":"Sure makes sense. Will edit that real quick","commit_id":"a2c275ae274d71aca5056453d7534bfd8c9b6f17"}]}