Skip to content

Commit

Permalink
Add TSAN timeout scale to both timers
Browse files Browse the repository at this point in the history
Signed-off-by: Yan Avlasov <yavlasov@google.com>
  • Loading branch information
yanavlasov committed Jun 29, 2023
1 parent c4d9186 commit da88ad4
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/integration/http2_flood_integration_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -956,13 +956,13 @@ TEST_P(Http2FloodMitigationTest, RstStreamOnStreamIdleTimeoutAfterResponseHeader
// Verify detection of overflowing outbound frame queue with the PING frames sent by the keep alive
// timer. The test verifies protocol constraint violation handling in the
// Http2::ConnectionImpl::sendKeepalive() method.
TEST_P(Http2FloodMitigationTest, KeepAliveTimeeTriggersFloodProtection) {
TEST_P(Http2FloodMitigationTest, KeepAliveTimerTriggersFloodProtection) {
config_helper_.addConfigModifier(
[](envoy::extensions::filters::network::http_connection_manager::v3::HttpConnectionManager&
hcm) {
auto* keep_alive = hcm.mutable_http2_protocol_options()->mutable_connection_keepalive();
keep_alive->mutable_interval()->set_nanos(500 * 1000 * 1000);
keep_alive->mutable_timeout()->set_seconds(1 * TSAN_TIMEOUT_FACTOR);
keep_alive->mutable_interval()->set_seconds(1 * TSAN_TIMEOUT_FACTOR);
keep_alive->mutable_timeout()->set_seconds(2 * TSAN_TIMEOUT_FACTOR);
});

prefillOutboundDownstreamQueue(AllFrameFloodLimit - 1);
Expand Down

0 comments on commit da88ad4

Please sign in to comment.