-
Notifications
You must be signed in to change notification settings - Fork 674
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
[Housekeeping] Test Flytectl & Flytekit on Windows #1561
Comments
cc: @evalsocket |
@kuspia, please go ahead! Thank you for picking this up. First, make sure the getting started guide works. Please share with us the screenshots of outputs of all the commands you run. Once you get the Flyte UI up and running, please send us a short video of an example workflow run. You can join our Slack #hacktoberfest2021 channel and ask us there if you have any questions. @evalsocket Is there anything specific we want to test on Windows? |
Hello. Sorry to distrupt or step in. But I wanted to find out too and ran the commands from the Getting Started guide on my Windows (which is my driver computer). It seems to work at a glance. Do I share my findings here as well? |
Hi, @martinlyra! Thanks for running the code. Please do share your findings. :) |
First I'll leave some specifications of my Windows version and installation of Python. Think that can help figuring out the differences if any.
Running
Running as module:
I'll try to do the next two getting-started when I get more time to. May take some while because this is completely new form of software for me to deal with. |
Oh sure! Thank you for doing this. :) Apologies for not mentioning this before: can you use Python 3.8 to test the code? Flyte should be compatible with that specific Python version. Also, if possible, please share with us the command prompt screenshots. cc: @kumare3 |
Sure! But why not hit two birds with one rock? ;) I've installed
Running getting started again, it seems to be no obvious difference between 3.8 and 3.9 when running the first getting-started example
If this is alright, I'll try to do the next ones with 3.8 like this. |
Awesome! You were able to install everything smoothly, right? |
Yes! Besides from the warnings about the modules not being installed to a folder included in my system PATH, which is intended anyways as far this 3.8 installation go. Either way, moving on to So I instead use the Bash shell that was included with the Git installation by default. It has MINGW64, which should have bash and utilities enough to run the commands in 2nd step. But with some modification:
So it installs into The next following commands gives:
...and
The next step is to get the cluster running, so I need to get both Docker and WSL (as required by Docker for Windows) installed and running. I'll be back when I have more news to share! At this point I would like to turn this into a blog, ha. |
Alright. I have good and bad news! Starting with the good. I have managed to go far enough to have the Flyte UI up and working. So I can interact and browse around in the window. It took a little while to get it running though. The sandbox commands work as they seem to would do. Other than the fact the progress logging works oddly on Windows: As it produces a stream of new lines rather than updating the output buffer. Resulting in every update being printed as new lines to the log. Now, the bad news. My troubles start with the
Looking up the documentation for
That might do it, ouch. I am willing to try volunteer to try fix this if possible, it is Hacktoberfest after all! |
@samhita-alla We only need to test getting started first and then we can test other commands. @martinlyra Awesome, @wild-endeavor Can help you with POSIX issue. |
@martinlyra This is uber-cool! We really appreciate the work you've been doing. :) Regarding POSIX, we'll be more than happy if you could contribute. Let's see what's the possible fix! |
I just need to know where I can find the right repository and folder, and maybe some know-how to test out the changes! Would it be in the flytekit repo? |
Hi, @martinlyra! Yes, it has to be in the Flytekit repo, most probably, the pyflyte.py file. Can you join our Slack #hacktoberfest2021 channel? We can talk more about it there. There's already a thread I have started to initiate the discussion. |
You'd have to install Flytekit in development mode to test your changes as described in the contribution guide. You can then run |
Hi, @martinlyra! How's it going? Have you hit any roadblocks? |
Have not met any roadblocks yet! Few errands popped up and I needed to go get them do done first, still got few more to do too. So my progress may be slower for this week. |
As a summary for prosperity because much of the discussion was deliberated in the Slack. So I'll summarize a bit quickly here. The PR to fix the issue with I've successfully registered the I'll continue testing Windows compatibility and functions, though I'd like to tell that I am a full-time student, and I'll be having an exam week. Meantime, the progress will be slower whenever I can help. |
Thank you for your contribution, @martinlyra! You pretty much tested getting started on Windows. That should resolve this issue. However, I'm going to keep this open as we can test other utilities. Take your time in doing so! And, can you fill in this form as part of Hacktoberfest? |
Of course! I am only half way through the getting started guide! Better get things done than leaving them unfinished. I've submitted the form, although more may come before end of the October. |
I could run the getting started code until Executing Workflows on a Flyte Cluster. Running this command:
Lead to this error:
python 3.9, fresh Any thoughts on resolving the issue? |
@rozsasarpi, you'll have to export the following two environment variables:
Make sure you modify the path to point to the correct flytectl config and k3s YAML file. |
@samhita-alla thanks, adding the environment variables solved the problem! (I should have read the text printed to the command window...) I successfully run all the Getting started code ( The
|
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! 🙏 |
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! 🙏 |
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 engage on it to decide if it is still applicable. |
Describe the Issue
We haven't yet thoroughly tested Flytekit and Flytectl on a windows OS. Would love someone with a windows OS to test this out.
What if we do not do this?
Flyte on windows may not be foolproof.
Related component
flytekit, flytectl
The text was updated successfully, but these errors were encountered: