Skip to content

tests/docker/azurite: run with a fork which does not leak conns#27018

Merged
nvartolomei merged 1 commit into
redpanda-data:devfrom
nvartolomei:nv/azurite-no-leaks
Jul 28, 2025
Merged

tests/docker/azurite: run with a fork which does not leak conns#27018
nvartolomei merged 1 commit into
redpanda-data:devfrom
nvartolomei:nv/azurite-no-leaks

Conversation

@nvartolomei
Copy link
Copy Markdown
Contributor

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v25.2.x
  • v25.1.x
  • v24.3.x
  • v24.2.x

Release Notes

  • none

@nvartolomei nvartolomei requested a review from a team as a code owner July 28, 2025 15:41
@nvartolomei nvartolomei requested review from andrewhsu and removed request for a team July 28, 2025 15:41
@nvartolomei
Copy link
Copy Markdown
Contributor Author

/ci-repeat 5
release

@nvartolomei nvartolomei requested a review from dotnwat July 28, 2025 18:19
@vbotbuildovich
Copy link
Copy Markdown
Collaborator

CI test results

test results on build#69768
test_class test_method test_arguments test_kind job_url test_status passed reason
EndToEndCloudTopicsTxTest test_write null integration https://buildkite.com/redpanda/redpanda/builds/69768#019851e1-56ee-4b94-8c24-4e50052fb6e7 FLAKY 18/21 upstream reliability is '94.79768786127167'. current run reliability is '85.71428571428571'. drift is 9.0834 and the allowed drift is set to 50. The test should PASS
FeaturesMultiNodeTest test_license_upload_and_query null integration https://buildkite.com/redpanda/redpanda/builds/69768#019851e1-56e8-48d9-bd81-ee3c6b84c52e FLAKY 17/21 upstream reliability is '97.18574108818011'. current run reliability is '80.95238095238095'. drift is 16.23336 and the allowed drift is set to 50. The test should PASS
RandomNodeOperationsTest test_node_operations {"cloud_storage_type": 1, "compaction_mode": "chunked_sliding_window", "enable_failures": true, "mixed_versions": false, "with_iceberg": false} integration https://buildkite.com/redpanda/redpanda/builds/69768#019851e1-56ea-4e0f-a6d2-8f23a6b8b8a6 FLAKY 19/21 upstream reliability is '100.0'. current run reliability is '90.47619047619048'. drift is 9.52381 and the allowed drift is set to 50. The test should PASS
RandomNodeOperationsTest test_node_operations {"cloud_storage_type": 2, "compaction_mode": "adjacent_merge", "enable_failures": true, "mixed_versions": true, "with_iceberg": false} integration https://buildkite.com/redpanda/redpanda/builds/69768#019851e1-56e8-479a-a60c-87daca048f5b FLAKY 20/21 upstream reliability is '99.49109414758269'. current run reliability is '95.23809523809523'. drift is 4.253 and the allowed drift is set to 50. The test should PASS

@WillemKauf
Copy link
Copy Markdown
Contributor

CI results look good?

@nvartolomei
Copy link
Copy Markdown
Contributor Author

@WillemKauf yes. I’d like to merge this to test drive for a few days.

@nvartolomei
Copy link
Copy Markdown
Contributor Author

Slighter longer description in Azure/Azurite#2575 (comment)

@nvartolomei nvartolomei merged commit e58fd7e into redpanda-data:dev Jul 28, 2025
14 checks passed

COPY --from=builder /opt/azurite/dist/ dist/

RUN npm pkg set scripts.prepare="echo no-prepare"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this? Should this replaced with adding --ignore-scripts to the npm ci commands?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copy & pasted the dockerfile from azurite/Dockerfile and just replaced the COPY dockerfile commands with curl.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants