3822 rejected transaction api service

.github/workflows/build-and-publish.yml

@@ -30,6 +30,9 @@ on:
  traces_api_facade_changed:
  required: true
  type: string
+ transaction_exclusion_api_changed:
+ required: true
+ type: string
  coordinator_image_tagged:
  required: true
  type: string
@@ -42,6 +45,9 @@ on:
  traces_api_facade_image_tagged:
  required: true
  type: string
+ transaction_exclusion_api_image_tagged:
+ required: true
+ type: string
  secrets:
  DOCKERHUB_USERNAME:
  required: true
@@ -96,3 +102,15 @@ jobs:
  untested_tag_suffix: ${{ inputs.untested_tag_suffix }}
  image_name: consensys/linea-traces-api-facade
  secrets: inherit
+
+ transaction_exclusion_api:
+ uses: ./.github/workflows/transaction-exclusion-api-build-and-publish.yml
+ if: ${{ always() && (inputs.transaction_exclusion_api_changed == 'true' || inputs.transaction_exclusion_api_image_tagged != 'true') }}
+ with:
+ commit_tag: ${{ inputs.commit_tag }}
+ last_commit_tag: ${{ inputs.last_commit_tag }}
+ common_ancestor_tag: ${{ inputs.common_ancestor_tag }}
+ develop_tag: ${{ inputs.develop_tag }}
+ untested_tag_suffix: ${{ inputs.untested_tag_suffix }}
+ image_name: consensys/linea-transaction-exclusion-api
+ secrets: inherit

.github/workflows/main.yml

@@ -18,8 +18,9 @@ jobs:
  postman: ${{ steps.filter.outputs.postman }}
  prover: ${{ steps.filter.outputs.prover }}
  traces-api-facade: ${{ steps.filter.outputs.traces-api-facade }}
+ transaction-exclusion-api: ${{ steps.filter.outputs.transaction-exclusion-api }}
  finalized-tag-updater: ${{ steps.filter.outputs.finalized-tag-updater }}
- no-changes: ${{ steps.filter.outputs.coordinator == 'false' && steps.filter.outputs.postman == 'false' && steps.filter.outputs.prover == 'false' && steps.filter.outputs.traces-api-facade == 'false' }}
+ no-changes: ${{ steps.filter.outputs.coordinator == 'false' && steps.filter.outputs.postman == 'false' && steps.filter.outputs.prover == 'false' && steps.filter.outputs.traces-api-facade == 'false' && steps.filter.outputs.transaction-exclusion-api == 'false' }}
  steps:
  - name: Checkout
  uses: actions/checkout@v3
@@ -85,6 +86,25 @@ jobs:
  - 'build.gradle'
  - 'gradle.properties'
  - 'settings.gradle'
+ transaction-exclusion-api:
+ - 'transaction-exclusion-api/**'
+ - 'jvm-libs/core/domain-models/**'
+ - 'jvm-libs/core/long-running-service/**'
+ - 'jvm-libs/json-rpc/**'
+ - 'jvm-libs/kotlin-extensions/**'
+ - 'jvm-libs/future-extensions/**'
+ - 'jvm-libs/vertx-helper/**'
+ - 'jvm-libs/metrics/**'
+ - 'jvm-libs/persistence/**'
+ - '.github/workflows/transaction-exclusion-api-*.yml'
+ - '.github/workflows/build-and-publish.yml'
+ - '.github/workflows/main.yml'
+ - '.github/workflows/reuse-*.yml'
+ - 'buildSrc/**'
+ - 'gradle/**'
+ - 'build.gradle'
+ - 'gradle.properties'
+ - 'settings.gradle'
  finalized-tag-updater:
  - 'jvm-libs/core/long-running-service/**'
  - 'jvm-libs/kotlin-extensions/**'
@@ -107,6 +127,7 @@ jobs:
  postman_changed: ${{ needs.filter-commit-changes.outputs.postman }}
  prover_changed: ${{ needs.filter-commit-changes.outputs.prover }}
  traces_api_facade_changed: ${{ needs.filter-commit-changes.outputs.traces-api-facade }}
+ transaction_exclusion_api_changed: ${{ needs.filter-commit-changes.outputs.transaction-exclusion-api }}
  secrets: inherit
 
  manual-docker-build-and-e2e-tests:
@@ -131,10 +152,12 @@ jobs:
  postman_changed: ${{ needs.filter-commit-changes.outputs.postman }}
  prover_changed: ${{ needs.filter-commit-changes.outputs.prover }}
  traces_api_facade_changed: ${{ needs.filter-commit-changes.outputs.traces-api-facade }}
+ transaction_exclusion_api_changed: ${{ needs.filter-commit-changes.outputs.transaction-exclusion-api }}
  coordinator_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_coordinator }}
  postman_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_postman }}
  prover_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_prover }}
  traces_api_facade_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_traces_api_facade }}
+ transaction_exclusion_api_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_transaction_exclusion_api }}
  secrets: inherit
 
  # Comment out the auto build and release step below as the plugin release should be
@@ -156,9 +179,11 @@ jobs:
  postman_changed: ${{ needs.filter-commit-changes.outputs.postman }}
  prover_changed: ${{ needs.filter-commit-changes.outputs.prover }}
  traces_api_facade_changed: ${{ needs.filter-commit-changes.outputs.traces-api-facade }}
+ transaction_exclusion_api_changed: ${{ needs.filter-commit-changes.outputs.transaction-exclusion-api }}
  coordinator_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_coordinator }}
  postman_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_postman }}
  traces_api_facade_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_traces_api_facade }}
+ transaction_exclusion_api_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_transaction_exclusion_api }}
  secrets: inherit
 
  run-e2e-tests-geth-tracing:
@@ -197,7 +222,7 @@ jobs:
  commit_tag: ${{ needs.store-image-name-and-tags.outputs.commit_tag }}
  develop_tag: ${{ needs.store-image-name-and-tags.outputs.develop_tag }}
  untested_tag_suffix: ${{ needs.store-image-name-and-tags.outputs.untested_tag_suffix }}
- image_names: '["consensys/linea-coordinator", "consensys/linea-postman", "consensys/linea-prover", "consensys/linea-traces-api-facade"]'
+ image_names: '["consensys/linea-coordinator", "consensys/linea-postman", "consensys/linea-prover", "consensys/linea-traces-api-facade", "consensys/linea-transaction-exclusion-api"]'
  secrets: inherit
 
  cleanup-deployments:

.github/workflows/reuse-check-images-tags-and-push.yml

@@ -29,6 +29,9 @@ on:
  traces_api_facade_changed:
  required: true
  type: string
+ transaction_exclusion_api_changed:
+ required: true
+ type: string
  outputs:
  image_tagged_coordinator:
  value: ${{ jobs.image_tag_push.outputs.image_tagged_coordinator }}
@@ -38,6 +41,8 @@ on:
  value: ${{ jobs.image_tag_push.outputs.image_tagged_postman }}
  image_tagged_traces_api_facade:
  value: ${{ jobs.image_tag_push.outputs.image_tagged_traces_api_facade }}
+ image_tagged_transaction_exclusion_api:
+ value: ${{ jobs.image_tag_push.outputs.image_tagged_transaction_exclusion_api }}
  secrets:
  DOCKERHUB_USERNAME:
  required: true
@@ -61,6 +66,8 @@ jobs:
  common_ancestor_commit_tag_exists_prover: ${{ steps.check_image_tags_exist_prover.outputs.common_ancestor_commit_tag_exists }}
  last_commit_tag_exists_traces_api_facade: ${{ steps.check_image_tags_exist_traces_api_facade.outputs.last_commit_tag_exists }}
  common_ancestor_commit_tag_exists_traces_api_facade: ${{ steps.check_image_tags_exist_traces_api_facade.outputs.common_ancestor_commit_tag_exists }}
+ last_commit_tag_exists_transaction_exclusion_api: ${{ steps.check_image_tags_exist_transaction_exclusion_api.outputs.last_commit_tag_exists }}
+ common_ancestor_commit_tag_exists_transaction_exclusion_api: ${{ steps.check_image_tags_exist_transaction_exclusion_api.outputs.common_ancestor_commit_tag_exists }}
  steps:
  - name: Checkout
  uses: actions/checkout@v4
@@ -108,6 +115,17 @@ jobs:
  image_name: consensys/linea-traces-api-facade
  docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
  docker_password: ${{ secrets.DOCKERHUB_TOKEN }}
+
+ - name: Check image tags exist for transaction-exclusion-api
+ uses: ./.github/actions/check-image-tags-exist
+ if: ${{ inputs.transaction_exclusion_api_changed == 'false' }}
+ id: check_image_tags_exist_transaction_exclusion_api
+ with:
+ last_commit_tag: ${{ inputs.last_commit_tag }}
+ common_ancestor_tag: ${{ inputs.common_ancestor_tag }}
+ image_name: consensys/linea-transaction-exclusion-api
+ docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
+ docker_password: ${{ secrets.DOCKERHUB_TOKEN }}
 
  image_tag_push:
  runs-on: [self-hosted, ubuntu-20.04, X64, small]
