From 6cb4b5d8ad6d5455ec7cfbcd36354159bc764993 Mon Sep 17 00:00:00 2001
From: Anton Kaliaev <anton.kalyaev@gmail.com>
Date: Thu, 18 Feb 2021 13:01:57 +0400
Subject: [PATCH] .github: [jepsen] use "bash -c" to execute lein run cmd
 (#6136)

Fixes https://github.com/tendermint/tendermint/runs/1925329781

```
OCI runtime exec failed: exec failed: container_linux.go:370: starting
container process caused: exec: "lein run test --nemesis none --workload
cas-register --concurrency 10 --tendermint-url
https://github.com/melekes/katas/releases/download/0.2.0/tendermint.tar.gz
--merkleeyes-url
https://github.com/melekes/katas/releases/download/0.2.0/merkleeyes_0.1.7.tar.gz":
stat lein run test --nemesis none --workload cas-register --concurrency
10 --tendermint-url
https://github.com/melekes/katas/releases/download/0.2.0/tendermint.tar.gz
--merkleeyes-url
https://github.com/melekes/katas/releases/download/0.2.0/merkleeyes_0.1.7.tar.gz:
no such file or directory: unknown
```
---
 .github/workflows/jepsen.yml | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/jepsen.yml b/.github/workflows/jepsen.yml
index 793491118..3178b97f1 100644
--- a/.github/workflows/jepsen.yml
+++ b/.github/workflows/jepsen.yml
@@ -11,21 +11,18 @@ on:
         description: 'Nemesis to use: (none | clocks | single-partitions | half-partitions | ring-partitions | split-dup-validators | peekaboo-dup-validators | changing-validators | crash | truncate-tendermint | truncate-merkleeyes)'
         required: true
         default: 'none'
-      dupValidators:
-        description: 'Whether to have multiple validators share the same key.'
+      dupOrSuperByzValidators:
+        description: '"--dup-validators" (multiple validators share the same key) and(or) "--super-byzantine-validators" (byzantine validators have just shy of 2/3 the voting weight)'
         required: false
-        default: 'false'
-      superByzantineValidators:
-        description: 'Should byzantine validators have just shy of 2/3 the voting weight?'
-        required: false
-        default: 'false'
+        default: ''
       concurrency:
         description: 'How many workers should we run? Must be an integer and >= 10, optionally followed by n (e.g. 3n) to multiply by the number of nodes.'
         required: true
         default: 10
       timeLimit:
         description: 'Excluding setup and teardown, how long should a test run for, in seconds?'
-        required: false
+        required: true
+        default: 60
       tendermintUrl:
         description: 'Where to grab the Tendermint binary (linux/amd64 tarball)?'
         required: true
@@ -49,4 +46,4 @@ jobs:
 
       - name: Run the test
         working-directory: tendermint
-        run: docker exec -i jepsen-control 'lein run test --nemesis ${{ github.event.inputs.nemesis }} --workload ${{ github.event.inputs.workload }} --concurrency ${{ github.event.inputs.concurrency }} --tendermint-url ${{ github.event.inputs.tendermintUrl }} --merkleeyes-url ${{ github.event.inputs.merkleeyesUrl }}'
+        run: docker exec -i jepsen-control bash -c 'lein run test --nemesis ${{ github.event.inputs.nemesis }} --workload ${{ github.event.inputs.workload }} --concurrency ${{ github.event.inputs.concurrency }} --tendermint-url ${{ github.event.inputs.tendermintUrl }} --merkleeyes-url ${{ github.event.inputs.merkleeyesUrl }} --time-limit ${{ github.event.inputs.timeLimit }} ${{ github.event.inputs.dupOrSuperByzValidators }}'