Skip to content

Commit

Permalink
issue-1350: increasing default IdleSessionTimeout to 1 hour - 5 minut…
Browse files Browse the repository at this point in the history
…es aren't enough if we deploy a bug which causes filestore-vhost to enter a crash loop or something like that, whereas 1 hour should be enough (#1813)

* issue-1350: increasing default IdleSessionTimeout to 1 hour - 5 minutes aren't enough if we deploy a bug which causes filestore-vhost to enter a crash loop or something like that, whereas 1 hour should be enough

* issue-1350: fixed ut after increasing default IdleSessionTimeout to 1 hour
  • Loading branch information
qkrorlqr authored Aug 19, 2024
1 parent 0bcc822 commit 21d66f8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cloud/filestore/libs/storage/core/config.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ using TAliases = NProto::TStorageConfig::TFilestoreAliases;
xxx(PipeClientMaxRetryTime, TDuration, TDuration::Seconds(4) )\
\
xxx(EstablishSessionTimeout, TDuration, TDuration::Seconds(30) )\
xxx(IdleSessionTimeout, TDuration, TDuration::Minutes(5) )\
xxx(IdleSessionTimeout, TDuration, TDuration::Hours(1) )\
\
xxx(WriteBatchEnabled, bool, false )\
xxx(WriteBatchTimeout, TDuration, TDuration::MilliSeconds(0) )\
Expand Down
2 changes: 1 addition & 1 deletion cloud/filestore/libs/storage/tablet/tablet_ut_counters.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,7 @@ Y_UNIT_TEST_SUITE(TIndexTabletTest_Counters)
});

Tablet->RebootTablet();
Tablet->AdvanceTime(TDuration::Minutes(10));
Tablet->AdvanceTime(TDuration::Hours(1));
Tablet->CleanupSessions();

Tablet->AdvanceTime(TDuration::Seconds(15));
Expand Down

0 comments on commit 21d66f8

Please sign in to comment.