@@ -118,6 +136,7 @@ jobs:
  image_tagged_prover: ${{ steps.image_tag_push_prover.outputs.image_tagged }}
  image_tagged_postman: ${{ steps.image_tag_push_postman.outputs.image_tagged }}
  image_tagged_traces_api_facade: ${{ steps.image_tag_push_traces_api_facade.outputs.image_tagged }}
+ image_tagged_transaction_exclusion_api: ${{ steps.image_tag_push_transaction_exclusion_api.outputs.image_tagged }}
  steps:
  - name: Checkout
  uses: actions/checkout@v4
@@ -185,3 +204,19 @@ jobs:
  common_ancestor_commit_tag_exists: ${{ needs.check_image_tags_exist.outputs.common_ancestor_commit_tag_exists_traces_api_facade }}
  docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
  docker_password: ${{ secrets.DOCKERHUB_TOKEN }}
+
+ - name: Tag and push transaction exclusion api image
+ id: image_tag_push_transaction_exclusion_api
+ uses: ./.github/actions/image-tag-and-push
+ if: ${{ inputs.transaction_exclusion_api_changed == 'false' }}
+ with:
+ commit_tag: ${{ inputs.commit_tag }}
+ last_commit_tag: ${{ inputs.last_commit_tag }}
+ common_ancestor_tag: ${{ inputs.common_ancestor_tag }}
+ develop_tag: ${{ inputs.develop_tag }}
+ untested_tag_suffix: ${{ inputs.untested_tag_suffix }}
+ image_name: consensys/linea-transaction-exclusion-api
+ last_commit_tag_exists: ${{ needs.check_image_tags_exist.outputs.last_commit_tag_exists_transaction_exclusion_api }}
+ common_ancestor_commit_tag_exists: ${{ needs.check_image_tags_exist.outputs.common_ancestor_commit_tag_exists_transaction_exclusion_api }}
+ docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
+ docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

.github/workflows/reuse-run-e2e-tests.yml

@@ -29,6 +29,11 @@ on:
  required: false
  type: boolean
  default: true
+ e2e-tests-for-tx-exclusion:
+ description: Run end to end tests for transaction exclusion
+ required: false
+ type: boolean
+ default: false
  workflow_call:
  inputs:
  commit_tag:
@@ -56,6 +61,11 @@ on:
  required: false
  type: boolean
  default: true
+ e2e-tests-for-tx-exclusion:
+ description: Run end to end tests for transaction exclusion
+ required: false
+ type: boolean
+ default: false
  outputs:
  tests_outcome:
  value: ${{ jobs.run-e2e-tests.outputs.tests_outcome }}
@@ -72,9 +82,10 @@ jobs:
  POSTMAN_TAG: ${{ inputs.commit_tag }}-${{ inputs.untested_tag_suffix }}
  PROVER_TAG: ${{ inputs.commit_tag }}-${{ inputs.untested_tag_suffix }}
  TRACES_API_TAG: ${{ inputs.commit_tag }}-${{ inputs.untested_tag_suffix }}
+ TRANSACTION_EXCLUSION_API_TAG: ${{ inputs.commit_tag }}-${{ inputs.untested_tag_suffix }}
  GITHUB_TOKEN: ${{ secrets._GITHUB_TOKEN_RELEASE_ACCESS }}
  outputs:
- tests_outcome: ${{ steps.run_e2e_tests.outcome }}
+ tests_outcome: ${{ (inputs.e2e-tests-for-tx-exclusion && steps.run_e2e_tests_tx_exclusion.outcome) || (!inputs.e2e-tests-for-tx-exclusion && steps.run_e2e_tests.outcome) }}
  runs-on: [self-hosted, ubuntu-22.04, X64, large]
  steps:
  - name: Setup upterm session
@@ -113,7 +124,7 @@ jobs:
  command: |
  make pull-all-images
  - name: Spin up fresh environment with geth tracing with retry
- if: ${{ inputs.tracing-engine == 'geth' }}
+ if: ${{ inputs.tracing-engine == 'geth' && !inputs.e2e-tests-for-tx-exclusion }}
  uses: nick-fields/retry@v2
  with:
  max_attempts: 10
@@ -125,7 +136,7 @@ jobs:
  on_retry_command: |
  make clean-environment
  - name: Spin up fresh environment with besu tracing with retry
- if: ${{ inputs.tracing-engine == 'besu' }}
+ if: ${{ inputs.tracing-engine == 'besu' && !inputs.e2e-tests-for-tx-exclusion }}
  uses: nick-fields/retry@v2
  with:
  max_attempts: 10
@@ -136,6 +147,30 @@ jobs:
  make start-all-traces-v2
  on_retry_command: |
  make clean-environment
