diff --git a/pkg/cloudprovider/resources.go b/pkg/cloudprovider/resources.go
index 8dd222ed4b..41c79143cf 100644
--- a/pkg/cloudprovider/resources.go
+++ b/pkg/cloudprovider/resources.go
@@ -38,7 +38,6 @@ type ICloudResource interface {
 	Refresh() error
 
 	IsEmulated() bool
-	//	GetMetadata() *jsonutils.JSONDict
 
 	GetSysTags() map[string]string
 	GetTags() (map[string]string, error)
diff --git a/pkg/multicloud/aliyun/disk.go b/pkg/multicloud/aliyun/disk.go
index 4380d09e83..42a31c1f79 100644
--- a/pkg/multicloud/aliyun/disk.go
+++ b/pkg/multicloud/aliyun/disk.go
@@ -74,26 +74,6 @@ type SDisk struct {
 	ZoneId                        string
 }
 
-func (self *SDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	// The pricingInfo key structure is 'RegionId::DiskCategory::DiskType
-	priceKey := fmt.Sprintf("%s::%s::%s", self.RegionId, self.Category, self.Type)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-
-	data.Add(jsonutils.NewString(api.HYPERVISOR_ALIYUN), "hypervisor")
-
-	return data
-}
-
-func (self *SDisk) GetSysTags() map[string]string {
-	priceKey := fmt.Sprintf("%s::%s::%s", self.RegionId, self.Category, self.Type)
-	data := map[string]string{}
-	data["price_key"] = priceKey
-	data["hypervisor"] = api.HYPERVISOR_ALIYUN
-	return data
-}
-
 func (self *SRegion) GetDisks(instanceId string, zoneId string, category string, diskIds []string, offset int, limit int) ([]SDisk, int, error) {
 	if limit > 50 || limit <= 0 {
 		limit = 50
diff --git a/pkg/multicloud/aliyun/image.go b/pkg/multicloud/aliyun/image.go
index 56a86083a9..1cc14673cd 100644
--- a/pkg/multicloud/aliyun/image.go
+++ b/pkg/multicloud/aliyun/image.go
@@ -83,23 +83,6 @@ func (self *SImage) GetMinRamSizeMb() int {
 	return 0
 }
 
-func (self *SImage) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	if len(self.Architecture) > 0 {
-		data.Add(jsonutils.NewString(self.Architecture), "os_arch")
-	}
-	if len(self.OSType) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsType()), "os_name")
-	}
-	if len(self.Platform) > 0 {
-		data.Add(jsonutils.NewString(self.Platform), "os_distribution")
-	}
-	if len(self.OSName) > 0 {
-		data.Add(jsonutils.NewString(self.OSName), "os_version")
-	}
-	return data
-}
-
 func (self *SImage) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["os_arch"] = self.Architecture
diff --git a/pkg/multicloud/aliyun/instance.go b/pkg/multicloud/aliyun/instance.go
index d7a136cc80..efff036be5 100644
--- a/pkg/multicloud/aliyun/instance.go
+++ b/pkg/multicloud/aliyun/instance.go
@@ -181,34 +181,6 @@ func (self *SInstance) GetSecurityGroupIds() ([]string, error) {
 	return self.SecurityGroupIds.SecurityGroupId, nil
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	// The pricingInfo key structure is 'RegionId::InstanceType::NetworkType::OSType::IoOptimized'
-	optimized := "optimized"
-	if !self.IoOptimized {
-		optimized = "none"
-	}
-	priceKey := fmt.Sprintf("%s::%s::%s::%s::%s", self.RegionId, self.InstanceType, self.InstanceNetworkType, self.OSType, optimized)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-
-	_, tags, err := self.fetchTags()
-	if err != nil {
-		log.Errorln(err)
-	}
-	data.Update(jsonutils.Marshal(tags))
-
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-	if len(self.ImageId) > 0 {
-		if image, err := self.host.zone.region.GetImage(self.ImageId); err != nil {
-			log.Errorf("Failed to find image %s for instance %s", self.ImageId, self.GetName())
-		} else if meta := image.GetMetadata(); meta != nil {
-			data.Update(meta)
-		}
-	}
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	// The pricingInfo key structure is 'RegionId::InstanceType::NetworkType::OSType::IoOptimized'
diff --git a/pkg/multicloud/aliyun/securitygroup.go b/pkg/multicloud/aliyun/securitygroup.go
index da4c29d1e4..1fdfd4cf17 100644
--- a/pkg/multicloud/aliyun/securitygroup.go
+++ b/pkg/multicloud/aliyun/securitygroup.go
@@ -90,17 +90,6 @@ func (self *SSecurityGroup) GetVpcId() string {
 	return self.VpcId
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	if len(self.Tags.Tag) == 0 {
-		return nil
-	}
-	data := jsonutils.NewDict()
-	for _, value := range self.Tags.Tag {
-		data.Add(jsonutils.NewString(value.TagValue), value.TagKey)
-	}
-	return data
-}
-
 func (self *SSecurityGroup) GetTags() (map[string]string, error) {
 	tags := map[string]string{}
 	for _, value := range self.Tags.Tag {
diff --git a/pkg/multicloud/apsara/dbinstance.go b/pkg/multicloud/apsara/dbinstance.go
index b2c9b98f82..05952b3c13 100644
--- a/pkg/multicloud/apsara/dbinstance.go
+++ b/pkg/multicloud/apsara/dbinstance.go
@@ -778,20 +778,6 @@ func (region *SRegion) RenewDBInstance(instanceId string, bc billing.SBillingCyc
 	return err
 }
 
-func (rds *SDBInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	tags, err := rds.region.ListResourceTags(APSARA_PRODUCT_RDS, "INSTANCE", []string{rds.GetId()})
-	if err != nil {
-		log.Errorf(`[err:%s]rds.region.FetchResourceTags("slb", "instance", []string{rds.GetId()})`, err.Error())
-		return nil
-	}
-	if _, ok := tags[rds.GetId()]; !ok {
-		return nil
-	}
-	data.Update(jsonutils.Marshal(tags[rds.GetId()]))
-	return data
-}
-
 func (rds *SDBInstance) GetTags() (map[string]string, error) {
 	tags, err := rds.region.ListResourceTags("rds", "INSTANCE", []string{rds.GetId()})
 	if err != nil {
diff --git a/pkg/multicloud/apsara/disk.go b/pkg/multicloud/apsara/disk.go
index 61dea2fae7..25afd4c918 100644
--- a/pkg/multicloud/apsara/disk.go
+++ b/pkg/multicloud/apsara/disk.go
@@ -75,28 +75,6 @@ type SDisk struct {
 	ZoneId                        string
 }
 
-func (self *SDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	// The pricingInfo key structure is 'RegionId::DiskCategory::DiskType
-	priceKey := fmt.Sprintf("%s::%s::%s", self.RegionId, self.Category, self.Type)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-
-	data.Add(jsonutils.NewString(api.HYPERVISOR_APSARA), "hypervisor")
-
-	return data
-}
-
-func (self *SDisk) GetSysTags() map[string]string {
-	data := map[string]string{}
-	// The pricingInfo key structure is 'RegionId::DiskCategory::DiskType
-	priceKey := fmt.Sprintf("%s::%s::%s", self.RegionId, self.Category, self.Type)
-	data["price_key"] = priceKey
-	data["hypervisor"] = api.HYPERVISOR_APSARA
-
-	return data
-}
-
 func (self *SRegion) GetDisks(instanceId string, zoneId string, category string, diskIds []string, offset int, limit int) ([]SDisk, int, error) {
 	if limit > 50 || limit <= 0 {
 		limit = 50
diff --git a/pkg/multicloud/apsara/elasticcache_instance.go b/pkg/multicloud/apsara/elasticcache_instance.go
index 96c5b7fe2e..43fae416df 100644
--- a/pkg/multicloud/apsara/elasticcache_instance.go
+++ b/pkg/multicloud/apsara/elasticcache_instance.go
@@ -923,20 +923,6 @@ func (self *SElasticcache) GetICloudElasticcacheBackup(backupId string) (cloudpr
 	return nil, cloudprovider.ErrNotFound
 }
 
-func (instance *SElasticcache) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	tags, err := instance.region.ListResourceTags(APSARA_PRODUCT_KVSTORE, "INSTANCE", []string{instance.GetId()})
-	if err != nil {
-		log.Errorf(`[err:%s]instance.region.FetchResourceTags("kvs", "instance", []string{instance.GetId()})`, err.Error())
-		return nil
-	}
-	if _, ok := tags[instance.GetId()]; !ok {
-		return nil
-	}
-	data.Update(jsonutils.Marshal(tags[instance.GetId()]))
-	return data
-}
-
 func (instance *SElasticcache) GetTags() (map[string]string, error) {
 	tags, err := instance.region.ListResourceTags("kvs", "INSTANCE", []string{instance.GetId()})
 	if err != nil {
diff --git a/pkg/multicloud/apsara/image.go b/pkg/multicloud/apsara/image.go
index 7804bbd4c1..fe2dc77965 100644
--- a/pkg/multicloud/apsara/image.go
+++ b/pkg/multicloud/apsara/image.go
@@ -83,23 +83,6 @@ func (self *SImage) GetMinRamSizeMb() int {
 	return 0
 }
 
-func (self *SImage) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	if len(self.Architecture) > 0 {
-		data.Add(jsonutils.NewString(self.Architecture), "os_arch")
-	}
-	if len(self.OSType) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsType()), "os_name")
-	}
-	if len(self.Platform) > 0 {
-		data.Add(jsonutils.NewString(self.Platform), "os_distribution")
-	}
-	if len(self.OSName) > 0 {
-		data.Add(jsonutils.NewString(self.OSName), "os_version")
-	}
-	return data
-}
-
 func (self *SImage) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["os_arch"] = self.Architecture
diff --git a/pkg/multicloud/apsara/instance.go b/pkg/multicloud/apsara/instance.go
index fd6301545f..a76751a353 100644
--- a/pkg/multicloud/apsara/instance.go
+++ b/pkg/multicloud/apsara/instance.go
@@ -222,34 +222,6 @@ func (self *SInstance) GetSecurityGroupIds() ([]string, error) {
 	return self.SecurityGroupIds.SecurityGroupId, nil
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	// The pricingInfo key structure is 'RegionId::InstanceType::NetworkType::OSType::IoOptimized'
-	optimized := "optimized"
-	if !self.IoOptimized {
-		optimized = "none"
-	}
-	priceKey := fmt.Sprintf("%s::%s::%s::%s::%s", self.RegionId, self.InstanceType, self.InstanceNetworkType, self.OSType, optimized)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-
-	tags, err := self.host.zone.region.fetchTags("instance", self.InstanceId)
-	if err != nil {
-		log.Errorln(err)
-	}
-	data.Update(tags)
-
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-	if len(self.ImageId) > 0 {
-		if image, err := self.host.zone.region.GetImage(self.ImageId); err != nil {
-			log.Errorf("Failed to find image %s for instance %s", self.ImageId, self.GetName())
-		} else if meta := image.GetMetadata(); meta != nil {
-			data.Update(meta)
-		}
-	}
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	// The pricingInfo key structure is 'RegionId::InstanceType::NetworkType::OSType::IoOptimized'
diff --git a/pkg/multicloud/apsara/loadbalancer.go b/pkg/multicloud/apsara/loadbalancer.go
index 5ec2156900..470c91c672 100644
--- a/pkg/multicloud/apsara/loadbalancer.go
+++ b/pkg/multicloud/apsara/loadbalancer.go
@@ -102,20 +102,6 @@ func (lb *SLoadbalancer) GetStatus() string {
 	return api.LB_STATUS_DISABLED
 }
 
-func (lb *SLoadbalancer) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	tags, err := lb.region.ListResourceTags(APSARA_PRODUCT_SLB, "instance", []string{lb.GetId()})
-	if err != nil {
-		log.Errorf(`[err:%s]lb.region.FetchResourceTags("slb", "instance", []string{lb.GetId()})`, err.Error())
-		return nil
-	}
-	if _, ok := tags[lb.GetId()]; !ok {
-		return nil
-	}
-	data.Update(jsonutils.Marshal(tags[lb.GetId()]))
-	return data
-}
-
 func (lb *SLoadbalancer) GetTags() (map[string]string, error) {
 	tags, err := lb.region.ListResourceTags("slb", "instance", []string{lb.GetId()})
 	if err != nil {
diff --git a/pkg/multicloud/apsara/securitygroup.go b/pkg/multicloud/apsara/securitygroup.go
index 6d4d527a81..bec2daf30e 100644
--- a/pkg/multicloud/apsara/securitygroup.go
+++ b/pkg/multicloud/apsara/securitygroup.go
@@ -90,17 +90,6 @@ func (self *SSecurityGroup) GetVpcId() string {
 	return self.VpcId
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	if len(self.Tags.Tag) == 0 {
-		return nil
-	}
-	data := jsonutils.NewDict()
-	for _, value := range self.Tags.Tag {
-		data.Add(jsonutils.NewString(value.TagValue), value.TagKey)
-	}
-	return data
-}
-
 func (self *SSecurityGroup) GetTags() (map[string]string, error) {
 	tags := map[string]string{}
 	for _, value := range self.Tags.Tag {
diff --git a/pkg/multicloud/aws/disk.go b/pkg/multicloud/aws/disk.go
index bad4ef2257..6557fbb0f5 100644
--- a/pkg/multicloud/aws/disk.go
+++ b/pkg/multicloud/aws/disk.go
@@ -124,19 +124,6 @@ func (self *SDisk) IsEmulated() bool {
 	return false
 }
 
-func (self *SDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	data.Add(jsonutils.NewString(api.HYPERVISOR_AWS), "hypervisor")
-
-	return data
-}
-
-func (self *SDisk) GetSysTags() map[string]string {
-	data := map[string]string{}
-	data["hypervisor"] = api.HYPERVISOR_AWS
-	return data
-}
-
 func (self *SDisk) GetBillingType() string {
 	// todo: implement me
 	return billing.BILLING_TYPE_POSTPAID
diff --git a/pkg/multicloud/aws/image.go b/pkg/multicloud/aws/image.go
index 710116137d..62050e6ee5 100644
--- a/pkg/multicloud/aws/image.go
+++ b/pkg/multicloud/aws/image.go
@@ -273,23 +273,6 @@ func (self *SImage) IsEmulated() bool {
 	return false
 }
 
-func (self *SImage) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	if len(self.Architecture) > 0 {
-		data.Add(jsonutils.NewString(self.Architecture), "os_arch")
-	}
-	if len(self.OSType) > 0 {
-		data.Add(jsonutils.NewString(self.OSType), "os_name")
-	}
-	if len(self.OSDist) > 0 {
-		data.Add(jsonutils.NewString(self.OSDist), "os_distribution")
-	}
-	if len(self.OSVersion) > 0 {
-		data.Add(jsonutils.NewString(self.OSVersion), "os_version")
-	}
-	return data
-}
-
 func (self *SImage) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["os_arch"] = self.Architecture
diff --git a/pkg/multicloud/aws/instance.go b/pkg/multicloud/aws/instance.go
index 07c829b01c..2b568b6591 100644
--- a/pkg/multicloud/aws/instance.go
+++ b/pkg/multicloud/aws/instance.go
@@ -210,50 +210,6 @@ func (self *SInstance) GetSecurityGroupIds() ([]string, error) {
 	return self.SecurityGroupIds.SecurityGroupId, nil
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	// todo: add price_key here
-	// 格式 ：regionId::instanceType::osName::os_license::preInstall::tenancy::usageType
-	// 举例 ： cn-northwest-1::c3.2xlarge::linux::NA::NA::shared::boxusage
-	// 注意：除了空用大写NA.其他一律用小写格式
-	priceKey := fmt.Sprintf("%s::%s::%s::NA::NA::shared::boxusage", self.RegionId, self.InstanceType, strings.ToLower(self.OSType))
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-	ec2Client, err := self.host.zone.region.getEc2Client()
-	if err != nil {
-		return data
-	}
-	tags, err := FetchTags(ec2Client, self.InstanceId)
-	if err != nil {
-		log.Errorln(err)
-	} else {
-		data.Update(tags)
-	}
-
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-
-	if strings.Contains(strings.ToLower(self.OSType), "window") {
-		if loginKey, err := self.host.zone.region.getPasswordData(self.GetId()); err == nil {
-			data.Add(jsonutils.NewString(loginKey), "login_key")
-		}
-	}
-
-	// no need to sync image metadata
-	/*
-		if len(self.ImageId) > 0 {
-			image, err := self.host.zone.region.GetImage(self.ImageId)
-			if err != nil {
-				log.Errorf("Failed to find image %s for instance %s zone %s", self.ImageId, self.GetId(), self.ZoneId)
-			} else {
-				meta := image.GetMetadata()
-				if meta != nil {
-					data.Update(meta)
-				}
-			}
-		}
-	*/
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	priceKey := fmt.Sprintf("%s::%s::%s::NA::NA::shared::boxusage", self.RegionId, self.InstanceType, strings.ToLower(self.OSType))
diff --git a/pkg/multicloud/aws/loadbalancer.go b/pkg/multicloud/aws/loadbalancer.go
index 779f6d3ad1..0f02c64e59 100644
--- a/pkg/multicloud/aws/loadbalancer.go
+++ b/pkg/multicloud/aws/loadbalancer.go
@@ -113,32 +113,6 @@ func (self *SElb) IsEmulated() bool {
 	return false
 }
 
-func (self *SElb) GetMetadata() *jsonutils.JSONDict {
-	metadata := jsonutils.NewDict()
-	metadata.Add(jsonutils.NewString(self.Type), "loadbalance_type")
-
-	attrs, err := self.region.getElbAttributesById(self.GetId())
-	if err != nil {
-		log.Errorf("SElb GetMetadata %s", err)
-		return metadata
-	}
-
-	for k, v := range attrs {
-		metadata.Add(jsonutils.NewString(v), k)
-	}
-
-	tags, err := self.region.FetchElbTags(self.LoadBalancerArn)
-	if err != nil {
-		log.Errorf("self.region.FetchElbTags() %s", err)
-		return metadata
-	}
-	for k, v := range tags {
-		metadata.Add(jsonutils.NewString(v), k)
-	}
-
-	return metadata
-}
-
 func (self *SElb) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["loadbalance_type"] = self.Type
diff --git a/pkg/multicloud/aws/loadbalancerbackend.go b/pkg/multicloud/aws/loadbalancerbackend.go
index b666270a58..65704fff52 100644
--- a/pkg/multicloud/aws/loadbalancerbackend.go
+++ b/pkg/multicloud/aws/loadbalancerbackend.go
@@ -18,8 +18,6 @@ import (
 	"context"
 	"fmt"
 
-	"yunion.io/x/jsonutils"
-
 	api "yunion.io/x/onecloud/pkg/apis/compute"
 	"yunion.io/x/onecloud/pkg/multicloud"
 )
@@ -68,10 +66,6 @@ func (self *SElbBackend) IsEmulated() bool {
 	return false
 }
 
-func (self *SElbBackend) GetMetadata() *jsonutils.JSONDict {
-	return jsonutils.NewDict()
-}
-
 func (self *SElbBackend) GetProjectId() string {
 	return ""
 }
diff --git a/pkg/multicloud/aws/loadbalancerbackendgroup.go b/pkg/multicloud/aws/loadbalancerbackendgroup.go
index caff89b927..21d6e8d1e2 100644
--- a/pkg/multicloud/aws/loadbalancerbackendgroup.go
+++ b/pkg/multicloud/aws/loadbalancerbackendgroup.go
@@ -104,15 +104,6 @@ func (self *SElbBackendGroup) IsEmulated() bool {
 	return false
 }
 
-func (self *SElbBackendGroup) GetMetadata() *jsonutils.JSONDict {
-	metadata := jsonutils.NewDict()
-	metadata.Add(jsonutils.NewInt(self.Port), "port")
-	metadata.Add(jsonutils.NewString(self.TargetType), "target_type")
-	metadata.Add(jsonutils.NewString(strings.ToLower(self.HealthCheckProtocol)), "health_check_protocol")
-	metadata.Add(jsonutils.NewInt(int64(self.HealthCheckIntervalSeconds)), "health_check_interval")
-	return metadata
-}
-
 func (self *SElbBackendGroup) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["port"] = strconv.FormatInt(self.Port, 10)
diff --git a/pkg/multicloud/aws/loadbalancercert.go b/pkg/multicloud/aws/loadbalancercert.go
index c367ab2643..4f76de3cd8 100644
--- a/pkg/multicloud/aws/loadbalancercert.go
+++ b/pkg/multicloud/aws/loadbalancercert.go
@@ -81,10 +81,6 @@ func (self *SElbCertificate) IsEmulated() bool {
 	return false
 }
 
-func (self *SElbCertificate) GetMetadata() *jsonutils.JSONDict {
-	return jsonutils.NewDict()
-}
-
 func (self *SElbCertificate) GetProjectId() string {
 	return ""
 }
diff --git a/pkg/multicloud/aws/loadbalancerlistener.go b/pkg/multicloud/aws/loadbalancerlistener.go
index 2b0fa5eef5..a874401ff7 100644
--- a/pkg/multicloud/aws/loadbalancerlistener.go
+++ b/pkg/multicloud/aws/loadbalancerlistener.go
@@ -92,10 +92,6 @@ func (self *SElbListener) IsEmulated() bool {
 	return false
 }
 
-func (self *SElbListener) GetMetadata() *jsonutils.JSONDict {
-	return jsonutils.NewDict()
-}
-
 func (self *SElbListener) GetProjectId() string {
 	return ""
 }
diff --git a/pkg/multicloud/aws/loadbalancerlistenerrule.go b/pkg/multicloud/aws/loadbalancerlistenerrule.go
index be9aeec69c..461ca42c1e 100644
--- a/pkg/multicloud/aws/loadbalancerlistenerrule.go
+++ b/pkg/multicloud/aws/loadbalancerlistenerrule.go
@@ -116,10 +116,6 @@ func (self *SElbListenerRule) IsEmulated() bool {
 	return false
 }
 
-func (self *SElbListenerRule) GetMetadata() *jsonutils.JSONDict {
-	return jsonutils.NewDict()
-}
-
 func (self *SElbListenerRule) GetProjectId() string {
 	return ""
 }
diff --git a/pkg/multicloud/aws/network.go b/pkg/multicloud/aws/network.go
index 7421a9bf58..2673920744 100644
--- a/pkg/multicloud/aws/network.go
+++ b/pkg/multicloud/aws/network.go
@@ -86,27 +86,6 @@ func (self *SNetwork) IsEmulated() bool {
 	return false
 }
 
-func (self *SNetwork) GetMetadata() *jsonutils.JSONDict {
-	meta := jsonutils.NewDict()
-	routes, _ := self.wire.vpc.region.GetRouteTablesByNetworkId(self.GetId())
-	if len(routes) == 0 {
-		routes, _ = self.wire.vpc.region.GetRouteTables(self.VpcId, true)
-	}
-
-	support_eip := false
-	if len(routes) >= 1 {
-		for i := range routes[0].Routes {
-			route := routes[0].Routes[i]
-			if route.GetNextHopType() == api.Next_HOP_TYPE_INTERNET {
-				support_eip = true
-			}
-		}
-	}
-
-	meta.Set("support_eip", jsonutils.NewBool(support_eip))
-	return meta
-}
-
 func (self *SNetwork) GetSysTags() map[string]string {
 	data := map[string]string{}
 	routes, _ := self.wire.vpc.region.GetRouteTablesByNetworkId(self.GetId())
diff --git a/pkg/multicloud/aws/securitygroup.go b/pkg/multicloud/aws/securitygroup.go
index c50f8a1d13..1b2292a40f 100644
--- a/pkg/multicloud/aws/securitygroup.go
+++ b/pkg/multicloud/aws/securitygroup.go
@@ -102,26 +102,12 @@ func (self *SSecurityGroup) IsEmulated() bool {
 	return false
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	if len(self.Tags.Tag) == 0 {
-		return nil
-	}
-	data := jsonutils.NewDict()
-	for _, value := range self.Tags.Tag {
-		data.Add(jsonutils.NewString(value.TagValue), value.TagKey)
-	}
-	return data
-}
-
-func (self *SSecurityGroup) GetSysTags() map[string]string {
-	if len(self.Tags.Tag) == 0 {
-		return nil
-	}
+func (self *SSecurityGroup) GetTags() (map[string]string, error) {
 	data := map[string]string{}
 	for _, value := range self.Tags.Tag {
 		data[value.TagKey] = value.TagValue
 	}
-	return data
+	return data, nil
 }
 
 func (self *SSecurityGroup) GetDescription() string {
diff --git a/pkg/multicloud/azure/classic_disk.go b/pkg/multicloud/azure/classic_disk.go
index 5200e9e8a6..877aafa6c6 100644
--- a/pkg/multicloud/azure/classic_disk.go
+++ b/pkg/multicloud/azure/classic_disk.go
@@ -20,7 +20,6 @@ import (
 	"strings"
 	"time"
 
-	"yunion.io/x/jsonutils"
 	"yunion.io/x/pkg/errors"
 
 	billing_api "yunion.io/x/onecloud/pkg/apis/billing"
@@ -53,18 +52,6 @@ type SClassicDisk struct {
 	Properties ClassicProperties
 }
 
-func (self *SClassicDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	data.Add(jsonutils.NewString(api.HYPERVISOR_AZURE), "hypervisor")
-	return data
-}
-
-func (self *SClassicDisk) GetSysTags() map[string]string {
-	data := map[string]string{}
-	data["hypervisor"] = api.HYPERVISOR_AZURE
-	return data
-}
-
 func (self *SClassicDisk) CreateISnapshot(ctx context.Context, name, desc string) (cloudprovider.ICloudSnapshot, error) {
 	return nil, cloudprovider.ErrNotSupported
 }
diff --git a/pkg/multicloud/azure/classic_instance.go b/pkg/multicloud/azure/classic_instance.go
index 698c7b46a0..bea669d212 100644
--- a/pkg/multicloud/azure/classic_instance.go
+++ b/pkg/multicloud/azure/classic_instance.go
@@ -155,14 +155,6 @@ func (self *SClassicInstance) GetSecurityGroupIds() ([]string, error) {
 	return secgroupIds, nil
 }
 
-func (self *SClassicInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	priceKey := fmt.Sprintf("%s::%s", self.Properties.HardwareProfile.Size, self.host.zone.region.Name)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-	return data
-}
-
 func (self *SClassicInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	priceKey := fmt.Sprintf("%s::%s", self.Properties.HardwareProfile.Size, self.host.zone.region.Name)
diff --git a/pkg/multicloud/azure/classic_secruitygroup.go b/pkg/multicloud/azure/classic_secruitygroup.go
index c1c78624d2..b3a2e03e76 100644
--- a/pkg/multicloud/azure/classic_secruitygroup.go
+++ b/pkg/multicloud/azure/classic_secruitygroup.go
@@ -120,14 +120,6 @@ func (self *SClassicSecurityGroup) GetVpcId() string {
 	return "classic"
 }
 
-func (self *SClassicSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	if len(self.Tags) == 0 {
-		return nil
-	}
-	data := jsonutils.Marshal(self.Tags).(*jsonutils.JSONDict)
-	return data
-}
-
 func (self *SClassicSecurityGroup) GetTags() (map[string]string, error) {
 	return self.Tags, nil
 }
diff --git a/pkg/multicloud/azure/image.go b/pkg/multicloud/azure/image.go
index be3ffe2f8c..e86b25b7f3 100644
--- a/pkg/multicloud/azure/image.go
+++ b/pkg/multicloud/azure/image.go
@@ -103,15 +103,6 @@ func (self *SImage) GetMinRamSizeMb() int {
 	return 0
 }
 
-func (self *SImage) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	osType := string(self.Properties.StorageProfile.OsDisk.OsType)
-	if len(osType) > 0 {
-		data.Add(jsonutils.NewString(osType), "os_name")
-	}
-	return data
-}
-
 func (self *SImage) GetSysTags() map[string]string {
 	data := map[string]string{}
 	osType := string(self.Properties.StorageProfile.OsDisk.OsType)
diff --git a/pkg/multicloud/azure/instance.go b/pkg/multicloud/azure/instance.go
index 36dcf0a6a1..c145f851ea 100644
--- a/pkg/multicloud/azure/instance.go
+++ b/pkg/multicloud/azure/instance.go
@@ -226,29 +226,6 @@ func (self *SInstance) GetSecurityGroupIds() ([]string, error) {
 	return secgroupIds, nil
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	tags := jsonutils.NewDict()
-	for k, v := range self.Tags {
-		tags.Set(k, jsonutils.NewString(v))
-	}
-	data.Update(tags)
-	if osDistribution := self.Properties.StorageProfile.ImageReference.Publisher; len(osDistribution) > 0 {
-		data.Add(jsonutils.NewString(osDistribution), "os_distribution")
-	}
-	if loginAccount := self.Properties.OsProfile.AdminUsername; len(loginAccount) > 0 {
-		data.Add(jsonutils.NewString(loginAccount), "login_account")
-	}
-	if loginKey := self.Properties.OsProfile.AdminPassword; len(loginKey) > 0 {
-		data.Add(jsonutils.NewString(loginKey), "login_key")
-	}
-
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-	priceKey := fmt.Sprintf("%s::%s", self.Properties.HardwareProfile.VMSize, self.host.zone.region.Name)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	if osDistribution := self.Properties.StorageProfile.ImageReference.Publisher; len(osDistribution) > 0 {
diff --git a/pkg/multicloud/azure/securitygroup.go b/pkg/multicloud/azure/securitygroup.go
index 05455661a2..15c2ebb2d6 100644
--- a/pkg/multicloud/azure/securitygroup.go
+++ b/pkg/multicloud/azure/securitygroup.go
@@ -78,14 +78,6 @@ type SSecurityGroup struct {
 	Tags       map[string]string
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	if len(self.Tags) == 0 {
-		return nil
-	}
-	data := jsonutils.Marshal(self.Tags).(*jsonutils.JSONDict)
-	return data
-}
-
 func (self *SSecurityGroup) GetTags() (map[string]string, error) {
 	return self.Tags, nil
 }
diff --git a/pkg/multicloud/ctyun/disk.go b/pkg/multicloud/ctyun/disk.go
index b7f2e4a510..2329aeab89 100644
--- a/pkg/multicloud/ctyun/disk.go
+++ b/pkg/multicloud/ctyun/disk.go
@@ -186,13 +186,6 @@ func (self *SDisk) IsEmulated() bool {
 	return false
 }
 
-func (self *SDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	data.Add(jsonutils.NewString(api.HYPERVISOR_CTYUN), "hypervisor")
-
-	return data
-}
-
 func (self *SDisk) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["hypervisor"] = api.HYPERVISOR_CTYUN
diff --git a/pkg/multicloud/ctyun/image.go b/pkg/multicloud/ctyun/image.go
index d2cc99aa25..087e68b547 100644
--- a/pkg/multicloud/ctyun/image.go
+++ b/pkg/multicloud/ctyun/image.go
@@ -86,23 +86,6 @@ func (self *SImage) IsEmulated() bool {
 	return false
 }
 
-func (self *SImage) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	if self.OSBit > 0 {
-		data.Add(jsonutils.NewString(self.GetOsArch()), "os_arch")
-	}
-	if len(self.OSType) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsType()), "os_name")
-	}
-	if len(self.Platform) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsDist()), "os_distribution")
-	}
-
-	data.Add(jsonutils.NewString(self.GetOsVersion()), "os_version")
-
-	return data
-}
-
 func (self *SImage) GetSysTags() map[string]string {
 	data := map[string]string{}
 	if self.OSBit > 0 {
diff --git a/pkg/multicloud/ctyun/instance.go b/pkg/multicloud/ctyun/instance.go
index 8f66fad0bf..5562ac569d 100644
--- a/pkg/multicloud/ctyun/instance.go
+++ b/pkg/multicloud/ctyun/instance.go
@@ -181,26 +181,6 @@ func (self *SInstance) IsEmulated() bool {
 	return false
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	lowerOs := self.GetOSType()
-	if strings.HasPrefix(lowerOs, "win") {
-		lowerOs = "win"
-	}
-	priceKey := fmt.Sprintf("%s::%s::%s", self.host.zone.region.GetId(), self.GetInstanceType(), lowerOs)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-
-	image, _ := self.GetImage()
-	if image != nil {
-		if meta := image.GetMetadata(); meta != nil {
-			data.Update(meta)
-		}
-	}
-
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	lowerOs := self.GetOSType()
diff --git a/pkg/multicloud/ctyun/securitygroup.go b/pkg/multicloud/ctyun/securitygroup.go
index eb03dc2e87..74aeabe5b3 100644
--- a/pkg/multicloud/ctyun/securitygroup.go
+++ b/pkg/multicloud/ctyun/securitygroup.go
@@ -135,10 +135,6 @@ func (self *SSecurityGroup) IsEmulated() bool {
 	return false
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	return jsonutils.NewDict()
-}
-
 func (self *SSecurityGroup) GetDescription() string {
 	return ""
 }
diff --git a/pkg/multicloud/esxi/virtualmachine.go b/pkg/multicloud/esxi/virtualmachine.go
index 3b390c1c65..88616322c7 100644
--- a/pkg/multicloud/esxi/virtualmachine.go
+++ b/pkg/multicloud/esxi/virtualmachine.go
@@ -97,27 +97,6 @@ func (self *SVirtualMachine) GetSecurityGroupIds() ([]string, error) {
 	return []string{}, cloudprovider.ErrNotSupported
 }
 
-func (self *SVirtualMachine) GetMetadata() *jsonutils.JSONDict {
-	meta := jsonutils.NewDict()
-	meta.Set("datacenter", jsonutils.NewString(self.GetDatacenterPathString()))
-	rp, _ := self.getResourcePool()
-	if rp != nil {
-		rpPath := rp.GetPath()
-		rpOffset := -1
-		for i := range rpPath {
-			if rpPath[i] == "Resources" {
-				if i > 0 {
-					meta.Set("cluster", jsonutils.NewString(rpPath[i-1]))
-					rpOffset = i
-				}
-			} else if rpOffset >= 0 && i > rpOffset {
-				meta.Set(fmt.Sprintf("pool%d", i-rpOffset-1), jsonutils.NewString(rpPath[i]))
-			}
-		}
-	}
-	return meta
-}
-
 func (self *SVirtualMachine) GetSysTags() map[string]string {
 	meta := map[string]string{}
 	meta["datacenter"] = self.GetDatacenterPathString()
diff --git a/pkg/multicloud/huawei/disk.go b/pkg/multicloud/huawei/disk.go
index 8655224f62..4f90c451cc 100644
--- a/pkg/multicloud/huawei/disk.go
+++ b/pkg/multicloud/huawei/disk.go
@@ -184,21 +184,6 @@ func (self *SDisk) IsEmulated() bool {
 	return false
 }
 
-func (self *SDisk) GetMetadata() *jsonutils.JSONDict {
-	// todo: add price key
-	data := jsonutils.NewDict()
-	data.Add(jsonutils.NewString(api.HYPERVISOR_HUAWEI), "hypervisor")
-
-	return data
-}
-
-func (self *SDisk) GetSysTags() map[string]string {
-	data := map[string]string{}
-	data["hypervisor"] = api.HYPERVISOR_HUAWEI
-
-	return data
-}
-
 func (self *SDisk) getResourceDetails() *SResourceDetail {
 	if self.details != nil {
 		return self.details
diff --git a/pkg/multicloud/huawei/image.go b/pkg/multicloud/huawei/image.go
index f3861146b5..af498dfa63 100644
--- a/pkg/multicloud/huawei/image.go
+++ b/pkg/multicloud/huawei/image.go
@@ -204,23 +204,6 @@ func (self *SImage) IsEmulated() bool {
 	return false
 }
 
-func (self *SImage) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	if len(self.OSBit) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsArch()), "os_arch")
-	}
-	if len(self.OSType) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsType()), "os_name")
-	}
-	if len(self.Platform) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsDist()), "os_distribution")
-	}
-	if len(self.OSVersion) > 0 {
-		data.Add(jsonutils.NewString(self.GetOsVersion()), "os_version")
-	}
-	return data
-}
-
 func (self *SImage) GetSysTags() map[string]string {
 	data := map[string]string{}
 	if len(self.OSBit) > 0 {
diff --git a/pkg/multicloud/huawei/instance.go b/pkg/multicloud/huawei/instance.go
index 16396f621d..cb5784c070 100644
--- a/pkg/multicloud/huawei/instance.go
+++ b/pkg/multicloud/huawei/instance.go
@@ -253,26 +253,6 @@ func (self *SInstance) GetSecurityGroupIds() ([]string, error) {
 	return self.host.zone.region.GetInstanceSecrityGroupIds(self.GetId())
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	// cn-north-1::et2.2xlarge.16::win
-	lowerOs := self.GetOSType()
-	if strings.HasPrefix(lowerOs, "win") {
-		lowerOs = "win"
-	}
-	priceKey := fmt.Sprintf("%s::%s::%s", self.host.zone.region.GetId(), self.GetInstanceType(), lowerOs)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-	if len(self.Metadata.MeteringImageID) > 0 {
-		if image, err := self.host.zone.region.GetImage(self.Metadata.MeteringImageID); err != nil {
-			log.Errorf("Failed to find image %s for instance %s zone %s", self.Metadata.MeteringImageID, self.GetId(), self.OSEXTAZAvailabilityZone)
-		} else if meta := image.GetMetadata(); meta != nil {
-			data.Update(meta)
-		}
-	}
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	// cn-north-1::et2.2xlarge.16::win
diff --git a/pkg/multicloud/huawei/loadbalancer.go b/pkg/multicloud/huawei/loadbalancer.go
index 9445122efe..de9662d281 100644
--- a/pkg/multicloud/huawei/loadbalancer.go
+++ b/pkg/multicloud/huawei/loadbalancer.go
@@ -127,12 +127,6 @@ func (self *SLoadbalancer) IsEmulated() bool {
 	return false
 }
 
-func (self *SLoadbalancer) GetMetadata() *jsonutils.JSONDict {
-	meta := jsonutils.NewDict()
-
-	return meta
-}
-
 func (self *SLoadbalancer) GetProjectId() string {
 	return self.ProjectID
 }
diff --git a/pkg/multicloud/huawei/securitygroup.go b/pkg/multicloud/huawei/securitygroup.go
index 3c06ea7577..6b11d58aaa 100644
--- a/pkg/multicloud/huawei/securitygroup.go
+++ b/pkg/multicloud/huawei/securitygroup.go
@@ -124,11 +124,6 @@ func (self *SSecurityGroup) IsEmulated() bool {
 	return false
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	return data
-}
-
 func (self *SSecurityGroup) GetDescription() string {
 	if self.Description == self.VpcID {
 		return ""
diff --git a/pkg/multicloud/openstack/disk.go b/pkg/multicloud/openstack/disk.go
index fdabbe8d34..1a71bcbea2 100644
--- a/pkg/multicloud/openstack/disk.go
+++ b/pkg/multicloud/openstack/disk.go
@@ -120,19 +120,6 @@ type SDisk struct {
 	VolumeType          string
 }
 
-func (disk *SDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	data.Add(jsonutils.NewString(api.HYPERVISOR_OPENSTACK), "hypervisor")
-	return data
-}
-
-func (disk *SDisk) GetSysTags() map[string]string {
-	data := map[string]string{}
-	data["hypervisor"] = api.HYPERVISOR_OPENSTACK
-	return data
-}
-
 func (region *SRegion) GetDisks() ([]SDisk, error) {
 	disks := []SDisk{}
 	resource := "/volumes/detail"
diff --git a/pkg/multicloud/qcloud/disk.go b/pkg/multicloud/qcloud/disk.go
index 7ea7b52000..027de633e2 100644
--- a/pkg/multicloud/qcloud/disk.go
+++ b/pkg/multicloud/qcloud/disk.go
@@ -84,28 +84,6 @@ func (v SDiskSet) Less(i, j int) bool {
 	return false
 }
 
-func (self *SDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	// // The pricingInfo key structure is 'RegionId::DiskCategory::DiskType
-	// priceKey := fmt.Sprintf("%s::%s::%s", self.RegionId, self.Category, self.Type)
-	// data.Add(jsonutils.NewString(priceKey), "price_key")
-
-	data.Add(jsonutils.NewString(api.HYPERVISOR_QCLOUD), "hypervisor")
-	return data
-}
-
-func (self *SDisk) GetSysTags() map[string]string {
-	data := map[string]string{}
-
-	// // The pricingInfo key structure is 'RegionId::DiskCategory::DiskType
-	// priceKey := fmt.Sprintf("%s::%s::%s", self.RegionId, self.Category, self.Type)
-	// data.Add(jsonutils.NewString(priceKey), "price_key")
-
-	data["hypervisor"] = api.HYPERVISOR_QCLOUD
-	return data
-}
-
 func (self *SRegion) GetDisks(instanceId string, zoneId string, category string, diskIds []string, offset int, limit int) ([]SDisk, int, error) {
 	if limit > 50 || limit <= 0 {
 		limit = 50
diff --git a/pkg/multicloud/qcloud/elasticcache_instance.go b/pkg/multicloud/qcloud/elasticcache_instance.go
index b31663187c..0b9d581aee 100644
--- a/pkg/multicloud/qcloud/elasticcache_instance.go
+++ b/pkg/multicloud/qcloud/elasticcache_instance.go
@@ -241,25 +241,6 @@ func (self *SElasticcache) IsEmulated() bool {
 	return false
 }
 
-func (self *SElasticcache) GetMetadata() *jsonutils.JSONDict {
-	meta := jsonutils.NewDict()
-	tags, err := self.region.FetchResourceTags("redis", "instance", []string{self.GetId()})
-	if err != nil {
-		log.Errorf(`[err:%s]self.region.FetchResourceTags("cdb", "instanceId", []string{self.GetId()})`, err.Error())
-		return nil
-	}
-	if _, ok := tags[self.GetId()]; !ok {
-		return meta
-	}
-	resourceTag := tags[self.GetId()]
-	if resourceTag != nil {
-		for k, v := range *resourceTag {
-			meta.Add(jsonutils.NewString(v), k)
-		}
-	}
-	return meta
-}
-
 func (self *SElasticcache) GetTags() (map[string]string, error) {
 	tags, err := self.region.FetchResourceTags("redis", "instance", []string{self.GetId()})
 	if err != nil {
diff --git a/pkg/multicloud/qcloud/instance.go b/pkg/multicloud/qcloud/instance.go
index 656b14711c..8c548e4125 100644
--- a/pkg/multicloud/qcloud/instance.go
+++ b/pkg/multicloud/qcloud/instance.go
@@ -166,32 +166,6 @@ func (self *SInstance) GetSecurityGroupIds() ([]string, error) {
 	return self.SecurityGroupIds, nil
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	if self.image == nil {
-		image, err := self.host.zone.region.GetImage(self.ImageId)
-		if err == nil {
-			self.image = image
-		}
-	}
-
-	if self.image != nil {
-		data.Add(jsonutils.NewString(self.image.OsName), "os_distribution")
-	}
-
-	priceKey := fmt.Sprintf("%s::%s", self.host.zone.Zone, self.InstanceType)
-	data.Add(jsonutils.NewString(priceKey), "price_key")
-
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-
-	tags, _ := self.getCloudMetadata()
-	for k, v := range tags {
-		data.Add(jsonutils.NewString(v), k)
-	}
-
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	if self.image == nil {
diff --git a/pkg/multicloud/qcloud/loadbalancer.go b/pkg/multicloud/qcloud/loadbalancer.go
index 92ad2152e5..fec993da45 100644
--- a/pkg/multicloud/qcloud/loadbalancer.go
+++ b/pkg/multicloud/qcloud/loadbalancer.go
@@ -272,27 +272,22 @@ func (self *SLoadbalancer) IsEmulated() bool {
 	return false
 }
 
-func (self *SLoadbalancer) GetMetadata() *jsonutils.JSONDict {
-	meta := jsonutils.NewDict()
-	meta.Add(jsonutils.NewInt(int64(self.Forward)), "Forward")
-	meta.Add(jsonutils.NewInt(self.OpenBGP), "OpenBGP")
-	meta.Add(jsonutils.NewString(self.Domain), "Domain")
-	meta.Add(jsonutils.NewInt(self.ProjectID), "ProjectID")
+func (self *SLoadbalancer) GetTags() (map[string]string, error) {
 	tags, err := self.region.FetchResourceTags("clb", "clb", []string{self.GetId()})
 	if err != nil {
-		log.Errorf(`[err:%s]self.region.FetchResourceTags("clb", "clb", []string{self.GetId()})`, err.Error())
-		return nil
+		return nil, errors.Wrapf(err, "FetchResourceTags")
 	}
+	ret := map[string]string{}
 	if _, ok := tags[self.GetId()]; !ok {
-		return meta
+		return ret, nil
 	}
 	resourceTag := tags[self.GetId()]
 	if resourceTag != nil {
 		for k, v := range *resourceTag {
-			meta.Add(jsonutils.NewString(v), k)
+			ret[k] = v
 		}
 	}
-	return meta
+	return ret, nil
 }
 
 func (self *SLoadbalancer) GetSysTags() map[string]string {
diff --git a/pkg/multicloud/qcloud/localdisk.go b/pkg/multicloud/qcloud/localdisk.go
index 5ebccc9dc1..7644cbc815 100644
--- a/pkg/multicloud/qcloud/localdisk.go
+++ b/pkg/multicloud/qcloud/localdisk.go
@@ -18,8 +18,6 @@ import (
 	"context"
 	"time"
 
-	"yunion.io/x/jsonutils"
-
 	api "yunion.io/x/onecloud/pkg/apis/compute"
 	"yunion.io/x/onecloud/pkg/cloudprovider"
 	"yunion.io/x/onecloud/pkg/multicloud"
@@ -35,12 +33,6 @@ type SLocalDisk struct {
 	DiskUsage string
 }
 
-func (self *SLocalDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	data.Add(jsonutils.NewString(api.HYPERVISOR_QCLOUD), "hypervisor")
-	return data
-}
-
 func (self *SLocalDisk) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["hypervisor"] = api.HYPERVISOR_QCLOUD
diff --git a/pkg/multicloud/qcloud/rds_mysql.go b/pkg/multicloud/qcloud/rds_mysql.go
index ebab181b06..abd9cfe53c 100644
--- a/pkg/multicloud/qcloud/rds_mysql.go
+++ b/pkg/multicloud/qcloud/rds_mysql.go
@@ -897,25 +897,6 @@ func (self *SMySQLInstance) CreateIBackup(opts *cloudprovider.SDBInstanceBackupC
 	return self.region.CreateMySQLBackup(self.InstanceId, tables)
 }
 
-func (self *SMySQLInstance) GetMetadata() *jsonutils.JSONDict {
-	meta := jsonutils.NewDict()
-	tags, err := self.region.FetchResourceTags("cdb", "instanceId", []string{self.GetId()})
-	if err != nil {
-		log.Errorf(`[err:%s]self.region.FetchResourceTags("cdb", "instanceId", []string{self.GetId()})`, err.Error())
-		return nil
-	}
-	if _, ok := tags[self.GetId()]; !ok {
-		return meta
-	}
-	resourceTag := tags[self.GetId()]
-	if resourceTag != nil {
-		for k, v := range *resourceTag {
-			meta.Add(jsonutils.NewString(v), k)
-		}
-	}
-	return meta
-}
-
 func (self *SMySQLInstance) GetTags() (map[string]string, error) {
 	tags, err := self.region.FetchResourceTags("cdb", "instanceId", []string{self.GetId()})
 	if err != nil {
diff --git a/pkg/multicloud/ucloud/disk.go b/pkg/multicloud/ucloud/disk.go
index 99014d375b..fdab49543f 100644
--- a/pkg/multicloud/ucloud/disk.go
+++ b/pkg/multicloud/ucloud/disk.go
@@ -115,14 +115,6 @@ func (self *SDisk) IsEmulated() bool {
 	return false
 }
 
-func (self *SDisk) GetMetadata() *jsonutils.JSONDict {
-	// todo: add price key
-	data := jsonutils.NewDict()
-	data.Add(jsonutils.NewString(api.HYPERVISOR_UCLOUD), "hypervisor")
-
-	return data
-}
-
 func (self *SDisk) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["hypervisor"] = api.HYPERVISOR_UCLOUD
diff --git a/pkg/multicloud/ucloud/image.go b/pkg/multicloud/ucloud/image.go
index cd5744003f..fc0fc30cde 100644
--- a/pkg/multicloud/ucloud/image.go
+++ b/pkg/multicloud/ucloud/image.go
@@ -92,24 +92,6 @@ func (self *SImage) IsEmulated() bool {
 	return false
 }
 
-func (self *SImage) GetMetadata() *jsonutils.JSONDict {
-	imageInfo := imagetools.NormalizeImageInfo(self.ImageName, "", "", "", "")
-	data := jsonutils.NewDict()
-	if len(imageInfo.OsArch) > 0 {
-		data.Add(jsonutils.NewString(imageInfo.OsArch), "os_arch")
-	}
-	if len(imageInfo.OsType) > 0 {
-		data.Add(jsonutils.NewString(imageInfo.OsType), "os_name")
-	}
-	if len(imageInfo.OsDistro) > 0 {
-		data.Add(jsonutils.NewString(imageInfo.OsDistro), "os_distribution")
-	}
-	if len(imageInfo.OsVersion) > 0 {
-		data.Add(jsonutils.NewString(imageInfo.OsVersion), "os_version")
-	}
-	return data
-}
-
 func (self *SImage) GetSysTags() map[string]string {
 	imageInfo := imagetools.NormalizeImageInfo(self.ImageName, "", "", "", "")
 	data := map[string]string{}
diff --git a/pkg/multicloud/ucloud/instance.go b/pkg/multicloud/ucloud/instance.go
index 71390249bb..d8f2024b51 100644
--- a/pkg/multicloud/ucloud/instance.go
+++ b/pkg/multicloud/ucloud/instance.go
@@ -182,21 +182,6 @@ func (self *SInstance) IsEmulated() bool {
 	return false
 }
 
-func (self *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	// todo: add price key here
-	data.Add(jsonutils.NewString(self.host.zone.GetGlobalId()), "zone_ext_id")
-	if len(self.BasicImageID) > 0 {
-		if image, err := self.host.zone.region.GetImage(self.BasicImageID); err != nil {
-			log.Errorf("Failed to find image %s for instance %s", self.BasicImageID, self.GetName())
-		} else if meta := image.GetMetadata(); meta != nil {
-			data.Update(meta)
-		}
-	}
-
-	return data
-}
-
 func (self *SInstance) GetSysTags() map[string]string {
 	data := map[string]string{}
 	// todo: add price key here
diff --git a/pkg/multicloud/ucloud/project.go b/pkg/multicloud/ucloud/project.go
index 36ed7a66f9..b95a304f3a 100644
--- a/pkg/multicloud/ucloud/project.go
+++ b/pkg/multicloud/ucloud/project.go
@@ -15,8 +15,6 @@
 package ucloud
 
 import (
-	"yunion.io/x/jsonutils"
-
 	"yunion.io/x/onecloud/pkg/multicloud"
 )
 
@@ -57,10 +55,6 @@ func (self *SProject) IsEmulated() bool {
 	return false
 }
 
-func (self *SProject) GetMetadata() *jsonutils.JSONDict {
-	return jsonutils.NewDict()
-}
-
 func (self *SUcloudClient) FetchProjects() ([]SProject, error) {
 	params := NewUcloudParams()
 	projects := make([]SProject, 0)
diff --git a/pkg/multicloud/ucloud/securitygroup.go b/pkg/multicloud/ucloud/securitygroup.go
index 2ea0bb98eb..4efc4fcf85 100644
--- a/pkg/multicloud/ucloud/securitygroup.go
+++ b/pkg/multicloud/ucloud/securitygroup.go
@@ -91,11 +91,6 @@ func (self *SSecurityGroup) IsEmulated() bool {
 	return false
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	return data
-}
-
 func (self *SSecurityGroup) GetDescription() string {
 	return self.Remark
 }
diff --git a/pkg/multicloud/zstack/disk.go b/pkg/multicloud/zstack/disk.go
index d1dea8badc..8116fd1d9c 100644
--- a/pkg/multicloud/zstack/disk.go
+++ b/pkg/multicloud/zstack/disk.go
@@ -135,13 +135,6 @@ func (region *SRegion) GetDisks(storageId string, diskIds []string, diskType str
 	return disks, region.client.listAll("volumes", params, &disks)
 }
 
-func (disk *SDisk) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	data.Add(jsonutils.NewString(api.HYPERVISOR_ZSTACK), "hypervisor")
-	return data
-}
-
 func (disk *SDisk) GetSysTags() map[string]string {
 	data := map[string]string{}
 	data["hypervisor"] = api.HYPERVISOR_ZSTACK
diff --git a/pkg/multicloud/zstack/image.go b/pkg/multicloud/zstack/image.go
index b975aece95..7f6e1dc88e 100644
--- a/pkg/multicloud/zstack/image.go
+++ b/pkg/multicloud/zstack/image.go
@@ -74,11 +74,6 @@ func (image *SImage) GetMinRamSizeMb() int {
 	return 0
 }
 
-func (image *SImage) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	return data
-}
-
 func (image *SImage) GetId() string {
 	return image.UUID
 }
diff --git a/pkg/multicloud/zstack/instance.go b/pkg/multicloud/zstack/instance.go
index c113c8cd5c..f3aa08dabc 100644
--- a/pkg/multicloud/zstack/instance.go
+++ b/pkg/multicloud/zstack/instance.go
@@ -107,12 +107,6 @@ func (instance *SInstance) GetSecurityGroupIds() ([]string, error) {
 	return ids, nil
 }
 
-func (instance *SInstance) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-
-	return data
-}
-
 func (instance *SInstance) GetIHost() cloudprovider.ICloudHost {
 	return instance.host
 }
diff --git a/pkg/multicloud/zstack/securitygroup.go b/pkg/multicloud/zstack/securitygroup.go
index 8d67006263..1d3ecbc50a 100644
--- a/pkg/multicloud/zstack/securitygroup.go
+++ b/pkg/multicloud/zstack/securitygroup.go
@@ -86,11 +86,6 @@ func (self *SSecurityGroup) GetVpcId() string {
 	return api.NORMAL_VPC_ID
 }
 
-func (self *SSecurityGroup) GetMetadata() *jsonutils.JSONDict {
-	data := jsonutils.NewDict()
-	return data
-}
-
 func (self *SSecurityGroup) GetId() string {
 	return self.UUID
 }