Skip to content

Commit

Permalink
Change monitor intervals from milliseconds to seconds (#1314)
Browse files Browse the repository at this point in the history
For the fabric tests the monitor intervals were set to single
millisecond values such as 3, 4, 5. This adds a massive overhead to the
running of the integration tests and isn't needed, so changing them to
be seconds instead.

Signed-off-by: D <d_kelsey@uk.ibm.com>

Co-authored-by: D <d_kelsey@uk.ibm.com>
  • Loading branch information
davidkel and davidkel committed Apr 15, 2022
1 parent 7f4d374 commit 57d8d46
Show file tree
Hide file tree
Showing 6 changed files with 15 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -37,5 +37,5 @@ monitors:
resource:
- module: process
options:
interval: 3
interval: 3000
processes: [{ command: 'node', arguments: 'caliper.js', multiOutput: 'avg' }]
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,9 @@ monitors:
resource:
- module: process
options:
interval: 3
interval: 3000
processes: [{ command: 'node', arguments: 'caliper.js', multiOutput: 'avg' }]
- module: docker
options:
interval: 4
interval: 4000
containers: ['peer0.org1.example.com', 'peer0.org2.example.com', 'orderer0.example.com', 'orderer1.example.com']
Original file line number Diff line number Diff line change
Expand Up @@ -51,12 +51,12 @@ monitors:
messageLevel: info
- module: prometheus-push
options:
pushInterval: 5
pushInterval: 5000
pushUrl: "http://localhost:9091"
resource:
- module: prometheus
options:
interval: 5
interval: 5000
url: "http://localhost:9090"
metrics:
include: [peer*, dev*]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,12 @@ monitors:
transaction:
- module: prometheus-push
options:
pushInterval: 5
pushInterval: 5000
pushUrl: "http://localhost:9091"
resource:
- module: prometheus
options:
interval: 5
interval: 5000
url: "http://localhost:9090"
metrics:
include: [peer*, dev*]
Expand All @@ -71,9 +71,9 @@ monitors:
metrics: [all]
- module: process
options:
interval: 3
interval: 3000
processes: [{ command: 'node', arguments: 'caliper.js', multiOutput: 'avg' }]
- module: docker
options:
interval: 4
interval: 4000
containers: ['peer0.org1.example.com', 'peer0.org2.example.com', 'orderer0.example.com', 'orderer1.example.com']
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,12 @@ monitors:
transaction:
- module: prometheus-push
options:
pushInterval: 5
pushInterval: 5000
pushUrl: "http://localhost:9091"
resource:
- module: prometheus
options:
interval: 5
interval: 5000
url: "http://localhost:9090"
metrics:
include: [peer*, dev*]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,12 @@ monitors:
transaction:
- module: prometheus-push
options:
pushInterval: 5
pushInterval: 5000
pushUrl: "http://localhost:9091"
resource:
- module: prometheus
options:
interval: 5
interval: 5000
url: "http://localhost:9090"
metrics:
include: [peer*, dev*]
Expand All @@ -74,10 +74,10 @@ monitors:
metrics: [all]
- module: process
options:
interval: 3
interval: 3000
processes: [{ command: 'node', arguments: 'caliper.js', multiOutput: 'avg' }]
- module: docker
options:
interval: 4
interval: 4000
containers: ['peer0.org1.example.com', 'peer0.org2.example.com', 'orderer0.example.com', 'orderer1.example.com']

0 comments on commit 57d8d46

Please sign in to comment.