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
KEP-3998: Uptdate JobSuccessPolicy API Name and Validations #4537
base: master
Are you sure you want to change the base?
KEP-3998: Uptdate JobSuccessPolicy API Name and Validations #4537
Conversation
/cc @atiratree |
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: alculquicondor, tenzen-y The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold |
033db39
to
b5d00b7
Compare
b5d00b7
to
6c8d5c5
Compare
/lgtm |
I will add some status validation |
6c8d5c5
to
a22a618
Compare
@alculquicondor @mimowo @soltysh @atiratree I updated the design based on actual implementations. |
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.
Thanks!
/lgtm
a22a618
to
8f4c50a
Compare
@atiratree I addressed all comments. PTAL, thanks. |
Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com>
Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com>
8f4c50a
to
026cf09
Compare
/lgtm |
@soltysh Could you review this? Thanks. |
@@ -5,7 +5,7 @@ authors: | |||
owning-sig: sig-apps | |||
participating-sigs: | |||
- sig-apps | |||
status: implementable | |||
status: implemented |
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.
AIUI we use implemented
once the feature is generally available.
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.
But this is still alpha
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.
Thank you for checking this!
I didn't know that. I'm updating it!
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 dropped this commit.
Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com>
026cf09
to
b2f1695
Compare
New changes are detected. LGTM label has been removed. |
@soltysh Hi, Maciej, could you please check this PR? Thanks in advance. |
status.conditions
.rules[*].succeededCount
.