Skip to content
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.

Sign up for GitHub

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

Jump to bottom

HTTP 400 returned when attempting to retrieve data for NodeExecution child of a Dynamic Task #51

Closed
georgesnelling opened this issue Nov 19, 2019 · 1 comment
Closed

HTTP 400 returned when attempting to retrieve data for NodeExecution child of a Dynamic Task #51

georgesnelling opened this issue Nov 19, 2019 · 1 comment
Assignees
@schottra
Labels
bug Something isn't working pri2 ui Admin console user interface
Milestone
0.4.0

Comments

@georgesnelling
Copy link
Contributor

Update:

This is a UI bug. We should not attempt to retrieve inputs if no inputsUri is set, and should not attempt to retrieve outputs if closure.outputsUri is unset. 

Direct child

[https://flyte.lyft.net/api/v1/data/node_executions/flytekit/production/y9n8xi9amd/task1-b0e1be7f74-h-task-sqb5710215b84d56d6770b72f5e3cd4f797910c6e6-0-0]

Grandchild (nested subtask)

[https://flyte.lyft.net/api/v1/data/node_executions/flytekit/production/y9n8xi9amd/task1-b0e1be7f74-h-task-sqb5710215b84d56d6770b72f5e3cd4f797910c6e6-0-0-78d085b30a--sub-taskb5710215b84d56d6770b72f5e3cd4f797910c6e6-0-0]

The above URLs should both return NodeExecution data for the ids provided, but instead they return an error "invalid URI".

 
@georgesnelling georgesnelling added bug Something isn't working pri2 Trust labels Nov 19, 2019
@schottra schottra added the ui Admin console user interface label Dec 11, 2019
@kumare3
Copy link
Contributor

kumare3 commented Jun 10, 2020

This has been fixed

@kumare3 kumare3 added this to the 0.4.0 milestone Jun 10, 2020
@kumare3 kumare3 closed this as completed Jun 10, 2020
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 6, 2022
@wild-endeavor
Node executor abort to call finalize even on error (flyteorg#51)
66b40c7 
In cases where the abort call fails, we should still call finalize as this is the intended behavior of the finalize construct.
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 6, 2022
@katrogan
Implement get and delete for attributes (flyteorg#51)
1f516ba
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 6, 2022
@migueltol22
Update FlyteIDL Version (flyteorg#51)
5b6ba17 
* sync with release

* upd to 0.17.22
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
@honnix @EngHabu
Fix go module change (flyteorg#51)
c0c53d1 
workaround golang/go#30515
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
@EngHabu
[AWS Array] Use flytek8s logic to compute resource requirements (flyt…
e45fc46 
…eorg#51)

* [AWS Array] Use flytek8s logic to compute resource requirements

* Fix test

* fix unit tests
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 9, 2023
@wild-endeavor
Node executor abort to call finalize even on error (flyteorg#51)
1933683 
In cases where the abort call fails, we should still call finalize as this is the intended behavior of the finalize construct.
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
@honnix @EngHabu
Fix go module change (flyteorg#51)
4e01a7f 
workaround golang/go#30515
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
@katrogan
Implement get and delete for attributes (flyteorg#51)
acf091e
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
@EngHabu
[AWS Array] Use flytek8s logic to compute resource requirements (flyt…
9c98dd5 
…eorg#51)

* [AWS Array] Use flytek8s logic to compute resource requirements

* Fix test

* fix unit tests
eapolinario pushed a commit that referenced this issue Sep 8, 2023
@migueltol22
Update FlyteIDL Version (#51)
411737c 
* sync with release

* upd to 0.17.22
eapolinario pushed a commit that referenced this issue Sep 12, 2023
@yindia
Added end2end test agaist latest 2 release (#51)
2b5fb6f 
* Added end2end test

Signed-off-by: Yuvraj <[email protected]>
eapolinario added a commit that referenced this issue Sep 12, 2023
@eapolinario @yindia
Revert 51 (#55)
5a528ef 
* Revert "Added end2end test agaist latest 2 release (#51)"

This reverts commit 2b5fb6f.

Signed-off-by: Eduardo Apolinario <[email protected]>

* make end2end_execute

Signed-off-by: Eduardo Apolinario <[email protected]>

* small fix

Signed-off-by: Yuvraj <[email protected]>

Co-authored-by: Eduardo Apolinario <[email protected]>
Co-authored-by: Yuvraj <[email protected]>
eapolinario pushed a commit that referenced this issue Sep 13, 2023
@migueltol22 @eapolinario
Update FlyteIDL Version (#51)
73334e0 
* sync with release

* upd to 0.17.22

Signed-off-by: Eduardo Apolinario <[email protected]>
pvditt pushed a commit that referenced this issue Dec 29, 2023
@migueltol22
Update FlyteIDL Version (#51)
55b86bb 
* sync with release

* upd to 0.17.22
pmahindrakar-oss pushed a commit that referenced this issue Feb 5, 2024
@katrogan
Rewrite GetExecutionData path additional bindings for org (#4816) (#51)
a866023 
Signed-off-by: Katrina Rogan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@schottra schottra

Labels
bug Something isn't working pri2 ui Admin console user interface
Projects
None yet
Milestone
0.4.0
Development

No branches or pull requests
3 participants
@georgesnelling @schottra @kumare3