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 | |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 80 | def execute_command(self, step, msg): |
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) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 105 | |
| 106 | # Workaround of exit code 0 from salt in case of failures |
| 107 | failed = 0 |
Dennis Dmitriev | 4db5bf2 | 2017-05-13 19:31:17 +0300 | [diff] [blame] | 108 | for s in result['stdout'] + result['stderr']: |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 109 | if s.startswith("Failed:"): |
| 110 | failed += int(s.split("Failed:")[1]) |
Dennis Dmitriev | 68671a6 | 2017-05-13 16:40:32 +0300 | [diff] [blame] | 111 | if 'Minion did not return. [No response]' in s: |
| 112 | failed += 1 |
Dennis Dmitriev | f854617 | 2017-07-20 21:57:05 +0300 | [diff] [blame] | 113 | if 'Minion did not return. [Not connected]' in s: |
| 114 | failed += 1 |
Dennis Dmitriev | 68671a6 | 2017-05-13 16:40:32 +0300 | [diff] [blame] | 115 | if s.startswith("[CRITICAL]"): |
| 116 | failed += 1 |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 117 | |
| 118 | if result.exit_code != 0: |
| 119 | time.sleep(retry_delay) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 120 | elif failed != 0: |
| 121 | LOG.error( |
| 122 | " === SALT returned exit code = 0 while " |
| 123 | "there are failed modules! ===") |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 124 | time.sleep(retry_delay) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 125 | else: |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 126 | break |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 127 | |
| 128 | if x == 1 and skip_fail is False: |
| 129 | # In the last retry iteration, raise an exception |
| 130 | raise Exception("Step '{0}' failed" |
| 131 | .format(description)) |
| 132 | |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 133 | def command2(self, step, msg): |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 134 | # Required fields |
| 135 | do = step['do'] |
| 136 | target = step['target'] |
| 137 | state = step.get('state') |
| 138 | states = step.get('states') |
| 139 | # Optional fields |
| 140 | args = step.get('args') |
| 141 | kwargs = step.get('kwargs') |
| 142 | description = step.get('description', do) |
| 143 | retry = step.get('retry', {'count': 1, 'delay': 1}) |
| 144 | retry_count = retry.get('count', 1) |
| 145 | retry_delay = retry.get('delay', 1) |
| 146 | skip_fail = step.get('skip_fail', False) |
| 147 | |
| 148 | if not bool(state) ^ bool(states): |
| 149 | raise ValueError("You should use state or states in step") |
| 150 | |
| 151 | for x in range(retry_count, 0, -1): |
| 152 | time.sleep(3) |
| 153 | |
Dennis Dmitriev | ce646fb | 2018-03-21 09:10:00 +0200 | [diff] [blame] | 154 | retry_msg = (' (try {0} of {1}, skip_fail={2}, target={3})' |
| 155 | .format(retry_count - x + 1, |
| 156 | retry_count, |
| 157 | skip_fail, |
| 158 | target)) |
Dennis Dmitriev | c83b3d4 | 2018-03-16 00:59:18 +0200 | [diff] [blame] | 159 | LOG.info("\n\n{0}\n{1}".format( |
| 160 | msg + retry_msg, '=' * len(msg + retry_msg))) |
| 161 | |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 162 | method = getattr(self._salt, self._salt._map[do]) |
| 163 | command_ret = method(tgt=target, state=state or states, |
| 164 | args=args, kwargs=kwargs) |
| 165 | command_ret = command_ret if \ |
| 166 | isinstance(command_ret, list) else [command_ret] |
| 167 | results = [(r['return'][0], f) for r, f in command_ret] |
| 168 | |
| 169 | # FIMME: Change to debug level |
| 170 | LOG.info(" === States output =======================\n" |
| 171 | "{}\n" |
| 172 | " =========================================".format( |
| 173 | pretty_repr([r for r, f in results]))) |
| 174 | |
| 175 | all_fails = [f for r, f in results if f] |
| 176 | if all_fails: |
| 177 | LOG.error("States finished with failures.\n{}".format( |
| 178 | all_fails)) |
| 179 | time.sleep(retry_delay) |
Dmitry Tyzhnenko | 2b730a0 | 2017-04-07 19:31:32 +0300 | [diff] [blame] | 180 | else: |
| 181 | break |
| 182 | |
| 183 | if x == 1 and skip_fail is False: |
| 184 | # In the last retry iteration, raise an exception |
| 185 | raise Exception("Step '{0}' failed" |
| 186 | .format(description)) |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 187 | |
| 188 | def action_upload(self, step): |
| 189 | """Upload from local host to environment node |
| 190 | |
| 191 | Example: |
| 192 | |
| 193 | - description: Upload a file |
| 194 | upload: |
| 195 | local_path: /tmp/ |
| 196 | local_filename: cirros*.iso |
| 197 | remote_path: /tmp/ |
| 198 | node_name: ctl01 |
| 199 | skip_fail: False |
| 200 | """ |
| 201 | node_name = step.get('node_name') |
| 202 | local_path = step.get('upload', {}).get('local_path', None) |
| 203 | local_filename = step.get('upload', {}).get('local_filename', None) |
| 204 | remote_path = step.get('upload', {}).get('remote_path', None) |
| 205 | description = step.get('description', local_path) |
| 206 | skip_fail = step.get('skip_fail', False) |
| 207 | |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 208 | if not local_path or not remote_path: |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 209 | raise Exception("Step '{0}' failed: please specify 'local_path', " |
| 210 | "'local_filename' and 'remote_path' correctly" |
| 211 | .format(description)) |
| 212 | |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 213 | if not local_filename: |
| 214 | # If local_path is not specified then uploading a directory |
| 215 | with self.__underlay.remote(node_name=node_name) as remote: |
| 216 | LOG.info("Uploading directory {0} to {1}:{2}" |
| 217 | .format(local_path, node_name, remote_path)) |
Dmitry Tyzhnenko | 8f6a63e | 2017-09-05 15:37:23 +0300 | [diff] [blame] | 218 | remote.upload(source=local_path.rstrip(), |
| 219 | target=remote_path.rstrip()) |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 220 | return |
| 221 | |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 222 | result = {} |
| 223 | with self.__underlay.local() as local: |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 224 | result = local.execute('cd {0} && find . -type f -name "{1}"' |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 225 | .format(local_path, local_filename)) |
| 226 | LOG.info("Found files to upload:\n{0}".format(result)) |
| 227 | |
| 228 | if not result['stdout'] and not skip_fail: |
| 229 | raise Exception("Nothing to upload on step {0}" |
| 230 | .format(description)) |
| 231 | |
| 232 | with self.__underlay.remote(node_name=node_name) as remote: |
| 233 | file_names = result['stdout'] |
| 234 | for file_name in file_names: |
Dennis Dmitriev | 9dada8a | 2017-08-30 17:38:55 +0300 | [diff] [blame] | 235 | source_path = local_path + file_name.rstrip() |
| 236 | destination_path = remote_path.rstrip() + file_name.rstrip() |
| 237 | LOG.info("Uploading file {0} to {1}:{2}" |
| 238 | .format(source_path, node_name, remote_path)) |
| 239 | remote.upload(source=source_path, target=destination_path) |
Dennis Dmitriev | 2dfb8ef | 2017-07-21 20:19:38 +0300 | [diff] [blame] | 240 | |
| 241 | def action_download(self, step): |
| 242 | """Download from environment node to local host |
| 243 | |
| 244 | Example: |
| 245 | |
| 246 | - description: Download a file |
| 247 | download: |
| 248 | remote_path: /tmp/ |
| 249 | remote_filename: report*.html |
| 250 | local_path: /tmp/ |
| 251 | node_name: ctl01 |
| 252 | skip_fail: False |
| 253 | """ |
| 254 | node_name = step.get('node_name') |
| 255 | remote_path = step.get('download', {}).get('remote_path', None) |
| 256 | remote_filename = step.get('download', {}).get('remote_filename', None) |
| 257 | local_path = step.get('download', {}).get('local_path', None) |
| 258 | description = step.get('description', remote_path) |
| 259 | skip_fail = step.get('skip_fail', False) |
| 260 | |
| 261 | if not remote_path or not remote_filename or not local_path: |
| 262 | raise Exception("Step '{0}' failed: please specify 'remote_path', " |
| 263 | "'remote_filename' and 'local_path' correctly" |
| 264 | .format(description)) |
| 265 | |
| 266 | with self.__underlay.remote(node_name=node_name) as remote: |
| 267 | |
| 268 | result = remote.execute('find {0} -type f -name {1}' |
| 269 | .format(remote_path, remote_filename)) |
| 270 | LOG.info("Found files to download:\n{0}".format(result)) |
| 271 | |
| 272 | if not result['stdout'] and not skip_fail: |
| 273 | raise Exception("Nothing to download on step {0}" |
| 274 | .format(description)) |
| 275 | |
| 276 | file_names = result['stdout'] |
| 277 | for file_name in file_names: |
| 278 | LOG.info("Downloading {0}:{1} to {2}" |
| 279 | .format(node_name, file_name, local_path)) |
| 280 | remote.download(destination=file_name.rstrip(), |
| 281 | target=local_path) |