Skip to content

Commit 5d8f163

Browse files
committed
Fix tests
1 parent f31328e commit 5d8f163

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

pkg/capabilities/base_trigger_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ func TestResolveBaseTriggerRetryInterval(t *testing.T) {
2121
t.Run("nil getter uses defaults", func(t *testing.T) {
2222
d, err := ResolveBaseTriggerRetryInterval(ctx, nil, lggr)
2323
require.NoError(t, err)
24-
require.Equal(t, 30*time.Second, d)
24+
require.Zero(t, d, "default BaseTriggerRetransmitEnabled is false, so retry interval is disabled")
2525
})
2626

2727
t.Run("global JSON enables interval", func(t *testing.T) {

pkg/settings/cresettings/defaults.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ VaultOrgIdAsSecretOwnerEnabled = 'false'
77
GatewayHTTPGlobalRate = '500rps:500'
88
GatewayHTTPPerNodeRate = '100rps:100'
99
TriggerRegistrationStatusUpdateTimeout = '0s'
10-
BaseTriggerRetransmitEnabled = 'true'
10+
BaseTriggerRetransmitEnabled = 'false'
1111
BaseTriggerRetryInterval = '30s'
1212
VaultCiphertextSizeLimit = '2kb'
1313
VaultShareSizeLimit = '600b'

pkg/settings/cresettings/settings.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ var Default = Schema{
6161
GatewayHTTPGlobalRate: Rate(rate.Limit(500), 500),
6262
GatewayHTTPPerNodeRate: Rate(rate.Limit(100), 100),
6363
TriggerRegistrationStatusUpdateTimeout: Duration(0 * time.Second),
64-
BaseTriggerRetransmitEnabled: Bool(true),
64+
BaseTriggerRetransmitEnabled: Bool(false),
6565
BaseTriggerRetryInterval: Duration(30 * time.Second),
6666

6767
// DANGER(cedric): Be extremely careful changing these vault limits as they act as a default value

0 commit comments

Comments
 (0)