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

[Docs] What is the distinction between flytectl sandbox and flytectl demo #2503

Closed
SmritiSatyanV opened this issue May 17, 2022 · 11 comments
Closed
Labels
documentation Improvements or additions to documentation

Comments

@SmritiSatyanV
Copy link
Contributor

SmritiSatyanV commented May 17, 2022

Description

As seen https://docs.flyte.org/projects/flytectl/en/latest/gen/flytectl.html#id1, there are two sandbox nouns. What is the difference between flytectl sandbox and flytectl demo?
According to the discussion: demo is for beginners, and sandbox is for advanced users who wish to update certain attributes, perform end to end testing and gain better control over it.

Goal: We need to provide clear distinction between flytectl sandbox and flytectl demo , or delete one of them.

Marc: We need a clarification of the differences between demo and sandbox to help the user choose the best

What are each best for? When should the user choose one or the other?
What is included in each one? What is missing?
Proposed places to add this content:

https://docs.flyte.org/projects/cookbook/en/latest/userguide_setup.html#create-a-local-demo-flyte-cluster

https://docs.flyte.org/projects/flytectl/en/latest/sandbox.html

https://docs.flyte.org/projects/flytectl/en/latest/demo.html

See also

https://flyte-org.slack.com/archives/CP2HDHKE1/p1655871238925919

@SmritiSatyanV SmritiSatyanV added documentation Improvements or additions to documentation untriaged This issues has not yet been looked at by the Maintainers labels May 17, 2022
@flixr
Copy link
Contributor

flixr commented Jun 18, 2022

Yes, please clarify this... What is different between them? And when should I choose which?

@SmritiSatyanV
Copy link
Contributor Author

SmritiSatyanV commented Jun 20, 2022

Hi @flixr

flytectl sandbox

  • Uses all Flyte released images in addition to postgres and minio
  • May glitch incase of bad network
  • Uses port 30081
  • Up-to-date with production environment setup

flytectl demo

  • Uses single binary to bundle all the Flyte components thereby reducing the time to spin up the cluster because it has lesser dependencies to download
  • Other dependencies include postgres and minio
  • Uses port 30080

If you are looking to play around or you are a beginner, start with demo, else use sandbox.
Hope this answers your question.
We have a comprehensive document on all the differences between sandbox and demo coming soon.

@rupurt
Copy link

rupurt commented May 31, 2023

Any chance there are updates on this? I can only start the demo. I can't start the sandbox as it doesn't have the k3s file that is symlinked /var/lib/flyte/config/kubeconfig

@tekumara
Copy link

The advantage of the demo is that it comes with a docker registry to store images that can be run on the cluster.

@kumare3
Copy link
Contributor

kumare3 commented May 31, 2023

I think we want to make both the same. Cc @jeevb / @eapolinario

@jeevb
Copy link
Contributor

jeevb commented May 31, 2023

Generally, we should all be using flytectl demo now. Personally, I can't think of a reason to use flytectl sandbox. Do folks frequently experiment with individually bumped components in the sandbox?

@eapolinario: Please confirm, but I believe that the near-term goal is to drop the current implementation of flytectl sandbox completely, move flytectl demo to flytectl sandbox, and just add demo as an alias of sandbox for backward compatibility.

@rupurt
Copy link

rupurt commented May 31, 2023

Is there a way to remove the default project from the demo?

@jeevb
Copy link
Contributor

jeevb commented May 31, 2023

Is there a way to remove the default project from the demo?

You should be able to override the projects by specifying this in your own values.yaml file for flyte-binary.

However, this is frozen in the public sandbox, by default. You will need to override with the following configuration in ~/.flyte/sandbox/config.yaml:

flyte:
  admin:
    seedProjects:
      - myproject

The flytesnacks project might still persist. If so, you will need to wipe your sandbox state with flytectl demo teardown -v then restart it again: flytectl demo start

Screenshot 2023-05-31 at 1 13 14 PM

@rupurt rupurt mentioned this issue Jun 2, 2023
2 tasks
@rupurt
Copy link

rupurt commented Jun 2, 2023

@jeevb sweet. That worked. Thank you 🙏

Is it possible to make that file come from somewhere local to the repository?

@jeevb
Copy link
Contributor

jeevb commented Jun 2, 2023

@jeevb sweet. That worked. Thank you 🙏

Is it possible to make that file come from somewhere local to the repository?

Not atm, afaik. But we should make the config directory configurable for sure.

@pingsutw pingsutw removed the untriaged This issues has not yet been looked at by the Maintainers label Dec 22, 2023
@kumare3
Copy link
Contributor

kumare3 commented Dec 22, 2023

there is no difference any more

@kumare3 kumare3 closed this as completed Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

7 participants