@Zhi Yan Liu About your fix, why did you renamed configure_add in configure, and removed the sheepdog.Store stubs in tests ? It seems to make thoses tests fail:
Also, I think it misses a expect OSError after the processutils.execute. Without shell=True, execute raises OSError instead of ProcessExecutionError and it makes this test fail:
@Zhi Yan Liu About your fix, why did you renamed configure_add in configure, and removed the sheepdog.Store stubs in tests ? It seems to make thoses tests fail:
- glance. tests.functiona l.store. test_filesystem .TestFilesystem Store tests.functiona l.store. test_http. TestHTTPStore tests.unit. test_store_ base.TestStoreB ase tests.unit. test_store_ location. TestStoreLocati on
- glance.
- glance.
- glance.
Also, I think it misses a expect OSError after the processutils. execute. Without shell=True, execute raises OSError instead of ProcessExecutio nError and it makes this test fail:
- glance. tests.functiona l.store. test_sheepdog. TestSheepdogSto re
Do you have thoses errors as well ?