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.

Sign up for GitHub

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

Jump to bottom

Getting "Entity too large" error while rendering multi layer chart #1

Open
Eroyi opened this issue Jun 19, 2023 · 12 comments
Open

Getting "Entity too large" error while rendering multi layer chart #1

Eroyi opened this issue Jun 19, 2023 · 12 comments
Assignees
@tobiashochguertel
Labels
bug Something isn't working question Further information is requested

Comments

@Eroyi
Copy link

Eroyi commented Jun 19, 2023

I was trying to render Gitlab Official Helm Chart, editor work just fine, but getting error in rendering.
@tobiashochguertel
Copy link
Owner

@Eroyi Thanks for provide the feedback.

Can you tell me where I find the Chart?
I searched quickly and found this: https://gitlab.com/gitlab-org/charts/gitlab/-/tree/master/charts/gitlab

@tobiashochguertel tobiashochguertel self-assigned this Jun 22, 2023
@tobiashochguertel tobiashochguertel added bug Something isn't working question Further information is requested labels Jun 22, 2023
@tobiashochguertel
Copy link
Owner

tobiashochguertel commented Jun 24, 2023 •
edited
Loading

I had to add/increase the limit for the express middleware... Fixed

Released a new version of the template container image https://hub.docker.com/r/tobiashochguertel/th-helm-playground-template/tags

In the case of the following Chart: https://gitlab.com/gitlab-org/charts/gitlab/-/tree/master/charts/gitlab I see then that helm is missing some templates, but that is not related to the editor that is related to the chart.

Works now. Can you give it a try @Eroyi ?

@Eroyi
Copy link
Author

Eroyi commented Jul 3, 2023

Oops sry for the late reply, just got the chance to check my mailbox.
Ive tested the latest version, now im getting a different error

<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="utf-8">
<title>Error</title>
</head>
<body>
<pre>SyntaxError: Bad Unicode escape in JSON at position 2539709<br> &nbsp; &nbsp;at JSON.parse (&lt;anonymous&gt;)<br> &nbsp; &nbsp;at parse (/app/node_modules/body-parser/lib/types/json.js:92:19)<br> &nbsp; &nbsp;at /app/node_modules/body-parser/lib/read.js:128:18<br> &nbsp; &nbsp;at AsyncResource.runInAsyncScope (node:async_hooks:203:9)<br> &nbsp; &nbsp;at invokeCallback (/app/node_modules/raw-body/index.js:238:16)<br> &nbsp; &nbsp;at done (/app/node_modules/raw-body/index.js:227:7)<br> &nbsp; &nbsp;at IncomingMessage.onEnd (/app/node_modules/raw-body/index.js:287:7)<br> &nbsp; &nbsp;at IncomingMessage.emit (node:events:513:28)<br> &nbsp; &nbsp;at endReadableNT (node:internal/streams/readable:1359:12)<br> &nbsp; &nbsp;at process.processTicksAndRejections (node:internal/process/task_queues:82:21)</pre>
</body>
</html>

And also the left side could really use a handler to resize the area, it is really hard to navigate.
image

and abou the charts, helm pull gitlab/gitlab --untar

@tobiashochguertel
Copy link
Owner

I will work on it at the weekend @Eroyi, thanks for providing the feedback about the helm chart.

@tobiashochguertel
Copy link
Owner

@Eroyi Can you provide me your custom configuration for this chart? I don't get this error. I used helm pull gitlab/gitlab --untar as chart, and after setting
image

image

it renders the chart:

image

I need:

  • your myValues.yaml (Editor -> Set Configuration Values)
  • the selected filename/path

@tobiashochguertel
Copy link
Owner

Add added some handles so that the editor has resizeable Files and Folder Tree. The Editor (Split Editor) has this feature now, too, and also the "My Configuration:" is resizeable.

But I need more details to find the error which you have @Eroyi. Are you using it on Windows?

@Eroyi
Copy link
Author

Eroyi commented Jul 10, 2023

Add added some handles so that the editor has resizeable Files and Folder Tree. The Editor (Split Editor) has this feature now, too, and also the "My Configuration:" is resizeable.

But I need more details to find the error which you have @Eroyi. Are you using it on Windows?

Nah, I'm using it on macOS Ventura, with Safari v16.5

Im gonna try the latest image later with Safari and Chrome, see if the issue is caused by Browser or my helm charts.

@Eroyi
Copy link
Author

Eroyi commented Jul 10, 2023

Tried the latest version with fresh pull helm chart, but getting this new error from "Dashboard Template - Bootstrap v5.3"
image

Im running with:
Apple M1 Max

  • Docker Desktop 4.2.0
    • Engine 20.10.10
    • Compose 1.29.2
  • macOS Ventura 13.4
    • Safari 16.5
    • Chrome 114.0
> docker inspect image 64b85c4736bc

............
        "Architecture": "arm64",
        "Os": "linux",
............

tobiashochguertel added a commit that referenced this issue Aug 7, 2023
@tobiashochguertel
Fixs Bug #1
a16aa05
@tobiashochguertel
Copy link
Owner

@Eroyi I figured out what the "Oops!..." error caused and fixed it.

There is now a new version at https://hub.docker.com/u/tobiashochguertel

@Eroyi
Copy link
Author

Eroyi commented Aug 10, 2023

@Eroyi I figured out what the "Oops!..." error caused and fixed it.

There is now a new version at https://hub.docker.com/u/tobiashochguertel

'preciate the work, i ll be run another test tmr, and feedback asap

@Eroyi
Copy link
Author

Eroyi commented Aug 14, 2023

Tested on

  • WSL on Windows 11 with Ubuntu 22.04.2 LTS
  • Podman 3.4.4
  • Chrome 115.0.5790.171
  • Bitnami Charts Repo

Two issues was founded:

  1. Getting
    <!-- a padding to disable MSIE and Chrome friendly error page -->
    on browser, and

    21#21: *29 client intended to send too large body: 44698916 bytes, client: 10.89.0.1, server: localhost, request: "POST /template HTTP/1.1", host: "localhost:4173", referrer: "http://localhost:4173/"

    in webapp, but can be fix by putting the client_max_body_size 100M; in nginx.conf.

  2. After I patched the body_size error, getting this in browser instead.

    {
  "result": {
    "error": {
      "code": 1,
      "killed": false,
      "signal": null,
      "cmd": "helm template ./public/chart -f ./public/myValues.yaml"
    },
    "stderr": "Error: Chart.yaml file is missing\n"
  }
}

@tobiashochguertel
Copy link
Owner

Okay, I will check this over the week. Thanks for the help and the Information about the environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tobiashochguertel tobiashochguertel

Labels
bug Something isn't working question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tobiashochguertel @Eroyi