Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Test registrar on Ubuntu 24.04 #639

Closed
2 tasks
feanil opened this issue Aug 14, 2024 · 2 comments · Fixed by #646
Closed
2 tasks

Test registrar on Ubuntu 24.04 #639

feanil opened this issue Aug 14, 2024 · 2 comments · Fixed by #646
Labels
maintenance Routine upkeep necessary for the health of the platform

Comments

@feanil
Copy link
Contributor

feanil commented Aug 14, 2024

This repository is using Ubuntu 20.04 for testing. That version of Ubuntu will be out of support before Teak. Therefore this repo needs to be updated to testing with Ubuntu 24.04 before Sumac is cut to allow everyone sufficient time to switch to the new version.

Update this repository to test with Ubuntu 24.04 so that we can make the switch.

  • Tests are run and passing on Ubuntu 20.04 and 24.04
  • If major changes were needed and this is a library, a new version is published to PyPI or NPM

Note: In some cases, it may not make sense to test with both the old and the new version. For example, if the workflow is running linting or publishing to a package manager. In these cases, simply update the workflow to run on the newer version or opt to set it to ubuntu-latest instead if it doesn't matter what version it's running on. If you're unsure, reach out to the maintenance working group in #wg-maintenance in slack for guidance.

Known affected workflow files:

  • .github/workflows/mysql8-check-migrations.yml
  • .github/workflows/ci.yml
@feanil feanil added the maintenance Routine upkeep necessary for the health of the platform label Aug 14, 2024
@feanil feanil mentioned this issue Aug 14, 2024
Open
feanil added a commit that referenced this issue Sep 9, 2024
@feanil
build: Switch to ubuntu-latest for builds
b648876 
This code does not have any dependencies that are specific to any specific
version of ubuntu.  So instead of testing on a specific version and then needing
to do work to keep the versions up-to-date, we switch to the ubuntu-latest
target which should be sufficient for testing purposes.

This work is being done as a part of openedx/platform-roadmap#377

closes #639
@feanil feanil mentioned this issue Sep 9, 2024
Merged
@kdmccormick
Copy link
Contributor

Closing as not necessary, since Registrar will be archived on 19 Sep openedx/public-engineering#276

FYI @Zacharis278 , just in case you want to carry this change over to the edx registrar fork.

@kdmccormick kdmccormick closed this as not planned Won't fix, can't repro, duplicate, stale Sep 11, 2024
kdmccormick pushed a commit that referenced this issue Sep 11, 2024
@feanil
build: Switch to ubuntu-latest for builds (#646)
959e362 
This code does not have any dependencies that are specific to any specific
version of ubuntu.  So instead of testing on a specific version and then needing
to do work to keep the versions up-to-date, we switch to the ubuntu-latest
target which should be sufficient for testing purposes.

This work is being done as a part of openedx/platform-roadmap#377

closes #639
@kdmccormick
Copy link
Contributor

(I saw that there was already a PR for this, so I just merged it.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
maintenance Routine upkeep necessary for the health of the platform
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

build: Switch to ubuntu-latest for builds openedx-unsupported/registrar
2 participants
@feanil @kdmccormick