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.

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

Relaunch button doesn't pass arguments from the previous run correctly #533

Closed
1 of 20 tasks
datability-io opened this issue Oct 2, 2020 · 1 comment
Closed
1 of 20 tasks
Assignees
Labels
bug Something isn't working ui Admin console user interface
Milestone

Comments

@datability-io
Copy link

Describe the bug
I have seen several times that an empty array was passed in as None. Here is an example execution: https://flyte.lyft.net/console/projects/setl/domains/production/executions/ft7yxneh7l

The argument loading_tbl_full_names should be an empty array instead of None.

image

Expected behavior
The argument loading_tbl_full_names should be an empty array instead of None.

Flyte component

  • Overall
  • Flyte Setup and Installation scripts
  • Flyte Documentation
  • Flyte communication (slack/email etc)
  • FlytePropeller
  • FlyteIDL (Flyte specification language)
  • Flytekit (Python SDK)
  • FlyteAdmin (Control Plane service)
  • FlytePlugins
  • DataCatalog
  • FlyteStdlib (common libraries)
  • FlyteConsole (UI)
  • Other

To Reproduce
Steps to reproduce the behavior:
Click and relaunch button from the provided execution.

Screenshots
See above

Environment
Flyte component

  • Sandbox (local or on one machine)
  • Cloud hosted
    • AWS
    • GCP
    • Azure
  • Baremetal
  • Other

Additional context
Add any other context about the problem here.

@datability-io datability-io added bug Something isn't working untriaged This issues has not yet been looked at by the Maintainers labels Oct 2, 2020
@kumare3 kumare3 added this to the 0.10.0 milestone Oct 15, 2020
@schottra schottra added ui Admin console user interface and removed untriaged This issues has not yet been looked at by the Maintainers labels Nov 16, 2020
@schottra
Copy link
Contributor

Fixed by flyteorg/flyteconsole#121
Available in flyteconsole v0.17.7

eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
chore: fix test

Signed-off-by: Nastya Rusina <nastya@union.ai>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Jul 24, 2023
…rg#533)

* Bump propeller

Signed-off-by: Kevin Su <pingsutw@apache.org>

* Fixed slice bounds out of range

Signed-off-by: Kevin Su <pingsutw@apache.org>

* Fixed slice bounds out of range

Signed-off-by: Kevin Su <pingsutw@apache.org>

* update go.mod

Signed-off-by: Kevin Su <pingsutw@apache.org>

* update go.mod

Signed-off-by: Kevin Su <pingsutw@apache.org>

* update

Signed-off-by: Kevin Su <pingsutw@apache.org>

* Add tests

Signed-off-by: Kevin Su <pingsutw@apache.org>

* lint

Signed-off-by: Kevin Su <pingsutw@apache.org>

---------

Signed-off-by: Kevin Su <pingsutw@apache.org>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
…rg#533)

* Bump propeller

Signed-off-by: Kevin Su <pingsutw@apache.org>

* Fixed slice bounds out of range

Signed-off-by: Kevin Su <pingsutw@apache.org>

* Fixed slice bounds out of range

Signed-off-by: Kevin Su <pingsutw@apache.org>

* update go.mod

Signed-off-by: Kevin Su <pingsutw@apache.org>

* update go.mod

Signed-off-by: Kevin Su <pingsutw@apache.org>

* update

Signed-off-by: Kevin Su <pingsutw@apache.org>

* Add tests

Signed-off-by: Kevin Su <pingsutw@apache.org>

* lint

Signed-off-by: Kevin Su <pingsutw@apache.org>

---------

Signed-off-by: Kevin Su <pingsutw@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui Admin console user interface
Projects
None yet
Development

No branches or pull requests

3 participants