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

RPS count drops when master and slaves drift in time #38

Closed
hrosenhorn opened this issue Oct 17, 2012 · 5 comments
Closed

RPS count drops when master and slaves drift in time #38

hrosenhorn opened this issue Oct 17, 2012 · 5 comments

Comments

@hrosenhorn
Copy link

It looks like the master trusts the slaves to report the correct time and if the system clock drifts the rps value will be reported incorrectly.

During a 24 hour run, running 10k rps this value will drop to about 6k rps.

Doing a ntpdate ntp.kth.se on master and the slaves restores the rps count back to 10k

@heyman
Copy link
Member

heyman commented Oct 19, 2012

I think it would be hard to remove the requirement that the different machines should have synced system clocks. The alternative would be to rely on the time when the "reports" arrive to the master from the slave nodes, which I think would be less reliable.

@cgbystrom
Copy link
Member

Sounds reasonable.

@hjlarsson Do you recall how much the clock drifted during those 24 hours? It seems odd that the clock should be drifting that much between master/slaves in the first place.

@Jahaja
Copy link
Member

Jahaja commented Oct 28, 2012

Obviously the simple way of handling this is having ntpd setup properly. However I see the following options as a way forward on this issue.

  1. Making the slaves report in their local time when they connect to the master and warn if a significant clock skew is detected.
  2. In addition to 1, the master then compensates for the detected clock skew when receiving reports.

With the latter option feeling quite naive and error prone, I'd suggest doing just the first - which in the worst case, horrendous connect time, will result in a false warning.

@cgbystrom
Copy link
Member

Sticking to KISS, I think it is fair to expect the OS to provide an accurate system clock.

@heyman
Copy link
Member

heyman commented Nov 27, 2012

I'm closing this since we agree that doing some auto compensation is a bad idea.

Issuing a warning in the master node, if a slave connects and reports a time that is off by more than a certain timespan, isn't related to the reported problem and can be discussed in a new ticket if we think it should be done :).

@heyman heyman closed this as completed Nov 27, 2012
pancaprima added a commit to pancaprima/locust that referenced this issue May 14, 2018
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

No branches or pull requests

4 participants