From f570be3b5a33c9be38517ca7d2d60ca4df4f8344 Mon Sep 17 00:00:00 2001
From: fmacleal <frederico.macielleal@iovlabs.org>
Date: Wed, 17 Jul 2024 17:56:03 +0200
Subject: [PATCH] Increasing the attempt and timeout to try to make the tes
 pass

+ Adding a final version of the README.md for the action
---
 .github/README.md                             | 19 +++++++++++++------
 .github/action.yml                            |  4 ++--
 .../images/scripts/configure_rit_locally.sh   |  4 ++--
 3 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/.github/README.md b/.github/README.md
index 21655e7c..a52eada6 100644
--- 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
index 66ccd453..37811e8e 100644
--- 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
index 444d7d63..63308bd0 100755
--- 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"