Merge branch 'master' into remove_mistral

Author: arm4b

CHANGELOG.rst

@@ -31,6 +31,8 @@ Changed
   that does not exist. It now reports a simplified log for cleaner reading. Addresses and Fixes #4979. (improvement) #4981
 
   Contributed by Justin Sostre (@saucetray)
+* The built-in ``st2.action.file_writen`` trigger has been renamed to ``st2.action.file_written``
+  to fix the typo (bug fix) #4992
 
 Fixed
 ~~~~~
@@ -56,6 +58,8 @@ Fixed
   up correctly, specifically when specifying a bastion host / jump box. (bug fix) #4973
 
   Contributed by Nick Maludy (@nmaludy Encore Technologies)
+* Fixed a bytes/string encoding bug in the ``linux.dig`` action so it should work on Python 3
+  (bug fix) #4993
 
 * Fixed a bug where a python3 sensor using ssl needs to be monkey patched earlier. See also #4832, #4975 and gevent/gevent#1016 (bug fix) #4976
 
@@ -68,12 +72,9 @@ Fixed
   Contributed by Bradley Bishop (@bishopbm1 Encore Technologies)
 * Fix a regression when updated ``dnspython`` pip dependency resulted in
   st2 services unable to connect to mongodb remote host (bug fix) #4997
+* Fixed a regression in the ``linux.dig`` action on Python 3. (bug fix) #4993
 
-Changed
-~~~~~~~
-
-* The built-in ``st2.action.file_writen`` trigger has been renamed to ``st2.action.file_written``
-  to fix the typo (bug fix) #4992
+  Contributed by @blag
 
 Removed
 ~~~~~~~

OWNERS.md

@@ -32,9 +32,11 @@ Have deep platform knowledge & experience and demonstrate technical leadership a
 
 # Maintainers *
 ###### 1 vote points
