-
Notifications
You must be signed in to change notification settings - Fork 27
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
✨dynamic-sidecar now logs into DockerHub (⚠️devops) #6081
Conversation
…o-dockerhub-in-dy-sidecars
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ 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
Flags with carried forward coverage won't be shown. Click here to find out more.
|
…o-dockerhub-in-dy-sidecars
Thank you very much 🙏 Let's first merge |
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.
thx. Left some suggestions for your consideration
...r-v2/src/simcore_service_director_v2/modules/dynamic_sidecar/docker_service_specs/sidecar.py
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
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.
./osparc-config
changes are ready to be merged. Thank you 🙏
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.
a few comments. we can discuss. thanks a lot!
services/director-v2/tests/unit/with_dbs/test_modules_dynamic_sidecar_docker_service_specs.py
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
…o-dockerhub-in-dy-sidecars
…o-dockerhub-in-dy-sidecars
…o-dockerhub-in-dy-sidecars
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.
thanks!
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Outdated
Show resolved
Hide resolved
services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/registry.py
Show resolved
Hide resolved
…o-dockerhub-in-dy-sidecars
…o-dockerhub-in-dy-sidecars
Quality Gate passedIssues Measures |
What do these changes do?
When the
dynamic-sidecar
runsdocker 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.DIRECTOR_V2_DOCKER_HUB_REGISTRY
, if present the dynamic-sidecar will login into DockerHubRelated issue/s
How to test
Dev-ops checklist