Skip to content

Conversation

@GitHK
Copy link
Contributor

@GitHK GitHK commented Jan 8, 2025

What do these changes do?

Since the ownership of a service's information is the catalog, a service's labels will now be provided by the catalog and should not be recovered via the director service.

Related issue/s

How to test

Dev-ops checklist

@GitHK GitHK added this to the Event Horizon milestone Jan 8, 2025
@GitHK GitHK self-assigned this Jan 8, 2025
@codecov
Copy link

codecov bot commented Jan 8, 2025

Codecov Report

Attention: Patch coverage is 95.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 71.30%. Comparing base (0963820) to head (6fc086b).
Report is 1 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (0963820) and HEAD (6fc086b). Click for more details.

HEAD has 32 uploads less than BASE
Flag BASE (0963820) HEAD (6fc086b)
integrationtests 7 6
unittests 34 3
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7016       +/-   ##
===========================================
- Coverage   86.94%   71.30%   -15.65%     
===========================================
  Files        1649      727      -922     
  Lines       64791    34937    -29854     
  Branches     2041      262     -1779     
===========================================
- Hits        56335    24912    -31423     
- Misses       8118     9965     +1847     
+ Partials      338       60      -278     
Flag Coverage Δ
integrationtests 64.87% <79.16%> (+0.27%) ⬆️
unittests 84.65% <87.50%> (-2.04%) ⬇️
Components Coverage Δ
api ∅ <ø> (∅)
pkg_aws_library ∅ <ø> (∅)
pkg_dask_task_models_library ∅ <ø> (∅)
pkg_models_library ∅ <ø> (∅)
pkg_notifications_library ∅ <ø> (∅)
pkg_postgres_database ∅ <ø> (∅)
pkg_service_integration ∅ <ø> (∅)
pkg_service_library ∅ <ø> (∅)
pkg_settings_library ∅ <ø> (∅)
pkg_simcore_sdk 77.37% <ø> (-8.02%) ⬇️
agent ∅ <ø> (∅)
api_server ∅ <ø> (∅)
autoscaling ∅ <ø> (∅)
catalog 90.66% <100.00%> (+0.08%) ⬆️
clusters_keeper ∅ <ø> (∅)
dask_sidecar ∅ <ø> (∅)
datcore_adapter ∅ <ø> (∅)
director 76.49% <ø> (+0.08%) ⬆️
director_v2 91.39% <91.66%> (-0.02%) ⬇️
dynamic_scheduler ∅ <ø> (∅)
dynamic_sidecar 89.74% <ø> (ø)
efs_guardian ∅ <ø> (∅)
invitations ∅ <ø> (∅)
osparc_gateway_server ∅ <ø> (∅)
payments ∅ <ø> (∅)
resource_usage_tracker ∅ <ø> (∅)
storage ∅ <ø> (∅)
webclient ∅ <ø> (∅)
webserver 59.24% <ø> (-24.81%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0963820...6fc086b. Read the comment docs.

@GitHK GitHK added a:catalog catalog service a:director-v2 issue related with the director-v2 service t:maintenance Some planned maintenance work labels Jan 8, 2025
@GitHK GitHK marked this pull request as ready for review January 8, 2025 13:54
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.

very nice, just a few questions regarding the availability of these labels:

  • does it make sense to return all the labels? they are used in the resources api endpoint
  • would it make more sense to return just what is necessary so that we do not start to use that endpoint in parallel to the resources one?

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
dont forget to increase the minor version of the catalog as well

make version-minor

@GitHK GitHK requested a review from sanderegg January 10, 2025 06:45
@sonarqubecloud
Copy link

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.

still a bit worried about making all the static labels available like so. but let's see what comes afterwards.

@GitHK GitHK merged commit a8ca793 into ITISFoundation:master Jan 10, 2025
90 of 93 checks passed
@GitHK GitHK deleted the pr-osparc-move-service-labels-to-catalog-client branch January 10, 2025 10:39
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Jan 15, 2025
58 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:catalog catalog service a:director-v2 issue related with the director-v2 service t:maintenance Some planned maintenance work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants