)]}'
{"keystone/cmd/cli.py":[{"author":{"_account_id":27621,"name":"Vishakha Agarwal","email":"agarwalvishakha18@gmail.com","username":"Vishakha"},"change_message_id":"09c5f81693cef44c6591721492d829752caae0b3","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        if self.password is None:"},{"line_number":155,"context_line":"            print(_(\u0027ERROR: Either --bootstrap-password argument or \u0027"},{"line_number":156,"context_line":"                    \u0027OS_BOOTSTRAP_PASSWORD must be set.\u0027))"},{"line_number":157,"context_line":"            exit(1)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"        self.bootstrapper.admin_password \u003d self.password"},{"line_number":160,"context_line":"        self.bootstrapper.admin_username \u003d self.username"}],"source_content_type":"text/x-python","patch_set":2,"id":"bfb3d3c7_566455e4","line":157,"range":{"start_line":157,"start_character":12,"end_line":157,"end_character":17},"updated":"2019-05-23 06:26:43.000000000","message":"Better to use sys.exit(1)?","commit_id":"9c807cd23d1d9901379dde6324a3e963f0c30ceb"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"841c1e040b713a461cc3654dd374f9f3c608dc68","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        if self.password is None:"},{"line_number":155,"context_line":"            print(_(\u0027ERROR: Either --bootstrap-password argument or \u0027"},{"line_number":156,"context_line":"                    \u0027OS_BOOTSTRAP_PASSWORD must be set.\u0027))"},{"line_number":157,"context_line":"            exit(1)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"        self.bootstrapper.admin_password \u003d self.password"},{"line_number":160,"context_line":"        self.bootstrapper.admin_username \u003d self.username"}],"source_content_type":"text/x-python","patch_set":2,"id":"bfb3d3c7_e4bb4c6f","line":157,"range":{"start_line":157,"start_character":12,"end_line":157,"end_character":17},"in_reply_to":"bfb3d3c7_566455e4","updated":"2019-05-23 14:37:39.000000000","message":"Done","commit_id":"9c807cd23d1d9901379dde6324a3e963f0c30ceb"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"a49cd874434b8a42cb154800b2924a672ac97ded","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        if self.password is None:"},{"line_number":155,"context_line":"            print(_(\u0027ERROR: Either --bootstrap-password argument or \u0027"},{"line_number":156,"context_line":"                    \u0027OS_BOOTSTRAP_PASSWORD must be set.\u0027))"},{"line_number":157,"context_line":"            sys.exit(1)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"        self.bootstrapper.admin_password \u003d self.password"},{"line_number":160,"context_line":"        self.bootstrapper.admin_username \u003d self.username"}],"source_content_type":"text/x-python","patch_set":3,"id":"bfb3d3c7_7554a4fe","line":157,"updated":"2019-05-28 18:39:56.000000000","message":"An alternative would be to return 1 here and then do sys.exit(status) in main() the way we do for domain config upload https://opendev.org/openstack/keystone/src/branch/master/keystone/cmd/cli.py#L1100-L1102","commit_id":"295b07cc76c8387adc2fd9a8f3efb769f260ff02"}]}