treeverse / lakeFS

lakeFS - Data version control for your data lake | Git for data
https://docs.lakefs.io
Apache License 2.0
4.46k stars 359 forks source link

[Bug]: Importing fails if a pre commit hook is defined #8319

Closed Jonathan-Rosenberg closed 3 weeks ago

Jonathan-Rosenberg commented 3 weeks ago

What happened?

If a pre commit hook is defined, import operations will fail.

Steps to Reproduce:

  1. Define a pre commit hook
  2. Import data

Root cause

Import uses multiple async Go routines to import the data. Those Go routines start their own contexts, thus dropping the middleware-initialized contexts with a username field in them. This results in a user not found error when the import operation tries to commit (and the pre commit hook validates the user that ran the import).

Expected behavior

No response

lakeFS version

1.39.2

How lakeFS is installed

No response

Affected clients

No response

Relevant log output

No response

Contact details

No response