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

✨Clusters-keeper: add CLUSTERS_KEEPER_COMPUTATIONAL_BACKEND_DOCKER_IMAGE_TAG ENV #5066

Merged

Conversation

sanderegg
Copy link
Member

@sanderegg sanderegg commented Nov 22, 2023

What do these changes do?

This PR introduces the CLUSTERS_KEEPER_COMPUTATIONAL_BACKEND_DOCKER_IMAGE_TAG environment variable that allows to set the image tag of the deployed dask-sidecar in the external clusters.

Related issue/s

How to test

Dev Checklist

DevOps Checklist

@sanderegg sanderegg self-assigned this Nov 22, 2023
@sanderegg sanderegg added this to the 7peaks milestone Nov 22, 2023
@sanderegg sanderegg marked this pull request as ready for review November 22, 2023 09:04
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.

Thank you! Don't forget to update ops-config 🚀

Copy link

codecov bot commented Nov 22, 2023

Codecov Report

Merging #5066 (d624135) into master (9afb78a) will decrease coverage by 5.3%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head d624135 differs from pull request most recent head 7153464. Consider uploading reports for the commit 7153464 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #5066      +/-   ##
=========================================
- Coverage    87.2%   82.0%    -5.3%     
=========================================
  Files        1237     647     -590     
  Lines       51409   25497   -25912     
  Branches     1115     618     -497     
=========================================
- Hits        44842   20909   -23933     
+ Misses       6328    4456    -1872     
+ Partials      239     132     -107     
Flag Coverage Δ
integrationtests 76.8% <ø> (+12.0%) ⬆️
unittests 83.4% <ø> (-1.6%) ⬇️

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

Files Coverage Δ
...c/simcore_service_clusters_keeper/core/settings.py 97.5% <ø> (ø)

... and 721 files with indirect coverage changes

@sanderegg sanderegg force-pushed the add-env-for-dask-sidecar-image-tag branch from bf2662a to 52dfc05 Compare November 22, 2023 09:14
Copy link

codeclimate bot commented Nov 22, 2023

Code Climate has analyzed commit d624135 and detected 0 issues on this pull request.

View more on Code Climate.

@sanderegg sanderegg force-pushed the add-env-for-dask-sidecar-image-tag branch from d624135 to 7153464 Compare November 22, 2023 10:16
@sanderegg sanderegg merged commit 9f2bcc0 into ITISFoundation:master Nov 22, 2023
@sanderegg sanderegg deleted the add-env-for-dask-sidecar-image-tag branch November 22, 2023 10:17
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants