Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 1 | # Copyright 2020 The Pigweed Authors |
| 2 | # |
| 3 | # Licensed under the Apache License, Version 2.0 (the "License"); you may not |
| 4 | # use this file except in compliance with the License. You may obtain a copy of |
| 5 | # the License at |
| 6 | # |
| 7 | # https://www.apache.org/licenses/LICENSE-2.0 |
| 8 | # |
| 9 | # Unless required by applicable law or agreed to in writing, software |
| 10 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 11 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 12 | # License for the specific language governing permissions and limitations under |
| 13 | # the License. |
Rob Mohr | d39ea9e | 2020-09-22 15:01:07 -0700 | [diff] [blame] | 14 | """Utility functions for rollers.""" |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 15 | |
| 16 | import pprint |
| 17 | import re |
Rob Mohr | cbee3fc | 2020-11-30 14:07:42 -0800 | [diff] [blame] | 18 | import urlparse |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 19 | |
| 20 | import attr |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 21 | import enum |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 22 | from recipe_engine import recipe_api |
| 23 | |
| 24 | # If we're embedding the original commit message, prepend 'Original-' to lines |
| 25 | # which begin with these tags. |
| 26 | ESCAPE_TAGS = [ |
| 27 | 'Bug:', |
| 28 | 'Fixed:', |
Rob Mohr | 2ec49a0 | 2020-12-15 08:37:21 -0800 | [diff] [blame] | 29 | 'Fixes:', |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 30 | 'Requires:', |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 31 | 'Reviewed-on:', |
| 32 | ] |
| 33 | |
| 34 | # If we're embedding the original commit message, remove lines which contain |
| 35 | # these tags. |
| 36 | FILTER_TAGS = [ |
| 37 | 'API-Review:', |
| 38 | 'Acked-by:', |
| 39 | 'CC:', |
| 40 | 'CQ-Do-Not-Cancel-Tryjobs:', |
| 41 | 'Change-Id:', |
| 42 | 'Commit-Queue:', |
Rob Mohr | 4c6a538 | 2021-02-25 08:50:44 -0800 | [diff] [blame] | 43 | 'Cq-Cl-Tag:', |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 44 | 'Reviewed-by:', |
Rob Mohr | 4c6a538 | 2021-02-25 08:50:44 -0800 | [diff] [blame] | 45 | 'Roller-URL:', |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 46 | 'Signed-off-by:', |
| 47 | 'Testability-Review:', |
| 48 | 'Tested-by:', |
Rob Mohr | 7cb53c4 | 2020-11-18 10:45:13 -0800 | [diff] [blame] | 49 | 'Auto-Submit', |
| 50 | re.compile(r'^\w+-Auto-Submit:'), |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 51 | ] |
| 52 | |
| 53 | |
Rob Mohr | 7cb53c4 | 2020-11-18 10:45:13 -0800 | [diff] [blame] | 54 | def _match_tag(line, tag): |
| 55 | if hasattr(tag, 'match'): |
| 56 | return tag.match(line) |
| 57 | return line.startswith(tag) |
| 58 | |
| 59 | |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 60 | def _sanitize_message(message): |
| 61 | """Sanitize lines of a commit message. |
| 62 | |
| 63 | Prepend 'Original-' to lines which begin with ESCAPE_TAGS. Filter |
| 64 | out lines which begin with FILTER_TAGS. |
| 65 | """ |
| 66 | return '\n'.join( |
| 67 | "Original-" + line |
| 68 | if any((line.startswith(tag) for tag in ESCAPE_TAGS)) |
| 69 | else line |
| 70 | for line in message.splitlines() |
Rob Mohr | 7cb53c4 | 2020-11-18 10:45:13 -0800 | [diff] [blame] | 71 | if not any((_match_tag(line, tag) for tag in FILTER_TAGS)) |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 72 | ) |
| 73 | |
| 74 | |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 75 | class _Direction(enum.Enum): |
| 76 | CURRENT = 'CURRENT' |
| 77 | FORWARD = 'FORWARD' |
| 78 | BACKWARD = 'BACKWARD' |
| 79 | REBASE = 'REBASE' |
| 80 | |
| 81 | |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 82 | @attr.s |
| 83 | class Commit(object): |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 84 | hash = attr.ib() |
| 85 | message = attr.ib() |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 86 | |
| 87 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 88 | @attr.s |
| 89 | class Roll(object): |
| 90 | _api = attr.ib() |
| 91 | project_name = attr.ib(type=str) |
| 92 | old_revision = attr.ib(type=str) |
| 93 | new_revision = attr.ib(type=str) |
| 94 | proj_dir = attr.ib(type=str) |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 95 | direction = attr.ib(type=str) |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 96 | _commit_data = attr.ib(default=None) |
Rob Mohr | cbee3fc | 2020-11-30 14:07:42 -0800 | [diff] [blame] | 97 | _remote = attr.ib(type=str, default=None) |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 98 | |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 99 | @direction.validator |
| 100 | def check(self, _, value): # pragma: no cover |
| 101 | if value not in _Direction: |
| 102 | raise ValueError('invalid direction: {}'.format(value)) |
| 103 | if value == _Direction.CURRENT: |
| 104 | raise ValueError('attempt to do a no-op roll') |
| 105 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 106 | @property |
| 107 | def commits(self): |
| 108 | if self._commit_data: |
| 109 | return self._commit_data |
| 110 | |
| 111 | with self._api.context(cwd=self.proj_dir): |
| 112 | with self._api.step.nest(self.project_name): |
| 113 | commits = [] |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 114 | if ( |
Rob Mohr | 476a704 | 2020-11-30 13:33:52 -0800 | [diff] [blame] | 115 | _is_hash(self.old_revision) |
| 116 | and self.direction == _Direction.FORWARD |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 117 | ): |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 118 | base = self.old_revision |
| 119 | else: |
| 120 | base = '{}~5'.format(self.new_revision) |
| 121 | |
| 122 | for commit in ( |
| 123 | self._api.git( |
| 124 | 'git log', |
| 125 | 'log', |
| 126 | '{}..{}'.format(base, self.new_revision), |
| 127 | '--pretty=format:%H %B', |
| 128 | # Separate entries with null bytes since most entries |
| 129 | # will contain newlines ("%B" is the full commit |
| 130 | # message, not just the first line.) |
| 131 | '-z', |
| 132 | stdout=self._api.raw_io.output(), |
| 133 | ) |
| 134 | .stdout.strip('\0') |
| 135 | .split('\0') |
| 136 | ): |
| 137 | hash, message = commit.split(' ', 1) |
| 138 | commits.append(Commit(hash, message)) |
| 139 | |
| 140 | self._commit_data = tuple(commits) |
| 141 | return self._commit_data |
| 142 | |
Rob Mohr | cbee3fc | 2020-11-30 14:07:42 -0800 | [diff] [blame] | 143 | @property |
| 144 | def remote(self): |
| 145 | if self._remote: |
| 146 | return self._remote |
| 147 | |
| 148 | api = self._api |
| 149 | |
| 150 | with api.step.nest('remote'), api.context(cwd=self.proj_dir): |
| 151 | name = api.git( |
| 152 | 'name', |
| 153 | 'remote', |
| 154 | stdout=api.raw_io.output(), |
| 155 | step_test_data=lambda: api.raw_io.test_api.stream_output( |
| 156 | 'origin' |
| 157 | ), |
| 158 | ).stdout.strip() |
| 159 | |
| 160 | remote = api.git( |
| 161 | 'url', |
| 162 | 'remote', |
| 163 | 'get-url', |
| 164 | name, |
| 165 | stdout=api.raw_io.output(), |
| 166 | step_test_data=lambda: api.raw_io.test_api.stream_output( |
| 167 | 'sso://pigweed/pigweed/pigweed' |
| 168 | ), |
| 169 | ).stdout.strip() |
| 170 | |
| 171 | self._remote = api.sso.sso_to_https(remote) |
| 172 | return self._remote |
| 173 | |
| 174 | @property |
| 175 | def gerrit_name(self): |
| 176 | return urlparse.urlparse(self.remote).netloc.split('.')[0] |
| 177 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 178 | |
| 179 | @attr.s |
| 180 | class Message(object): |
| 181 | name = attr.ib(type=str) |
| 182 | template = attr.ib(type=str) |
| 183 | kwargs = attr.ib(type=dict) |
| 184 | num_commits = attr.ib(type=int) |
| 185 | footer = attr.ib(type=tuple, default=()) |
| 186 | |
| 187 | def render(self, with_footer=True): |
| 188 | result = [self.template.format(**self.kwargs)] |
| 189 | if with_footer: |
| 190 | result.extend(x for x in self.footer) |
| 191 | return '\n'.join(result) |
| 192 | |
| 193 | |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 194 | def _is_hash(value): |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 195 | return re.match(r'^[0-9a-fA-F]{40}', value) |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 196 | |
| 197 | |
Rob Mohr | d39ea9e | 2020-09-22 15:01:07 -0700 | [diff] [blame] | 198 | class RollUtilApi(recipe_api.RecipeApi): |
Rob Mohr | 1eb8f91 | 2020-10-07 13:17:17 -0700 | [diff] [blame] | 199 | def __init__(self, props, *args, **kwargs): |
| 200 | super(RollUtilApi, self).__init__(*args, **kwargs) |
| 201 | self.labels_to_set = {x.label: x.value for x in props.labels_to_set} |
| 202 | self.labels_to_wait_on = props.labels_to_wait_on |
Rob Mohr | 1c1ef30 | 2021-03-08 12:23:14 -0800 | [diff] [blame] | 203 | self.footer = [] |
Rob Mohr | 1eb8f91 | 2020-10-07 13:17:17 -0700 | [diff] [blame] | 204 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 205 | def _single_commit_roll_message(self, roll): |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 206 | template = """ |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 207 | [roll {project_name}] {sanitized_message} |
| 208 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 209 | {project_name} Rolled-Commits: {old_revision:.15}..{new_revision:.15} |
| 210 | """.strip() |
| 211 | |
| 212 | commit = roll.commits[0] |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 213 | |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 214 | kwargs = { |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 215 | 'project_name': roll.project_name, |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 216 | 'original_message': commit.message, |
| 217 | 'sanitized_message': _sanitize_message(commit.message), |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 218 | 'old_revision': roll.old_revision, |
| 219 | 'new_revision': roll.new_revision, |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 220 | } |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 221 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 222 | message = Message( |
| 223 | name=roll.project_name, |
| 224 | template=template, |
| 225 | kwargs=kwargs, |
| 226 | num_commits=1, |
| 227 | footer=tuple(self.footer), |
| 228 | ) |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 229 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 230 | with self.m.step.nest( |
| 231 | 'message for {}'.format(roll.project_name) |
| 232 | ) as pres: |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 233 | pres.logs['template'] = template |
| 234 | pres.logs['kwargs'] = pprint.pformat(kwargs) |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 235 | pres.logs['message'] = message.render() |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 236 | |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 237 | return message |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 238 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 239 | def _multiple_commits_roll_message(self, roll): |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 240 | template = """ |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 241 | [{project_name}] Roll {num_commits} commits |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 242 | |
| 243 | {one_liners} |
| 244 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 245 | {project_name} Rolled-Commits: {old_revision:.15}..{new_revision:.15} |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 246 | """.strip() |
| 247 | |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 248 | one_liners = [ |
| 249 | '{:.15} {}'.format(commit.hash, commit.message.splitlines()[0][:50]) |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 250 | for commit in roll.commits |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 251 | ] |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 252 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 253 | num_commits = len(roll.commits) |
| 254 | if not _is_hash(roll.old_revision): |
| 255 | num_commits = 'multiple' |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 256 | one_liners.append('...') |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 257 | |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 258 | kwargs = { |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 259 | 'project_name': roll.project_name, |
| 260 | 'num_commits': num_commits, |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 261 | 'one_liners': '\n'.join(one_liners), |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 262 | 'old_revision': roll.old_revision, |
| 263 | 'new_revision': roll.new_revision, |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 264 | } |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 265 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 266 | message = Message( |
| 267 | name=roll.project_name, |
| 268 | template=template, |
| 269 | kwargs=kwargs, |
| 270 | num_commits=num_commits, |
| 271 | footer=tuple(self.footer), |
| 272 | ) |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 273 | |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 274 | with self.m.step.nest('message') as pres: |
| 275 | pres.logs['template'] = template |
| 276 | pres.logs['kwargs'] = pprint.pformat(kwargs) |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 277 | pres.logs['message'] = message.render() |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 278 | |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 279 | return message |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 280 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 281 | def _single_roll_message(self, roll): |
| 282 | if len(roll.commits) > 1: |
| 283 | return self._multiple_commits_roll_message(roll) |
| 284 | return self._single_commit_roll_message(roll) |
Rob Mohr | 092a446 | 2020-06-16 10:23:54 -0700 | [diff] [blame] | 285 | |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 286 | def _multiple_rolls_message(self, *rolls): |
| 287 | rolls = sorted(rolls, key=lambda x: x.project_name) |
| 288 | |
| 289 | messages = [] |
| 290 | for roll in rolls: |
| 291 | messages.append(self._single_roll_message(roll)) |
| 292 | |
| 293 | texts = [ |
| 294 | '[roll {}] Roll {} commits'.format( |
| 295 | ', '.join(x.name for x in messages), |
| 296 | sum(x.num_commits for x in messages), |
| 297 | ) |
| 298 | ] |
| 299 | texts.extend(x.render(with_footer=False) for x in messages) |
| 300 | texts.append('\n'.join('{}'.format(x) for x in self.footer)) |
| 301 | |
| 302 | return '\n\n'.join(texts) |
| 303 | |
| 304 | def Roll(self, **kwargs): |
| 305 | """Create a Roll. See Roll class above for details.""" |
| 306 | return Roll(api=self.m, **kwargs) |
| 307 | |
| 308 | def message(self, *rolls): |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 309 | with self.m.step.nest('roll message'): |
Rob Mohr | f6ee7cb | 2020-10-26 14:05:06 -0700 | [diff] [blame] | 310 | if len(rolls) > 1: |
| 311 | return self._multiple_rolls_message(*rolls) |
| 312 | return self._single_roll_message(*rolls).render() |
Rob Mohr | d39ea9e | 2020-09-22 15:01:07 -0700 | [diff] [blame] | 313 | |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 314 | Direction = _Direction |
| 315 | |
Rob Mohr | 476a704 | 2020-11-30 13:33:52 -0800 | [diff] [blame] | 316 | def get_roll_direction(self, git_dir, old, new, name='get roll direction'): |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 317 | """Return Direction of roll.""" |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 318 | if old == new: |
| 319 | with self.m.step.nest(name) as pres: |
| 320 | pres.step_summary_text = 'up-to-date' |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 321 | return self.Direction.CURRENT |
Rob Mohr | d39ea9e | 2020-09-22 15:01:07 -0700 | [diff] [blame] | 322 | |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 323 | with self.m.context(git_dir): |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 324 | with self.m.step.nest(name) as pres: |
| 325 | forward = self.m.git( |
| 326 | 'is forward', |
| 327 | 'merge-base', |
| 328 | '--is-ancestor', |
| 329 | old, |
| 330 | new, |
| 331 | ok_ret=(0, 1), |
| 332 | ) |
Rob Mohr | d39ea9e | 2020-09-22 15:01:07 -0700 | [diff] [blame] | 333 | |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 334 | backward = self.m.git( |
| 335 | 'is backward', |
| 336 | 'merge-base', |
| 337 | '--is-ancestor', |
| 338 | new, |
| 339 | old, |
| 340 | ok_ret=(0, 1), |
| 341 | ) |
Rob Mohr | d39ea9e | 2020-09-22 15:01:07 -0700 | [diff] [blame] | 342 | |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 343 | if ( |
| 344 | forward.exc_result.retcode == 0 |
| 345 | and backward.exc_result.retcode != 0 |
| 346 | ): |
| 347 | pres.step_summary_text = 'forward' |
| 348 | return self.Direction.FORWARD |
Rob Mohr | d39ea9e | 2020-09-22 15:01:07 -0700 | [diff] [blame] | 349 | |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 350 | if ( |
| 351 | forward.exc_result.retcode != 0 |
| 352 | and backward.exc_result.retcode == 0 |
| 353 | ): |
| 354 | pres.step_summary_text = 'backward' |
| 355 | return self.Direction.BACKWARD |
| 356 | |
Rob Mohr | cbee3fc | 2020-11-30 14:07:42 -0800 | [diff] [blame] | 357 | # If new and old are ancestors of each other then this is the |
| 358 | # same commit. We should only hit this during testing because |
| 359 | # the comparison at the top of the function should have caught |
| 360 | # this situation. |
| 361 | if ( |
| 362 | forward.exc_result.retcode == 0 |
| 363 | and backward.exc_result.retcode == 0 |
| 364 | ): |
| 365 | with self.m.step.nest(name) as pres: |
| 366 | pres.step_summary_text = 'up-to-date' |
| 367 | return self.Direction.CURRENT |
| 368 | |
Rob Mohr | f7e3015 | 2020-11-18 09:56:30 -0800 | [diff] [blame] | 369 | # If old is not an ancestor of new and new is not an ancestor |
| 370 | # of old then history was rewritten in some manner but we still |
| 371 | # need to update the pin. |
| 372 | pres.step_summary_text = 'rebase' |
| 373 | return self.Direction.REBASE |
| 374 | |
| 375 | def can_roll(self, direction): |
| 376 | return direction in (self.Direction.FORWARD, self.Direction.REBASE) |
| 377 | |
| 378 | def skip_roll_step(self, remote, old_revision, new_revision): |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 379 | with self.m.step.nest('cancelling roll') as pres: |
| 380 | fmt = ( |
| 381 | 'not updating from {old} to {new} because {old} is newer ' |
| 382 | 'than {new}' |
| 383 | ) |
| 384 | if old_revision == new_revision: |
Rob Mohr | ad16bc1 | 2020-09-28 06:39:55 -0700 | [diff] [blame] | 385 | fmt = ( |
| 386 | 'not updating from {old} to {new} because they are ' |
| 387 | 'identical' |
| 388 | ) |
Rob Mohr | 5720460 | 2020-09-23 08:41:18 -0700 | [diff] [blame] | 389 | pres.step_summary_text = fmt.format( |
| 390 | old=old_revision[0:7], new=new_revision[0:7] |
| 391 | ) |
| 392 | pres.links[old_revision] = '{}/+/{}'.format(remote, old_revision) |
| 393 | pres.links[new_revision] = '{}/+/{}'.format(remote, new_revision) |