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

balance scheduler service number when start scheduler #134

Merged
merged 1 commit into from
Aug 11, 2022

Conversation

sonyafenge
Copy link
Collaborator

Part of issue: #120. after this fix, scheduler has balanced number of service on each machine, but the RegisterClientDuration still have big difference:

file name RegisterClientDuration
grs-down-dismt-client-us-central1-a-mig-f361.log.0 365.287036ms
grs-down-dismt-client-us-central1-a-mig-f361.log.1 521.043594ms
grs-down-dismt-client-us-central1-a-mig-f361.log.2 342.378137ms
grs-down-dismt-client-us-central1-a-mig-f361.log.3 368.195141ms
grs-down-dismt-client-us-central1-a-mig-f361.log.4 360.4061ms
grs-down-dismt-client-us-central1-a-mig-f361.log.5 315.15387ms
grs-down-dismt-client-us-central1-a-mig-f361.log.6 320.079994ms
grs-down-dismt-client-us-central1-a-mig-f361.log.7 320.47048ms
grs-down-dismt-client-us-central1-a-mig-f361.log.8 374.838644ms
grs-down-dismt-client-us-central1-a-mig-f361.log.9 313.702576ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.0 5.46519ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.1 6.91962ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.2 6.453254ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.3 5.610316ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.4 5.197985ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.5 5.04598ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.6 5.100392ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.7 5.334791ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.8 5.047824ms
grs-down-dismt-client-us-central1-a-mig-z8mf.log.9 5.13324ms

@sonyafenge sonyafenge requested a review from yb01 August 10, 2022 23:31
@yb01
Copy link
Collaborator

yb01 commented Aug 11, 2022

this pr is just for fixed number ( or ensure they are evenly assigned on client machines). we will keep the issue open for investigating the registration duration differences. seems one machine is much slower than they other one

Copy link
Collaborator

@yb01 yb01 left a comment

Choose a reason for hiding this comment

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

/lgtm

@yb01 yb01 merged commit 75a0891 into CentaurusInfra:main Aug 11, 2022
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.

2 participants