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

Ui hung on detailed run and pathPrefix issue #2968

Closed
BartNetJS opened this issue Jul 19, 2023 · 18 comments · Fixed by #3127
Closed

Ui hung on detailed run and pathPrefix issue #2968

BartNetJS opened this issue Jul 19, 2023 · 18 comments · Fixed by #3127
Labels
bug Something isn't working triage requires triage user-request

Comments

@BartNetJS
Copy link

I've updated to the latest version V0.13.0
When I click on a detailed run, the page is not responsive anymore.
I got
image
If I wait long enough I got a response back

Also pathPrefix is not working for the liveness route, it return 404. I see in the yaml that liveness is on the prefix, but got 404 in the kubenretes events.
See also an old issue here #2907

@BartNetJS BartNetJS added bug Something isn't working triage requires triage labels Jul 19, 2023
@jorgeepc
Copy link
Contributor

jorgeepc commented Jul 19, 2023

Hey @BartNetJS thank you for opening this issue. I am not able to reproduce the problem. Could you share more details about your test? The test definition file could help.

Also, I am curious about error logs in the browser console, the size of the response body of your trigger and the number of spans.

@BartNetJS
Copy link
Author

It's a very simple test:

type: Test
spec:
  id: integration-das-liveness
  name: DAS
  trigger:
    type: http
    httpRequest:
      url: http://someapiroute....
      method: GET
      headers:
      - key: Content-Type
        value: application/json
  specs:
  - selector: span[tracetest.span.type="general" name="Tracetest trigger"]
    assertions:
    - attr:tracetest.response.status = 200

First try:

image

Past the url direct in another browser (chrome):

image

Also tested in incognito: same result

Here i got a response after a long time:
image

Somethimes i got this:

image

Logs from server seams ok

image

@BartNetJS
Copy link
Author

Maybe this one is also interesting:
image

@jorgeepc
Copy link
Contributor

@BartNetJS could you try using a clean install of tracetest v0.13.0 and see if the bug is reproducible there?

A couple of questions:

  • are you using docker or kubernetes?
  • could you share the response from the GET /api/tests/{TESTID}/run/{RUNID} endpoint

@BartNetJS
Copy link
Author

@jorgeepc

I tried a clean install. Same issue.
Running on kubernetes in docker windows, but also updated the version on azure AKS with the same results

The http call gives an immediate response:

`
GET http://localhost:11633/test/casechanged-test-1/run/2 HTTP/1.1
User-Agent: vscode-restclient
accept-encoding: gzip, deflate

HTTP/1.1 200 OK
Content-Encoding: gzip
Content-Type: text/html; charset=utf-8
Vary: Accept-Encoding
Date: Wed, 19 Jul 2023 18:11:51 GMT
Content-Length: 851
Connection: close

<!doctype html><title>Tracetest</title><script>const parser={toArray(e){try{return JSON.parse(e)||[]}catch(e){return[]}},toBoolean:e=>"true"===e,toObject(e){try{return JSON.parse(e)||{}}catch(e){return{}}},toString:e=>e};function getTemplateValue(e,t,r){var a=String.fromCharCode(123),n=${a}${a};return!e||e.startsWith(n)?r(t):r(e)}function injectVariables(){window.ENV={measurementId:getTemplateValue("X2vwYODI1vb8g6QpzL9OBxuv5vK7dGan","",parser.toString),analyticsEnabled:getTemplateValue("true","false",parser.toBoolean),serverPathPrefix:getTemplateValue("/","/",parser.toString),serverID:getTemplateValue("dc2da8af7a","",parser.toString),appVersion:getTemplateValue("v0.13.0","",parser.toString),env:getTemplateValue("main","",parser.toString),experimentalFeatures:getTemplateValue("[]","[]",parser.toArray),isTracetestDev:getTemplateValue("false","false",parser.toBoolean)};var e=document.createElement("base");e.href=window.ENV.serverPathPrefix,document.getElementsByTagName("head")[0].appendChild(e)}injectVariables()</script><script defer="defer" src="./static/js/main.98b94208.js"></script>You need to enable JavaScript to run this app.


`

@BartNetJS
Copy link
Author

BartNetJS commented Jul 20, 2023

@jorgeepc I've downgraded to v0.12.2 and it runs smoothly again

for reference, these are the helm chart i'm using:

kubeshop/tracetest 0.2.69 v0.13.0 A Helm chart for Trace Test
kubeshop/tracetest 0.2.68 v0.12.2 A Helm chart for Trace Test

@jorgeepc
Copy link
Contributor

Thank you @BartNetJS for the details. We are reviewing the problem and trying to duplicate it in our environments. We will let you know once we have more information.

@BartNetJS
Copy link
Author

BartNetJS commented Aug 1, 2023

@jorgeepc any update on this issue?
I've tested it also via a colleague, and he has the same issue
If it could be a help, we can setup a meeting

@jorgeepc
Copy link
Contributor

jorgeepc commented Aug 1, 2023

Hey @BartNetJS we have been testing this and we are not able to reproduce the issue. I think a meeting could be really helpful to debug the problem. Please let me know your availability and we can schedule something.

In the meantime, if you could share with us an screenshot of your network tab (with the All filter applied), we can see if there is anything weird there. I'm interested in seeing the status of the requests or if they stay in progress (pending) for a long time.

Screenshot 2023-08-01 at 11 11 10

@BartNetJS
Copy link
Author

Hi @jorgeepc

Can i send you a mp4 file directly by mail or link?

@BartNetJS
Copy link
Author

@jorgeepc today is my last working day before holiday. If you send me a meeting request today (09:00-17:00 CET) I can show you the issue.

@jorgeepc
Copy link
Contributor

jorgeepc commented Aug 3, 2023

@BartNetJS I sent you an invite

@BartNetJS
Copy link
Author

BartNetJS commented Aug 23, 2023

HI @jorgeepc I'm back, but didn't got an invite. Can you try to send a new one?
I've also tried the latest version, still the same issue

@jorgeepc
Copy link
Contributor

Hey @BartNetJS please send me your contact details to [email protected] I'll schedule something for this week. Thank you!

@jorgeepc
Copy link
Contributor

jorgeepc commented Sep 6, 2023

Hey @BartNetJS I found the issue. I opened a PR #3127 with the fix. I'll notify you when we have it in a new release.

@jorgeepc
Copy link
Contributor

jorgeepc commented Sep 6, 2023

Reopening until we can validate it in the release

@jorgeepc jorgeepc reopened this Sep 6, 2023
@jorgeepc
Copy link
Contributor

jorgeepc commented Sep 7, 2023

Hey @BartNetJS we release our v0.13.7 version. Could you please double check if the issue is fixed? Thank you.

@jorgeepc
Copy link
Contributor

The problem was fixed in v.0.13.7. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage requires triage user-request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants