Skip to content

Commit

Permalink
cadvisor test: fix (#18671)
Browse files Browse the repository at this point in the history
* influxdb module: add postStart

* cadvisor module: increase TimeoutStartSec

Under high load, the cadvisor module can take longer than the default 90
seconds to start. This change should hopefully fix the test on Hydra.
  • Loading branch information
ericsagnes authored and domenkozar committed Sep 16, 2016
1 parent 52307fd commit 2d2c311
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 10 deletions.
13 changes: 9 additions & 4 deletions nixos/modules/services/databases/influxdb.nix
Original file line number Diff line number Diff line change
Expand Up @@ -66,16 +66,16 @@ let
enabled = false;
}];

collectd = {
collectd = [{
enabled = false;
typesdb = "${pkgs.collectd}/share/collectd/types.db";
database = "collectd_db";
port = 25826;
};
}];

opentsdb = {
opentsdb = [{
enabled = false;
};
}];

continuous_queries = {
enabled = true;
Expand Down Expand Up @@ -171,6 +171,11 @@ in
mkdir -m 0770 -p ${cfg.dataDir}
if [ "$(id -u)" = 0 ]; then chown -R ${cfg.user}:${cfg.group} ${cfg.dataDir}; fi
'';
postStart = mkBefore ''
until ${pkgs.curl.bin}/bin/curl -s -o /dev/null 'http://127.0.0.1${toString configOptions.http.bind-address}'/ping; do
sleep 1;
done
'';
};

users.extraUsers = optional (cfg.user == "influxdb") {
Expand Down
1 change: 1 addition & 0 deletions nixos/modules/services/monitoring/cadvisor.nix
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,7 @@ in {
${optionalString cfg.storageDriverSecure "-storage_driver_secure"}
''}
'';
TimeoutStartSec=300;
};
};

Expand Down
3 changes: 0 additions & 3 deletions nixos/tests/cadvisor.nix
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,6 @@ import ./make-test.nix ({ pkgs, ... } : {
$influxdb->waitForUnit("influxdb.service");
# Wait until influxdb admin interface is available
$influxdb->waitUntilSucceeds("curl -f 127.0.0.1:8083");
# create influxdb database
$influxdb->succeed(q~
curl -XPOST http://localhost:8086/query --data-urlencode "q=CREATE DATABASE root"
Expand Down
3 changes: 0 additions & 3 deletions nixos/tests/influxdb.nix
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,6 @@ import ./make-test.nix ({ pkgs, ...} : {
$one->waitForUnit("influxdb.service");
# Check if admin interface is avalible
$one->waitUntilSucceeds("curl -f 127.0.0.1:8083");
# create database
$one->succeed(q~
curl -XPOST http://localhost:8086/query --data-urlencode "q=CREATE DATABASE test"
Expand Down

0 comments on commit 2d2c311

Please sign in to comment.