backup/drivers/test_backup_s3.py should not be using moto

Bug #1921586 reported by Thomas Goirand
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Invalid
Undecided
Unassigned

Bug Description

Hi,

The new recent driver for backups over s3 leads, in its test suite, is using the moto library. This library contains a huge chain of dependencies, which I would strongly recommend against. It leads to package "jose" (ie: https://pypi.org/project/jose/) which is Python 2 only.

Please get rid of the moto library ASAP.

Cheers,

Thomas Goirand (zigo)

Revision history for this message
Eric Harney (eharney) wrote :

It looks like the unit test only uses the s3 part of moto, which does not import the jose library.

The jose library appears to be listed as an extra dep in moto, so I don't think we need to change this in Cinder.

Changed in cinder:
status: New → Incomplete
Eric Harney (eharney)
Changed in cinder:
status: Incomplete → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.