From ac64a4dc9e28c4885f8f9b41d6640096c970a35d Mon Sep 17 00:00:00 2001 From: "Kyle D. McCormick" Date: Wed, 5 Jun 2024 15:34:05 -0400 Subject: [PATCH] feat: use pull request creator from repo-tools, not testeng-ci testeng-ci is deprecated in favor of repo-tools [1] This includes some cosmetic changes to the upgrade PRs. Before: * branch == jenkins/upgrade-python-requirements-{id} * title == Python Requirements Update * commit == chore: Updating Python Requirements After: * branch == repo-tools/upgrade-python-requirements-{id} * title == chore: Upgrade Python requirements * commit == chore: Upgrade Python requirements NOTE: This leaves behind a testeng-ci reference in the repo-health job. We need to figure out whether to fix the repo health dashboard or deprecate it. This will a follow up task after [1] is closed. [1] https://github.com/openedx/testeng-ci/issues/419 --- .github/workflows/bulk_repo_update.yml | 18 +++++++++--------- .../workflows/upgrade-python-requirements.yml | 16 +++++++++------- 2 files changed, 18 insertions(+), 16 deletions(-) diff --git a/.github/workflows/bulk_repo_update.yml b/.github/workflows/bulk_repo_update.yml index f332913..4bc7732 100644 --- a/.github/workflows/bulk_repo_update.yml +++ b/.github/workflows/bulk_repo_update.yml @@ -97,7 +97,7 @@ jobs: uses: actions/setup-python@v5 with: python-version: ${{ github.event.inputs.python_version }} - + - name: Setup Nodejs uses: actions/setup-node@v4 with: @@ -110,11 +110,11 @@ jobs: - name: run script run: ${{ github.event.inputs.script }} - - name: setup testeng-ci + - name: setup pull_request_creator run: | - git clone https://github.com/edx/testeng-ci.git - cd $GITHUB_WORKSPACE/testeng-ci - pip install -r requirements/base.txt + git clone https://github.com/openedx/repo-tools.git + cd $GITHUB_WORKSPACE/repo-tools + pip install -e '.[pull_request_creator]' - name: setup draft flag run: echo "draftflag=$(if ${{ github.event.inputs.draft }}; then echo '--draft'; else echo ''; fi)" >> $GITHUB_ENV @@ -122,16 +122,16 @@ jobs: - name: setup force delete flag run: echo "force_delete_old_prs_flag=$(if ${{ github.event.inputs.force_delete_old_prs }}; then echo '--force-delete-old-prs'; else echo '--no-force-delete-old-prs'; fi)" >> $GITHUB_ENV - - name: ignore testeng-ci - run: echo "testeng-ci" >> .git/info/exclude + - name: ignore repo-tools + run: echo "repo-tools" >> .git/info/exclude - name: create pull request env: GITHUB_TOKEN: ${{ secrets.requirements_bot_github_token }} GITHUB_USER_EMAIL: ${{ secrets.requirements_bot_github_email }} run: | - cd $GITHUB_WORKSPACE/testeng-ci - python -m jenkins.pull_request_creator --repo-root=$GITHUB_WORKSPACE \ + cd $GITHUB_WORKSPACE/repo-tools + pull_request_creator --repo-root=$GITHUB_WORKSPACE \ --target-branch="${{ env.BRANCH }}" --base-branch-name="${{ github.event.inputs.branch }}" \ --commit-message="${{ github.event.inputs.commit_message }}" \ --pr-title="${{ github.event.inputs.commit_message }}" \ diff --git a/.github/workflows/upgrade-python-requirements.yml b/.github/workflows/upgrade-python-requirements.yml index c064942..929bfbc 100644 --- a/.github/workflows/upgrade-python-requirements.yml +++ b/.github/workflows/upgrade-python-requirements.yml @@ -74,11 +74,12 @@ jobs: cd $GITHUB_WORKSPACE make upgrade - - name: setup testeng-ci + - name: setup pull_request_creator run: | - git clone https://github.com/edx/testeng-ci.git - cd $GITHUB_WORKSPACE/testeng-ci - pip install -r requirements/base.txt + git clone https://github.com/openedx/repo-tools + cd $GITHUB_WORKSPACE/repo-tools + pip install -e '.[pull_request_creator]' + - name: create pull request id: createpullrequest env: @@ -88,10 +89,11 @@ jobs: USER_REVIEWERS: ${{ inputs.user_reviewers }} TEAM_REVIEWERS: ${{ inputs.team_reviewers }} run: | - cd $GITHUB_WORKSPACE/testeng-ci - python -m jenkins.pull_request_creator --repo-root=$GITHUB_WORKSPACE \ + cd $GITHUB_WORKSPACE/repo-tools + pull_request_creator --repo-root=$GITHUB_WORKSPACE \ --target-branch="$TARGET_BRANCH" --base-branch-name="upgrade-python-requirements" \ - --commit-message="chore: Updating Python Requirements" --pr-title="Python Requirements Update" \ + --commit-message="chore: Upgrade Python requirements" \ + --pr-title="chore: Upgrade Python requirements" \ --pr-body="Python requirements update. Please review the [changelogs](https://openedx.atlassian.net/wiki/spaces/TE/pages/1001521320/Python+Package+Changelogs) for the upgraded packages." \ --user-reviewers="$USER_REVIEWERS" \ --team-reviewers="$TEAM_REVIEWERS" \