fix: improve memory monitor error message#1019
fix: improve memory monitor error message#1019devin-ai-integration[bot] wants to merge 1 commit into
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksTesting This CDK VersionYou can test this version of the CDK using the following: # Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/airbyte-python-cdk.git@devin/1778380600-memory-monitor-error-message#egg=airbyte-python-cdk[dev]' --help
# Update a connector to use the CDK from this branch ref:
cd airbyte-integrations/connectors/source-example
poe use-cdk-branch devin/1778380600-memory-monitor-error-messagePR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
|
↩️ Triggering Reason: CDK PR is linked to a connector oncall issue, CI is passing or already has local/full pytest evidence, and no AI review marker is present. |
|
Correction: attempted to trigger the CDK This PR remains marked for human review/next-step decision. |
Summary
AirbyteTracedExceptionmessage deterministic by removing the runtime memory percentage.internal_messagefor debugging.Resolves https://github.com/airbytehq/airbyte-internal-issues/issues/16386
Related to https://github.com/airbytehq/oncall/issues/12486
Review & Testing Checklist for Human
system_error.FailureType.system_erroris correct for CDK source-container memory exhaustion.internal_message.Notes
poetry run pytest unit_tests/utils/test_memory_monitor.py::test_raises_when_cgroup_critical_and_anon_share_of_usage_above_threshold -q.Link to Devin session: https://app.devin.ai/sessions/fb2a62909bbf4de3b9149b01c76ea48c