ceph / ceph/s3-tests

syntax error in tox.ini

Open
#497 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
372
Forks
328
PR merge metrics
No merged PRs in 30d

Description

Running `S3TEST_CONF=s3tests.conf tox` generates the error:
```ShellSession
$ tox --version
4.4.6 from /home/tuffli/.local/lib/python3.10/site-packages/tox/__init__.py
$ uname -srvmio
Linux 5.15.79.1-microsoft-standard-WSL2 #1 SMP Wed Nov 23 01:01:46 UTC 2022 x86_64 x86_64 GNU/Linux
$ S3TEST_CONF=s3tests.conf tox
py: failed with pass_env values cannot contain whitespace, use comma to have multiple values in a single line, invalid values found 'S3TEST_CONF S3_USE_SIGV4'
py: FAIL code 1 (0.00 seconds)
evaluation failed :( (0.06 seconds)
```
Adding a comma between the variables fixes this for me:
```Diff
diff --git a/tox.ini b/tox.ini
index 93fde38..4f2aaa8 100644
--- a/tox.ini
+++ b/tox.ini
@@ -3,5 +3,5 @@ envlist = py

[testenv]
deps = -rrequirements.txt
-passenv = S3TEST_CONF S3_USE_SIGV4
+passenv = S3TEST_CONF,S3_USE_SIGV4
commands = pytest {posargs}
```

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.