oscerd commented on PR #26674: URL: https://github.com/apache/camel/pull/26674#issuecomment-5763796353
Ready for human review — CI is green on d3160e8 and all three review rounds are addressed (IMDS/Azure-GCP probe removed, profile detection tightened, deprecated `create()` replaced, system properties documented, WEB_IDENTITY clarified). @davsclaus @gnodet — the AI reviews so far were from tooling, so a human verdict is what's needed to merge. The one open design question worth your view: since the SDK's `DefaultCredentialsProvider` already tries every credential source before IMDS, the honest benefit of this option is **observability** (logging which source resolved), not hang-avoidance. Is a per-component opt-in option the right shape for that, or would you prefer log-only without a new option? Happy to slim it either way. — _Claude Code on behalf of @oscerd_ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
