----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53914/#review156445 -----------------------------------------------------------
Fix it, then Ship it! contrib/utils/perf/deploy-gce-perf-cluster.py (line 220) <https://reviews.apache.org/r/53914/#comment226639> why do we need this sleep? - Dmitro Lisnichenko On Nov. 19, 2016, 2:12 a.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53914/ > ----------------------------------------------------------- > > (Updated Nov. 19, 2016, 2:12 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko, Dmytro Sen, Sumit Mohanty, Sid > Wagle, and Vitalyi Brodetskyi. > > > Bugs: AMBARI-18940 > https://issues.apache.org/jira/browse/AMBARI-18940 > > > Repository: ambari > > > Description > ------- > > The script to create VMs on GCE for the perf cluster has a bug where each VM > will have the exact same agent-multiplier config file with start=1 and > num=50, thereby preventing more than 50 agents. > This is because a single server.sh and agent.sh file is written out, and > copied to all VMs. > We need to create a custom config file for each VM instead. > > > Diffs > ----- > > contrib/utils/perf/deploy-gce-perf-cluster.py da45f8f > > Diff: https://reviews.apache.org/r/53914/diff/ > > > Testing > ------- > > Verified by deploying a cluster to GCE. > > > Thanks, > > Alejandro Fernandez > >