Skip to content

Commit d40d97d

Browse files
committed
test(nat): Add debug logs for vpc-routing, flow-lookup in NAT test
Additional logs turned out to be helpful to understand what was going on during the test. Let's keep them on in order to obtain more information if this test ever fails again. Given that we need to remove the "#[traced_test]" annotation, we re-add logs for target "stateful-nat" manually. Signed-off-by: Quentin Monnet <[email protected]>
1 parent 1084147 commit d40d97d

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

nat/Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ tracing = { workspace = true }
3131
# internal
3232
lpm = { workspace = true, features = ["testing"] }
3333
net = { workspace = true, features = ["bolero"] }
34+
tracectl = { workspace = true }
3435

3536
# external
3637
bolero = { workspace = true, default-features = false, features = ["alloc"] }

nat/src/stateful/test.rs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ mod tests {
4343
use std::net::{IpAddr, Ipv4Addr};
4444
use std::str::FromStr;
4545
use std::time::Duration;
46+
use tracectl::get_trace_ctl;
4647
use tracing_test::traced_test;
4748

4849
const FIVE_MINUTES: Duration = Duration::from_secs(5 * 60);
@@ -1229,9 +1230,12 @@ mod tests {
12291230
}
12301231

12311232
#[test]
1232-
#[traced_test]
12331233
#[allow(clippy::too_many_lines)]
12341234
fn test_full_config_unidirectional_nat_overlapping_destination() {
1235+
get_trace_ctl()
1236+
.setup_from_string("vpc-routing=debug,flow-lookup=debug,stateful-nat=debug")
1237+
.unwrap();
1238+
12351239
let mut config =
12361240
build_sample_config(build_overlay_3vpcs_unidirectional_nat_overlapping_addr());
12371241
config.validate().unwrap();

0 commit comments

Comments
 (0)