-Being part of Technical Steering Committee (TSC) [@StackStorm/maintainers](https://github.com/orgs/StackStorm/teams/maintainers) provide significant and reliable value to the project helping it grow and improve through development and maintenance.
+Being part of Technical Steering Committee (TSC) [@StackStorm/maintainers](https://github.com/orgs/StackStorm/teams/maintainers) provide significant and reliable value to the project helping it grow and improve through development and maintenance. See [Maintainer Responsibilities](https://github.com/StackStorm/st2/blob/master/GOVERNANCE.md#maintainer-responsibilities) for more info.
+* Amanda McGuinness ([@amanda11](https://github.com/amanda11)) <<amanda.mcguinness@ammeonsolutions.com>>
+  - Ansible, Core, deb/rpm packages, CI/CD, Deployments, StackStorm Exchange, Documentation.
 * JP Bourget ([@punkrokk](https://github.com/punkrokk)) <<jp.bourget@gmail.com>>
-  - Systems, deb/rpm, Deployments, Community, StackStorm Exchange, SecOps, CircleCi
+  - Systems, deb/rpm, Deployments, Community, StackStorm Exchange, SecOps, CircleCI.
 * Mick McGrath ([@mickmcgrath13](https://github.com/mickmcgrath13)) <<mick@bitovi.com>>
   - Systems, ST2 Exchange. [Case Study](https://stackstorm.com/case-study-bitovi/).
 * Nick Maludy ([@nmaludy](https://github.com/nmaludy)) <<nmaludy@gmail.com>>
@@ -44,9 +46,8 @@ Being part of Technical Steering Committee (TSC) [@StackStorm/maintainers](https
 
 # Contributors
 Contributors are using and occasionally contributing back to the project, might be active in conversations or express their opinion on the project’s direction.
-They're not part of the TSC voting process, but appreciated for their contribution, involvement and may become Maintainers in future.
-<!-- [@StackStorm/contributors](https://github.com/orgs/StackStorm/teams/contributors) are invited to StackStorm Github organization and we appreciate their contribution and involvement. -->
-* Amanda McGuinness ([@amanda11](https://github.com/amanda11)) - StackStorm Exchange, Ansible, ChatOps, Documentation, core.
+They're not part of the TSC voting process, but appreciated for their contribution, involvement and may become Maintainers in the future depending on their effort and involvement. See [How to become a Maintainer?](https://github.com/StackStorm/st2/blob/master/GOVERNANCE.md#how-to-become-a-maintainer)
+[@StackStorm/contributors](https://github.com/orgs/StackStorm/teams/contributors) are invited to StackStorm Github organization and have permissions to help triage the Issues and review PRs.
 * Carlos ([@nzlosh](https://github.com/nzlosh)) - Chatops, Errbot, Community, Discussions, StackStorm Exchange.
 * Hiroyasu Ohyama ([@userlocalhost](https://github.com/userlocalhost)) - Orquesta, Workflows, st2 Japan Community. [Case Study](https://stackstorm.com/case-study-dmm/).
 * Jon Middleton ([@jjm](https://github.com/jjm)) - StackStorm Exchange, Core, Discussions.

contrib/linux/actions/dig.py

@@ -16,9 +16,10 @@
 # limitations under the License.
 
 import errno
+import locale
 import subprocess
 import random
-import re
+import sys
 
 from st2common.runners.base_action import Action
 
@@ -34,26 +35,33 @@ def run(self, rand, count, nameserver, hostname, queryopts):
             nameserver = '@' + nameserver
             cmd_args.append(nameserver)
 
-        if re.search(',', queryopts):
+        if isinstance(queryopts, str) and ',' in queryopts:
             opt_list = queryopts.split(',')
         else:
             opt_list.append(queryopts)
-        for k, v in enumerate(opt_list):
-            cmd_args.append('+' + v)
+
+        cmd_args.extend(['+' + option for option in opt_list])
 
         cmd_args.append(hostname)
 
         try:
-            result_list = filter(None, subprocess.Popen(cmd_args,
-                                                        stderr=subprocess.PIPE,
-                                                        stdout=subprocess.PIPE)
-                                 .communicate()[0]
-                                 .split('\n'))
+            raw_result = subprocess.Popen(cmd_args,
+                                          stderr=subprocess.PIPE,
+                                          stdout=subprocess.PIPE).communicate()[0]
+
+            if sys.version_info >= (3,):
+                # This function might call getpreferred encoding unless we pass
+                # do_setlocale=False.
+                encoding = locale.getpreferredencoding(do_setlocale=False)
+                result_list_str = raw_result.decode(encoding)
+            else:
+                result_list_str = str(raw_result)
+
+            result_list = list(filter(None, result_list_str.split('\n')))
 
         # NOTE: Python3 supports the FileNotFoundError, the errono.ENOENT is for py2 compat
         # for Python3:
         # except FileNotFoundError as e:
-
         except OSError as e:
             if e.errno == errno.ENOENT:
                 return False, "Can't find dig installed in the path (usually /usr/bin/dig). If " \

contrib/linux/tests/test_action_dig.py

@@ -0,0 +1,55 @@
+#!/usr/bin/env python
+
+# Copyright 2020 The StackStorm Authors.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+from __future__ import absolute_import
+from st2tests.base import BaseActionTestCase
+
+from dig import DigAction
+
+
+class DigActionTestCase(BaseActionTestCase):
+    action_cls = DigAction
+
+    def test_run_with_empty_hostname(self):
+        action = self.get_action_instance()
+
+        # Use the defaults from dig.yaml
+        result = action.run(rand=False, count=0, nameserver=None, hostname='', queryopts='short')
+        self.assertIsInstance(result, list)
+        self.assertEqual(len(result), 0)
+
+    def test_run_with_empty_queryopts(self):
+        action = self.get_action_instance()
+
+        results = action.run(rand=False, count=0, nameserver=None, hostname='google.com',
+                             queryopts='')
+        self.assertIsInstance(results, list)
+
+        for result in results:
+            self.assertIsInstance(result, str)
+            self.assertGreater(len(result), 0)
+
+    def test_run(self):
+        action = self.get_action_instance()
+
+        results = action.run(rand=False, count=0, nameserver=None, hostname='google.com',
+                             queryopts='short')
+        self.assertIsInstance(results, list)
+        self.assertGreater(len(results), 0)
+
+        for result in results:
+            self.assertIsInstance(result, str)
+            self.assertGreater(len(result), 0)