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

✨dynamic-sidecar now logs into DockerHub (⚠️devops) #6081

Merged

Conversation

GitHK
Copy link
Contributor

@GitHK GitHK commented Jul 18, 2024

⚠️devops

  • ⚠️ close all sidecars before deploying

What do these changes do?

When the dynamic-sidecar runs docker compose pull it can fetch some images from Docker Hub. In combination with autoscaling machines and how our networking works, each time a new autoscaling machine is started it lowers the pull cap for the entire deployment by 1.

  • ✨ dynamic-sidecar now logs into docker hub when starting (also checks DockerHub for availability)
  • ✨ added new env var DIRECTOR_V2_DOCKER_HUB_REGISTRY, if present the dynamic-sidecar will login into DockerHub

Related issue/s

How to test

Dev-ops checklist

@GitHK GitHK self-assigned this Jul 18, 2024
@GitHK GitHK changed the title ✨dynamic-sidecar now logs into DockeHub ✨dynamic-sidecar now logs into DockeHub (⚠️devops) Jul 18, 2024
Copy link

codecov bot commented Jul 18, 2024

Codecov Report

Attention: Patch coverage is 97.29730% with 1 line in your changes missing coverage. Please review.

Project coverage is 88.1%. Comparing base (cafbf96) to head (1dba98b).
Report is 447 commits behind head on master.

Files Patch % Lines
...c/simcore_service_dynamic_sidecar/core/registry.py 94.7% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #6081      +/-   ##
=========================================
+ Coverage    84.5%   88.1%    +3.5%     
=========================================
  Files          10    1464    +1454     
  Lines         214   60736   +60522     
  Branches       25    1484    +1459     
=========================================
+ Hits          181   53556   +53375     
- Misses         23    6867    +6844     
- Partials       10     313     +303     
Flag Coverage Δ
integrationtests 64.9% <72.4%> (?)
unittests 86.1% <97.2%> (+1.5%) ⬆️

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

Files Coverage Δ
...settings-library/src/settings_library/utils_cli.py 85.5% <100.0%> (ø)
...ngs-library/src/settings_library/utils_encoders.py 100.0% <100.0%> (ø)
...2/src/simcore_service_director_v2/core/settings.py 100.0% <100.0%> (ø)
...es/dynamic_sidecar/docker_service_specs/sidecar.py 87.5% <100.0%> (ø)
...tor-v2/src/simcore_service_director_v2/utils/db.py 100.0% <100.0%> (ø)
...imcore_service_dynamic_sidecar/core/application.py 98.1% <ø> (ø)
...rvice_dynamic_sidecar/core/docker_compose_utils.py 96.6% <100.0%> (ø)
...vice_dynamic_sidecar/core/external_dependencies.py 100.0% <100.0%> (ø)
...c/simcore_service_dynamic_sidecar/core/settings.py 97.2% <100.0%> (ø)
...ervice_dynamic_sidecar/modules/service_liveness.py 100.0% <ø> (ø)
... and 2 more

... and 1407 files with indirect coverage changes

@YuryHrytsuk
Copy link
Contributor

Thank you very much 🙏

Let's first merge ./osparc-config changes

@GitHK GitHK marked this pull request as ready for review July 19, 2024 08:44
Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

thx. Left some suggestions for your consideration

Copy link
Contributor

@YuryHrytsuk YuryHrytsuk left a comment

Choose a reason for hiding this comment

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

./osparc-config changes are ready to be merged. Thank you 🙏

Copy link
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

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

a few comments. we can discuss. thanks a lot!

@GitHK GitHK requested a review from sanderegg August 1, 2024 07:22
Copy link
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

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

thanks!

@GitHK GitHK enabled auto-merge (squash) August 19, 2024 08:51
@GitHK GitHK disabled auto-merge August 19, 2024 09:55
@GitHK GitHK enabled auto-merge (squash) August 19, 2024 12:08
Copy link

@GitHK GitHK changed the title ✨dynamic-sidecar now logs into DockeHub (⚠️devops) ✨dynamic-sidecar now logs into DockerHub (⚠️devops) Aug 19, 2024
@GitHK GitHK merged commit 58114c8 into ITISFoundation:master Aug 19, 2024
57 checks passed
@GitHK GitHK deleted the pr-osparc-login-to-dockerhub-in-dy-sidecars branch August 19, 2024 13:41
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.

login dynamic-sidecars to docekerhub
5 participants