Cleanup jenkins scripts #7648
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| [*.sh] | ||
| indent_style = space | ||
| indent_size = 4 | ||
| shell_variant = bash | ||
| binary_next_line = false | ||
| switch_case_indent = true | ||
| space_redirects = true | ||
| keep_padding = true | ||
| function_next_line = false |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| # Copyright 2025-Present Couchbase, Inc. | ||
| # | ||
| # Use of this software is governed by the Business Source License included | ||
| # in the file licenses/BSL-Couchbase.txt. As of the Change Date specified | ||
| # in that file, in accordance with the Business Source License, use of this | ||
| # software will be governed by the Apache License, Version 2.0, included in | ||
| # the file licenses/APL2.txt. | ||
|
|
||
| name: shell | ||
| permissions: | ||
| contents: read | ||
|
|
||
| on: | ||
| push: | ||
| # Only run when we change a shell script | ||
| paths: | ||
| - "*.sh" | ||
| branches: | ||
| - "main" | ||
| - "release/*" | ||
| - "feature/*" | ||
| - "CBG*" | ||
| - "ci-*" | ||
| pull_request: | ||
| # Only run when we change a shell script | ||
| paths: | ||
| - "*.sh" | ||
| branches: | ||
| - "main" | ||
| - "release/*" | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: ${{ !contains(github.ref, 'release/')}} | ||
|
|
||
| jobs: | ||
| shellcheck: | ||
| name: Shellcheck | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v5 | ||
| - name: Run ShellCheck | ||
| uses: ludeeus/action-shellcheck@master | ||
| with: | ||
| ignore_paths: | | ||
| ./build.sh | ||
| ./bootstrap.sh | ||
| ./bench.sh | ||
| ./integration-test/service-test.sh | ||
| ./integration-test/service-install-tests.sh | ||
| ./rewrite-manifest.sh | ||
| ./snap-manifest.sh | ||
| ./set-version-stamp.sh | ||
| ./service/sync_gateway_service_install.sh | ||
| ./service/sync_gateway_service_uninstall.sh | ||
| ./service/sync_gateway_service_upgrade.sh | ||
| ./test-integration-init.sh | ||
| ./test.sh |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| #!/bin/bash | ||
|
|
||
| # Copyright 2024-Present Couchbase, Inc. | ||
| # | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -10,32 +10,42 @@ | |||||
| DEFAULT_PACKAGE_TIMEOUT="45m" | ||||||
|
|
||||||
| set -u | ||||||
| set -e # Abort on errors | ||||||
|
|
||||||
| if [ "${1:-}" == "-m" ]; then | ||||||
| echo "Running in automated master integration mode" | ||||||
| # Set automated setting parameters | ||||||
| TARGET_PACKAGE="..." | ||||||
| TARGET_TEST="ALL" | ||||||
| RUN_WALRUS="true" | ||||||
| DETECT_RACES="false" | ||||||
| SG_EDITION="EE" | ||||||
| RUN_COUNT="1" | ||||||
| # CBS server settings | ||||||
| COUCHBASE_SERVER_PROTOCOL="couchbase" | ||||||
| COUCHBASE_SERVER_VERSION="enterprise-7.6.6" | ||||||
| export SG_TEST_BUCKET_POOL_DEBUG="true" | ||||||
| GSI="true" | ||||||
| TLS_SKIP_VERIFY="false" | ||||||
| SG_CBCOLLECT_ALWAYS="false" | ||||||
| fi | ||||||
|
|
||||||
| REQUIRED_VARS=( | ||||||
| COUCHBASE_SERVER_PROTOCOL | ||||||
| COUCHBASE_SERVER_VERSION | ||||||
| GSI | ||||||
| TARGET_TEST | ||||||
| TARGET_PACKAGE | ||||||
| TLS_SKIP_VERIFY | ||||||
| SG_EDITION | ||||||
| ) | ||||||
|
|
||||||
| for var in "${REQUIRED_VARS[@]}"; do | ||||||
| if [ -z "${!var:-}" ]; then | ||||||
| echo "${var} environment variable is required to be set." | ||||||
| exit 1 | ||||||
| fi | ||||||
| done | ||||||
| # Use Git SSH and define private repos | ||||||
| git config --global --replace-all url."[email protected]:".insteadOf "https://github.com/" | ||||||
| export GOPRIVATE=github.com/couchbaselabs/go-fleecedelta | ||||||
|
|
@@ -44,37 +54,21 @@ export GOPRIVATE=github.com/couchbaselabs/go-fleecedelta | |||||
| SG_COMMIT_HASH=$(git rev-parse HEAD) | ||||||
| echo "Sync Gateway git commit hash: $SG_COMMIT_HASH" | ||||||
|
|
||||||
| echo "Downloading tool dependencies..." | ||||||
| go install github.com/axw/gocov/gocov@latest | ||||||
| go install github.com/AlekSi/gocov-xml@latest | ||||||
| go install gotest.tools/gotestsum@latest | ||||||
|
|
||||||
| if [ "${SG_TEST_X509:-}" == "true" ] && [ "${COUCHBASE_SERVER_PROTOCOL}" != "couchbases" ]; then | ||||||
| echo "Setting SG_TEST_X509 requires using couchbases:// protocol, aborting integration tests" | ||||||
| exit 1 | ||||||
| fi | ||||||
|
|
||||||
| # Set environment vars | ||||||
| GO_TEST_FLAGS=(-v -p 1 "-count=${RUN_COUNT:-1}") | ||||||
| INT_LOG_FILE_NAME="verbose_int" | ||||||
|
|
||||||
| export PATH=$PATH:~/go/bin | ||||||
| echo "PATH: $PATH" | ||||||
|
|
||||||
| if [ "${TEST_DEBUG:-}" == "true" ]; then | ||||||
|
|
@@ -83,38 +77,38 @@ if [ "${TEST_DEBUG:-}" == "true" ]; then | |||||
| fi | ||||||
|
|
||||||
| if [ "${TARGET_TEST}" != "ALL" ]; then | ||||||
| GO_TEST_FLAGS+=(-run "${TARGET_TEST}") | ||||||
| fi | ||||||
|
|
||||||
| if [ "${PACKAGE_TIMEOUT:-}" != "" ]; then | ||||||
| GO_TEST_FLAGS+=(-test.timeout="${PACKAGE_TIMEOUT}") | ||||||
| else | ||||||
| echo "Defaulting package timeout to ${DEFAULT_PACKAGE_TIMEOUT}" | ||||||
| GO_TEST_FLAGS+=(-test.timeout="${DEFAULT_PACKAGE_TIMEOUT}") | ||||||
| fi | ||||||
|
|
||||||
| if [ "${DETECT_RACES:-}" == "true" ]; then | ||||||
| GO_TEST_FLAGS=(-race) | ||||||
|
||||||
| GO_TEST_FLAGS=(-race) | |
| GO_TEST_FLAGS+=(-race) |
Member
There was a problem hiding this comment.
I think this comment was an accurate one FWIW
torcolvin marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
torcolvin marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why
docker exec curlhere and not on line 88?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I probably hit this with accidental debugging, I'd rather have it call curl from inside the container than outside given we don't know the network constraints.