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

Split libvcx alpine image build in 2 phases #174

Merged
merged 1 commit into from
Nov 3, 2020
Merged

Split libvcx alpine image build in 2 phases #174

merged 1 commit into from
Nov 3, 2020

Conversation

Patrik-Stas
Copy link
Contributor

@Patrik-Stas Patrik-Stas commented Oct 26, 2020

Everytime libvcx image (we have cache miss) we are currently rebuilding it from scratch. This includes building libindy and postgres plugin. This can takes more than 30 minutes and is blocking other jobs.

In this PR, we build separately alpine base image with libindy and posgres, which wouldn't change as often. Then on top of that, we'll build libvcx.

Signed-off-by: Patrik Stas patrik.stas@absa.africa

@Patrik-Stas Patrik-Stas force-pushed the ci/optimize branch 5 times, most recently from 89f9119 to f48f569 Compare October 26, 2020 11:30
@codecov-io
Copy link

codecov-io commented Oct 26, 2020

Codecov Report

Merging #174 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #174      +/-   ##
==========================================
+ Coverage   51.08%   51.10%   +0.01%     
==========================================
  Files         151      151              
  Lines       22923    22923              
  Branches     6053     6053              
==========================================
+ Hits        11711    11714       +3     
+ Misses       7606     7603       -3     
  Partials     3606     3606              
Flag Coverage Δ
unittests 51.10% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
libvcx/src/lib.rs 33.48% <0.00%> (+0.09%) ⬆️

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 bca8470...a81138d. Read the comment docs.

@Patrik-Stas Patrik-Stas force-pushed the ci/optimize branch 10 times, most recently from 0c46d73 to a5e52b7 Compare October 30, 2020 15:05
@Patrik-Stas Patrik-Stas added the ci label Oct 30, 2020
Copy link
Contributor

@mirgee mirgee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just please resolve the merge conflict, remove unused build arg and GTG

.github/workflows/main.yml Outdated Show resolved Hide resolved
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
@mirgee mirgee merged commit 5407e99 into master Nov 3, 2020
@mirgee mirgee deleted the ci/optimize branch November 3, 2020 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants