[TEST] Ensure-green inference system indices in AuthorizationTaskExecutorIT

[dimitris-athanasiou]

There are failures in the tests in AuthorizationTaskExecutorIT. We get the all shards fail error some times when we try to get models with ModelRegistry.getAllModels. After looking into this, I believe the cause is that the inference system indices may not manage to initialize properly by the time the test runs.

This calls the ensureGreen() method before any of those tests run. This should fix this potential issue.

Closes #140802
Closes #140849
Closes #143845
Relates #137911
Relates #138012

[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: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: a61d5cd7-06d6-406d-9db1-c83e1950caa5

📥 Commits

Reviewing files that changed from the base of the PR and between 1e07b3a and d610c73.

📒 Files selected for processing (1)

• x-pack/plugin/inference/src/internalClusterTest/java/org/elasticsearch/xpack/inference/integration/AuthorizationTaskExecutorIT.java

---

📝 Walkthrough

Walkthrough

This change modifies the AuthorizationTaskExecutorIT test class to add environment readiness checks during test component creation. The update calls ensureGreen on InferenceIndex.INDEX_PATTERN and InferenceSecretsIndex.INDEX_PATTERN to verify indices are ready before proceeding. Two new imports are added for InferenceIndex and InferenceSecretsIndex. No public API signatures are affected.

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	PR adds ensureGreen calls for inference indices before tests run, directly addressing the intermittent 'all shards failed' failures reported in issues #140802, #140849, #143845.
Out of Scope Changes check	✅ Passed	All changes are scoped to test initialization in AuthorizationTaskExecutorIT, directly supporting the objective of stabilizing the failing tests.

_{✏️ 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

• 🛠️ Update Documentation: Commit on current branch
• 🛠️ Update Documentation: Create PR

---

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

[DonalEvans]

@dimitris-athanasiou It looks like I just missed the boat on this, but should the muted tests be unmuted? Both #140802 and #143845 have muted tests associated with them.

[dimitris-athanasiou]

ah, I'll follow up. thanks for catching this!