marvin: use pycryptodome instead of PyCrypt#13594
Conversation
|
@blueorangutan package |
|
@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress. |
There was a problem hiding this comment.
Pull request overview
This PR updates Marvin’s Python packaging metadata to depend on pycryptodome instead of PyCrypt, aligning the Marvin install requirements with the Crypto.* module usage found in the integration test suite.
Changes:
- Replace
PyCryptwithpycryptodomeintools/marvin/setup.pyinstall_requires.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #13594 +/- ##
=========================================
Coverage 19.47% 19.47%
Complexity 19351 19351
=========================================
Files 6296 6296
Lines 568368 568368
Branches 69622 69622
=========================================
+ Hits 110673 110702 +29
+ Misses 445618 445586 -32
- Partials 12077 12080 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 18541 |
|
@blueorangutan test matrix |
|
@weizhouapache a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smoke tests |
|
[SF] Trillian Build Failed (tid-16532) |
|
[SF] Trillian Build Failed (tid-16534) |
|
[SF] Trillian Build Failed (tid-16531) |
|
[SF] Trillian Build Failed (tid-16535) |
|
[SF] Trillian test result (tid-16537)
|
|
[SF] Trillian test result (tid-16536)
|
Description
This PR...
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?