)]}'
{"tempest/common/compute.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"f97bd429df38f5060cebcfc33fffa98f94a242c6","unresolved":false,"context_lines":[{"line_number":122,"context_line":"            # to aid in debugging ssh failures."},{"line_number":123,"context_line":"            script \u003d \u0027\u0027\u0027"},{"line_number":124,"context_line":"                     #!/bin/sh"},{"line_number":125,"context_line":"                     ip addr"},{"line_number":126,"context_line":"                     ip route"},{"line_number":127,"context_line":"                     \u0027\u0027\u0027"},{"line_number":128,"context_line":"            script_b64 \u003d base64.b64encode(textwrap.dedent(script))"},{"line_number":129,"context_line":"            kwargs[\u0027user_data\u0027] \u003d script_b64"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a1ff146_29104201","line":126,"range":{"start_line":125,"start_character":0,"end_line":126,"end_character":29},"updated":"2017-03-15 22:34:19.000000000","message":"NIT: I\u0027m not sure what the impact of a failing user_data script would be, but just in case, you may want to || true each line, just to make sure we don\u0027t fail boot on a system where \"ip\" is not in the PATH.","commit_id":"48eb95676b87c0cce699478bf4af8ea22cbc9e86"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"001c9408652b6584a58d4b99cf00e50ee71d90a3","unresolved":false,"context_lines":[{"line_number":122,"context_line":"            # to aid in debugging ssh failures."},{"line_number":123,"context_line":"            script \u003d b\u0027\u0027\u0027"},{"line_number":124,"context_line":"                     #!/bin/sh"},{"line_number":125,"context_line":"                     ip addr || true"},{"line_number":126,"context_line":"                     ip route || true"},{"line_number":127,"context_line":"                     \u0027\u0027\u0027"},{"line_number":128,"context_line":"            script_b64 \u003d base64.b64encode(textwrap.dedent(script))"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a1ff146_659faf50","line":125,"updated":"2017-03-16 09:07:39.000000000","message":"Those info are already dumped by cloudinit on the console log. The console log is dumped when there\u0027s a ssh failure so I am not sure I understand why we need this.","commit_id":"60d61cfe20ae2f568c2e0b237be5ed562b1a68e2"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"4803114cbac4b095a48ce4e2dbf71005933dd99d","unresolved":false,"context_lines":[{"line_number":123,"context_line":"            script \u003d b\u0027\u0027\u0027"},{"line_number":124,"context_line":"                     #!/bin/sh"},{"line_number":125,"context_line":"                     echo \"Printing cirros user authorized keys\""},{"line_number":126,"context_line":"                     cat /home/cirros/.ssh/authorized_keys || true"},{"line_number":127,"context_line":"                     \u0027\u0027\u0027"},{"line_number":128,"context_line":"            script_b64 \u003d base64.b64encode(textwrap.dedent(script))"},{"line_number":129,"context_line":"            kwargs[\u0027user_data\u0027] \u003d script_b64"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_ece67ae7","line":126,"updated":"2017-03-16 19:27:45.000000000","message":"Can you print all user\u0027s authorized keys ?","commit_id":"dbefb5a719a416b299e6f3e3e27a4abeaea116fc"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"7d758eb758946f29b21ce038e62379aa51d63ca1","unresolved":false,"context_lines":[{"line_number":125,"context_line":"                     echo \"Printing cirros user authorized keys\""},{"line_number":126,"context_line":"                     cat /home/cirros/.ssh/authorized_keys || true"},{"line_number":127,"context_line":"                     \u0027\u0027\u0027"},{"line_number":128,"context_line":"            script_b64 \u003d base64.b64encode(textwrap.dedent(script))"},{"line_number":129,"context_line":"            kwargs[\u0027user_data\u0027] \u003d script_b64"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    if volume_backed:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_4c514e62","line":128,"updated":"2017-03-16 19:32:15.000000000","message":"AS I remember pep8 allows you to move the extra lines  close the edge, and we do not need to post modifying a literal at run-time...","commit_id":"dbefb5a719a416b299e6f3e3e27a4abeaea116fc"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"107de54c8dc31070b0e3d63823b9d388cdc1eba1","unresolved":false,"context_lines":[{"line_number":123,"context_line":"            script \u003d b\u0027\u0027\u0027"},{"line_number":124,"context_line":"                     #!/bin/sh"},{"line_number":125,"context_line":"                     echo \"Printing cirros user authorized keys\""},{"line_number":126,"context_line":"                     cat /home/cirros/.ssh/authorized_keys || true"},{"line_number":127,"context_line":"                     \u0027\u0027\u0027"},{"line_number":128,"context_line":"            script_b64 \u003d base64.b64encode(textwrap.dedent(script).lstrip())"},{"line_number":129,"context_line":"            kwargs[\u0027user_data\u0027] \u003d script_b64"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a1ced50_f94a7282","line":126,"updated":"2017-03-20 07:28:29.000000000","message":"\u0027cirros\u0027 should not be hard-coded here, we could use the CONF.validation.image_ssh_user option, in order support other then cirros images.","commit_id":"a0a4d6483cbd02b0038b628ec0de48f135e189ab"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"62a9bcfdecbc56201d13da8d92f146f89ff094f5","unresolved":false,"context_lines":[{"line_number":122,"context_line":"            # to aid in debugging ssh failures."},{"line_number":123,"context_line":"            script \u003d \u0027\u0027\u0027"},{"line_number":124,"context_line":"                     #!/bin/sh"},{"line_number":125,"context_line":"                     echo \"Printing {user} user authorized keys\""},{"line_number":126,"context_line":"                     cat ~{user}/.ssh/authorized_keys || true"},{"line_number":127,"context_line":"                     \u0027\u0027\u0027.format(user\u003dCONF.validation.image_ssh_user)"},{"line_number":128,"context_line":"            script_clean \u003d textwrap.dedent(script).lstrip().encode(\u0027utf8\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a1ced50_34b630f9","line":125,"updated":"2017-03-23 10:06:42.000000000","message":"Where is it supposed to be printed ? Sorry I can\u0027t find it. A link maybe ?","commit_id":"844180e127ed0c5d64b0f2018424c0e25cae275f"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"4a315c2d1cfdb03d751ff101e61478d9bf1c2bd8","unresolved":false,"context_lines":[{"line_number":122,"context_line":"            # to aid in debugging ssh failures."},{"line_number":123,"context_line":"            script \u003d \u0027\u0027\u0027"},{"line_number":124,"context_line":"                     #!/bin/sh"},{"line_number":125,"context_line":"                     echo \"Printing {user} user authorized keys\""},{"line_number":126,"context_line":"                     cat ~{user}/.ssh/authorized_keys || true"},{"line_number":127,"context_line":"                     \u0027\u0027\u0027.format(user\u003dCONF.validation.image_ssh_user)"},{"line_number":128,"context_line":"            script_clean \u003d textwrap.dedent(script).lstrip().encode(\u0027utf8\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a1ced50_66947a8a","line":125,"in_reply_to":"1a1ced50_34b630f9","updated":"2017-03-23 12:17:27.000000000","message":"It\u0027s going on the console log. So you can see it in case of ssh failure.","commit_id":"844180e127ed0c5d64b0f2018424c0e25cae275f"}]}