Skip to content
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

Migrate CI workflow from CircleCI #919

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sau226
Copy link
Member

@sau226 sau226 commented Dec 13, 2022

This PR uses a modified output from the GitHub Actions Importer to migrate all workflows from CircleCI to GitHub Actions itself.

By using GitHub Actions, we can ensure that CI can be managed adjacent to code without needing to depend on a third party service (CircleCI), and integrates it natively into GitHub (on PRs etc).

You'll need to create 3 actions secrets to make this config work. These secrets are stored as MASTER_KEY, COVERALLS_REPO_TOKEN and DEV_SSH_KEY in CircleCI env vars. Add the actions secrets with same name and value as the CircleCI env vars.

As this is a test of the migration tool, the migrated workflow will still be using CircleCI docker images, at least for now. We can migrate to alternatives once we feel comfortable with having integrated actions in GitHub.

@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from 6530f40 to 8214d01 Compare December 14, 2022 06:27
@cellio cellio requested a review from ArtOfCode- December 23, 2022 03:03
@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from 8214d01 to c3d3685 Compare December 30, 2022 04:54
@ArtOfCode-
Copy link
Member

Info: is there a particular reason we should move out of Circle?

@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from c3d3685 to 2a08e71 Compare February 4, 2023 11:35
@sau226
Copy link
Member Author

sau226 commented Feb 4, 2023

Info: is there a particular reason we should move out of Circle?

There's been a general trend to simplify Codidact's operations and reduce dependencies on external services/credentials, which add additional complexity and are used for only a limited purpose.

In early Jan, CircleCI had a security incident impacting GitHub tokens: https://circleci.com/blog/jan-4-2023-incident-report/

In light of the above information, I think it may be prudent to either move out of CircleCI or at least give sufficient consideration to that course of action.

@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from 2a08e71 to b9a01f8 Compare April 25, 2023 04:12
@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from b9a01f8 to f525636 Compare May 26, 2023 01:35
@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from f525636 to 3c92d4f Compare June 9, 2023 23:48
@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from 3c92d4f to 539675b Compare June 21, 2023 21:53
@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from 539675b to 694559e Compare August 2, 2023 12:31
@ArtOfCode-
Copy link
Member

This has diverged from the Circle workflow now - we also have system tests, and both tests and system tests are run against ruby 2.7 and 3.1.

@Taeir
Copy link
Contributor

Taeir commented Aug 3, 2023

@ArtOfCode- I think it may still be valuable to switch over to GitHub actions. Updating this PR to match should not be too difficult as it contains sort of the same information (and still uses CircleCI's docker images).

@sau226 sau226 force-pushed the convert-codidact-qpixel-to-actions-20221213-061700 branch from 694559e to b906e1e Compare August 31, 2023 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants