Skip to content

Commit 6f3e160

Browse files
committed
chore: Make telemetry changes consistent with the other operators
1 parent 1a79e82 commit 6f3e160

File tree

3 files changed

+11
-9
lines changed

3 files changed

+11
-9
lines changed

Cargo.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,10 @@ edition = "2021"
1010
repository = "https://github.com/stackabletech/hdfs-operator"
1111

1212
[workspace.dependencies]
13-
stackable-versioned = { git = "https://github.com/stackabletech/operator-rs.git", features = ["k8s"], tag = "stackable-versioned-0.7.1" }
14-
stackable-telemetry = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-telemetry-0.4.0" }
15-
stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.89.1" }
1613
product-config = { git = "https://github.com/stackabletech/product-config.git", tag = "0.7.0" }
14+
stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.89.1" }
15+
stackable-telemetry = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-telemetry-0.4.0" }
16+
stackable-versioned = { git = "https://github.com/stackabletech/operator-rs.git", features = ["k8s"], tag = "stackable-versioned-0.7.1" }
1717

1818
anyhow = "1.0"
1919
built = { version = "0.7", features = ["chrono", "git2"] }

rust/operator-binary/Cargo.toml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ repository.workspace = true
99
publish = false
1010

1111
[dependencies]
12-
stackable-versioned.workspace = true
13-
stackable-telemetry.workspace = true
14-
stackable-operator.workspace = true
1512
product-config.workspace = true
13+
stackable-operator.workspace = true
14+
stackable-telemetry.workspace = true
15+
stackable-versioned.workspace = true
1616

1717
anyhow.workspace = true
1818
clap.workspace = true

rust/operator-binary/src/main.rs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,9 @@ mod built_info {
5050

5151
pub const OPERATOR_NAME: &str = "hdfs.stackable.tech";
5252

53+
// TODO (@NickLarsenNZ): Change the variable to `CONSOLE_LOG`
54+
pub const ENV_VAR_CONSOLE_LOG: &str = "HDFS_OPERATOR_LOG ";
55+
5356
#[derive(clap::Parser)]
5457
#[clap(about, author)]
5558
struct Opts {
@@ -73,8 +76,7 @@ async fn main() -> anyhow::Result<()> {
7376
// TODO (@Techassi): This should be a constant
7477
.service_name("hbase-operator")
7578
.with_console_output((
76-
// TODO (@Techassi): Change to CONSOLE_LOG, create constant
77-
"HDFS_OPERATOR_LOG",
79+
ENV_VAR_CONSOLE_LOG,
7880
LevelFilter::INFO,
7981
!telemetry_arguments.no_console_output,
8082
))
@@ -91,7 +93,7 @@ async fn main() -> anyhow::Result<()> {
9193

9294
Settings::builder()
9395
// TODO (@Techassi): Change to CONSOLE_LOG or FILE_LOG, create constant
94-
.with_environment_variable("HDFS_OPERATOR_LOG")
96+
.with_environment_variable(ENV_VAR_CONSOLE_LOG)
9597
.with_default_level(LevelFilter::INFO)
9698
.file_log_settings_builder(log_directory, "tracing-rs.log")
9799
.with_rotation_period(rotation_period)

0 commit comments

Comments
 (0)