Skip to content

Commit

Permalink
update: fixed aws related issue
Browse files Browse the repository at this point in the history
  • Loading branch information
heemankv committed Nov 7, 2024
1 parent 5b1cbc2 commit 103e462
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 8 deletions.
4 changes: 2 additions & 2 deletions .env.test
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ MADARA_ORCHESTRATOR_AWS_S3_BUCKET_NAME=madara-orchestrator-test-bucket
## OTEL ##

MADARA_ORCHESTRATOR_OTEL_SERVICE_NAME=orchestrator
MADARA_ORCHESTRATOR_OTEL_COLLECTOR_ENDPOINT=http://localhost:4317



#### SERVER ####
Expand All @@ -96,7 +96,7 @@ MADARA_ORCHESTRATOR_PORT=3000

MADARA_ORCHESTRATOR_MAX_BLOCK_NO_TO_PROCESS=1000
MADARA_ORCHESTRATOR_MIN_BLOCK_NO_TO_PROCESS=0
MADARA_ORCHESTRATOR_MADARA_RPC_URL=http://localhost:9955
MADARA_ORCHESTRATOR_MADARA_RPC_URL=http://localhost:9944


#### SNOS ####
Expand Down
2 changes: 1 addition & 1 deletion crates/orchestrator/src/cli/instrumentation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ use url::Url;

/// Parameters used to config instrumentation.
#[derive(Debug, Clone, Args)]
#[group(requires_all = ["otel_service_name", "otel_collector_endpoint", "log_level"])]
#[group(requires_all = ["log_level"])]
pub struct InstrumentationCliArgs {
/// The name of the instrumentation service.
#[arg(env = "MADARA_ORCHESTRATOR_OTEL_SERVICE_NAME", long, default_value = "orchestrator")]
Expand Down
7 changes: 2 additions & 5 deletions e2e-tests/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ use rstest::rstest;
use serde::{Deserialize, Serialize};
use serde_json::json;
use starknet::core::types::{Felt, MaybePendingStateUpdate};
use utils::env_utils::get_env_var_or_panic;
use utils::env_utils::{get_env_var_optional, get_env_var_or_panic};
use uuid::Uuid;

extern crate e2e_tests;
Expand Down Expand Up @@ -584,10 +584,7 @@ fn set_env_vars() -> Vec<(String, String)> {
"MADARA_ORCHESTRATOR_OTEL_SERVICE_NAME".to_string(),
get_env_var_or_panic("MADARA_ORCHESTRATOR_OTEL_SERVICE_NAME"),
),
(
"MADARA_ORCHESTRATOR_OTEL_COLLECTOR_ENDPOINT".to_string(),
get_env_var_or_panic("MADARA_ORCHESTRATOR_OTEL_COLLECTOR_ENDPOINT"),
),

// Server
("MADARA_ORCHESTRATOR_HOST".to_string(), get_env_var_or_panic("MADARA_ORCHESTRATOR_HOST")),
("MADARA_ORCHESTRATOR_PORT".to_string(), get_env_var_or_panic("MADARA_ORCHESTRATOR_PORT")),
Expand Down

0 comments on commit 103e462

Please sign in to comment.