Differential D4810

FUSE: tests: remove temporary sleep(2) hack
ClosedPublic
Actions

Authored by haltode on Jan 6 2021, 10:45 AM.

Details

Reviewers

seirl

Group Reviewers

Reviewers

Maniphest Tasks

T2830: FUSE: history/by-date/.status file implementation does not seem reliable

Commits

rDFUSEa75335972e7c: tests: revision: remove temporary sleep(2) hack
rDFUSEc2f1b24904f9: fuse: invalidate kernel cache when removing inode

Summary

Close T2830.

Diff Detail

Repository

rDFUSE FUSE virtual file system

Branch

feature/remove-temporary-hack-sleep

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 18185
Build 28068: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 28067: arc lint + arc unit

Event Timeline

haltode created this revision.Jan 6 2021, 10:45 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptJan 6 2021, 10:45 AM

Build is green

Patch application report for D4810 (id=17045)

Rebasing onto 26ee05f3bd...

Current branch diff-target is up to date.

Changes applied before test

commit 10f1ecdd78956a01ac6a7ccbbbd8dff5bc22715f
Author: Thibault Allançon <haltode@gmail.com>
Date:   Wed Jan 6 10:43:32 2021 +0100

    tests: revision: remove temporary sleep(2) hack
    
    Now that the .status file behavior is consistent, we can safely rely on
    it during unit tests.
    
    Close T2830.

commit 0cb9842406308f1476963f30d4d6d7b9e2d6031a
Author: Thibault Allançon <haltode@gmail.com>
Date:   Wed Jan 6 10:42:59 2021 +0100

    fuse: invalidate kernel cache when removing inode

See https://jenkins.softwareheritage.org/job/DFUSE/job/tests-on-diff/261/ for more details.

Harbormaster completed remote builds in B18185: Diff 17045.Jan 6 2021, 10:48 AM

haltode requested review of this revision.Jan 6 2021, 10:48 AM

Remove TODO about invalidate_inode

Build is green

Patch application report for D4810 (id=17057)

Rebasing onto 26ee05f3bd...

Current branch diff-target is up to date.

Changes applied before test

commit 02dadeb85069a99b45bd3cf8c5476eaacf0ec968
Author: Thibault Allançon <haltode@gmail.com>
Date:   Wed Jan 6 10:43:32 2021 +0100

    tests: revision: remove temporary sleep(2) hack
    
    Now that the .status file behavior is consistent, we can safely rely on
    it during unit tests.
    
    Close T2830.

commit c2f1b24904f93404494a6f8a5fcac71637c02962
Author: Thibault Allançon <haltode@gmail.com>
Date:   Wed Jan 6 10:42:59 2021 +0100

    fuse: invalidate kernel cache when removing inode

See https://jenkins.softwareheritage.org/job/DFUSE/job/tests-on-diff/262/ for more details.

Harbormaster completed remote builds in B18197: Diff 17057.Jan 6 2021, 2:17 PM

haltode added a reviewer: seirl.Jan 6 2021, 4:28 PM

seirl accepted this revision.Jan 6 2021, 4:32 PM

This revision is now accepted and ready to land.Jan 6 2021, 4:32 PM

Remove redundant assert

Build is green

Patch application report for D4810 (id=17060)

Rebasing onto 26ee05f3bd...

Current branch diff-target is up to date.

Changes applied before test

commit a75335972e7ce9b2f7d4d96fa349967c6aff1742
Author: Thibault Allançon <haltode@gmail.com>
Date:   Wed Jan 6 10:43:32 2021 +0100

    tests: revision: remove temporary sleep(2) hack
    
    Now that the .status file behavior is consistent, we can safely rely on
    it during unit tests.
    
    Close T2830.

commit c2f1b24904f93404494a6f8a5fcac71637c02962
Author: Thibault Allançon <haltode@gmail.com>
Date:   Wed Jan 6 10:42:59 2021 +0100

    fuse: invalidate kernel cache when removing inode

See https://jenkins.softwareheritage.org/job/DFUSE/job/tests-on-diff/263/ for more details.

Harbormaster completed remote builds in B18200: Diff 17060.Jan 6 2021, 4:39 PM

Closed by commit rDFUSEc2f1b24904f9: fuse: invalidate kernel cache when removing inode (authored by haltode). · Explain WhyJan 6 2021, 4:41 PM

This revision was automatically updated to reflect the committed changes.

haltode added a commit: rDFUSEc2f1b24904f9: fuse: invalidate kernel cache when removing inode.

haltode added a commit: rDFUSEa75335972e7c: tests: revision: remove temporary sleep(2) hack.