Increasing the attempt and timeout to try to make the tes pass

+ Adding a final version of the README.md for the action
rsksmart · Jul 31, 2024 · a2a019b · a2a019b
1 parent d4d85e7
commit a2a019b
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 10 deletions.
diff --git a/.github/README.md b/.github/README.md
@@ -14,32 +14,39 @@ That way, we will add steps on each repository to run the integration tests with
 No matter if it's a tag, a branch or a specific commit.
 
 ## Inputs
+By default, all the inputs are pointed to the `master/main` branch of the repositories. But, ideally, we will adapt
+the action to receive the branches  received to be the one in a branch or tag that we want to test.
 
 ### `rskj-branch`
 
-**Optional** The rskj branch to checkout. Default is `master`.
+The rskj branch to checkout. If no branch or tag passed, it will be used the default `master`.
 
 ### `powpeg-node-branch`
 
-**Optional** The powpeg-node branch to checkout. Default is `master`.
+The powpeg-node branch to checkout. If no branch or tag passed, it will be used the default `master`.
 
 ### `rit-branch`
 
-**Optional** The rootstock-integration-tests branch to checkout. Default is `main`.
+**Optional** The rootstock-integration-tests branch to checkout. This one it's optional, because it will be
+very unlikely that we need to use a different branch for the rootstock-integration-test. It's offered the possibility
+to use a different one, but the default and most frequently used, should be `main`.
 
 ### `rit-log-level`
 
 **Optional** Log level for the rootstock-integration-tests. Default is `info`.
 
 ## Outputs
-
+The output of the action are basically two values, one is the status of the integration tests, and the other is the message.
+I
 ### `status`
 
-The status of the integration tests.
+The status of the integration tests.  It would be `0` for success and `1` for failure.
 
 ### `message`
 
-The output message of the integration tests.
+The output message of the integration tests. It will be:
+- In case of success: `Rootstock Integration Tests Status: PASSED`
+- In case of error: `Rootstock Integration Tests Status: FAILED`
 
 ## Example usage
 

diff --git a/.github/action.yml b/.github/action.yml
@@ -5,11 +5,11 @@ author: 'Rootstock Labs'
 inputs:
   rskj-branch:
     description: 'The rskj branch to checkout'
-    required: false
+    required: true
     default: 'master'
   powpeg-node-branch:
     description: 'The powpeg-node branch to checkout'
-    required: false
+    required: true
     default: 'master'
   rit-branch:
     description: 'The rootstock-integration-tests branch to checkout'

diff --git a/.github/images/scripts/configure_rit_locally.sh b/.github/images/scripts/configure_rit_locally.sh
@@ -9,8 +9,8 @@ CONFIG_FILE=/usr/src/rit/config/regtest.js
 LOG_HOME=/usr/src/rit/logs
 BITCOIND_BIN_PATH=/usr/local/bin/bitcoind
 BITCOIN_DATA_DIR=/usr/src/bitcoindata
-WAIT_FOR_BLOCK_ATTEMPT_TIME_MILLIS=600
-WAIT_FOR_BLOCK_MAX_ATTEMPTS=1000
+WAIT_FOR_BLOCK_ATTEMPT_TIME_MILLIS=800
+WAIT_FOR_BLOCK_MAX_ATTEMPTS=1200
 EOF
 
 echo -e  "\n\n---------- Configuring RIT to run the tests locally -----------\n\n"