Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow startuptime to be reported when it isn't exceeded. #2500

Merged

Conversation

danlapid
Copy link
Collaborator

@danlapid danlapid commented Aug 8, 2024

LimitEnforcer's enterStartup* functions now receive a reference to a OneOf an exception or a duration.
Which can be populated by the elapsed duration if the limit was not exceeded.

@danlapid danlapid requested review from a team as code owners August 8, 2024 23:44
@danlapid danlapid changed the title startuptime is now reported when it isn't exceeded. Allow startuptime to be reported when it isn't exceeded. Aug 8, 2024
@danlapid danlapid force-pushed the dlapid/EW-8620-report-correct-startuptime-from-validator branch 2 times, most recently from 01a05fa to ca661ae Compare August 9, 2024 10:07
@danlapid danlapid requested review from a team as code owners August 9, 2024 10:07
Copy link
Contributor

@dom96 dom96 left a comment

Choose a reason for hiding this comment

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

Should the base of this PR be different? It looks like it contains @anonrig's commit which is main so shouldn't show up here

@danlapid
Copy link
Collaborator Author

danlapid commented Aug 9, 2024

Should the base of this PR be different? It looks like it contains @anonrig's commit which is main so shouldn't show up here

Hmm, you're right, that wasn't here yesterday though.
I'll try to rebase

Update: fixed

@danlapid danlapid force-pushed the dlapid/EW-8620-report-correct-startuptime-from-validator branch 2 times, most recently from 2ea664b to d14701a Compare August 12, 2024 11:48
@danlapid danlapid force-pushed the dlapid/EW-8620-report-correct-startuptime-from-validator branch from d14701a to ecbb6ea Compare August 12, 2024 13:39
@jasnell
Copy link
Member

jasnell commented Aug 12, 2024

Marking with needs-internal-pr to ensure that this is tested internally before merging. If the internal CI has already been done and passed, the label can be removed.

limitenforcer's enterStartup* functions now receive a reference to a
OneOf an exception or a duration. Which can be populated by the elapsed
duration if the limit was not exceeded.
@danlapid danlapid force-pushed the dlapid/EW-8620-report-correct-startuptime-from-validator branch from ecbb6ea to 1ead2e9 Compare August 12, 2024 15:58
@danlapid danlapid merged commit 8a427ad into main Aug 12, 2024
9 of 10 checks passed
@danlapid danlapid deleted the dlapid/EW-8620-report-correct-startuptime-from-validator branch August 12, 2024 16:50
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.

3 participants