-
Notifications
You must be signed in to change notification settings - Fork 272
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
Clean up ci-runner dockerfiles & upgrade to nodejs 14.19.1 #2030
Clean up ci-runner dockerfiles & upgrade to nodejs 14.19.1 #2030
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2030 +/- ##
=========================================
Coverage 94.23% 94.23%
Complexity 25 25
=========================================
Files 198 198
Lines 3902 3902
Branches 29 29
=========================================
Hits 3677 3677
Misses 219 219
Partials 6 6
Continue to review full report at Codecov.
|
docker/ci/dockerfiles/test.rockylinux8.opensearch-dashboards.x64.arm64.dockerfile
Outdated
Show resolved
Hide resolved
I can perform some sanity check on the new images when they are ready. @peterzhuamazon |
Everything is built right now all the images above updated to v2. |
Image too big change to multi-stage now.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please optimize the image sizes before merging.
Size fixed now:
|
Signed-off-by: Peter Zhu <[email protected]>
7a2caab
to
e410f3d
Compare
Ignore failure as there is no cypress test present. Thanks. |
Rebuilt with cmake installed so no module compilation failure, still the sandbox test seems error out due to we are building a docker image inside a docker, but binary built. |
@tianleh please review. |
Hey @peterzhuamazon I see |
It is always like that and I understand where you come with that question. Let's not change now as release is coming up and a lot of coding is depending on these (yes there are actual code dependencies on these names). Plus even with the help of Jenkins these images build in 3 hours due to multiple compilation on the fly through binary translation (which is by design to do so). So I would not start that now. There is an issue in this repo already pointing that out. Thanks. |
Hey @peterzhuamazon if you have issue for enhancement for ci images can you add this example as well? opensearch-project/opensearch-ci#107 |
Not necessarily the same as we have other script to setup env before dockerfile being built, thus cannot directly run docker build from github actions. Please the compiling is intense and it would crash if we do it on the 2 core machine of github runners. It would also take more than 6 hours even if it runs. |
Signed-off-by: Peter Zhu [email protected]
Description
Clean up ci-runner dockerfiles & upgrade to nodejs 14.19.1.
Issues Resolved
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.