change version to 2.52.1

[elliot-barn]

for 2.52.1 release

[gemini-code-assist]

Code Review

This pull request primarily focuses on bumping the version to 2.52.1 and introducing a new token-based authentication mechanism. The changes are comprehensive, including significant security enhancements like using the secrets module for token generation, extensive documentation for the new authentication feature, and refactoring of CI/build configurations. The code quality is high, with good test coverage for the new features. I have one suggestion to improve the CLI user experience by using a more appropriate exception type for error handling.

[cursor]

Bug: Double period in authentication error messages

The AuthenticationError.__str__ method prepends . Ensure... to the message, but the error message constants in authentication_constants.py (like TOKEN_AUTH_ENABLED_BUT_NO_TOKEN_FOUND_ERROR_MESSAGE and TOKEN_INVALID_ERROR_MESSAGE) already end with periods. This results in double periods (..) in error output, e.g., "Token authentication is enabled but no authentication token was found.. Ensure that you have...". Either the constants should not end with periods, or __str__ should not prepend one.

python/ray/exceptions.py#L496-L501

ray/python/ray/exceptions.py

Lines 496 to 501 in 8337334

	def __str__(self) -> str:
	return self.message + (
	". Ensure that you have `RAY_AUTH_MODE=token` set and the token for the cluster is available as the `RAY_AUTH_TOKEN` environment variable or a local file. "
	"For more information, see: https://docs.ray.io/en/latest/ray-security/auth.html"
	)

python/ray/_private/authentication/authentication_constants.py#L1-L6

ray/python/ray/_private/authentication/authentication_constants.py

Lines 1 to 6 in 8337334

	# Authentication error messages
	TOKEN_AUTH_ENABLED_BUT_NO_TOKEN_FOUND_ERROR_MESSAGE = (
	"Token authentication is enabled but no authentication token was found."
	)
	TOKEN_INVALID_ERROR_MESSAGE = "Token authentication is enabled but the authentication token is invalid or incorrect." # noqa: E501

 

---