[change] Move python dependencies to requirements.txt file
Checklist
- [x] I have read the OpenWISP Contributing Guidelines.
- [x] I have manually tested the changes proposed in this pull request.
- [ ] I have written new test cases for new code and/or updated existing tests for changes to existing code.
- [ ] I have updated the documentation.
Reference to Existing Issue
Closes #507
Description of Changes
This PR here implements automated dependency management using Dependabot by extracting Python dependencies from pip.yml into a dedicated requirements.txt file.
Key Changes:
-
New Requirements File
- Created
files/requirements.txtcontaining all Python dependencies - Organized dependencies by category with clear comments
- Maintained version constraints where necessary
- Created
-
Updated Package Installation
- Modified
tasks/pip.ymlto use the new requirements file - Implemented version constraints using pip's constraint feature
- Preserved compatibility with existing Ansible variables
- Maintained support for conditional package installation
- Modified
-
Dependabot Configuration
- Added Python package ecosystem configuration
- Set weekly update schedule
- Configured PR labeling and commit message format
Testing Details:
-
Installation Testing
Tested with various configurations:
molecule test -s local -
Integration Testing
- Validated Dependabot configuration
- Tested requirements file parsing
- Verified dependency resolution
Benefits:
- Automated dependency updates via Dependabot
- Improved dependency management
- Maintained backward compatibility
- Better organized package requirements
Screenshots
Hey @nemesifier ,
The main issue was that the requirements.txt file cannot be found during the installation process. The error message was consistent across different Ubuntu and Debian environments I've made the possible fix and did tested it properly this time
can you please review my PR
Hey @pandafy @nemesifier can you please reivew this PR ? pls let me know if any changes are required, Thanks!
Hello @pandafy I've implemented the suggested changes in my recent commits, can you please check that once, Thanks!
Hey @pandafy @nemesifier
I've made the suggested changes as for ravan & now it usees the correct prefix format [deps]
Implemented in #529, thanks for trying!