hyperledger / fabric-x-committer

91%

main: 91%

LAST BUILD ON BRANCH fix/696-integration-port-reservation

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• add-live-connection-monitoring
• additional-db-test
• admin-db-init
• apt-ci-fix
• auto-merge-fix
• backoff-v5
• bccsp-race
• bench-report-metrics
• bft-delivery
• bft-metrics
• bob-reduce-context
• bob-rules
• buffer-race
• build-unify
• channel-race
• ci-gotestsum
• cli-refactor
• cli-refs
• client-test-race
• client-tls-bug
• commit-config-immediately
• config-docs-yaml
• configurable-coordinator-parameters
• configurable-retry-sig-vc-managers
• connection-test-dep
• coord-queue-deadlock
• coordinator-docs
• coordinator-manager-metrics
• coordinator-manager-parity
• coordinator-metadata-fix
• coordinator-test-no-db
• coordinator-vc-api-split
• coveralls
• create-multiple-namespaces-test
• create-new-sys-ns
• cryptogen-recreate-support
• dedup-templ
• default-test-node-startup-setup
• default-viper
• deliver-check-block-order
• deliver-seek-tls-cert
• deliver-seek-tls-cert-fix
• deliverorderer-tests
• delivery-data-blocks
• dependabot/go_modules/go_modules-9c5197dcb8
• dependabot/go_modules/go_modules-d35c03d4db
• dependabot/go_modules/go_modules-dc1255629f
• dependabot/go_modules/go_modules-eee72e131a
• dependencygraph-race
• deployment-strategy
• doc-loadgen-artifacts
• docker-moby-update
• dynamic-root-ca
• dynamic-tls
• enable-pprof
• expose-ledger-api
• external-server-config
• extract-prometheus-startup-and-grpc-server-start
• fabric-lib-go-v1.1.3
• fabric-logging
• fix-automatic-env
• fix-bft-detection
• fix-connection-reset
• fix-container-image
• fix-container-test-macos
• fix-db-test
• fix-falky-svmupdate-test
• fix-flaky
• fix-flaky-conn-metrics
• fix-flaky-loadgen-coord
• fix-master-tserver-rf
• fix-metrics
• fix-mkdoc-url
• fix-port-container-test
• fix-release-path
• fix-sign
• fix-test-node-with-insecure
• fix-test-wait
• fix-waiting-tx-count
• fix/690-notifier-metric-assertions
• fix/696-integration-port-reservation
• fix/712-grpcretry-port-reservation
• fix/coordinator-waiting-status-count
• fix/todo-001
• fix/todo-002
• fix/todo-005
• fix/todo-017
• flaky-coordinator-test-615
• flaky-test-orderer
• getpeersmspdirs-no-hardcode
• go-fix
• grpc-logger-fix
• grpcservice-refactor
• healthz-check
• http-endpoint
• inline-commands
• iss-640
• keep-alive-parameters-testing
• ledger-service-wait-behaviour
• lifecycle-policy
• limit-block-apis
• limit-inputs-block-apis
• llm-adjust
• load-config-block-certs
• loadgen-config-cleanup
• loadgen-const-distribution-shorthand
• loadgen-index-addressable-keys
• loadgen-manual-msp
• loadgen-probability-validation
• loadgen-remove-ctx
• loadgen-remove-query
• main
• make-ci-local
• make-fixes
• makefile-lints
• makefile-test-flags
• metrics-bob-rules
• metrics-tls
• metrics-url-race
• migrate-to-postgres-official-image
• minimize-logs
• mkdocs
• mock-orderer-direct-config
• mock-orderer-dynamic-tls
• mock-orderer-features
• mock-orderer-stream-holder
• mock-verifier-race
• move-retry-package
• move-testcrypto-to-common
• namespace-policy-docs
• notifier-metrics
• notify-full-block
• notify-tx-metadata
• optimize-map-block
• orderer-config-race
• pgx5
• postgres-renaming
• prom-refactor
• publish-common-packages
• qs-pen-test-fix
• race-detection
• rate-limit-sidecar
• refactor-channel
• refactor-cmd
• refactor-test
• release-1.0
• remove-common-root-cas
• remove-file
• remove-ifrit-gomega
• remove-loadgen-distribution
• remove-non-maintained-pkg
• rename
• rename-materials
• rename-util-files
• retry-infinite-max-elapsed-time
• review-ai-prompt
• rm-gpl-tools
• rm-reployment
• rm-unused
• root-ca-workaround-fix
• service-manager
• sidecar-block-store-code-fix
• sidecar-diagram-mod
• sidecar-diagrm
• sidecar-limit-active-txids-notifier
• sidecar-process-snapshot-checkpoint
• sidecar-race
• sidecar-receiver-unwrap-envelope-lite
• sidecar-remove-chan-id
• sidecar-unified
• skill-commit
• skill-dev
• skills-fork-pr-workflow
• skills-omit-verification-from-pr-body
• snapshot-creation
• snapshot-hash
• snapshot-reject-if-inprogress
• split-tablets
• start-server-readiness-context
• sustain-config
• sync-block
• template-tls-per-endpoint
• test-container-port-binding
• test-coverage-connection
• test-coverage-improve
• test-coverage-latency
• test-coverage-tests
• test-limiter-max-wait
• test-minor-fixes
• test-query-status
• test-tls-flatten
• testcrypto-race
• testcrypto-refactor
• tls-docs-update
• tx-metadata
• unused-code
• update-common
• update-common-dep
• update-dep
• update-dep-4
• update-depenencies-3
• update-deps
• update-dev-skills
• update-to-latest-common
• update-to-yugabytedb-latest-lts
• upgrade-common-dep
• upgrade-pg
• utils-channel-optional-writer
• vc-docs
• vc-manager-tracker-key
• vc-race
• vc-resource-leak
• verifier-snapshot-checkpoint
• verifier-test-remove-gomega
• verifier-timeout
• viper-default-annot
• worktree-notification-docs

Build # 30749695901

Build Type

Pull #728

github

Committed by liran-funaro

Commit Message

[integration-test] Reserve service ports across restarts to fix port races

- Bug fix
- Improvement (improvement to code, performance, etc)

- Allocate each service's gRPC/HTTP ports directly into the process that
  owns them, holding a listener on each from allocation until the subprocess
  binds it, and re-reserving the moment the subprocess stops (`Stop`) so a
  parallel test cannot claim a just-freed port while the service is down.
- Drop the central reservation pool and the allocate-then-transfer machinery
  (`claim`/`claimAll`/`closeAll`); defer config-file writing to a second pass
  once every service endpoint is known, since a template renders the whole
  `SystemConfig`.
- Pre-allocate the load generators in `NewRuntime` like the other services:
  the primary reserves its ports, while the distributed client binds its own
  ephemeral port (nothing dials it). Their config is rendered in `startLoadGen`,
  where the template and worker count are known from the service flags.
- Add `OrdererEnv.ReserveListeners`/`ReleaseListeners` so the orderer test
  environment follows the same stop-then-reserve pattern.

- resolves #696

Signed-off-by: Liran Funaro <liran.funaro@gmail.com>

Pull Request Pull Request #728: [integration-test] Reserve service ports across restarts to fix port races

Coverage Stats

10552 of 11569 relevant lines covered (91.21%)

33650.53 hits per line