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

Error generating query summary after click generate summary on the dashboard #42

Open
ymo-do opened this issue Feb 7, 2025 · 5 comments

Comments

@ymo-do
Copy link

ymo-do commented Feb 7, 2025

Hi, Really appreciate your great work here! I encountered an issue when I tried to implement this on my dashboard. I manually tested my cloud run app, it works. and I built the dashboard_summarization.js and uploaded to my looker project. I'm able to see the extension and am able to click generate summary. However, I didn't see any response back. By checking the source code on the Devtool Console. I found the error: Error generating query summary: Error: Error: Unable to parse request url "/generateSummary" I did update the js file a bit since the endpoint url should be the cloud run api endpoint/generateSummary but not generateQuerySummary. I tested this before and after the change. I got the same error anyway. Please let me know if i made anything wrong here. Thanks!

@tendres
Copy link

tendres commented Feb 9, 2025

Hi @ymo-do
I too am having the exact same issue as you.
Looking at the GC logs I am seeing the following error in the dashboard-summary-service Cloud Run logs:
POST 403 251 B6 ms Ruby - indicates a Forbidden error occurred for a POST request to the Cloud Run service dashboard-summary-service.

Please provide an update if you are able to find a fix - thanks!

@ymo-do
Copy link
Author

ymo-do commented Feb 10, 2025

Hi @ymo-do I too am having the exact same issue as you. Looking at the GC logs I am seeing the following error in the dashboard-summary-service Cloud Run logs: POST 403 251 B6 ms Ruby - indicates a Forbidden error occurred for a POST request to the Cloud Run service dashboard-summary-service.

Please provide an update if you are able to find a fix - thanks!

Could you check your looker version? I think it is related. the Looker server must be at version 7.21 or above.

@tendres
Copy link

tendres commented Feb 10, 2025

Hi @ymo-do I too am having the exact same issue as you. Looking at the GC logs I am seeing the following error in the dashboard-summary-service Cloud Run logs: POST 403 251 B6 ms Ruby - indicates a Forbidden error occurred for a POST request to the Cloud Run service dashboard-summary-service.

Could you check your looker version? I think it is related. the Looker server must be at version 7.21 or above.

I am running version 25.0.23 which is the latest.
I'm starting to think the error is in the dashboard_summarization.js file.
I made the same adjustment with "/generateSummary" and also noticed that the GENAI_CLIENT_SECRET was missing the last two characters in the value. But adjusting that did not change any behavior.

Really hoping to get this working.

@ymo-do
Copy link
Author

ymo-do commented Feb 11, 2025

Hi @ymo-do I too am having the exact same issue as you. Looking at the GC logs I am seeing the following error in the dashboard-summary-service Cloud Run logs: POST 403 251 B6 ms Ruby - indicates a Forbidden error occurred for a POST request to the Cloud Run service dashboard-summary-service.

Could you check your looker version? I think it is related. the Looker server must be at version 7.21 or above.

I am running version 25.0.23 which is the latest. I'm starting to think the error is in the dashboard_summarization.js file. I made the same adjustment with "/generateSummary" and also noticed that the GENAI_CLIENT_SECRET was missing the last two characters in the value. But adjusting that did not change any behavior.

Really hoping to get this working.

In this case, I think we might had the same issue. The functions fetchProxy() and serverProxy() don't work in the looker server. In this case instead of bringing the true looker version to compare, it always returns "not found" and this breaks the check. Please use it wisely. I override these functions and found no issue. Here is the code I added to the Nc() function. if (!t) {console.error("🚨 API URL (t) is undefined! Setting fallback."); t = "YOUR CLOUD RUN URL";} It works on my end.

@tendres
Copy link

tendres commented Feb 12, 2025

@ymo-do Hey super appreciate your help with this. Could you confirm the following for anyone else with this issue? I'm still working on it...

  1. Older blogs and instructions that link to this repository mention a looker.ini file. It is not relevant in this version.
  2. After building dashboard_summarization.js, open and update the file's endpoint url to the cloud run api from generateQuerySummary to generateSummary.
  3. Due to possible looker version mismatch, the functions fetchProxy() and serverProxy() need to be overridden by
    adding to the Nc() function. if (!t) {console.error("🚨 API URL (t) is undefined! Setting fallback."); t = "YOUR CLOUD RUN URL";}. Use with caution.

** Notice the emoji above - could you re-enter the code you used? Anything else? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants