-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[bug] condition sample stuck in running state with argo v3.1.0 #5937
Comments
|
workflow controller logs related to this pipeline run:
|
After some iterations, the minimal reproducing example is like: import kfp
from kfp import compiler
from kfp import components
from kfp import dsl
flip_coin_op = components.load_component_from_text(
'''
name: flip
inputs:
- name: force_flip_result
type: String
default: 'heads'
outputs:
- {name: flip_result, type: String}
implementation:
container:
image: alpine
command: [sh, -c]
args:
- |
mkdir -p "$(dirname $1)"
echo $0 > $1
- {inputValue: force_flip_result}
- {outputPath: flip_result}
'''
)
print_op = components.load_component_from_text(
'''
name: print
inputs:
- {name: text, type: String}
implementation:
container:
image: alpine
command: [sh, -c]
args:
- 'echo $0'
- {inputValue: text}
'''
)
# stuck
@dsl.pipeline(name='single-condition-pipeline')
def my_pipeline_4():
flip1 = flip_coin_op()
print_op(flip1.output)
with dsl.Condition(flip1.output == 'heads'):
print_op(flip1.output)
if __name__ == '__main__':
pipeline = my_pipeline_4
compiler.Compiler().compile(pipeline, package_path='condition.yaml')
# kfp.Client().create_run_from_pipeline_func(
# pipeline,
# arguments={'force_flip_result': 'heads'},
# # mode=kfp.dsl.PipelineExecutionMode.V2_COMPATIBLE
# ) |
Created upstream issue in argo after reproducing with a native argo workflow: |
I verified the bug has been fixed in argo v3.1.1 |
What steps did you take
What happened:
What did you expect to happen:
the sample should finish as normal
Environment:
Anything else you would like to add:
Here's status of the argo workflow, it's also stuck in running. So this is not a problem with persistence agent.
Labels
/area sdk
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.
The text was updated successfully, but these errors were encountered: