[For Test Only]: add debug log in idle extension loop#1180
[For Test Only]: add debug log in idle extension loop#1180litianningdatadog wants to merge 1 commit intomainfrom
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
litianningdatadog
requested review from
Copilot and
duncanista
and removed request for
Copilot and
duncanista
litianningdatadog
changed the title
|
@codex review |
|
To use Codex here, create a Codex account and connect to github. |
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4068636bd6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| match extension::next_event(client, &r.extension_id, &aws_config.runtime_api).await { | ||
| Ok(_) => { | ||
| debug!("Extension is idle, skipping next event"); | ||
| debug!("This is a test"); |
There was a problem hiding this comment.
This adds a non-actionable "This is a test" message inside extension_loop_idle, which executes continuously while the extension is idle. In debug-enabled environments, this will generate repetitive noise on every idle event and make it harder to diagnose real issues from logs; it appears to be test-only output rather than production telemetry.
Useful? React with 👍 / 👎.
1 participant
Summary
debug!("This is a test")log line in the idle extension loop handlerTest plan
RUST_LOG=debugis set🤖 Generated with Claude Code