Skip to content

Commit

Permalink
Add xtrace shell command tracing when running action in GitHub Debu…
Browse files Browse the repository at this point in the history
…g mode (#817)

We already do this in _some_ places, make it consistent everywhere.

Raised from [PR
feedback](#816 (comment)).
  • Loading branch information
JackPGreen authored Nov 5, 2024
1 parent 07d456a commit 7a6ba59
Show file tree
Hide file tree
Showing 13 changed files with 16 additions and 14 deletions.
2 changes: 1 addition & 1 deletion .github/scripts/abstract-simple-smoke-test.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/env bash

set -o errexit -o nounset -o pipefail
set -o errexit -o nounset -o pipefail ${RUNNER_DEBUG:+-x}

# Performs simple validation tests on an already-running Hazelcast instance
# Abstract as could be from Docker, Homebrew, local binary etc
Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/build.functions.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash

set -euo pipefail
set -euo pipefail ${RUNNER_DEBUG:+-x}

# Checks if we should build the OSS docker image.
# Returns "yes" if we should build it or "no" if we shouldn't.
Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/build.functions_tests.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/env bash

set -eu
set -eu ${RUNNER_DEBUG:+-x}

SCRIPT_DIR="$(dirname "$(readlink -f "$0")")"

Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/ee-build.functions.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash

set -euo pipefail
set -euo pipefail ${RUNNER_DEBUG:+-x}

# This is a simple script imitating what maven does for snapshot versions. We are not using maven because currently Docker Buildx and QEMU on Github Actions
# don't work with Java on architectures ppc64le and s390x. When the problem is fixed we will revert back to using maven.
Expand Down
3 changes: 2 additions & 1 deletion .github/scripts/ee-build.functions_tests.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#!/usr/bin/env bash

set -eu
set -eu ${RUNNER_DEBUG:+-x}

function find_script_dir() {
CURRENT=$PWD

Expand Down
3 changes: 2 additions & 1 deletion .github/scripts/get-tags-to-push_tests.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#!/usr/bin/env bash

set -eu
set -eu ${RUNNER_DEBUG:+-x}


SCRIPT_DIR="$(dirname "$(readlink -f "$0")")"

Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/maven.functions_tests.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/env bash

set -eu
set -eu ${RUNNER_DEBUG:+-x}

function find_script_dir() {
CURRENT=$PWD
Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/oss-build.functions.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash

set -euo pipefail
set -euo pipefail ${RUNNER_DEBUG:+-x}

function get_hz_dist_zip() {
local hz_variant=$1
Expand Down
3 changes: 2 additions & 1 deletion .github/scripts/oss-build.functions_tests.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#!/usr/bin/env bash

set -eu
set -eu ${RUNNER_DEBUG:+-x}

function find_script_dir() {
CURRENT=$PWD

Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/simple-smoke-test.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/usr/bin/env bash

set -o errexit
set -o errexit ${RUNNER_DEBUG:+-x}

# shellcheck source=../.github/scripts/abstract-simple-smoke-test.sh
. .github/scripts/abstract-simple-smoke-test.sh
Expand Down
3 changes: 1 addition & 2 deletions .github/scripts/smoke-test.sh
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
#!/bin/bash

set -e
set -o pipefail
set -e -o pipefail ${RUNNER_DEBUG:+-x}

# Fill the variables before running the script
WORKDIR=$1
Expand Down
2 changes: 1 addition & 1 deletion hazelcast-enterprise/maven.functions.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash

set -euo pipefail
set -euo pipefail ${RUNNER_DEBUG:+-x}

# THIS FILE IS DUPLICATED AND MUST BE KEPT IN SYNC MANUALLY
# Docker requires any included script to be in the current folder, hence we must duplicate this script for OS and EE
Expand Down
2 changes: 1 addition & 1 deletion hazelcast-oss/maven.functions.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash

set -euo pipefail
set -euo pipefail ${RUNNER_DEBUG:+-x}

# THIS FILE IS DUPLICATED AND MUST BE KEPT IN SYNC MANUALLY
# Docker requires any included script to be in the current folder, hence we must duplicate this script for OS and EE
Expand Down

0 comments on commit 7a6ba59

Please sign in to comment.