#!/usr/bin/env python3 # Copyright 2019 Google LLC # # Licensed under the the Apache License v2.0 with LLVM Exceptions (the "License"); # you may not use this file except in compliance with the License. # You may obtain a copy of the License at # # https://llvm.org/LICENSE.txt # # 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. import argparse import datetime import logging import os import re import subprocess import sys from typing import List, Optional, Tuple, Dict import backoff from buildkite_utils import annotate, feedback_url, upload_file import git from phabricator import Phabricator """URL of upstream LLVM repository.""" LLVM_GITHUB_URL = 'ssh://git@github.com/llvm/llvm-project' FORK_REMOTE_URL = 'ssh://git@github.com/llvm-premerge-tests/llvm-project' class ApplyPatch: """Apply a diff from Phabricator on local working copy. This script is a rewrite of `arc patch` to accommodate for dependencies that have already landed, but could not be identified by `arc patch`. For a given diff_id, this class will get the dependencies listed on Phabricator. For each dependency D it will check the it's status: - if D is closed, skip it. - If D is not closed, it will download the patch for D and try to apply it locally. Once this class has applied all dependencies, it will apply the original diff. This script must be called from the root folder of a local checkout of https://github.com/llvm/llvm-project or given a path to clone into. """ def __init__(self, path: str, diff_id: int, token: str, url: str, git_hash: str, phid: str, push_branch: bool = False): self.push_branch = push_branch # type: bool self.conduit_token = token # type: Optional[str] self.host = url # type: Optional[str] self.diff_id = diff_id # type: int self.phid = phid # type: str if not self.host.endswith('/api/'): self.host += '/api/' self.phab = self.create_phab() self.base_revision = git_hash # type: str self.branch_base_hexsha = '' self.apply_diff_counter = 0 self.build_dir = os.getcwd() self.revision_id = '' if not os.path.isdir(path): logging.info(f'{path} does not exist, cloning repository...') self.repo = git.Repo.clone_from(FORK_REMOTE_URL, path) else: logging.info('repository exist, will reuse') self.repo = git.Repo(path) # type: git.Repo self.repo.remote('origin').set_url(FORK_REMOTE_URL) os.chdir(path) logging.info(f'working dir {os.getcwd()}') @property def branch_name(self): """Name used for the git branch.""" return f'phab-diff-{self.diff_id}' def run(self): """try to apply the patch from phabricator """ try: diff = self.get_diff(self.diff_id) revision = self.get_revision(diff.revisionID) url = f"https://reviews.llvm.org/D{revision['id']}?id={diff['id']}" annotate(f"Patching changes [{url}]({url})", style='info', context='patch_diff') self.reset_repository() self.revision_id = revision['id'] dependencies = self.get_dependencies(revision) dependencies.reverse() # Now revisions will be from oldest to newest. if len(dependencies) > 0: logging.info('This diff depends on: {}'.format(revision_list_to_str(dependencies))) plan = [] for r in dependencies: if r['statusName'] == 'Closed': logging.info(f'skipping revision {r["id"]} - it is closed, assuming it has landed') continue d = self.get_diff(r['diffs'][0]) plan.append((r, d)) plan.append((revision, diff)) logging.info('Planning to apply in order:') for (r, d) in plan: logging.info(f"https://reviews.llvm.org/D{r['id']}?id={d['id']}") # Pick the newest known commit as a base for patches. base_commit = None for (r, d) in plan: c = self.find_commit(d['sourceControlBaseRevision']) if c is None: logging.warning(f"D{r['id']}#{d['id']} commit {d['sourceControlBaseRevision']} does not exist") continue if base_commit is None: logging.info(f"D{r['id']}#{d['id']} commit {c.hexsha} exists") base_commit = c elif c.committed_datetime > base_commit.committed_datetime: logging.info(f"D{r['id']}#{d['id']} commit {c.hexsha} has a later commit date then" f"{base_commit.hexsha}") base_commit = c if self.base_revision != 'auto': logging.info(f'Base revision "{self.base_revision}" is set by command argument. Will use ' f'instead of resolved "{base_commit}"') base_commit = self.find_commit(self.base_revision) if base_commit is None: base_commit = self.repo.heads['main'].commit annotate(f"Cannot find a base git revision. Will use current HEAD.", style='warning', context='patch_diff') self.create_branch(base_commit) for (r, d) in plan: if not self.apply_diff(d, r): return 1 if self.push_branch: self.repo.git.push('--force', 'origin', self.branch_name) annotate(f"Created branch [{self.branch_name}]" f"(https://github.com/llvm-premerge-tests/llvm-project/tree/{self.branch_name}).\n\n" f"To checkout locally, run in your copy of llvm-project directory:\n\n" "```shell\n" "git remote add premerge git@github.com:llvm-premerge-tests/llvm-project.git #first time\n" f"git fetch premerge {self.branch_name}\n" f"git checkout -b {self.branch_name} --track premerge/{self.branch_name}\n" "```", style='success', context='patch_diff') logging.info('Branch {} has been pushed'.format(self.branch_name)) return 0 except Exception as e: annotate(f":bk-status-failed: Unexpected error. Consider [creating a bug]({feedback_url()}).", style='error', context='patch_diff') logging.error(f'exception: {e}') return 1 @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def reset_repository(self): """Update local git repo and origin. As origin is disjoint from upstream, it needs to be updated by this script. """ # Remove index lock just in case. lock_file = f"{self.repo.working_tree_dir}/.git/index.lock" try: os.remove(lock_file) logging.info(f"removed {lock_file}") except FileNotFoundError: logging.info(f"{lock_file} does not exist") logging.info('Syncing local, origin and upstream...') if 'upstream' not in self.repo.remotes: self.repo.create_remote('upstream', url=LLVM_GITHUB_URL) self.repo.remotes.upstream.fetch() self.repo.git.clean('-ffxdq') self.repo.git.reset('--hard') self.repo.git.fetch('--all') if self.find_commit('main') is None: origin = self.repo.remotes.origin self.repo.create_head('main', origin.refs.main) self.repo.heads.main.set_tracking_branch(origin.refs.main) self.repo.heads.main.checkout() self.repo.git.pull('origin', 'main') self.repo.git.pull('upstream', 'main') if self.push_branch: self.repo.git.push('origin', 'main') @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def find_commit(self, rev): try: return self.repo.commit(rev) except: return None @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def create_branch(self, base_commit: git.Commit): if self.branch_name in self.repo.heads: self.repo.delete_head('--force', self.branch_name) logging.info(f'creating branch {self.branch_name} at {base_commit.hexsha}') new_branch = self.repo.create_head(self.branch_name, base_commit.hexsha) self.repo.head.reference = new_branch self.repo.head.reset(index=True, working_tree=True) self.branch_base_hexsha = self.repo.head.commit.hexsha logging.info('Base branch revision is {}'.format(self.repo.head.commit.hexsha)) annotate(f"Branch {self.branch_name} base revision is `{self.branch_base_hexsha}`.", style='info', context='patch_diff') @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def commit(self, revision: Dict, diff: Dict): """Commit the current state and annotates with the revision info.""" self.repo.git.add('-A') diff.setdefault('authorName', 'unknown') diff.setdefault('authorEmail', 'unknown') author = git.Actor(name=diff['authorName'], email=diff['authorEmail']) message = (f"{revision['title']}\n\n" f"Automated commit created by applying diff {self.diff_id}\n" f"\n" f"Phabricator-ID: {self.phid}\n" f"Review-ID: {diff_to_str(revision['id'])}\n") self.repo.index.commit(message=message, author=author) @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def create_phab(self): phab = Phabricator(token=self.conduit_token, host=self.host) phab.update_interfaces() return phab @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def get_diff(self, diff_id: int): """Get a diff from Phabricator based on its diff id.""" return self.phab.differential.getdiff(diff_id=diff_id) @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def get_revision(self, revision_id: int): """Get a revision from Phabricator based on its revision id.""" return self.phab.differential.query(ids=[revision_id])[0] @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def get_revisions(self, *, phids: List[str] = None): """Get a list of revisions from Phabricator based on their PH-IDs.""" if phids is None: raise Exception('_get_revisions phids is None') if not phids: # Handle an empty query locally. Otherwise the connection # will time out. return [] return self.phab.differential.query(phids=phids) def get_dependencies(self, revision: Dict) -> List[Dict]: """Recursively resolves dependencies of the given revision. They are listed in reverse chronological order - from most recent to least recent.""" dependency_ids = revision['auxiliary']['phabricator:depends-on'] revisions = self.get_revisions(phids=dependency_ids) result = [] for r in revisions: result.append(r) sub = self.get_dependencies(r) result.extend(sub) return result def apply_diff(self, diff: Dict, revision: Dict) -> bool: """Download and apply a diff to the local working copy.""" logging.info(f"Applying {diff['id']} for revision {revision['id']}...") patch = self.get_raw_diff(str(diff['id'])) self.apply_diff_counter += 1 patch_file = f"{self.apply_diff_counter}_{diff['id']}.patch" with open(os.path.join(self.build_dir, patch_file), 'wt') as f: f.write(patch) # For annotate to properly link this file it must exist before the upload. upload_file(self.build_dir, patch_file) logging.debug(f'raw patch:\n{patch}') proc = subprocess.run('git apply -', input=patch, shell=True, text=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE) if proc.returncode != 0: logging.info(proc.stdout) logging.error(proc.stderr) message = f":bk-status-failed: Failed to apply [{patch_file}](artifact://{patch_file}).\n\n" if self.revision_id != revision['id']: message += f"**Attention! D{revision['id']} is one of the dependencies of the target " \ f"revision D{self.revision_id}.**\n\n" message += (f"No testing is possible because we couldn't apply the patch.\n\n" f"---\n\n" '### Troubleshooting\n\n' 'More information is available in the log of of *create branch* step. ' f"All patches applied are available as *Artifacts*.\n\n" f":bulb: The patch may not apply if it includes only the most recent of " f"multiple local commits. Try to upload a patch with\n" f"```shell\n" f"arc diff `git merge-base HEAD origin` --update D{revision['id']}\n" f"```\n\n" f"to include all local changes.\n\n" '---\n\n' f"If this case could have been handled better, please [create a bug]({feedback_url()}).") annotate(message, style='error', context='patch_diff') return False self.commit(revision, diff) return True @backoff.on_exception(backoff.expo, Exception, max_tries=5, logger='', factor=3) def get_raw_diff(self, diff_id: str) -> str: return self.phab.differential.getrawdiff(diffID=diff_id).response def diff_to_str(diff: int) -> str: """Convert a diff id to a string with leading "D".""" return 'D{}'.format(diff) def revision_list_to_str(diffs: List[Dict]) -> str: """Convert list of diff ids to a comma separated list, prefixed with "D".""" return ', '.join([diff_to_str(d['id']) for d in diffs]) if __name__ == "__main__": parser = argparse.ArgumentParser(description='Apply Phabricator patch to working directory.') parser.add_argument('diff_id', type=int) parser.add_argument('--path', type=str, help='repository path', default=os.getcwd()) parser.add_argument('--token', type=str, default=None, help='Conduit API token') parser.add_argument('--url', type=str, default='https://reviews.llvm.org', help='Phabricator URL') parser.add_argument('--commit', dest='commit', type=str, default='auto', help='Use this commit as a base. For "auto" tool tries to pick the base commit itself') parser.add_argument('--push-branch', action='store_true', dest='push_branch', help='choose if branch shall be pushed to origin') parser.add_argument('--phid', type=str, default=None, help='Phabricator ID of the review this commit pertains to') parser.add_argument('--log-level', type=str, default='INFO') args = parser.parse_args() logging.basicConfig(level=args.log_level, format='%(levelname)-7s %(message)s') patcher = ApplyPatch(args.path, args.diff_id, args.token, args.url, args.commit, args.phid, args.push_branch) sys.exit(patcher.run())