refactor!: unify FrontendOptions and DatanodeOptions by using GrpcOptions

[realtaobo]

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

As mentioned in the comment at #3957 (comment), try to unify FrontendOptions and DatanodeOptions by using GrpcOptions.

What's changed and what's your intention?

• move GrpcOptions to servers/src/grpc
• refactor DatanodeOptions
• wrapping arguments like rpc_addr by Option. e.g. rpc_addr: Option, set them to None by default and print a warning if people set it(refactor!: unify FrontendOptions and DatanodeOptions by using GrpcOptions #4088 (comment)). The final result is as follows：

 *  Executing task: cargo test --package cmd --lib -- datanode::tests::test_deprecated_cli_options --exact --show-output 
...
running 1 test
2024-06-17T15:10:58.675193Z  INFO common_telemetry::logging: logs dir = /tmp/__unittest_logs
2024-06-17T15:10:58.682675Z  WARN cmd::datanode: Use the deprecated attribute `DatanodeOptions.rpc_addr`, please use `grpc.addr` instead.
2024-06-17T15:10:58.682740Z  WARN cmd::datanode: Use the deprecated attribute `DatanodeOptions.rpc_hostname`, please use `grpc.hostname` instead.
test datanode::tests::test_depreacted_cli_options ... ok

Checklist

• I have written the necessary rustdoc comments.
• I have added the necessary unit tests and integration tests.
• This PR requires documentation updates.

[codecov]

Codecov Report

Attention: Patch coverage is 86.36364% with 12 lines in your changes missing coverage. Please review.

Project coverage is 84.85%. Comparing base (0fc18b6) to head (df5e284).
Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4088      +/-   ##
==========================================
- Coverage   84.92%   84.85%   -0.07%     
==========================================
  Files        1023     1020       -3     
  Lines      179298   179296       -2     
==========================================
- Hits       152272   152146     -126     
- Misses      27026    27150     +124     

[zyy17]

Most of the changes are look good to me, except:

1. It's an breaking changes and it's worth for it? Or we should make it compatible?
2. The PR failed the fuzz tests;

[evenyag]

Since many people may already set the old rpc_addr, we might still keep these options but notify users that they are deprecated.

[realtaobo]

Since many people may already set the old rpc_addr, we might still keep these options but notify users that they are deprecated.

Yes, I'm aware of this, but still exist some other questions for fuzz tests within distributed mode. I'll ping again when everything is ready.

[evenyag]

Since many people may already set the old rpc_addr, we might still keep these options but notify users that they are deprecated.

Yes, I'm aware of this, but still exist some other questions for fuzz tests within distributed mode. I'll ping again when everything is ready.

I guess some of the fuzz tests use the old rpc options.

[realtaobo]

Hi, all, please take a look when you have some free time.

1. It's an breaking changes and it's worth for it? Or we should make it compatible?

This PR moves GrpcOptions to servers/src/grpc and refactors the gRPC-related configurations of DatanodeOptions. Additionally, I have tagged rpc_addr, rpc_runtime_size, rpc_hostname, and rpc_max_recv/send_message_size as deprecated fields, setting them to None by default and printing a warning if they are set (The execution result example can be found in the description). Moreover, if the old RPC options are set, we read them first for compatibility.

2. The PR failed the fuzz tests;

It has been resolved. I guess the distributed fuzz tests failed previously because they were not compatible with the greptimedb cluster's configuration used by CI.

[evenyag]

Looks good.

[killme2008]

Great work! Thanks a lot.