Correctly include endpoint id in log msg in AuthorizationPoller

[dimitris-athanasiou]

This fixes debug log messages added in #143567 where the endpoint id is not correctly included.

Summary by CodeRabbit

• Style
  • Updated logging format consistency in authorization-related debug messages.

[elasticsearchmachine]

Pinging @elastic/search-inference-team (Team:Search - Inference)

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: dc6c9375-7ae9-4826-a41e-c4f7ecbdd7e7

📥 Commits

Reviewing files that changed from the base of the PR and between bcc0ae3 and 0cb15e3.

📒 Files selected for processing (1)

• x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/authorization/AuthorizationPoller.java

---

📝 Walkthrough

Walkthrough

Three debug log messages in the AuthorizationPoller's shouldPersistEndpoint method were updated to replace {} format placeholders with %s format specifiers in logging statements. No functional logic changes were made.

Changes

Cohort / File(s)	Summary
Logging Format Updates x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/authorization/AuthorizationPoller.java	Updated three debug log message format placeholders from {} to %s in shouldPersistEndpoint method when endpoint does not exist, fingerprint has changed, or version is higher.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops through logs so bright,
Changing brackets—{} to %s in sight,
No logic bent, just format's dance,
A polish fine, a cleaning glance! ✨🐰

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: fixing log message format to correctly include endpoint id using proper placeholder syntax in AuthorizationPoller.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}