You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2021/02/19 08:45:12 UTC

[GitHub] [ignite] anton-vinogradov commented on a change in pull request #8686: IGNITE-14023 Password based authentication support in ducktape tests

anton-vinogradov commented on a change in pull request #8686:
URL: https://github.com/apache/ignite/pull/8686#discussion_r579017427



##########
File path: modules/ducktests/tests/ignitetest/services/utils/control_utility.py
##########
@@ -313,6 +305,41 @@ def __parse_output(raw_output):
     def __alives(self):
         return [node for node in self._cluster.nodes if self._cluster.alive(node)]
 
+    def _parse_ssl_params(self, user, globals_dict, **kwargs):

Review comment:
       @Sega76, Are you sure that writing something twice simplifies it?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org