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

BF(TST,workaround): just xfail failing archives test on NFS #6912

Merged
merged 1 commit into from
Aug 3, 2022

Conversation

yarikoptic
Copy link
Member

A workaround for #4101
with troubleshooting and trying various approaches done within
#6751

An ideal approach would be to check if create_tree does create a tree (and
archives with files it expect) matching the specification of the tree. Somehow
on NFS it seems to need some times a few seconds (sleep 3 before creating
archive helped according to testing in #6751) before filesystem actually reflect
creation of all files desired.

I found no robust solution which would not require "too much work" so decided
to just announce this an xfail.

For better or for worse, the loop of 100 invocations had them all PASS and none
XFAIL in https://app.travis-ci.com/github/datalad/datalad/builds/253793240 so
the effectiveness of this is not exactly confirmed yet ;)

A workaround for datalad#4101
with troubleshooting and trying various approaches done within
datalad#6751

An ideal approach would be to check if create_tree does create a tree (and
archives with files it expect) matching the specification of the tree. Somehow
on NFS it seems to need some times a few seconds (sleep 3 before creating
archive helped according to testing in datalad#6751) before filesystem actually reflect
creation of all files desired.

I found no robust solution which would not require "too much work" so decided
to just announce this an xfail.

For better or for worse, the loop of 100 invocations had them all PASS and none
XFAIL in https://app.travis-ci.com/github/datalad/datalad/builds/253793240 so
the effectiveness of this is not exactly confirmed yet ;)
@yarikoptic yarikoptic added tests Add or improve existing tests semver-tests Changes only affect tests, no impact on version labels Aug 3, 2022
@yarikoptic
Copy link
Member Author

travis - 3.10 , another PR to fix it... all good for this xfail AFAIK

@yarikoptic yarikoptic merged commit 5e8c831 into datalad:maint Aug 3, 2022
@yarikoptic yarikoptic deleted the bf-tst-xfail-nfs branch October 14, 2022 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-tests Changes only affect tests, no impact on version tests Add or improve existing tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant