Skip to content

feat: add validation for values of step rerun fields - #350

Merged
SashaLsnko merged 1 commit into
mainfrom
add-more-validation-to-step-reruns
Aug 26, 2025
Merged

feat: add validation for values of step rerun fields#350
SashaLsnko merged 1 commit into
mainfrom
add-more-validation-to-step-reruns

Conversation

@SashaLsnko

@SashaLsnko SashaLsnko commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

Jira

Description

Add validation for step rerun fields:

  • max_auto_reruns - value is between 1 and 5
  • auto_rerun_delay - value should be in a duration format, not longer than 10 minutes, and should conform to (10|[1-9])m|([1-9][0-9]*)s regex (be in the form of either minutes or seconds)

How to validate

unit tests

@SashaLsnko
SashaLsnko requested a review from a team August 25, 2025 18:42
@SashaLsnko
SashaLsnko force-pushed the add-more-validation-to-step-reruns branch from 50bdb29 to 30ba354 Compare August 26, 2025 17:36
@SashaLsnko

Copy link
Copy Markdown
Contributor Author

I updated the code to only allow delay value as minutes OR seconds

@SashaLsnko
SashaLsnko force-pushed the add-more-validation-to-step-reruns branch from 30ba354 to d656900 Compare August 26, 2025 18:43
@SashaLsnko
SashaLsnko merged commit d7d2f7e into main Aug 26, 2025
6 checks passed
@SashaLsnko
SashaLsnko deleted the add-more-validation-to-step-reruns branch August 26, 2025 18:58
This was referenced May 27, 2026
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