)]}'
{"kayobe/tests/unit/test_kolla_ansible.py":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"6a01cf9931d14f6bd4043845bea8f7bb723ece90","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        ]"},{"line_number":109,"context_line":"        expected_cmd \u003d \" \".join(expected_cmd)"},{"line_number":110,"context_line":"        expected_env \u003d {\"KAYOBE_VAULT_PASSWORD\": \"test-pass\"}"},{"line_number":111,"context_line":"#                        \"KAYOBE_CONFIG_PATH\": \"/path/to/config\"}"},{"line_number":112,"context_line":"        expected_calls \u003d ["},{"line_number":113,"context_line":"            mock.call([\"which\", \"kayobe-vault-password-helper\"],"},{"line_number":114,"context_line":"                      check_output\u003dTrue),"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_f2955a32","line":111,"range":{"start_line":111,"start_character":1,"end_line":111,"end_character":25},"updated":"2018-11-05 11:34:23.000000000","message":"nit: Can we remove this?","commit_id":"eca80db1e57291377fa0d8d2e8e555783da3b4d9"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"dac0ae6edc21e40d758227213a945f556d31813a","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        ]"},{"line_number":109,"context_line":"        expected_cmd \u003d \" \".join(expected_cmd)"},{"line_number":110,"context_line":"        expected_env \u003d {\"KAYOBE_VAULT_PASSWORD\": \"test-pass\"}"},{"line_number":111,"context_line":"#                        \"KAYOBE_CONFIG_PATH\": \"/path/to/config\"}"},{"line_number":112,"context_line":"        expected_calls \u003d ["},{"line_number":113,"context_line":"            mock.call([\"which\", \"kayobe-vault-password-helper\"],"},{"line_number":114,"context_line":"                      check_output\u003dTrue),"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_554e7839","line":111,"range":{"start_line":111,"start_character":1,"end_line":111,"end_character":25},"in_reply_to":"3f79a3b5_f2955a32","updated":"2018-11-05 12:06:51.000000000","message":"Done","commit_id":"eca80db1e57291377fa0d8d2e8e555783da3b4d9"}],"kayobe/tests/unit/test_vault.py":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"47785dc24b5c8486648b6e984c538e4d725d2575","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        parser \u003d argparse.ArgumentParser()"},{"line_number":50,"context_line":"        vault.add_args(parser)"},{"line_number":51,"context_line":"        parsed_args \u003d parser.parse_args([\"--vault-password-file\","},{"line_number":52,"context_line":"                                         \"/path/to/file\"])"},{"line_number":53,"context_line":"        self.assertRaises(SystemExit, vault.validate_args, parsed_args)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @mock.patch.object(vault.getpass, \u0027getpass\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_6b0dc234","line":52,"updated":"2018-11-06 08:03:36.000000000","message":"Doh, thanks for fixing Doug.","commit_id":"358b5c68828e6e7de8bfc7654c7418c02380052b"}],"kayobe/vault.py":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"6a01cf9931d14f6bd4043845bea8f7bb723ece90","unresolved":false,"context_lines":[{"line_number":101,"context_line":"    # prompt has been requested via --ask-vault-pass, ensure the environment"},{"line_number":102,"context_line":"    # variable is set, so that it can be referenced by playbooks to generate"},{"line_number":103,"context_line":"    # the kolla-ansible passwords.yml file."},{"line_number":104,"context_line":"    if VAULT_PASSWORD_ENV in env:"},{"line_number":105,"context_line":"        return"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"    vault_password \u003d None"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_b21cc2c3","line":104,"range":{"start_line":104,"start_character":7,"end_line":104,"end_character":25},"updated":"2018-11-05 11:34:23.000000000","message":"Could this cause hair loss if VAULT_PASSWORD_ENV has been previously set to the wrong password, or the password for a different Kayobe deployment, but the user has specified the correct password using one of the above options?","commit_id":"eca80db1e57291377fa0d8d2e8e555783da3b4d9"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"dac0ae6edc21e40d758227213a945f556d31813a","unresolved":false,"context_lines":[{"line_number":101,"context_line":"    # prompt has been requested via --ask-vault-pass, ensure the environment"},{"line_number":102,"context_line":"    # variable is set, so that it can be referenced by playbooks to generate"},{"line_number":103,"context_line":"    # the kolla-ansible passwords.yml file."},{"line_number":104,"context_line":"    if VAULT_PASSWORD_ENV in env:"},{"line_number":105,"context_line":"        return"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"    vault_password \u003d None"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_d50848d9","line":104,"range":{"start_line":104,"start_character":7,"end_line":104,"end_character":25},"in_reply_to":"3f79a3b5_b21cc2c3","updated":"2018-11-05 12:06:51.000000000","message":"Good question. That seems like something we could catch and error on.","commit_id":"eca80db1e57291377fa0d8d2e8e555783da3b4d9"}]}