Skip to content

Commit

Permalink
[Docs] Added docs on how to handle a large response when using FlyteR…
Browse files Browse the repository at this point in the history
…emote.sync (#2932)

* Update troubleshoot.rst

Added the troubleshoot for Flyteremote.sync problem.

Changes made from L115

Signed-off-by: LN <[email protected]>

* Update troubleshoot.rst

Changed the title of "FlyteRemote.sync" error

Signed-off-by: LN <[email protected]>

* Update rsts/community/troubleshoot.rst

Co-authored-by: Samhita Alla <[email protected]>
Signed-off-by: LN <[email protected]>
Signed-off-by: LN <[email protected]>

* Update rsts/community/troubleshoot.rst

Description update

Co-authored-by: Samhita Alla <[email protected]>
Signed-off-by: LN <[email protected]>
Signed-off-by: LN <[email protected]>

* Update troubleshoot.rst

Updated the description according to the suggestions

Signed-off-by: LN <[email protected]>
Signed-off-by: LN <[email protected]>

* Update troubleshoot.rst

Added the kubectl command to edit the flyte-admin-base-config map

Signed-off-by: Ln11211 [email protected]

Signed-off-by: LN <[email protected]>

* Update rsts/community/troubleshoot.rst

Co-authored-by: Samhita Alla <[email protected]>
Signed-off-by: LN <[email protected]>

* Update troubleshoot.rst

Removed repeating message lines

Signed-off-by: LN <[email protected]>

* Fix broken link

Fixed broken link in line 169

Signed-off-by: LN <[email protected]>

* Update data_management.rst

Fix link in L 169

Signed-off-by: LN <[email protected]>

* Update data_management.rst

Signed-off-by: LN <[email protected]>

* Update data_management.rst

Signed-off-by: LN <[email protected]>

* Update data_management.rst

Signed-off-by: LN <[email protected]>

* Update rsts/concepts/data_management.rst

Signed-off-by: Samhita Alla <[email protected]>

Signed-off-by: LN <[email protected]>
Signed-off-by: LN <[email protected]>
Signed-off-by: Samhita Alla <[email protected]>
Co-authored-by: Samhita Alla <[email protected]>
  • Loading branch information
Ln11211 and samhita-alla authored Oct 11, 2022
1 parent 4aaae1f commit fd464d6
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
4 changes: 4 additions & 0 deletions rsts/community/troubleshoot.rst
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,11 @@ Troubles With Inconsistent Names for Pods and Downstream Resources
- Flyte uses the format ``executionid-node-id-attempt`` from the node to assign a name to a Kubernetes pod or downstream resource.
- But if this is an invalid name for a Kubernetes pod, Flyte assigns a valid name of random characters instead.

Troubles with handling large responses in ``FlyteRemote.sync``
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

- ``Received message larger than max (xxx vs. 4194304)`` usually crops up when the message size is too large.
- To fix this, edit the ``flyte-admin-base-config`` config map using the command ``kubectl edit cm flyte-admin-base-config -n flyte`` to increase the ``maxMessageSizeBytes`` value.


I Still Need Help!
Expand Down
5 changes: 3 additions & 2 deletions rsts/concepts/data_management.rst
Original file line number Diff line number Diff line change
Expand Up @@ -166,8 +166,9 @@ Between Tasks
Bringing in Your Own Datastores for Raw Data
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Flytekit has a pluggable data persistence layer as explained in :std:ref:`data.extend:extend data persistence layer`. This is driven by PROTOCOL.
Flytekit has a pluggable `data persistence layer <https://docs.flyte.org/projects/flytekit/en/latest/data.extend.html>`__.
This is driven by PROTOCOL.
For example, it is theoretically possible to use S3 ``s3://`` for metadata and GCS ``gcs://`` for raw data. It is also possible to create your own protocol ``my_fs://``, to change how data is stored and accessed.
But for Metadata, the data should be accessible to Flyte control plane.

Data persistence is also pluggable. By default, it supports all major blob stores and uses an interface defined in Flytestdlib.
Data persistence is also pluggable. By default, it supports all major blob stores and uses an interface defined in Flytestdlib.

0 comments on commit fd464d6

Please sign in to comment.