-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Known Issues
The following dependency version conflicts have been identified:
-
Docker SDK (v7.1.0)
- Error:
TypeError: kwargs_from_env() got an unexpected keyword argument 'ssl_version' - Solution: Use Docker SDK v6.1.3
- Error:
-
Pytest (v8.3.5)
- Error:
fixture 'session_scoped_container_getter' not found - Solution: Use Pytest v6.2.3
- Error:
-
Pytest-rerunfailures (v14.0)
- Not compatible with Pytest v6.x.x
- Requires Pytest >=7.2
- Solution: Use Pytest-rerunfailures v10.2
-
Requests (v2.31.0)
- Issues with basic Docker operations
requests-unixsocketneeds explicit installation- Not automatically installed with current
docker_test_requirements.txt
Current Configuration
Dependencies are defined in two locations:
python_derivation_agent/setup.pypython_derivation_agent/docker_test_requirements.txt
Current dev dependencies in setup.py:
{
"dev": [
"testcontainers>=3.4.2",
"pytest>=6.2.3",
"pytest-docker-compose>=3.2.1",
"pytest-rerunfailures>=10.2"
]
}Recommendations
-
Version Specification
- Specify exact versions (==) instead of minimum versions (>=)
- This ensures consistent behavior across different environments
-
Dependency Organization
- Move test-related (dev) dependencies to
docker_test_requirements.txt - Remove dev option from
setup.py - This improves clarity and maintainability
- Move test-related (dev) dependencies to
Tested Version Combinations
The following versions have been tested and confirmed working:
| Package | Version |
|---|---|
| pytest | 6.2.3 |
| pytest-rerunfailures | 10.2 |
| docker | 6.1.3 |
| requests | 2.31.0 |
| requests-unixsocket | 0.3.0 |
Note: Earlier versions may work but have not been explicitly tested.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working