hyperledger / fabric-x-committer

91%

main: 91%

LAST BUILD ON BRANCH move-retry-to-common

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• abort-snapshot-tx
• acl/1-auth-service
• 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
• checkpoint-verification
• ci-gotestsum
• cli-refactor
• cli-refs
• client-test-race
• client-tls-bug
• clone-delete
• commit-config-immediately
• commit-skill-fix
• config-docs-yaml
• configurable-coordinator-parameters
• configurable-retry-sig-vc-managers
• connection-test-dep
• const-usage-service-metrics
• 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-db-init-when-starting-node-with-db
• 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
• dep-bump-flagged
• dep/fabric-x-common-v0.2.12
• dependabot/go_modules/go_modules-4d54e058aa
• dependabot/go_modules/go_modules-9c5197dcb8
• dependabot/go_modules/go_modules-a135cde889
• dependabot/go_modules/go_modules-c50f2ae065
• dependabot/go_modules/go_modules-c93f3641e6
• dependabot/go_modules/go_modules-d35c03d4db
• dependabot/go_modules/go_modules-dc1255629f
• dependabot/go_modules/go_modules-eee72e131a
• dependencygraph-race
• depgraph/size-scaling
• deployment-strategy
• doc-loadgen-artifacts
• docker-moby-update
• dynamic-root-ca
• dynamic-tls
• enable-pprof
• ensure-clone
• expose-ledger-api
• external-server-config
• extract-prometheus-startup-and-grpc-server-start
• fabric-lib-go-v1.1.3
• fabric-logging
• feat/automatic-tx-parsing-workers
• feat/grpc-flow-control-windows
• feat/loadgen-plain-nonce-ecdsa
• feat/loadgen-quick-asn1-digest
• feat/sidecar-decoded-tx-slab
• feat/sidecar-optional-txid-index
• feat/sidecar-parallel-parse
• feature/adding_release_binary_workflow
• fix-automatic-env
• fix-bft-detection
• fix-connection-reset
• fix-container-image
• fix-container-test-macos
• fix-db-test
• fix-deliverorderer-nil-config-panic
• fix-falky-svmupdate-test
• fix-flaky
• fix-flaky-conn-metrics
• fix-flaky-loadgen-coord
• fix-flaky-sv-manager-port-reuse
• fix-listener-ownership
• fix-master-tserver-rf
• fix-metrics
• fix-mkdoc-url
• fix-poll-condition-outer-t
• fix-port-container-test
• fix-release-path
• fix-sign
• fix-stale-metric-doc-refs
• fix-teardown-exited-container
• fix-test-node-with-insecure
• fix-test-wait
• fix-version-in-hash
• fix-waiting-tx-count
• fix/690-notifier-metric-assertions
• fix/696-integration-port-reservation
• fix/711-flaky-consistent-view
• fix/712-grpcretry-port-reservation
• fix/740-retired-view-batcher-race
• fix/752-config-block
• fix/config-list-defaults
• fix/coordinator-waiting-status-count
• fix/simple-manager-self-dependency
• fix/todo-001
• fix/todo-002
• fix/todo-005
• fix/todo-017
• fix/vulnerabilities
• flaky-coordinator-test-615
• flaky-test-orderer
• getpeersmspdirs-no-hardcode
• go-1.27
• 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-fast-block-prepare
• loadgen-index-addressable-keys
• loadgen-key-generation-metrics
• loadgen-manual-msp
• loadgen-probability-validation
• loadgen-query-client
• loadgen-remove-ctx
• loadgen-remove-query
• main
• make-ci-local
• make-fixes
• makefile-lints
• makefile-test-flags
• metrics-bob-rules
• metrics-doc-scan-all
• 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-hard-coded-monitoring-into-stat-handler
• move-retry-package
• move-retry-to-common
• move-testcrypto-to-common
• move-tx-status-batch
• 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
• query-tls-refresh-ignore-older-config
• queue-gauge-depgraph
• queue-gauge-notifier-stream
• queue-gauge-on-scrape
• queue-gauge-per-stream
• queue-gauge-vc
• race-detection
• rate-limit-sidecar
• refactor-channel
• refactor-cmd
• refactor-monitoring-dependencies
• 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-inflight-slots
• sidecar-limit-active-txids-notifier
• sidecar-mapping-no-per-tx-allocations
• sidecar-mapping-scaffolding
• sidecar-notifier-blockhash
• sidecar-process-snapshot-checkpoint
• sidecar-race
• sidecar-receiver-unwrap-envelope-lite
• sidecar-relay-single-owner-tracking
• sidecar-remove-chan-id
• sidecar-testing-panic-fix
• sidecar-unified
• skill-commit
• skill-dev
• skills-fork-pr-workflow
• skills-omit-verification-from-pr-body
• snapshot-creation
• snapshot-hash
• snapshot-recovery
• snapshot-reject-if-inprogress
• snapshot-service
• split-tablets
• start-server-readiness-context
• stat-handler-phase-two
• 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
• testsig-ecdsa-deprecation
• tls-docs-update
• tx-metadata
• unify-sidecar-service
• unused-code
• update-common
• update-common-dep
• update-common-version
• 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-configurable-parameters
• vc-docs
• vc-manager-tracker-key
• vc-race
• vc-resource-leak
• verifier-lock-wait-or-hold-duration-metrics
• verifier-snapshot-checkpoint
• verifier-test-remove-gomega
• verifier-timeout
• viper-default-annot
• worktree-notification-docs

Build # 40289478944

Build Type

Pull #864

github

Committed by dean-amar

Commit Message

[retry] Use the retry package from fabric-x-common

#### Type of change

- Improvement (improvement to code, performance, etc)

#### Description

- Remove `utils/retry` and import `github.com/hyperledger/fabric-x-common/utils/retry` instead.
- `ExecuteSQL` is Postgres-specific and was not moved to fabric-x-common. Its callers now
  call `retry.Execute` around `pool.Exec` directly.
- Update the `development` skill's reuse table to the new package location.

#### Additional details (Optional)

- `go.mod` temporarily replaces `fabric-x-common` with `dean-amar/fabric-x-common@4468fc9b1`.
  Once hyperledger/fabric-x-common#138 merges, the replace is dropped and `fabric-x-common`
  is bumped to the merged version.

#### Related issues

- resolves partly #675
- depends on hyperledger/fabric-x-common#138

Signed-off-by: Dean Amar <Dean.Amar@ibm.com>

Pull Request Pull Request #864: [dep] Use the retry package from fabric-x-common

Coverage Stats

11 of 12 new or added lines in 2 files covered. (91.67%)

9 existing lines in 6 files now uncovered.

9697 of 10696 relevant lines covered (90.66%)

13344.94 hits per line