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

fix(test-functional)_: fix docker file #6292

Closed
wants to merge 1 commit into from

Conversation

friofry
Copy link
Contributor

@friofry friofry commented Jan 29, 2025

since we are using :latest tag, the pkg manager is updated

  • apk -> apt-get

fixes #6291

@status-im-auto
Copy link
Member

status-im-auto commented Jan 29, 2025

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ b1983e7 #1 2025-01-29 09:20:00 ~2 min tests-rpc 📄log
✔️ b1983e7 #1 2025-01-29 09:22:26 ~4 min ios 📦zip
✔️ b1983e7 #1 2025-01-29 09:22:48 ~5 min macos 📦zip
✔️ b1983e7 #1 2025-01-29 09:23:09 ~5 min linux 📦zip
✔️ b1983e7 #1 2025-01-29 09:23:11 ~5 min windows 📦zip
✔️ b1983e7 #1 2025-01-29 09:23:17 ~5 min android 📦aar
✔️ b1983e7 #1 2025-01-29 09:24:04 ~6 min macos 📦zip
✖️ b1983e7 #2 2025-01-29 09:28:31 ~1 min tests-rpc 📄log
✔️ b1983e7 #1 2025-01-29 09:48:27 ~30 min tests 📄log
✔️ bf05110 #2 2025-01-29 18:39:55 ~3 min ios 📦zip
✔️ bf05110 #2 2025-01-29 18:40:09 ~3 min macos 📦zip
✔️ bf05110 #2 2025-01-29 18:40:30 ~4 min windows 📦zip
✔️ bf05110 #2 2025-01-29 18:41:26 ~5 min macos 📦zip
✔️ bf05110 #2 2025-01-29 18:41:39 ~5 min linux 📦zip
✔️ bf05110 #2 2025-01-29 18:42:20 ~6 min android 📦aar
✔️ bf05110 #3 2025-01-29 18:43:05 ~6 min tests-rpc 📄log
✔️ bf05110 #2 2025-01-29 19:06:25 ~30 min tests 📄log

Copy link
Collaborator

@igor-sirotin igor-sirotin left a comment

Choose a reason for hiding this comment

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

I think we should fix the version and not use latest, it will continue to lead to such issues.
cc @antdanchenko @fbarbu15

@fbarbu15
Copy link
Contributor

I think we should fix the version and not use latest, it will continue to lead to such issues. cc @antdanchenko @fbarbu15

When I tried to use apt-get a couple of days ago, I had other failures, so I tried this
Agree we should have a more fixed version

@friofry friofry force-pushed the ab/issue-6291-fix-functional-tests-docker branch from b1983e7 to bf05110 Compare January 29, 2025 18:35
@friofry
Copy link
Contributor Author

friofry commented Jan 29, 2025

I think we should fix the version and not use latest, it will continue to lead to such issues. cc @antdanchenko @fbarbu15

When I tried to use apt-get a couple of days ago, I had other failures, so I tried this Agree we should have a more fixed version

Great! I'll close this PR then

@friofry friofry closed this Jan 29, 2025
Copy link

codecov bot commented Jan 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.85%. Comparing base (c993c7f) to head (bf05110).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #6292      +/-   ##
===========================================
- Coverage    61.91%   61.85%   -0.07%     
===========================================
  Files          843      843              
  Lines       111287   111287              
===========================================
- Hits         68903    68834      -69     
- Misses       34449    34494      +45     
- Partials      7935     7959      +24     
Flag Coverage Δ
functional 21.56% <ø> (+0.02%) ⬆️
unit 60.31% <ø> (-0.11%) ⬇️

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

see 36 files with indirect coverage changes

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.

fix(tests-functional)_: Docker fails to install packages
4 participants