Refactor container CI. ARM Image

jcortejoso commented 1 year ago

Description

Added a new Dockerfile.arm64 supporting ARM builds. This image is different from the existing Dockerfile because:
- celo-blockchain does not compile on alpine ARM. I tried installing some dependencies but I could not find the problem
- For debian as base, there is one package (linux-headers and linux-headers-arm64) with different name for both architectures. It may be possible to add some logic to the Dockerfile to be able to share the same for linux/amd64 and linux/arm64 platforms, but really I don't see a lot of value on investing time there tbh.
Refactored the building process to use new composite actions at celo-org/reusable-workflows. Depending on PR https://github.com/celo-org/reusable-workflows/pull/72

Other changes

Removed old Dockerfile.arm64 expected for Android builds
At mycelo, skip the transfer ownsership of governance for local and loadTest mycelo templates

Tested

Images generated: https://github.com/celo-org/celo-blockchain/actions/runs/6315913589/workflow

Pending

Update the reference to the composite actions when https://github.com/celo-org/reusable-workflows/pull/72 is merged

github-actions[bot] commented 1 year ago

Coverage from tests in ./e2e_test/... for ./consensus/istanbul/... at commit 2c709330ac6d3d69cfad1896876c6ab336b00c08

coverage: 48.9% of statements across all listed packages

coverage:  63.0% of statements in consensus/istanbul
coverage:  40.0% of statements in consensus/istanbul/announce
coverage:  54.6% of statements in consensus/istanbul/backend
coverage:   0.0% of statements in consensus/istanbul/backend/backendtest
coverage:  24.3% of statements in consensus/istanbul/backend/internal/replica
coverage:  61.2% of statements in consensus/istanbul/core
coverage:  45.0% of statements in consensus/istanbul/db
coverage:   0.0% of statements in consensus/istanbul/proxy
coverage:  64.4% of statements in consensus/istanbul/uptime
coverage:  51.8% of statements in consensus/istanbul/validator
coverage:  79.2% of statements in consensus/istanbul/validator/random

github-actions[bot] commented 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Attention: 48 lines in your changes are missing coverage. Please review.

Comparison is base (0ed594e) 55.10% compared to head (8617a40) 55.14%. Report is 1 commits behind head on master.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #2190 +/- ## ========================================== + Coverage 55.10% 55.14% +0.04% ========================================== Files 680 681 +1 Lines 114370 114407 +37 ========================================== + Hits 63019 63088 +69 + Misses 47468 47438 -30 + Partials 3883 3881 -2 ``` | [Files](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org) | Coverage Δ | | |---|---|---| | [eth/state\_accessor.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-ZXRoL3N0YXRlX2FjY2Vzc29yLmdv) | `0.00% <0.00%> (ø)` | | | [les/api\_backend.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-bGVzL2FwaV9iYWNrZW5kLmdv) | `0.00% <0.00%> (ø)` | | | [eth/api\_backend.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-ZXRoL2FwaV9iYWNrZW5kLmdv) | `0.00% <0.00%> (ø)` | | | [core/state\_processor.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-Y29yZS9zdGF0ZV9wcm9jZXNzb3IuZ28=) | `80.00% <29.41%> (-1.07%)` | :arrow_down: | | [eth/tracers/api.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-ZXRoL3RyYWNlcnMvYXBpLmdv) | `24.17% <18.75%> (-0.36%)` | :arrow_down: | | [eth/celo\_state\_accessor.go](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org#diff-ZXRoL2NlbG9fc3RhdGVfYWNjZXNzb3IuZ28=) | `0.00% <0.00%> (ø)` | | ... and [25 files with indirect coverage changes](https://app.codecov.io/gh/celo-org/celo-blockchain/pull/2190/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=celo-org)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

celo-org / celo-blockchain