Closed discord9 closed 1 day ago
The changes across multiple files primarily involve removing aliases from the metasrv_addrs
field in the StartCommand
struct for both the datanode
and frontend
components. Additionally, the command line argument for passing server addresses in the Env
implementation tests has been updated from --metasrv-addr
to --metasrv-addrs
.
Files | Change Summary |
---|---|
src/cmd/src/datanode.rs |
Removed aliases = ["metasrv-addr"] from the metasrv_addrs field's attributes in the StartCommand struct. |
src/cmd/src/frontend.rs |
Removed aliases = ["metasrv-addr"] from the metasrv_addrs field's attributes in the StartCommand struct. |
tests/runner/src/env.rs |
Changed --metasrv-addr to --metasrv-addrs in argument construction within Env implementation tests. |
⚙️🌐
Aliases removed, gone from sight,
metasrv_addrs
, now just right. Commands refined, so sleek and neat, Addressing servers, a clearer feat.🐇✨
[!TIP]
AI model upgrade
## `gpt-4o` model for reviews and chat is now live OpenAI claims that this model is better at understanding and generating code than the previous models. Please join our [Discord Community](https://discord.com/invite/GsXnASn26c) to provide any feedback or to report any issues.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.69%. Comparing base (
c4db9e8
) to head (632a801
).
The CI still can't fetch the latest version operator(v0.1.0-alpha.26). cc @daviderli614 @zyy17
Log:
Release "greptimedb-operator" does not exist. Installing it now.
NAME: greptimedb-operator
LAST DEPLOYED: Wed Jul 3 04:01:17 2024
NAMESPACE: greptimedb-admin
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
***********************************************************************
Welcome to use greptimedb-operator
Chart version: 0.1.16
GreptimeDB Operator version: 0.1.0-alpha.25
***********************************************************************
https://github.com/GreptimeTeam/helm-charts/blob/cf2ab450b2cc81aec9682b616944562cd7f78c25/charts/greptimedb-operator/Chart.yaml#L5 Seems we also need to change this?
https://github.com/GreptimeTeam/helm-charts/blob/cf2ab450b2cc81aec9682b616944562cd7f78c25/charts/greptimedb-operator/Chart.yaml#L5 Seems we also need to change this?
Let me fix it.
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
https://github.com/GreptimeTeam/greptimedb-operator/pull/156 https://github.com/GreptimeTeam/docs/pull/1030 cherry-pick from https://github.com/GreptimeTeam/greptimedb/pull/4226
What's changed and what's your intention?
remove alias
metasrv-addr
formetasrv-addrs
Checklist
Summary by CodeRabbit
--metasrv-addr
to--metasrv-addrs
for consistency.