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] Clarify what Resource is used when doing FlyteFile/Directory.download() or current_context().working_dir #3516

Open
2 tasks done
Tracked by #4064 ...
cosmicBboy opened this issue Mar 22, 2023 · 11 comments
Assignees
Labels
documentation Improvements or additions to documentation hacktoberfest stale
Milestone

Comments

@cosmicBboy
Copy link
Contributor

Description

We need to mention in flytekit and flytesnacks that the Resource(ephemeral_storage=...) request is relevant when the user writes data to the task pod's disk.

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

  • Yes

Have you read the Code of Conduct?

  • Yes
@cosmicBboy cosmicBboy added documentation Improvements or additions to documentation untriaged This issues has not yet been looked at by the Maintainers labels Mar 22, 2023
@cosmicBboy cosmicBboy added this to the 1.6.0 milestone Mar 22, 2023
@cosmicBboy cosmicBboy removed the untriaged This issues has not yet been looked at by the Maintainers label Mar 22, 2023
@wild-endeavor
Copy link
Contributor

Thanks for making the ticket. Yes we also need to mention that the storage field in Resources doesn't do anything anymore. Actually it never did, iirc, we added it before K8s supported it and when support was added it came in the form of ephemeral-storage. but we are keeping it so as to not break people. I think we should add a deprecation message there as well unless we think it'll again be used one day?

@Kota-Karthik
Copy link

@samhita-alla @cosmicBboy
I am interested in doing this issue
So kindly assign me this issue
Thank You : )

@dado5688
Copy link

Hello, @cosmicBboy @wild-endeavor
I am interested in this issue, but I found that when I modify the flytekit/flytekit/types/directory/types.py it doesn't automatically modify the corresponding file via make html
flytekit/docs/build/html/_modules/flytekit/types/directory/types.html
and
flytekit/docs/build/html/generated/flytekit.types.directory.FlyteDirectory.html
I was wondering if there are some config file about make html that I can check if something is wrong.

@samhita-alla
Copy link
Contributor

samhita-alla commented Oct 16, 2023

@dado5688, are the pages not updating? Running make html should incorporate all the changes you made into the docs. Can you share the commands you ran to build docs locally?

@dado5688
Copy link

dado5688 commented Oct 16, 2023

Hi @samhita-alla , thanks for the response
I'm run the following steps
run make setup make doc-requirements.txt pip install -r doc-requirements.txt in the root directory in order
then run the make html in the docs directory after modify code
but it's only not update about this file's corresponding html file
since I also modify other file like this one and it's update to corresponding html file successfully.

but I just wondering that the description about paramters in here is also not show in here

BTW there have some error when I run make html, I don't know if this will be related.
https://drive.google.com/file/d/1UhbLY-wpSg_CbewwawRCzC-EUTOjBD5T/view?usp=sharing

@samhita-alla
Copy link
Contributor

The warnings can be ignored but the other errors need to be fixed. Not sure why remote launchplans are being gathered while building the documentation.

@samhita-alla
Copy link
Contributor

image ^^ This is the output I'm seeing in the terminal.

@dado5688
Copy link

got it ! thank you!

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 engage on it to decide if it is still applicable.
Thank you for your contribution and understanding! 🙏

@dec0dedd
Copy link

dec0dedd commented Oct 5, 2024

Hey, I'd like to work on this issue. Could it get assigned to me?

@davidmirror-ops
Copy link
Contributor

@dec0dedd sure! We merged a major change to the docs structure yesterday, so please ensure your fork is in sync with the main branch as many sections moved to different location. Thank you!

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 hacktoberfest stale
Projects
None yet
Development

No branches or pull requests

7 participants