Skip to content

test(monitor): add resource minotor logs for e2e test job#2222

Open
Junjiequan wants to merge 1 commit intomasterfrom
monitor-docker-container-stats-for-test-job
Open

test(monitor): add resource minotor logs for e2e test job#2222
Junjiequan wants to merge 1 commit intomasterfrom
monitor-docker-container-stats-for-test-job

Conversation

@Junjiequan
Copy link
Member

@Junjiequan Junjiequan commented Feb 16, 2026

Description

This PR adds Docker container resource monitor logs to the e2e tests job.
image

Motivation

Elasticsearch crashed silently during e2e tests due to insufficient memory.
These logs make it easier to diagnose such scenarios.

Fixes:

Please provide a list of the fixes implemented in this PR

  • Items added

Changes:

Please provide a list of the changes implemented by this PR

  • changes made

Tests included

  • Included for each change/fix?
  • Passing? (Merge will not be approved unless this is checked)

Documentation

  • swagger documentation updated [required]
  • official documentation updated [nice-to-have]

official documentation info

If you have updated the official documentation, please provide PR # and URL of the pages where the updates are included

Backend version

  • Does it require a specific version of the backend
  • which version of the backend is required:

Summary by Sourcery

Add resource monitoring to the e2e test GitHub Actions workflow to aid debugging of resource-related failures.

CI:

  • Add a background job in the e2e test workflow to periodically log system memory and Docker container stats.
  • Print the collected resource monitor log at the end of the e2e test job for inspection on failures.

@Junjiequan Junjiequan requested a review from a team as a code owner February 16, 2026 14:43
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@Junjiequan Junjiequan force-pushed the monitor-docker-container-stats-for-test-job branch from c7544be to 288091b Compare February 16, 2026 19:24
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.

2 participants

Comments