You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When passing -d development for example, sometimes the value gets applied to the --destination-dir instead of / as well as the --domain. To avoid this, short flags should be unique.
Expected behavior
Expected -d to be applied only to the --domain when running pyflyte and not the --destination-dir.
Additional context to reproduce
Run pyflyte register -p flytesnacks -d development my_workflow.py
Launch workflow via Flyte console (or other)
Note that in the Task Details section, the value for --dest-dir is no longer the default /root but development
Workflow crashes with the Destination path is required to download distribution and it should be a directory error (fast_registration.download_distribution())
Screenshots
No response
Are you sure this issue hasn't been raised already?
Yes
Have you read the Code of Conduct?
Yes
The text was updated successfully, but these errors were encountered:
mcloney-ddm
changed the title
[BUG] -d short flag allowed in pyflyte for both --domain *and* --destination-dir
[BUG] -d short flag allowed in pyflyte for both --domain and --destination-dir
Dec 21, 2022
Describe the bug
When passing
-d development
for example, sometimes the value gets applied to the--destination-dir
instead of / as well as the--domain
. To avoid this, short flags should be unique.Expected behavior
Expected
-d
to be applied only to the--domain
when runningpyflyte
and not the--destination-dir
.Additional context to reproduce
pyflyte register -p flytesnacks -d development my_workflow.py
--dest-dir
is no longer the default/root
butdevelopment
Destination path is required to download distribution and it should be a directory
error (fast_registration.download_distribution()
)Screenshots
No response
Are you sure this issue hasn't been raised already?
Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: