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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Lakectl Azure: fs upload with --pre-sign url fails #6426

Closed
N-o-Z opened this issue Aug 17, 2023 · 0 comments · Fixed by #6660
Closed

[Bug]: Lakectl Azure: fs upload with --pre-sign url fails #6426

N-o-Z opened this issue Aug 17, 2023 · 0 comments · Fixed by #6660
Assignees
@guy-har
Labels
area/lakectl Issues related to lakeFS' command line interface (lakectl) bug Something isn't working

Comments

@N-o-Z
Copy link
Member

N-o-Z commented Aug 17, 2023

What happened?

What actually happened, including error codes if applies.

Steps to Reproduce:

  1. Uploading a file using lakectl to a lakeFS instance configured with Azure using --pre-sign flag provides the following error:

% lakectl fs upload lakefs://test/test/04.pdf --source a.txt --pre-sign
upload request failed https://.blob.core.windows.net/container1/test/data/glohpnt4fd2m97cp8fe0/cjeucl54fd2m97cp8ffg?se=2023-08-17T09%3A35%3A20Z&sig=9xp9ouZFdoAnyPdO1OVqDKAEYdQAqPrjt680S9UcJl0%3D&ske=2023-08-17T10%3A34%3A29Z&skoid=d8193925-4108-40a0-8d76-0c6d3ceb1f61&sks=b&skt=2023-08-17T09%3A19%3A19Z&sktid=ed72fd32-966e-4e48-a5c9-abb43b8b9c8c&skv=2020-10-02&sp=raw&spr=https&sr=b&sv=2020-02-10: 400 An HTTP header that's mandatory for this request is not specified.

Expected behavior

Upload expected to work

lakeFS version

master over 106.2

How lakeFS is installed

local deployment

Affected clients

lakectl

Relevant log output

No response

Contact details

No response
@N-o-Z N-o-Z added bug Something isn't working area/lakectl Issues related to lakeFS' command line interface (lakectl) labels Aug 17, 2023
@itaiad200 itaiad200 self-assigned this Aug 31, 2023
@itaiad200 itaiad200 removed their assignment Sep 10, 2023
@guy-har guy-har assigned guy-har and unassigned guy-har Sep 24, 2023
@guy-har guy-har linked a pull request Oct 1, 2023 that will close this issue
Fix lakectl upload with pre-sign for Azure #6660
Merged
This was referenced Oct 1, 2023
Merged
Closed
@guy-har guy-har mentioned this issue Oct 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guy-har guy-har

Labels
area/lakectl Issues related to lakeFS' command line interface (lakectl) bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix lakectl upload with pre-sign for Azure treeverse/lakeFS
3 participants
@N-o-Z @itaiad200 @guy-har