foundationdb icon indicating copy to clipboard operation
foundationdb copied to clipboard

Rename error variable in go tests to err #8828

Open h4ck3r-04 opened this issue 1 year ago • 15 comments

Renamed the variables from e to err as mentioned in the https://go.dev/doc/effective_go

According to the issue number #8828 the variables in go code were named using good and effective go practices. Following the example link for reference mentioned in the issue the variables have been renamed.

h4ck3r-04 avatar Oct 15 '24 21:10 h4ck3r-04

Result of foundationdb-pr-clang-ide on Linux CentOS 7

  • Commit ID: 6a6a4e767c3e9b70a1060acacfae2ed8a0d5d2cd
  • Duration 0:21:18
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 20 '24 06:10 foundationdb-ci

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

  • Commit ID: 6a6a4e767c3e9b70a1060acacfae2ed8a0d5d2cd
  • Duration 0:42:31
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 20 '24 06:10 foundationdb-ci

Result of foundationdb-pr-macos on macOS Ventura 13.x

  • Commit ID: 6a6a4e767c3e9b70a1060acacfae2ed8a0d5d2cd
  • Duration 0:49:57
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 20 '24 06:10 foundationdb-ci

Result of foundationdb-pr-clang-arm on Linux CentOS 7

  • Commit ID: 6a6a4e767c3e9b70a1060acacfae2ed8a0d5d2cd
  • Duration 0:53:00
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 20 '24 06:10 foundationdb-ci

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

  • Commit ID: 6a6a4e767c3e9b70a1060acacfae2ed8a0d5d2cd
  • Duration 0:53:54
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)
  • Cluster Test Logs zip file of the test logs (available for 30 days)

foundationdb-ci avatar Oct 20 '24 06:10 foundationdb-ci

Result of foundationdb-pr-clang on Linux CentOS 7

  • Commit ID: 6a6a4e767c3e9b70a1060acacfae2ed8a0d5d2cd
  • Duration 1:04:23
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 20 '24 06:10 foundationdb-ci

Result of foundationdb-pr on Linux CentOS 7

  • Commit ID: 6a6a4e767c3e9b70a1060acacfae2ed8a0d5d2cd
  • Duration 1:04:26
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 20 '24 06:10 foundationdb-ci

There are still some e for element renamed to err, and the whitespaces change. Aside from that, LGTM; thanks!

gm42 avatar Oct 21 '24 07:10 gm42

Result of foundationdb-pr-clang-arm on Linux CentOS 7

  • Commit ID: cf190c9e8291e5a4da4af410148b29ef03688c6d
  • Duration 0:04:41
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 21 '24 15:10 foundationdb-ci

Result of foundationdb-pr-clang on Linux CentOS 7

  • Commit ID: cf190c9e8291e5a4da4af410148b29ef03688c6d
  • Duration 0:04:46
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 21 '24 15:10 foundationdb-ci

Result of foundationdb-pr on Linux CentOS 7

  • Commit ID: cf190c9e8291e5a4da4af410148b29ef03688c6d
  • Duration 0:04:49
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 21 '24 15:10 foundationdb-ci

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

  • Commit ID: cf190c9e8291e5a4da4af410148b29ef03688c6d
  • Duration 0:04:53
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)
  • Cluster Test Logs zip file of the test logs (available for 30 days)

foundationdb-ci avatar Oct 21 '24 15:10 foundationdb-ci

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

  • Commit ID: cf190c9e8291e5a4da4af410148b29ef03688c6d
  • Duration 0:06:55
  • Result: :x: FAILED
  • Error: Error while executing command: ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i ${HOME}/.ssh_key ec2-user@${MAC_EC2_HOST} /opt/homebrew/bin/bash --login -c ./build_pr_macos.sh. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 21 '24 15:10 foundationdb-ci

Result of foundationdb-pr-macos on macOS Ventura 13.x

  • Commit ID: cf190c9e8291e5a4da4af410148b29ef03688c6d
  • Duration 0:07:05
  • Result: :x: FAILED
  • Error: Error while executing command: ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i ${HOME}/.ssh_key ec2-user@${MAC_EC2_HOST} /usr/local/bin/bash --login -c ./build_pr_macos.sh. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 21 '24 15:10 foundationdb-ci

Result of foundationdb-pr-clang-ide on Linux CentOS 7

  • Commit ID: cf190c9e8291e5a4da4af410148b29ef03688c6d
  • Duration 0:21:12
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Oct 21 '24 16:10 foundationdb-ci

Result of foundationdb-pr on Linux CentOS 7

  • Commit ID: 04b995a13329e390920441423e719c0098e201d4
  • Duration 0:04:33
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 02 '24 01:11 foundationdb-ci

Result of foundationdb-pr-clang on Linux CentOS 7

  • Commit ID: 04b995a13329e390920441423e719c0098e201d4
  • Duration 0:04:41
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 02 '24 01:11 foundationdb-ci

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

  • Commit ID: 04b995a13329e390920441423e719c0098e201d4
  • Duration 0:04:40
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)
  • Cluster Test Logs zip file of the test logs (available for 30 days)

foundationdb-ci avatar Nov 02 '24 01:11 foundationdb-ci

Result of foundationdb-pr-clang-arm on Linux CentOS 7

  • Commit ID: 04b995a13329e390920441423e719c0098e201d4
  • Duration 0:04:40
  • Result: :x: FAILED
  • Error: Error while executing command: ninja -v -C build_output -j ${NPROC} all packages strip_targets. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 02 '24 01:11 foundationdb-ci

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

  • Commit ID: 04b995a13329e390920441423e719c0098e201d4
  • Duration 0:06:58
  • Result: :x: FAILED
  • Error: Error while executing command: ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i ${HOME}/.ssh_key ec2-user@${MAC_EC2_HOST} /opt/homebrew/bin/bash --login -c ./build_pr_macos.sh. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 02 '24 01:11 foundationdb-ci

Result of foundationdb-pr-macos on macOS Ventura 13.x

  • Commit ID: 04b995a13329e390920441423e719c0098e201d4
  • Duration 0:07:03
  • Result: :x: FAILED
  • Error: Error while executing command: ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i ${HOME}/.ssh_key ec2-user@${MAC_EC2_HOST} /usr/local/bin/bash --login -c ./build_pr_macos.sh. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 02 '24 01:11 foundationdb-ci

Result of foundationdb-pr-clang-ide on Linux CentOS 7

  • Commit ID: 04b995a13329e390920441423e719c0098e201d4
  • Duration 0:21:40
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 02 '24 02:11 foundationdb-ci

Looks like there are CI errors:

ninja: Entering directory `build_output'
[0/2] /usr/local/bin/cmake -P /codebuild/output/src805022784/src/github.com/apple/foundationdb/build_output/CMakeFiles/VerifyGlobs.cmake
ninja: error: '/codebuild/output/src805022784/src/github.com/apple/foundationdb/bindings/go/src/fdb/directory/directoryLayer.go', needed by 'bindings/go/src/github.com/apple/foundationdb/bindings/go/src/fdb/directory/directoryLayer.go', missing and no known rule to make it

bindings/go/CMakeLists.txt should be updated with file name changes.

jzhou77 avatar Nov 04 '24 20:11 jzhou77

Result of foundationdb-pr-clang-ide on Linux CentOS 7

  • Commit ID: 5659f310b1e1bce24633c9569d75e3748e4cbfc9
  • Duration 0:20:34
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 11 '24 09:11 foundationdb-ci

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

  • Commit ID: 5659f310b1e1bce24633c9569d75e3748e4cbfc9
  • Duration 0:45:12
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 11 '24 10:11 foundationdb-ci

Result of foundationdb-pr-macos on macOS Ventura 13.x

  • Commit ID: 5659f310b1e1bce24633c9569d75e3748e4cbfc9
  • Duration 0:50:04
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 11 '24 10:11 foundationdb-ci

Result of foundationdb-pr-clang-arm on Linux CentOS 7

  • Commit ID: 5659f310b1e1bce24633c9569d75e3748e4cbfc9
  • Duration 0:51:25
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 11 '24 10:11 foundationdb-ci

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

  • Commit ID: 5659f310b1e1bce24633c9569d75e3748e4cbfc9
  • Duration 0:56:13
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)
  • Cluster Test Logs zip file of the test logs (available for 30 days)

foundationdb-ci avatar Nov 11 '24 10:11 foundationdb-ci

Result of foundationdb-pr-clang on Linux CentOS 7

  • Commit ID: 5659f310b1e1bce24633c9569d75e3748e4cbfc9
  • Duration 0:57:20
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 11 '24 10:11 foundationdb-ci

Result of foundationdb-pr on Linux CentOS 7

  • Commit ID: 5659f310b1e1bce24633c9569d75e3748e4cbfc9
  • Duration 1:02:18
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Nov 11 '24 10:11 foundationdb-ci