tests: Drop bare-user on tmpfs negative test #3107
Conversation
As of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2daf18a7884dc03d5164ab9c7dc3f2ea70638469 the restriction is lifted, FINALLY.
|
Why not keep the test but verify that creating a bare-user repo on tmpfs doesn't error out now? |
Well, in theory longer term we want to be able to run our tests across multiple distributions. |
|
Ahhh, is support for it gated behind a config? We could grep for it and skip it if it's not on. |
I don't think so, see the linked commit. It changed to unconditional support. Is it really worth keeping our CI broken over this? |
There was a problem hiding this comment.
Ahhh, is support for it gated behind a config?
I don't think so, see the linked commit. It changed to unconditional support.
In that case, I don't understand #3107 (comment).
I'm just pointing out that this test can trivially be turned from a negative to a positive test.
Is it really worth keeping our CI broken over this?
Not really. I hadn't realized this was blocking CI. Feel free to merge!
Not if we want to run the tests on c9s, where that commit isn't backported right? |
As of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2daf18a7884dc03d5164ab9c7dc3f2ea70638469 the restriction is lifted, FINALLY.