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
docs: Document node field selector. Closes #2860 #2882
Conversation
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 @mark9white! Just some very minor changes
|
||
## Introduction | ||
|
||
If triggering an external job (eg an Amazon EMR job) from Argo that does not run to completion in a container, there are two options: |
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 doc is great
@@ -43,7 +43,7 @@ message WorkflowResubmitRequest { | |||
message WorkflowRetryRequest { | |||
string name = 1; | |||
string namespace = 2; | |||
bool restartSuccesful = 3; |
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.
Is this a breaking change? Has this gone GA yet?
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.
It is breaking, but it's part of v2.8 and hasn't gone GA yet.
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.
👍
docs/node-field-selector.md
Outdated
@@ -0,0 +1,42 @@ | |||
# Node Field Selectors | |||
|
|||
![GA](assets/ga.svg) |
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 think this should be alpha
or beta
.
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'd say 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.
@jessesuen could you please rubber-stamp the proto changes? This needs to be back-ported for v2.8.0-rc4
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.
Please change the tag for Node Field Selector to Alpha
, otherwise it LGTM
@simster7 done |
c5dc0e4
to
6864828
Compare
Please run |
@alexec lint issue should be fixed |
Co-Authored-By: Simon Behar <simbeh7@gmail.com>
Co-Authored-By: Simon Behar <simbeh7@gmail.com>
Co-Authored-By: Simon Behar <simbeh7@gmail.com>
Co-Authored-By: Simon Behar <simbeh7@gmail.com>
Co-Authored-By: Simon Behar <simbeh7@gmail.com>
f06d6fc
to
a342f36
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.