-
Notifications
You must be signed in to change notification settings - Fork 66
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
feat: Supports marking a job as failed after specifying a few retries… #362
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #362 +/- ##
==========================================
- Coverage 79.31% 77.05% -2.27%
==========================================
Files 23 24 +1
Lines 1750 2022 +272
==========================================
+ Hits 1388 1558 +170
- Misses 278 371 +93
- Partials 84 93 +9
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feature is generally LGTM. We can consider make it a configuration level properties.
chart/values.yaml
Outdated
@@ -12,6 +12,7 @@ controllerNamespace: "" | |||
|
|||
# "{\"nat\": \"true\"}" | |||
jobNodeSelector: "" | |||
jobBackoffLimit: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A default value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The default is the maximum value of int, compatible with the original default value
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about "" instead of just leaving it empty.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with your point of view.
Also please sign the DCO by |
…ubevela#361 Signed-off-by: 吴就业 <wujiuye@lizhi.fm>
b531b79
to
bb9fa1c
Compare
Signed-off-by: 吴就业 <wujiuye@lizhi.fm>
… #361