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

[Core feature] Remove Storage config requirement for fast registration #2263

Closed
2 tasks done
EngHabu opened this issue Mar 16, 2022 · 0 comments · Fixed by flyteorg/flytectl#296 or #2318
Closed
2 tasks done
Assignees
Labels
enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers

Comments

@EngHabu
Copy link
Contributor

EngHabu commented Mar 16, 2022

Motivation: Why do you think this is important?

Fast registration is one of the more convenient ways to iterate over code. Requiring local access to remote storage (s3, gs... etc.) is quite a barrier to use though.

Goal: What should the final outcome look like, ideally?

flytectl register shouldn't require storage config
pyflyte/flyteremote register shouldn't require storage config

Describe alternatives you've considered

Exposing a service to mint signed urls but the UX is still ugly (requires copy pasting... etc.)

Propose: Link/Inline OR Additional context

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes
@EngHabu EngHabu added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels Mar 16, 2022
@EngHabu EngHabu added this to the 1.0.0 - Phoenix! milestone Mar 16, 2022
@EngHabu EngHabu self-assigned this Mar 16, 2022
@EngHabu EngHabu reopened this Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers
Projects
None yet
1 participant