)]}'
{"config_tempest/services/services.py":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1b1a2251d4ef050b1109b4ec49969d0203ab37b7","unresolved":false,"context_lines":[{"line_number":218,"context_line":"        # goal right now"},{"line_number":219,"context_line":"        self._conf.set(\u0027compute-feature-enabled\u0027,"},{"line_number":220,"context_line":"                       \u0027attach_encrypted_volume\u0027,"},{"line_number":221,"context_line":"                       str(self.is_service(\u0027key-manager\u0027)))"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"    def set_supported_api_versions(self):"},{"line_number":224,"context_line":"        # set supported API versions for services with more of them"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_8da730a8","line":221,"range":{"start_line":221,"start_character":23,"end_line":221,"end_character":59},"updated":"2018-05-23 10:19:55.000000000","message":"Can we add key-manager service discovery support in a seperate patch?","commit_id":"73fd402ba674574d22d7d5885c9783e3a315c493"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b5fa3eb4e27f85bdfbde2602390db2d54a1cb6ef","unresolved":false,"context_lines":[{"line_number":218,"context_line":"        # goal right now"},{"line_number":219,"context_line":"        self._conf.set(\u0027compute-feature-enabled\u0027,"},{"line_number":220,"context_line":"                       \u0027attach_encrypted_volume\u0027,"},{"line_number":221,"context_line":"                       str(self.is_service(\u0027key-manager\u0027)))"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"    def set_supported_api_versions(self):"},{"line_number":224,"context_line":"        # set supported API versions for services with more of them"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_8d71700f","line":221,"range":{"start_line":221,"start_character":23,"end_line":221,"end_character":59},"in_reply_to":"5f7c97a3_4d17d8a4","updated":"2018-05-23 10:32:28.000000000","message":"Incase of normal deployment, Barbican service will be enabled but barbican tempest plugin is not installed there in that case if someone needs to run encryption tests that time they need to enable barbican manually. So it is better to add a seperate patch to check for barbican service instead of dependent on barbican tempest plugin.","commit_id":"73fd402ba674574d22d7d5885c9783e3a315c493"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"6dc8268bf4b3d3dc1fb82bb46c4c5b20ebc8654b","unresolved":false,"context_lines":[{"line_number":218,"context_line":"        # goal right now"},{"line_number":219,"context_line":"        self._conf.set(\u0027compute-feature-enabled\u0027,"},{"line_number":220,"context_line":"                       \u0027attach_encrypted_volume\u0027,"},{"line_number":221,"context_line":"                       str(self.is_service(\u0027key-manager\u0027)))"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"    def set_supported_api_versions(self):"},{"line_number":224,"context_line":"        # set supported API versions for services with more of them"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_4d17d8a4","line":221,"range":{"start_line":221,"start_character":23,"end_line":221,"end_character":59},"in_reply_to":"5f7c97a3_8da730a8","updated":"2018-05-23 10:28:24.000000000","message":"Is it required? Right now key-manager only set barbican to true, which is the default in barbican-tempest-tests plugin.","commit_id":"73fd402ba674574d22d7d5885c9783e3a315c493"}]}