github-actions[bot] commented on PR #4710:
URL: https://github.com/apache/solr/pull/4710#issuecomment-6824503123

   **Automated PR readiness judgements** (LLM-generated; may be wrong; not 
updated on later pushes).
   
   - ✅ [Jira issue in 
title](https://github.com/apache/solr/blob/main/CONTRIBUTING.md): already 
present
   - ✅ 
[Changelog](https://github.com/apache/solr/blob/main/dev-docs/changelog.adoc#_changelog_types_reference):
 type: fixed <!-- The change fixes a bug where &#96;bin&#47;​solr&#46;​cmd 
start&#96; on Windows would wait for all running Solr instances instead of just 
the one being started&#44; and also fixes the connection host resolution to use 
the advertised or bind host instead of always localhost&#44; which is a 
user&#45;visible fix for SSL and non&#45;loopback binds&#46;​ -->
   - ⚠️ [Reference Guide 
update](https://github.com/apache/solr/tree/main/solr/solr-ref-guide) **— 
probably not needed** (confidence: medium): The change modifies internal 
behavior of the CLI status tool and the Windows start script to correctly 
target the specific instance and resolve the correct host&#44; but does not 
introduce new configuration options&#44; parameters&#44; or user&#45;facing API 
changes that require Reference Guide documentation&#46;​
   - ✅ [Upgrade 
notes](https://github.com/apache/solr/tree/main/solr/solr-ref-guide/modules/upgrade-notes/pages):
 not needed <!-- The change is a bug fix that restores expected behavior 
&#40;waiting for the specific instance and using the correct host&#41; and does 
not remove or rename any parameters&#44; APIs&#44; or configuration 
elements&#44; nor does it change defaults in a way that requires user 
action&#46;​ -->
   - ✅ Tests: included <!-- The PR introduces new logic in 
&#96;SolrProcessManager&#96; for parsing system properties&#44; resolving local 
connect hosts &#40;including IPv6 and wildcard handling&#41;&#44; and waiting 
for processes&#44; as well as changes in &#96;StatusTool&#96; for timeout 
management&#44; which are non&#45;trivial production logic changes that warrant 
new or updated tests&#46;​ -->
   
   <!-- Assessed commit 1d81010cba3175117bbf87933a26271dd1b81523 -->
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to