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 to GitHub runners. #21

Merged
merged 8 commits into from
Dec 22, 2020
Merged

Migrate CI to GitHub runners. #21

merged 8 commits into from
Dec 22, 2020

Conversation

siyuan0322
Copy link
Collaborator

@siyuan0322 siyuan0322 commented Dec 21, 2020

What do these changes do?

Migrate some CI steps to GitHub runners. While two steps that requires k8s cluster still remains on self-hosted, because resource requested is exceed the standard configuration of GitHub runner (2 core CPU, 8G memory).

This PR also

  1. Parallels the building process of three engines,
  2. Reduce the size of artifacts.
  3. Add code coverage report for graphscope.

However, this PR also introduced several overhead, which is two additional Dockerfile for assemble artifact content into graphscope-runtime image, however, this is basically copied from the stage 2 of the normal Dockerfile. And the manipulation of upload and download artifact is more complicated than before (to preserve file permissions of artifact, one need to assemble all files into a tar ball)

@codecov-io
Copy link

codecov-io commented Dec 21, 2020

Codecov Report

❗ No coverage uploaded for pull request base (main@a570a31). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage        ?   78.50%           
=======================================
  Files           ?       60           
  Lines           ?     4978           
  Branches        ?        0           
=======================================
  Hits            ?     3908           
  Misses          ?     1070           
  Partials        ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a570a31...02a7ace. Read the comment docs.

@siyuan0322 siyuan0322 merged commit 8fd52b4 into main Dec 22, 2020
@siyuan0322 siyuan0322 deleted the zsy/debug4 branch December 22, 2020 06:54
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.

None yet

4 participants