Add pytz dependency to test requirements#32
Add pytz dependency to test requirements#32rajivmucheli wants to merge 2 commits intostable/2025.2-m3from
Conversation
required for audit-middleware
joker-at-work
left a comment
There was a problem hiding this comment.
I think requirements are installed in tests, too, so this means duplication.
If barbican doesn't need this, it shouldn't be part of barbican's requirements.txt. You could use our internal custom-requirements.txt but really, it should be part of audit-middleware's requirements.txt. afaik, lower-constraints.txt are not evaluated to get a list of packages to install when finding dependencies, so audit-middleware needs to properly define their dependencies. lower-constraints.txt are afaik also only evaluated when explicitly passed as contraint file to pip - which is not how we install audit-middleware in our images.
Hi @notque, could you implement the above ? i will close this PR then. |
|
Thanks @joker-at-work - replaced with sapcc/openstack-audit-middleware#129 |
required for audit-middleware