[SERVE-657] Replace special characters by underscore to fix logging failure during serve.deploy()

[ok-scale]

Why are these changes needed?

When creating a Serve deployment with a name containing a slash (such as TextGenerationModel.options(name="huawei-noah/TinyBERT_General_4L_312D")) leads to actor failures. The error occurs because the slash in the name is likely being used as a path separator in log files.

Error: FileNotFoundError: [Errno 2] No such file or directory: '/tmp/ray/session_2025-03-11_16-02-48_264712_45753/logs/serve/replica_default_huawei-noah/TinyBERT_General_4L_312D_eo3vqu7d.log'

In general, any other special character used may cause the logger to crash. This branch fixes the issue by replacing all special characters with _ for logging purposes.

Related issue number

https://anyscale1.atlassian.net/browse/SERVE-657

Checks

• I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
• I've run scripts/format.sh to lint the changes in this PR.
• I've included any doc changes needed for https://docs.ray.io/en/master/.
  • I've added any new APIs to the API Reference. For example, if I added a
    method in Tune, I've added it in doc/source/tune/api/ under the
    corresponding .rst file.
• I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
• Testing Strategy
  • Unit tests
  • Release tests
  • This PR is not tested :(

[abrarsheikh]

@zcin, does this have any side effects on the service's ability to reference the correct app log on anyscale ?

Is component_id user-provided? If not, maybe we can avoid applying the sub-operation on it.

[zcin]

Hmm yeah I don't know the full implications of this, I would be hesitant to do this unless it's fully scoped out what the effects are.

@GeneDer might know more about the logging issue specifically.

[ok-scale]

another way is to just provide some useful log statements to users that include WARN or something.

[ok-scale]

@GeneDer thoughts?

[GeneDer]

component_id is mostly auto-generated id, and most of time we just use pid for that process. Feel this would be no-op, but also unnecessary :)

[ok-scale]

Closing wrt #52702