Quick and dirty fixes to enable spackbot on multiple repos (#108)

* Quick fix to spackbot to enable usage on both core and packages

In lieu of a more thorough update to spackbot a quick fix to simply
enable the usage of spackbot from both of the repos maintained by the
spack organization and not an update to make spackbot configurable to
support workflows on other forks that wish to utilized the spackbot
tools. The configurations for the two repos are hard-coded, similarly to
before, for the "spack" and "spack-packages" projects in the PROJECTS
variable.

Labeling and package maintainer pinging have been dropped. These are now
handled by GitHub actions that run on pull request automatically as part
of the "triage" workflow.

ref. spack/spack-packages#507

Style is handled by configuring the tool to run for each project. This
is hardcoded based on the project. For "spack" project the tool is the
develop version of `spack style --fix` and for the "spack-packages"
project the tool is simply `black`.

* Update to use python3 version of awscli

* Update spackbotdev redeploy script

Author: kwryankrattiger

redeploy-spackbot-dev.sh

@@ -14,6 +14,6 @@ docker push ghcr.io/${gh_user}/spackbot:${image_tag}
 docker push ghcr.io/${gh_user}/spackbot-workers:${image_tag}
 
 # Rollout with the new containers
-kubectl rollout restart -n spack deployments/spackbotdev-spack-io
-kubectl rollout restart -n spack deployments/spackbotdev-workers
-kubectl rollout restart -n spack deployments/spackbotdev-lworkers
+kubectl -n spack rollout restart deployment spackbotdev-spack-io
+kubectl -n spack rollout restart deployment spackbotdev-workers
+kubectl -n spack rollout restart deployment spackbotdev-lworkers

requirements.txt

@@ -4,4 +4,4 @@ gidgethub
 python_dotenv
 rq
 sh
-aws
+awscli

spackbot/handlers/__init__.py

@@ -3,8 +3,5 @@
     run_pipeline_rebuild_all,
     close_pr_gitlab_branch,
 )
-from .labels import add_labels  # noqa
-from .reviewers import add_reviewers, add_issue_maintainers  # noqa
-from .reviewers import add_reviewers  # noqa
 from .style import style_comment, fix_style  # noqa
 from .mirrors import close_pr_mirror  # noqa

spackbot/handlers/gitlab.py

@@ -65,7 +65,8 @@ async def close_pr_gitlab_branch(event, gh):
     pr_branch = payload["pull_request"]["head"]["ref"]
     pr_branch_name = f"pr{pr_number}_{pr_branch}"
 
-    url = helpers.gitlab_spack_project_url
+    event_project = payload["repository"]["name"]
+    url = helpers.PROJECT[event_project].gitlab_project_url
     url = f"{url}/repository/branches/{pr_branch_name}"
 
     GITLAB_TOKEN = os.environ.get("GITLAB_TOKEN")

spackbot/handlers/labels.py

@@ -50,7 +50,8 @@ async def close_pr_mirror(event, gh):
         "pr_branch": pr_branch,
     }
 
-    if is_merged and base_branch == pr_expected_base:
+    # PR Graduation Mirror is disabled
+    if False and is_merged and base_branch == pr_expected_base:
         logger.info(
             f"PR {pr_number}/{pr_branch} merged to develop, graduating binaries"
         )