Arvados

16039-fuse-forward-slash-sub @ 487e8f756d63c6e68eb300a559eccb504f78c40b -- developer-run-tests: #1723

Tom Clegg wrote:

16039-fuse-forward-slash-sub @ 487e8f756d63c6e68eb300a559eccb504f78c40b -- developer-run-tests: #1723

If you run this against a pre-2.0 server it runs into trouble because there's no public config endpoint in 1.4.

It doesn't crash on mount, but when you do ask for a directory listing where a name has a slash, you get an I/O error, because fetching from the config endpoint is deferred until first use.

Arv-mount should either fail gracefully (don't support older versions) or (preferably) have a fallback behavior (substitute '_' like it did before).

2nd comment, it looks like SlashSubstitutionTest checks that writing to path that's been rewritten for ForwardSlashNameSubstitution doesn't fail, but it doesn't verify that it actually wrote to the correct collection. It should check that self.testcoll actually has a 'waz' file. Also 'foo-bar-baz' and 'foo/bar/baz' should have different file contents (to check that they arn't being confused for one another).

developer-run-tests: #1727

• adds test for name conflict case (which was broken... and is fixed)
• falls back to old behavior (replace with "_") if apiserver has no config endpoint, or FSNS entry is missing

developer-run-tests: #1730

• Touches various setup.py to work around dependency bug #16151

Tom Clegg wrote:

16039-fuse-forward-slash-sub @ a4f57793d8757c451330e6d861eb3c5af2bff3b8 -- developer-run-tests: #1730

• Touches various setup.py to work around dependency bug #16151

LGTM.