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

Launch form fails without error when using unsupported required inputs #291

Closed
1 of 13 tasks
schottra opened this issue Apr 23, 2020 · 1 comment
Closed
1 of 13 tasks
Assignees
Labels
bug Something isn't working ui Admin console user interface
Milestone

Comments

@schottra
Copy link
Contributor

schottra commented Apr 23, 2020

Describe the bug
When using an input type that is unsupported in Flyte Console (such as schema), is required by the workflow, and also has no default value, clicking on Launch seems to have no effect and provides no feedback to the user.

Expected behavior
An error should be displayed about an input being required, or the user should otherwise be notified that it's not possible to launch this workflow because of the required/unsupported value.

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:

  1. Browse to a workflow which uses an input type that is not supported by Flyte Console, such as Schema or Binary
  2. Click Launch Workflow
  3. Fill in any other required, supported values
  4. Click "Launch"
@schottra schottra added bug Something isn't working ui Admin console user interface untriaged This issues has not yet been looked at by the Maintainers labels Apr 23, 2020
@schottra schottra self-assigned this Apr 23, 2020
@schottra schottra changed the title [BUG] Launch form fails without error when using unsupported required inputs Launch form fails without error when using unsupported required inputs Jul 6, 2020
@schottra schottra assigned schottra and unassigned schottra Jul 6, 2020
@schottra schottra added this to the 0.7.0 milestone Jul 7, 2020
@schottra schottra removed the untriaged This issues has not yet been looked at by the Maintainers label Jul 15, 2020
@schottra
Copy link
Contributor Author

Fixed by flyteorg/flyteconsole#84. Available in flyteconsole v0.7.4

eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
…imit exceeded (flyteorg#291)

* Turn PhaseRetryableFailure into PhaseRetryLimitExceededFailure

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

* update

Signed-off-by: Kevin Su <[email protected]>

* update test

Signed-off-by: Kevin Su <[email protected]>

* lint

Signed-off-by: Kevin Su <[email protected]>

* update

Signed-off-by: Kevin Su <[email protected]>

* update tests

Signed-off-by: Kevin Su <[email protected]>

* lint

Signed-off-by: Kevin Su <[email protected]>

* wip

Signed-off-by: Kevin Su <[email protected]>

* udpate

Signed-off-by: Kevin Su <[email protected]>

* address comment

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

* fix tests

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

Signed-off-by: Kevin Su <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
flyteorg#291)

* chore: allow times timeline update, while execution in progress
* chore: add dynamic timeleine scale
* chore: update eslint and coverage settings
* chore: fix scroll
* fix: ensure that leveling is part of node distinguish
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
…imit exceeded (flyteorg#291)

* Turn PhaseRetryableFailure into PhaseRetryLimitExceededFailure

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

* update

Signed-off-by: Kevin Su <[email protected]>

* update test

Signed-off-by: Kevin Su <[email protected]>

* lint

Signed-off-by: Kevin Su <[email protected]>

* update

Signed-off-by: Kevin Su <[email protected]>

* update tests

Signed-off-by: Kevin Su <[email protected]>

* lint

Signed-off-by: Kevin Su <[email protected]>

* wip

Signed-off-by: Kevin Su <[email protected]>

* udpate

Signed-off-by: Kevin Su <[email protected]>

* address comment

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

* fix tests

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

Signed-off-by: Kevin Su <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Apr 30, 2024
austin362667 pushed a commit to austin362667/flyte that referenced this issue May 7, 2024
robert-ulbrich-mercedes-benz pushed a commit to robert-ulbrich-mercedes-benz/flyte that referenced this issue Jul 2, 2024
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

1 participant