450666 - Increase intervals for long-running tests

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Ryan Doherty (:rdoherty)]

In order to reduce the amount of data the graph server needs to store/retrieve, we need to increase the interval for long running tests.

First idea is tests that run for more than 10 minutes can have their intervals increased to once every 10 seconds?

Comment 1

[alice nodelman [:alice] [:anode]]

I'm running some tests on talos stage to only poll for memory/cpu information every 20 seconds.  I'll compare the test results taken with different granularities to determine if we are losing any valuable information by polling less.

Comment 2

[alice nodelman [:alice] [:anode]]

Attachment: [Checked in]sample memory/cpu only every 20 seconds instead of every second

I'm not seeing much difference in the numbers by sampling less frequently.  I think that this is pretty low risk and will reduce the amount of data sent to the graph server by a given talos cycle pretty dramatically.

Comment 3

[Ryan Doherty (:rdoherty)]

Comment on attachment 336106 [details] [diff] [review]
[Checked in]sample memory/cpu only every 20 seconds instead of every second

I'm not sure I should be reviewing this patch. I've never touched or seen this code before.

Comment 4

[alice nodelman [:alice] [:anode]]

Comment on attachment 336106 [details] [diff] [review]
[Checked in]sample memory/cpu only every 20 seconds instead of every second

/cvsroot/mozilla/testing/performance/talos/sample.config,v  <--  sample.config
new revision: 1.18; previous revision: 1.17
done

Comment 5

[alice nodelman [:alice] [:anode]]

Will need to keep an eye on this in case it radically alters the numbers collected.

Comment 6

[alice nodelman [:alice] [:anode]]

Looks like this got accidentally switched back to '1' at some point...

Comment 7

[alice nodelman [:alice] [:anode]]

Attachment: [Checked in]sample memory/cpu only every 20 seconds instead of every second (once again)

Must have happened with the patch that ensures that we run talos tests in a specific order.  Need to be more careful with this main config file, since it is run everywhere.

Comment 8

[[e-mail] (:bhearsum)]

Comment on attachment 353343 [details] [diff] [review]
[Checked in]sample memory/cpu only every 20 seconds instead of every second (once again)

Boy, it's confusing to have dashes as valid starts of lines.

Looks fine though.

Comment 9

[alice nodelman [:alice] [:anode]]

Comment on attachment 353343 [details] [diff] [review]
[Checked in]sample memory/cpu only every 20 seconds instead of every second (once again)

Checking in sample.config;
/cvsroot/mozilla/testing/performance/talos/sample.config,v  <--  sample.config
new revision: 1.22; previous revision: 1.21
done