[PR] AMBARI-26105: Fix TestPortAlert.py unit test failures in Ambari agent… [ambari]

[via GitHub]

JiaLiangC opened a new pull request, #3800:
URL: https://github.com/apache/ambari/pull/3800

   … due to Python 2 to 3 upgrade
   
   ## What changes were proposed in this pull request?
   After fixing the ZooKeeper port alert, this unit test failed due to changes 
in Python 3 behavior. In Python 3, socket.recv should return bytes instead of a 
string. The original unit test was returning a string, causing subsequent 
assertions to fail. This PR updates the test to handle the byte return type 
correctly, ensuring compatibility with Python 3 while maintaining the test's 
integrity.
   
![image](https://github.com/user-attachments/assets/9f862988-2baf-4135-8116-6260db1df151)
   
   (Please fill in changes proposed in this fix)
   
   ## How was this patch tested?
   manual test and CI test
   
![image](https://github.com/user-attachments/assets/77a1441e-3a11-4bbd-a549-558942681b95)
   
   (Please explain how this patch was tested. Ex: unit tests, manual tests)
   (If this patch involves UI changes, please attach a screen-shot; otherwise, 
remove this)
   
   Please review [Ambari Contributing 
Guide](https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute) 
before opening a pull request.


-- 
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: dev-unsubscr...@ambari.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@ambari.apache.org
For additional commands, e-mail: dev-h...@ambari.apache.org