Skip to content

Increase rack timeout in test environment too#5766

Closed
zachmargolis wants to merge 1 commit intomainfrom
margolis-timeout-test
Closed

Increase rack timeout in test environment too#5766
zachmargolis wants to merge 1 commit intomainfrom
margolis-timeout-test

Conversation

@zachmargolis
Copy link
Contributor

phone_setups_per_ip_track_only_mode: 'false'
piv_cac_verify_token_secret: 3ac13bfa23e22adae321194c083e783faf89469f6f85dcc0802b27475c94b5c3891b5657bd87d0c1ad65de459166440512f2311018db90d57b15d8ab6660748f
poll_rate_for_verify_in_seconds: '1'
rack_timeout_service_timeout_seconds: 9_999_999_999
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm less sure about doing this in test, I'd rather not have a test wait 300 years to fail 😛

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😬 so what should we set it to?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we haven't seen any issues in test, leaving the default seems fine for now?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought I was getting segfaults in test but I guess not! Will close this for now and we can always redo if others have issues

@aduth aduth deleted the margolis-timeout-test branch January 3, 2022 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants