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

Missing Credentials Error when deploying to Vercel #727

Closed
2 of 12 tasks
archiewood opened this issue Mar 31, 2023 · 11 comments · Fixed by #807
Closed
2 of 12 tasks

Missing Credentials Error when deploying to Vercel #727

archiewood opened this issue Mar 31, 2023 · 11 comments · Fixed by #807
Labels
bug Something isn't working high-priority Bug significantly impacting the Evidence UX for many users. Immediate fix

Comments

@archiewood
Copy link
Member

archiewood commented Mar 31, 2023

Bug Description

When attempting to deploy with duckdb to Vercel, Evidence fails to read in the database configuration.

image

image

Netlify does not have this issue.

Slack thread

Severity

  • High - I'm not able to use Evidence until this bug is fixed
  • Medium - I'm able to use Evidence, but a feature I need to use is not working
  • Low - I'm able to keep using Evidence as normal, but flagging this for the team to fix

Expected Behavior
It should deploy.

Workarounds
Use Netlify or Evidence Cloud to deploy.

Environment Information
Package versions: v15.0.1

Database:

  • Postgres
  • BigQuery
  • Snowflake
  • Redshift
  • MySQL
  • SQLite
  • DuckDB
  • CSV
  • Other (please specify)
@archiewood archiewood added bug Something isn't working to-review Evidence team to review labels Mar 31, 2023
@hughess hughess added medium-priority Significant bug with fix needed within a few weeks and removed to-review Evidence team to review labels Apr 4, 2023
@mcrascal mcrascal added high-priority Bug significantly impacting the Evidence UX for many users. Immediate fix and removed medium-priority Significant bug with fix needed within a few weeks labels Apr 6, 2023
@mcrascal
Copy link
Member

mcrascal commented Apr 6, 2023

To reproduce:

  1. Building Evidence from a sub directory, called reports (need to confirm it does work in root)
  2. Using duckdb where the database file is checked into the repo, per the docs

Suspicion is that this is a file path issue arising because of the subdirectory setting in vercel and the way we're reading the ddb instance. See below -- @archiewood repro'd without changing root directory.

@archiewood
Copy link
Member Author

archiewood commented Apr 6, 2023

I repro'ed the error when building in root last week, using this repo.

@archiewood
Copy link
Member Author

archiewood commented Apr 6, 2023

Looked into this in a bit more detail:

if I get vercel to run printenv DATABASE && printenv DUCKDB_FILENAME during the build process, it successfully returns

image

But then later in the build:
image

@archiewood
Copy link
Member Author

archiewood commented Apr 6, 2023

@archiewood
Copy link
Member Author

This appears to be a duckdb specific issue. Snowflake does not exhibit the same behaviour.

@ItsMeBrianD
Copy link
Member

/lib64/libm.so.6: version 'GLIBC_2.29' not found (required by /vercel/path0/node_modules/duckdb/lib/binding/duckdb.node)

Looks like this is related to the environment that vercel is running builds in; I'm not entirely sure how to proceed with this.

@archiewood
Copy link
Member Author

I’d at least try opening an issue with duckdb? They are pretty responsive and helpful.

@mcrascal
Copy link
Member

mcrascal commented Apr 14, 2023

@ItsMeBrianD

Good find

@archiewood
Copy link
Member Author

One thing I don't understand.
I'm reasonably sure that duckdb projects used to build on vercel pre SK1.0? ie with v 14.0.0 or similar

@ItsMeBrianD
Copy link
Member

I did a test on vercel to see which version of glibc is being used, looks like it's ldd (GNU libc) 2.26; so this might be a "duckdb upgraded their dep", rather than a SK breaking change

@ItsMeBrianD
Copy link
Member

We can track this here: duckdb/duckdb-node#15
Going to update docs and move on for the time being

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high-priority Bug significantly impacting the Evidence UX for many users. Immediate fix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants