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

[Feature] flyte-cli configuration file #412

Closed
1 of 13 tasks
wild-endeavor opened this issue Jul 14, 2020 · 3 comments
Closed
1 of 13 tasks

[Feature] flyte-cli configuration file #412

wild-endeavor opened this issue Jul 14, 2020 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers stale untriaged This issues has not yet been looked at by the Maintainers

Comments

@wild-endeavor
Copy link
Contributor

wild-endeavor commented Jul 14, 2020

Motivation: Why do you think this is important?
flytekit currently ships with a CLI called flyte-cli that references a config file is assumed to live in ~/.flyte/config We like this pattern as it mirrors kubectl's ~/.kube/config. However, when debugging, or when working with multiple deployments of Flyte (like we have staging and production, in addition to a local deployment), it's often helpful to be able to give it a file to reference instead of always assuming it's at ~/.flyte/config.

[platform]
url=flyte.mycorp.net
auth=True

[credentials]
client_id=abcdef123
redirect_uri=http://localhost:53593/callback
auth_mode=standard
authorization_metadata_key=flyte-authorization
scope=svc

Note, workflow repository configs look similar and have many of the same keys (workflow ones have more)
https://github.com/lyft/flytesnacks/blob/master/cookbook/sandbox.config

Goal: What should the final outcome look like, ideally?
If not specified, flyte-cli should look for a config file at the default location of ~/.flyte/config. If flyte-cli -f /other/file is given, then the file /other/file should be used instead.

Not sure if f is the best argument to use, it may already be taken.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

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

[Optional] Propose: Link/Inline
NA

Additional context
NA

Is this a blocker for you to adopt Flyte
NA

@wild-endeavor wild-endeavor added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers good first issue Good for newcomers labels Jul 14, 2020
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 6, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
* Fix assets path for index.js query

Signed-off-by: Alex Bain (Level 5 US)/Alex Bain <[email protected]>

* chore: remove item folder fof assets

Signed-off-by: Nastya Rusina <[email protected]>

Co-authored-by: Alex Bain (Level 5 US)/Alex Bain <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
@github-actions
Copy link

Hello 👋, This issue has been inactive for over 9 months. To help maintain a clean and focused backlog, we'll be marking this issue as stale and will close the issue if we detect no activity in the next 7 days. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot added the stale label Aug 26, 2023
@github-actions
Copy link

github-actions bot commented Sep 2, 2023

Hello 👋, This issue has been inactive for over 9 months and hasn't received any updates since it was marked as stale. We'll be closing this issue for now, but if you believe this issue is still relevant, please feel free to reopen it. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 2, 2023
@eapolinario eapolinario reopened this Nov 2, 2023
@kumare3
Copy link
Contributor

kumare3 commented Dec 22, 2023

this is not used any more. please use pyflyte or flytectl

@kumare3 kumare3 closed this as completed Dec 22, 2023
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Apr 30, 2024
* Remove flytekit image priming

Signed-off-by: Jeev B <[email protected]>

* Fix goreleaser

Signed-off-by: Jeev B <[email protected]>

---------

Signed-off-by: Jeev B <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Apr 30, 2024
* Remove flytekit image priming

Signed-off-by: Jeev B <[email protected]>

* Fix goreleaser

Signed-off-by: Jeev B <[email protected]>

---------

Signed-off-by: Jeev B <[email protected]>
austin362667 pushed a commit to austin362667/flyte that referenced this issue May 7, 2024
* Remove flytekit image priming

Signed-off-by: Jeev B <[email protected]>

* Fix goreleaser

Signed-off-by: Jeev B <[email protected]>

---------

Signed-off-by: Jeev B <[email protected]>
robert-ulbrich-mercedes-benz pushed a commit to robert-ulbrich-mercedes-benz/flyte that referenced this issue Jul 2, 2024
* Remove flytekit image priming

Signed-off-by: Jeev B <[email protected]>

* Fix goreleaser

Signed-off-by: Jeev B <[email protected]>

---------

Signed-off-by: Jeev B <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers stale untriaged This issues has not yet been looked at by the Maintainers
Projects
None yet
Development

No branches or pull requests

3 participants