+ - name: Spin up fresh environment for tx-exclusion e2e tests with geth tracing with retry
+ if: ${{ inputs.tracing-engine == 'geth' && inputs.e2e-tests-for-tx-exclusion }}
+ uses: nick-fields/retry@v2
+ with:
+ max_attempts: 10
+ retry_on: error
+ retry_wait_seconds: 30
+ timeout_minutes: 10
+ command: |
+ make start-all-tx-exclusion
+ on_retry_command: |
+ make clean-environment
+ - name: Spin up fresh environment for tx-exclusion e2e tests with besu tracing with retry
+ if: ${{ inputs.tracing-engine == 'besu' && inputs.e2e-tests-for-tx-exclusion }}
+ uses: nick-fields/retry@v2
+ with:
+ max_attempts: 10
+ retry_on: error
+ retry_wait_seconds: 30
+ timeout_minutes: 10
+ command: |
+ make start-all-tx-exclusion-traces-v2
+ on_retry_command: |
+ make clean-environment
  - name: List docker containers/images
  if: ${{ always() && inputs.e2e-tests-containers-list }}
  continue-on-error: true
@@ -145,12 +180,19 @@ jobs:
  - name: Run e2e tests
  id: run_e2e_tests
  timeout-minutes: 25
+ if: ${{ !inputs.e2e-tests-for-tx-exclusion }}
  run: |
  pnpm run -F e2e test:e2e:local
+ - name: Run e2e tests for tx exclusion
+ id: run_e2e_tests_tx_exclusion
+ timeout-minutes: 25
+ if: ${{ inputs.e2e-tests-for-tx-exclusion }}
+ run: |
+ pnpm run -F e2e test:e2e:tx-exclusion:local
  - name: Show e2e tests result
  if: always()
  run: |
- echo "E2E_TESTS_RESULT: ${{ steps.run_e2e_tests.outcome }}"
+ echo "E2E_TESTS_RESULT: ${{ (inputs.e2e-tests-for-tx-exclusion && steps.run_e2e_tests_tx_exclusion.outcome) || (!inputs.e2e-tests-for-tx-exclusion && steps.run_e2e_tests.outcome) }}"
  - name: Dump logs
  if: ${{ failure() && inputs.e2e-tests-logs-dump }}
  run: |
@@ -165,6 +207,7 @@ jobs:
  docker logs postman --since 1h &>> docker_logs/postman.txt
  docker logs traces-node --since 1h &>> docker_logs/traces-node.txt
  docker logs traces-node-v2 --since 1h &>> docker_logs/traces-node-v2.txt;
+ docker logs transaction-exclusion-api --since 1h &>> docker_logs/transaction-exclusion-api.txt
  docker logs sequencer --since 1h &>> docker_logs/sequencer.txt
  - name: Archive debug logs
  uses: actions/upload-artifact@v4

.github/workflows/testing.yml

@@ -18,6 +18,9 @@ on:
  traces_api_facade_changed:
  required: true
  type: string
+ transaction_exclusion_api_changed:
+ required: true
+ type: string
  coordinator_image_tagged:
  required: true
  type: string
@@ -27,6 +30,9 @@ on:
  traces_api_facade_image_tagged:
  required: true
  type: string
+ transaction_exclusion_api_image_tagged:
+ required: true
+ type: string
  secrets:
  DOCKERHUB_USERNAME:
  required: true
@@ -55,3 +61,8 @@ jobs:
  uses: ./.github/workflows/traces-api-facade-testing.yml
  if: ${{ always() && (inputs.traces_api_facade_changed == 'true' || inputs.traces_api_facade_image_tagged != 'true') }}
  secrets: inherit
+
+ transaction-exclusion-api:
+ uses: ./.github/workflows/transaction-exclusion-api-testing.yml
+ if: ${{ always() && (inputs.transaction_exclusion_api_changed == 'true' || inputs.transaction_exclusion_api_image_tagged != 'true') }}
+ secrets: inherit

.github/workflows/traces-api-facade-testing.yml

@@ -43,7 +43,7 @@ jobs:
  retry_on: error
  timeout_minutes: 20
  command: |
- ./gradlew traces-api-facade:app:buildNeeded jacocoRootReport
+ ./gradlew -V traces-api-facade:app:buildNeeded jacocoRootReport
  - name: Run tests without coverage
  uses: nick-fields/retry@v2
  if: ${{ !inputs.coverage }}
@@ -52,4 +52,4 @@ jobs:
  retry_on: error
  timeout_minutes: 20
  command: |
- ./gradlew traces-api-facade:app:buildNeeded
+ ./gradlew -V traces-api-facade:app:buildNeeded