Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 1 | |
| 2 | import time |
| 3 | |
| 4 | from tcp_tests import logger |
| 5 | from tcp_tests.helpers.log_helpers import pretty_repr |
| 6 | |
| 7 | LOG = logger.logger |
| 8 | |
| 9 | |
| 10 | class ExecuteCommandsMixin(object): |
| 11 | """docstring for ExecuteCommands""" |
| 12 | |
Dmitry Tyzhnenko | bc0f826 | 2017-04-28 15:39:26 +0300 | [diff] [blame] | 13 | __config = None |
| 14 | __underlay = None |
| 15 | |
| 16 | def __init__(self, config, underlay): |
| 17 | self.__config = config |
| 18 | self.__underlay = underlay |
| 19 | super(ExecuteCommandsMixin, self).__init__() |
| 20 | |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 21 | def execute_commands(self, commands, label="Command"): |
| 22 | """Execute a sequence of commands |
| 23 | |
| 24 | Main propose is to implement workarounds for salt formulas like: |
| 25 | - exit_code == 0 when there are actual failures |
| 26 | - salt_master and/or salt_minion stop working after executing a formula |
| 27 | - a formula fails at first run, but completes at next runs |
| 28 | |
| 29 | :param label: label of the current sequence of the commands, for log |
| 30 | :param commands: list of dicts with the following data: |
| 31 | commands = [ |
| 32 | ... |
| 33 | { |
| 34 | # Required: |
| 35 | 'cmd': 'shell command(s) to run', |
| 36 | 'node_name': 'name of the node to run the command(s)', |
| 37 | # Optional: |
| 38 | 'description': 'string with a readable command description', |
| 39 | 'retry': { |
| 40 | 'count': int, # How many times should be run the command |
| 41 | # until success |
| 42 | 'delay': int, # Delay between tries in seconds |
| 43 | }, |
| 44 | 'skip_fail': bool # If True - continue with the next step |
| 45 | # without failure even if count number |
| 46 | # is reached. |
| 47 | # If False - rise an exception (default) |
| 48 | }, |
| 49 | ... |
| 50 | ] |
| 51 | """ |
| 52 | for n, step in enumerate(commands): |
| 53 | # Required fields |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 54 | action_cmd = step.get('cmd') |
| 55 | action_do = step.get('do') |
| 56 | action_upload = step.get('upload') |
| 57 | action_download = step.get('download') |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 58 | # node_name = step.get('node_name') |
| 59 | # Optional fields |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 60 | description = step.get('description', action_cmd) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 61 | # retry = step.get('retry', {'count': 1, 'delay': 1}) |
| 62 | # retry_count = retry.get('count', 1) |
| 63 | # retry_delay = retry.get('delay', 1) |
| 64 | # skip_fail = step.get('skip_fail', False) |
| 65 | |
| 66 | msg = "[ {0} #{1} ] {2}".format(label, n + 1, description) |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 67 | log_msg = "\n\n{0}\n{1}".format(msg, '=' * len(msg)) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 68 | |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 69 | if action_cmd: |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 70 | self.execute_command(step, msg) |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 71 | elif action_do: |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 72 | self.command2(step, msg) |
Dennis Dmitriev | 707bfeb | 2018-03-15 17:50:28 -0500 | [diff] [blame] | 73 | elif action_upload: |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 74 | LOG.info(log_msg) |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 75 | self.action_upload(step) |
| 76 | elif action_download: |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 77 | LOG.info(log_msg) |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 78 | self.action_download(step) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 79 | |
Tatyana Leontovich | afe8f95 | 2018-06-20 15:33:03 +0300 | [diff] [blame^] | 80 | def execute_command(self, step, msg, return_res=None): |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 81 | # Required fields |
| 82 | cmd = step.get('cmd') |
| 83 | node_name = step.get('node_name') |
| 84 | # Optional fields |
| 85 | description = step.get('description', cmd) |
| 86 | retry = step.get('retry', {'count': 1, 'delay': 1}) |
| 87 | retry_count = retry.get('count', 1) |
| 88 | retry_delay = retry.get('delay', 1) |
| 89 | skip_fail = step.get('skip_fail', False) |
| 90 | |
Dmitry Tyzhnenko | bc0f826 | 2017-04-28 15:39:26 +0300 | [diff] [blame] | 91 | with self.__underlay.remote(node_name=node_name) as remote: |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 92 | |
| 93 | for x in range(retry_count, 0, -1): |
| 94 | time.sleep(3) |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 95 | |
Dennis Dmitriev | ce646fb | 2018-03-21 09:10:00 +0200 | [diff] [blame] | 96 | retry_msg = (' (try {0} of {1}, skip_fail={2}, node_name={3})' |
| 97 | .format(retry_count - x + 1, |
| 98 | retry_count, |
| 99 | skip_fail, |
| 100 | node_name)) |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 101 | LOG.info("\n\n{0}\n{1}".format( |
| 102 | msg + retry_msg, '=' * len(msg + retry_msg))) |
| 103 | |
Dennis Dmitriev | f015f9f | 2018-03-30 08:25:44 +0300 | [diff] [blame] | 104 | result = remote.execute(cmd, verbose=True) |
Tatyana Leontovich | afe8f95 | 2018-06-20 15:33:03 +0300 | [diff] [blame^] | 105 | if return_res: |
| 106 | return result |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 107 | |
| 108 | # Workaround of exit code 0 from salt in case of failures |
| 109 | failed = 0 |
Dennis Dmitriev | 4db5bf2 | 2017-05-13 19:31:17 +0300 | [diff] [blame] | 110 | for s in result['stdout'] + result['stderr']: |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 111 | if s.startswith("Failed:"): |
| 112 | failed += int(s.split("Failed:")[1]) |
Dennis Dmitriev | 68671a6 | 2017-05-13 16:40:32 +0300 | [diff] [blame] | 113 | if 'Minion did not return. [No response]' in s: |
| 114 | failed += 1 |
Dennis Dmitriev | f854617 | 2017-07-20 21:57:05 +0300 | [diff] [blame] | 115 | if 'Minion did not return. [Not connected]' in s: |
| 116 | failed += 1 |
Dennis Dmitriev | 68671a6 | 2017-05-13 16:40:32 +0300 | [diff] [blame] | 117 | if s.startswith("[CRITICAL]"): |
| 118 | failed += 1 |
Tatyana Leontovich | 835465f | 2018-06-14 16:42:23 +0300 | [diff] [blame] | 119 | if 'Fatal' in s: |
| 120 | failed += 1 |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 121 | |
| 122 | if result.exit_code != 0: |
| 123 | time.sleep(retry_delay) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 124 | elif failed != 0: |
| 125 | LOG.error( |
| 126 | " === SALT returned exit code = 0 while " |
| 127 | "there are failed modules! ===") |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 128 | time.sleep(retry_delay) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 129 | else: |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 130 | break |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 131 | |
| 132 | if x == 1 and skip_fail is False: |
| 133 | # In the last retry iteration, raise an exception |
| 134 | raise Exception("Step '{0}' failed" |
| 135 | .format(description)) |
| 136 | |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 137 | def command2(self, step, msg): |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 138 | # Required fields |
| 139 | do = step['do'] |
| 140 | target = step['target'] |
| 141 | state = step.get('state') |
| 142 | states = step.get('states') |
| 143 | # Optional fields |
| 144 | args = step.get('args') |
| 145 | kwargs = step.get('kwargs') |
| 146 | description = step.get('description', do) |
| 147 | retry = step.get('retry', {'count': 1, 'delay': 1}) |
| 148 | retry_count = retry.get('count', 1) |
| 149 | retry_delay = retry.get('delay', 1) |
| 150 | skip_fail = step.get('skip_fail', False) |
| 151 | |
| 152 | if not bool(state) ^ bool(states): |
| 153 | raise ValueError("You should use state or states in step") |
| 154 | |
| 155 | for x in range(retry_count, 0, -1): |
| 156 | time.sleep(3) |
| 157 | |
Dennis Dmitriev | ce646fb | 2018-03-21 09:10:00 +0200 | [diff] [blame] | 158 | retry_msg = (' (try {0} of {1}, skip_fail={2}, target={3})' |
| 159 | .format(retry_count - x + 1, |
| 160 | retry_count, |
| 161 | skip_fail, |
| 162 | target)) |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 163 | LOG.info("\n\n{0}\n{1}".format( |
| 164 | msg + retry_msg, '=' * len(msg + retry_msg))) |
| 165 | |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 166 | method = getattr(self._salt, self._salt._map[do]) |
| 167 | command_ret = method(tgt=target, state=state or states, |
| 168 | args=args, kwargs=kwargs) |
| 169 | command_ret = command_ret if \ |
| 170 | isinstance(command_ret, list) else [command_ret] |
| 171 | results = [(r['return'][0], f) for r, f in command_ret] |
| 172 | |
| 173 | # FIMME: Change to debug level |
| 174 | LOG.info(" === States output =======================\n" |
| 175 | "{}\n" |
| 176 | " =========================================".format( |
| 177 | pretty_repr([r for r, f in results]))) |
| 178 | |
| 179 | all_fails = [f for r, f in results if f] |
| 180 | if all_fails: |
| 181 | LOG.error("States finished with failures.\n{}".format( |
| 182 | all_fails)) |
| 183 | time.sleep(retry_delay) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 184 | else: |
| 185 | break |
| 186 | |
| 187 | if x == 1 and skip_fail is False: |
| 188 | # In the last retry iteration, raise an exception |
| 189 | raise Exception("Step '{0}' failed" |
| 190 | .format(description)) |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 191 | |
| 192 | def action_upload(self, step): |
| 193 | """Upload from local host to environment node |
| 194 | |
| 195 | Example: |
| 196 | |
| 197 | - description: Upload a file |
| 198 | upload: |
| 199 | local_path: /tmp/ |
| 200 | local_filename: cirros*.iso |
| 201 | remote_path: /tmp/ |
| 202 | node_name: ctl01 |
| 203 | skip_fail: False |
| 204 | """ |
| 205 | node_name = step.get('node_name') |
| 206 | local_path = step.get('upload', {}).get('local_path', None) |
| 207 | local_filename = step.get('upload', {}).get('local_filename', None) |
| 208 | remote_path = step.get('upload', {}).get('remote_path', None) |
| 209 | description = step.get('description', local_path) |
| 210 | skip_fail = step.get('skip_fail', False) |
| 211 | |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 212 | if not local_path or not remote_path: |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 213 | raise Exception("Step '{0}' failed: please specify 'local_path', " |
| 214 | "'local_filename' and 'remote_path' correctly" |
| 215 | .format(description)) |
| 216 | |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 217 | if not local_filename: |
| 218 | # If local_path is not specified then uploading a directory |
| 219 | with self.__underlay.remote(node_name=node_name) as remote: |
| 220 | LOG.info("Uploading directory {0} to {1}:{2}" |
| 221 | .format(local_path, node_name, remote_path)) |
Dmitry Tyzhnenko | 8f6a63e | 2017-09-05 15:37:23 +0300 | [diff] [blame] | 222 | remote.upload(source=local_path.rstrip(), |
| 223 | target=remote_path.rstrip()) |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 224 | return |
| 225 | |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 226 | result = {} |
| 227 | with self.__underlay.local() as local: |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 228 | result = local.execute('cd {0} && find . -type f -name "{1}"' |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 229 | .format(local_path, local_filename)) |
| 230 | LOG.info("Found files to upload:\n{0}".format(result)) |
| 231 | |
| 232 | if not result['stdout'] and not skip_fail: |
| 233 | raise Exception("Nothing to upload on step {0}" |
| 234 | .format(description)) |
| 235 | |
| 236 | with self.__underlay.remote(node_name=node_name) as remote: |
| 237 | file_names = result['stdout'] |
| 238 | for file_name in file_names: |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 239 | source_path = local_path + file_name.rstrip() |
| 240 | destination_path = remote_path.rstrip() + file_name.rstrip() |
| 241 | LOG.info("Uploading file {0} to {1}:{2}" |
| 242 | .format(source_path, node_name, remote_path)) |
| 243 | remote.upload(source=source_path, target=destination_path) |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 244 | |
| 245 | def action_download(self, step): |
| 246 | """Download from environment node to local host |
| 247 | |
| 248 | Example: |
| 249 | |
| 250 | - description: Download a file |
| 251 | download: |
| 252 | remote_path: /tmp/ |
| 253 | remote_filename: report*.html |
| 254 | local_path: /tmp/ |
| 255 | node_name: ctl01 |
| 256 | skip_fail: False |
| 257 | """ |
| 258 | node_name = step.get('node_name') |
| 259 | remote_path = step.get('download', {}).get('remote_path', None) |
| 260 | remote_filename = step.get('download', {}).get('remote_filename', None) |
| 261 | local_path = step.get('download', {}).get('local_path', None) |
| 262 | description = step.get('description', remote_path) |
| 263 | skip_fail = step.get('skip_fail', False) |
| 264 | |
| 265 | if not remote_path or not remote_filename or not local_path: |
| 266 | raise Exception("Step '{0}' failed: please specify 'remote_path', " |
| 267 | "'remote_filename' and 'local_path' correctly" |
| 268 | .format(description)) |
| 269 | |
| 270 | with self.__underlay.remote(node_name=node_name) as remote: |
| 271 | |
| 272 | result = remote.execute('find {0} -type f -name {1}' |
| 273 | .format(remote_path, remote_filename)) |
| 274 | LOG.info("Found files to download:\n{0}".format(result)) |
| 275 | |
| 276 | if not result['stdout'] and not skip_fail: |
| 277 | raise Exception("Nothing to download on step {0}" |
| 278 | .format(description)) |
| 279 | |
| 280 | file_names = result['stdout'] |
| 281 | for file_name in file_names: |
| 282 | LOG.info("Downloading {0}:{1} to {2}" |
| 283 | .format(node_name, file_name, local_path)) |
| 284 | remote.download(destination=file_name.rstrip(), |
| 285 | target=local_path) |