-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DEPR]: testeng-ci #265
Comments
@robrap I had originally mentioned on this ticket that 2U would probably need to fork testeng-ci from openedx-unsupported and continue maintaining it in the edx org, but I believe I was mistaken. I thought that this repo supported 2U jenkins infrastructure; my understanding now is that all the Jenkins stuff has already been removed. All that remains in testeng-ci today is the |
This also changes the branch prefix from `jenkins/` to `repo-tools/` Copied from https://github.com/openedx/testeng-ci/ Part of https://github.com/openedx/testeng-ci/issues/419
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
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
testeng-ci is deprecated in favor of repo-tools. See the DEPR [1] and the PR that moved pull_request_creator to repo-tools [2]. This also 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 the future of the edx-repo-health repo. This will be a future task [3] [1] openedx/public-engineering#265 [2] openedx/repo-tools#517 [3] #138
Replacement and removal is complete. |
Proposal Date
2024-05-14
Target Ticket Acceptance Date
2024-05-28
Earliest Open edX Named Release Without This Functionality
Redwood
Rationale
This is a tiny repo with a couple Python modules for interacting with GitHub. They are used in a handful of GitHub Actions workflows that open PRs. It could easily be merged into repo-tools.
Removal
This repository will be archived and it will not be part of the Open edX named release, starting with Redwood.
(the repo contains a redwood.master branch, but it will not get redwood.1 tag if this is accepted in time)
Replacement
This repo used to have a bunch of scripts used by edX's Jenkins server. Those have mostly all been replaced in a community-friendly way with a mix of GitHub Actions workflows, scripts in app repos, and Tutor.
The remaining module that Open edX still uses,
pull_request_creator
, will be moved into repo-tools: openedx/repo-tools#517Deprecation
I'll put a notice in the README.
Migration
Workflows that currently clone and install testeng-ci for the pull_request_creator tool should instead:
Additional Info
N/A
Task List
The text was updated successfully, but these errors were encountered: