-
Notifications
You must be signed in to change notification settings - Fork 5
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
Unable to query the Thimble database from our sweet dashboard. #232
Comments
This is what I reported in the Thimble bug, and it's just 503'ing:
|
The attempt to connect to postgres fails with an @jbuck Are we hitting a limit on the number of clients that can connect at once? I wonder how often clients are released back into the pool. |
The dashboard creates a new client connection for every request. That might cause issues if there's a limited number of clients connections available. |
@flukeout there was work to convert date types, which might be what this is? cc @gideonthomas |
@Pomax @gideonthomas It looks like some Thimble projects have made it into the What's the plan on this? Will they be the same ultimately? Should I account for this somehow in my dashboard code? cc @chadsansing |
@flukeout I'm working on a patch that will store them as datetime sql types so that should fix this. I'm not sure what caused it to have epoch-based time though (seems like they all have "X-Ray Goggles" in their name so maybe that has something to do with it :P) |
https://github.com/mozilla/goggles.webmaker.org/blob/master/views/publication.js#L278-L286 looks like the culprit. Another reason why publish.wmo, not clients, should control creation and update dates =) |
Okay, so it sounds like this is just a "unexpected data in the database" issue causing the query to not work, right? As opposed to not having access or something like that |
I think @gideonthomas will fix this when he lands his date handling patch. In the meantime, you could patch goggles to send date strings instead of unix timestamps, and use sql to convert the existing timestamps to iso strings. |
filed mozilla/goggles.mozilla.org#257 to track that over on the goggles repo |
The dashboard lives on: https://thimble-dashboard-staging.herokuapp.com/
Some queries seem to work, though. Check out below
projects
tablepublishedProjects
tableAre there different permissions set on this?
I'm also unable to connect to the database at all from my desktop database app.
@jbuck Are there some new security rules that are preventing my app from working?
cc @humphd @sedge
The text was updated successfully, but these errors were encountered: