hyperledger / fabric-x-committer

89%

main: 90%

LAST BUILD ON BRANCH fix-container-image

branch: fix-container-image

CHANGE BRANCH

x

Reset

• fix-container-image
• apt-ci-fix
• bft-delivery
• ci-gotestsum
• commit-config-immediately
• config-docs-yaml
• connection-test-dep
• coveralls
• cryptogen-recreate-support
• default-test-node-startup-setup
• deliver-check-block-order
• deliver-seek-tls-cert
• doc-loadgen-artifacts
• dynamic-root-ca
• enable-pprof
• expose-ledger-api
• fabric-logging
• fix-connection-reset
• fix-container-test-macos
• fix-db-test
• fix-falky-svmupdate-test
• fix-flaky-conn-metrics
• fix-flaky-loadgen-coord
• fix-master-tserver-rf
• fix-port-container-test
• fix-release-path
• fix-sign
• fix-test-node-with-insecure
• fix-waiting-tx-count
• go-fix
• ledger-service-wait-behaviour
• lifecycle-policy
• limit-block-apis
• limit-inputs-block-apis
• loadgen-remove-ctx
• main
• make-fixes
• metrics-tls
• migrate-to-postgres-official-image
• minimize-logs
• mock-orderer-features
• mock-orderer-stream-holder
• namespace-policy-docs
• optimize-map-block
• pgx5
• postgres-renaming
• qs-pen-test-fix
• rate-limit-sidecar
• root-ca-workaround-fix
• sidecar-block-store-code-fix
• sidecar-limit-active-txids-notifier
• sidecar-remove-chan-id
• split-tablets
• sustain-config
• sync-block
• template-tls-per-endpoint
• test-coverage-improve
• test-coverage-tests
• test-limiter-max-wait
• test-minor-fixes
• test-query-status
• test-tls-flatten
• testcrypto-refactor
• tls-docs-update
• update-to-latest-common
• update-to-yugabytedb-latest-lts
• upgrade-pg
• worktree-notification-docs

Build # 22315506234

Build Type

Pull #376

github

Committed by cendhu

Commit Message

fix TestCommitterReleaseImagesWithTLS

Root cause: ExposePort() used HostPort: "" with go-dockerclient's
PortBinding, which has json:"HostPort,omitempty". The omitempty
tag drops the empty string during JSON serialization, so Docker
never receives a HostPort field and defaults to binding the
container's own port number (5433) on the host. When parallel
subtests each try to bind host port 5433, only the first wins.

Fix: HostPort: "" → HostPort: "0". The string "0" explicitly
tells Docker to auto-assign an ephemeral port, and being non-empty,
it survives omitempty serialization.

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Pull Request Pull Request #376: fix TestCommitterReleaseImagesWithTLS

Run Details

9182 of 10335 relevant lines covered (88.84%)

1.29 hits per line