Method update to unanimous (#535)

Signed-off-by: lengrongfu <lenronfu@gmail.com>

cdnsystem/daemon/cdn/storage/storage_gc.go

@@ -48,16 +48,19 @@ func NewStorageCleaner(cfg *GCConfig, driver storedriver.Driver, storageMgr Mana
 }
 
 func (cleaner *Cleaner) GC(storagePattern string, force bool) ([]string, error) {
-	freeSpace, err := cleaner.driver.GetAvailSpace()
+	freeSpace, err := cleaner.driver.GetFreeSpace()
 	if err != nil {
 		if cdnerrors.IsFileNotExist(err) {
 			err = cleaner.driver.CreateBaseDir()
 			if err != nil {
 				return nil, err
 			}
-			freeSpace, _ = cleaner.driver.GetAvailSpace()
+			freeSpace, err = cleaner.driver.GetFreeSpace()
+			if err != nil {
+				return nil, fmt.Errorf("get free space: %v", err)
+			}
 		} else {
-			return nil, fmt.Errorf("get available space: %v", err)
+			return nil, fmt.Errorf("get free space: %v", err)
 		}
 	}
 	fullGC := force

cdnsystem/storedriver/driver.go

@@ -67,8 +67,8 @@ type Driver interface {
 	// If not, return the ErrFileNotExist.
 	Stat(raw *Raw) (*StorageInfo, error)
 
-	// GetAvailSpace returns the available disk space in B.
-	GetAvailSpace() (unit.Bytes, error)
+	// GetFreeSpace returns the available disk space in B.
+	GetFreeSpace() (unit.Bytes, error)
 
 	// GetTotalAndFreeSpace
 	GetTotalAndFreeSpace() (unit.Bytes, unit.Bytes, error)
@@ -253,9 +253,9 @@ func (s *driverPlugin) MoveFile(src string, dst string) error {
 	return s.instance.MoveFile(src, dst)
 }
 
-// GetAvailSpace returns the available disk space in B.
-func (s *driverPlugin) GetAvailSpace() (unit.Bytes, error) {
-	return s.instance.GetAvailSpace()
+// GetFreeSpace returns the available disk space in B.
+func (s *driverPlugin) GetFreeSpace() (unit.Bytes, error) {
+	return s.instance.GetFreeSpace()
 }
 
 func (s *driverPlugin) GetHomePath() string {

cdnsystem/storedriver/local/local_driver.go

@@ -316,8 +316,8 @@ func (ds *driver) Remove(raw *storedriver.Raw) error {
 	return err
 }
 
-// GetAvailSpace returns the available disk space in Byte.
-func (ds *driver) GetAvailSpace() (unit.Bytes, error) {
+// GetFreeSpace returns the available disk space in Byte.
+func (ds *driver) GetFreeSpace() (unit.Bytes, error) {
 	path := ds.BaseDir
 	lock(path, -1, true)
 	defer unLock(path, -1, true)

cdnsystem/storedriver/mock_driver.go

@@ -78,19 +78,19 @@ func (mr *MockDriverMockRecorder) Get(arg0 interface{}) *gomock.Call {
 	return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "Get", reflect.TypeOf((*MockDriver)(nil).Get), arg0)
 }
 
-// GetAvailSpace mocks base method.
-func (m *MockDriver) GetAvailSpace() (unit.Bytes, error) {
+// GetFreeSpace mocks base method.
+func (m *MockDriver) GetFreeSpace() (unit.Bytes, error) {
 	m.ctrl.T.Helper()
-	ret := m.ctrl.Call(m, "GetAvailSpace")
+	ret := m.ctrl.Call(m, "GetFreeSpace")
 	ret0, _ := ret[0].(unit.Bytes)
 	ret1, _ := ret[1].(error)
 	return ret0, ret1
 }
 
-// GetAvailSpace indicates an expected call of GetAvailSpace.
-func (mr *MockDriverMockRecorder) GetAvailSpace() *gomock.Call {
+// GetFreeSpace indicates an expected call of GetFreeSpace.
+func (mr *MockDriverMockRecorder) GetFreeSpace() *gomock.Call {
 	mr.mock.ctrl.T.Helper()
-	return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "GetAvailSpace", reflect.TypeOf((*MockDriver)(nil).GetAvailSpace))
+	return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "GetFreeSpace", reflect.TypeOf((*MockDriver)(nil).GetFreeSpace))
 }
 
 // GetBytes mocks base method.

cdnsystem/storedriver/store_mgr_test.go

@@ -60,7 +60,7 @@ func (m mockDriver) Stat(_ *Raw) (*StorageInfo, error) {
 	panic("implement me")
 }
 
-func (m mockDriver) GetAvailSpace() (unit.Bytes, error) {
+func (m mockDriver) GetFreeSpace() (unit.Bytes, error) {
 	panic("implement me")
 }
 

docs/zh-CN/design/architecture.md

@@ -40,6 +40,12 @@
 
 ![alt][download-process]
 
+### 子模块架构
+
+#### CDN模块
+
+![cdn](../images/cdn.png)
+
 ### 子系统功能
 
 **Manager**