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

Set object-store last-modified time as mtime in lakeFSFS direct access #8388

Open arielshaqed opened 19 hours ago

arielshaqed commented 19 hours ago

(This is not possible with presigned URLs, so obviously not done there.)

Closes #8339.

github-actions[bot] commented 19 hours ago

E2E Test Results - Quickstart

10 passed, 1 failed

Test failures:
  Quickstart › pull requests: common/quickstart.spec.ts

quickstart.spec.ts:102:9 pull requests

  [quickstart] › common/quickstart.spec.ts:102:9 › Quickstart › pull requests ──────────────────────

    Test timeout of 30000ms exceeded.

    Error: locator.click: Test timeout of 30000ms exceeded.
    Call log:
      - waiting for locator('#pulls-tabs-tab-closed')

       at poms/pullsPage.ts:44

      42 |
      43 |     async gotoPullsTab(id: string): Promise<void> {
    > 44 |         await this.page.locator(`#pulls-tabs-tab-${id}`).click();
         |                                                          ^
      45 |     }
      46 |
      47 |     async getFirstPullsRowDetails(): Promise<{title: string, description: string}> {

        at PullsPage.gotoPullsTab (/home/runner/work/lakeFS/lakeFS/webui/test/e2e/poms/pullsPage.ts:44:58)
        at /home/runner/work/lakeFS/lakeFS/webui/test/e2e/common/quickstart.spec.ts:142:25

    attachment #1: trace (application/zip) ─────────────────────────────────────────────────────────
    test-results/common-quickstart-Quickstart-pull-requests-quickstart/trace.zip
    Usage:

        npx playwright show-trace test-results/common-quickstart-Quickstart-pull-requests-quickstart/trace.zip

    ────────────────────────────────────────────────────────────────────────────────────────────────

This test report was produced by the test-summary action.  Made with ❤️ in Cambridge.
github-actions[bot] commented 19 hours ago

E2E Test Results - DynamoDB Local - Local Block Adapter

13 passed