Merge pull request #9115 from yousong/automated-cherry-pick-of-#9109-upstream-release-3.1

Automated cherry pick of #9109: fix(host): make sure it won't match vpc guests by accident
fix(host): make sure it won't match vpc guests by accident
2026-09-20 16:13:56 +08:00 · 2020-12-01 20:22:08 +08:00 · 2020-12-01 15:34:34 +08:00 · 2020-10-28 00:57:41 +08:00 · 2020-10-27 21:35:55 +08:00 · 2020-10-27 21:35:55 +08:00
2201 changed files with 34837 additions and 274214 deletions
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -13,7 +13,7 @@ jobs:
      - checkout
      - run:
          command: |
-            make -j3 check
+            make check
  build:
    requires:
      - check
@@ -38,7 +38,7 @@ jobs:
          command: |
            targets="$(circleci tests glob "cmd/*" | grep -v cmd/host-image | circleci tests split)"
            echo $targets | tr ' ' '\n'
-            make -j3 $targets
+            make $targets
            ls -lh _output/bin/
      - save_cache:
          key: onecloud-build-cache-{{ checksum "go.mod" }}-{{ checksum "Makefile" }}
@@ -55,7 +55,6 @@ jobs:
        environment:
          GO111MODULE: "on"
    working_directory: /root/go/onecloud
-    parallelism: 4
    steps:
      - checkout
      - restore_cache:
--- a/.github/pull_request_template.md
+++ b/.github/pull_request_template.md
@@ -1,15 +1,9 @@
 **这个 PR 实现什么功能/修复什么问题**:

-<!--
- [ ] 功能、bugfix描述
- [ ] 冒烟测试
- [ ] 单元测试编写
-->
-
 **是否需要 backport 到之前的 release 分支**:
-
 <!--
 如果不需要，填写 "NONE".
 如果需要，就以下面 item 的格式写 release 分支名，并提交对应的 cherry-pick PR:
- release/3.1
+- release/2.8.0
+- release/2.6.0
 -->
--- a/.github/workflows/docker.yml
+++ b/.github/workflows/docker.yml
@@ -36,41 +36,11 @@ jobs:
        make cmd/ansibleserver cmd/cloudnet cmd/notify
        make cmd/host-deployer && ./scripts/bundle_libraries.sh _output/bin/bundles/host-deployer _output/bin/host-deployer
        make cmd/baremetal-agent && ./scripts/bundle_libraries.sh _output/bin/bundles/baremetal-agent _output/bin/baremetal-agent
-        make cmd/cloudevent cmd/devtool cmd/cloudid
+        make cmd/cloudevent cmd/devtool
        make cmd/*cli
        make cmd/esxi-agent
        make docker-alpine-build F='-j4 cmd/host cmd/vpcagent cmd/region-dns'

-    - name: Image region
-      uses: elgohr/Publish-Docker-Github-Action@master
-      with:
-        name: registry.cn-beijing.aliyuncs.com/yunionio/region
-        username: ${{ secrets.DOCKER_USERNAME }}
-        password: ${{ secrets.DOCKER_PASSWORD }}
-        registry: registry.cn-beijing.aliyuncs.com
-        snapshot: true
-        dockerfile: build/docker/Dockerfile.region
-
-    - name: Image host
-      uses: elgohr/Publish-Docker-Github-Action@master
-      with:
-        name: registry.cn-beijing.aliyuncs.com/yunionio/host
-        username: ${{ secrets.DOCKER_USERNAME }}
-        password: ${{ secrets.DOCKER_PASSWORD }}
-        registry: registry.cn-beijing.aliyuncs.com
-        snapshot: true
-        dockerfile: build/docker/Dockerfile.host
-
-    - name: Image host-deployer
-      uses: elgohr/Publish-Docker-Github-Action@master
-      with:
-        name: registry.cn-beijing.aliyuncs.com/yunionio/host-deployer
-        username: ${{ secrets.DOCKER_USERNAME }}
-        password: ${{ secrets.DOCKER_PASSWORD }}
-        registry: registry.cn-beijing.aliyuncs.com
-        snapshot: true
-        dockerfile: build/docker/Dockerfile.host-deployer
-
    - name: Image baremetal-agent
      uses: elgohr/Publish-Docker-Github-Action@master
      with:
@@ -131,6 +101,16 @@ jobs:
        snapshot: true
        dockerfile: build/docker/Dockerfile.logger

+    - name: Image region
+      uses: elgohr/Publish-Docker-Github-Action@master
+      with:
+        name: registry.cn-beijing.aliyuncs.com/yunionio/region
+        username: ${{ secrets.DOCKER_USERNAME }}
+        password: ${{ secrets.DOCKER_PASSWORD }}
+        registry: registry.cn-beijing.aliyuncs.com
+        snapshot: true
+        dockerfile: build/docker/Dockerfile.region
+
    - name: Image scheduler
      uses: elgohr/Publish-Docker-Github-Action@master
      with:
@@ -201,6 +181,26 @@ jobs:
        snapshot: true
        dockerfile: build/docker/Dockerfile.notify

+    - name: Image host
+      uses: elgohr/Publish-Docker-Github-Action@master
+      with:
+        name: registry.cn-beijing.aliyuncs.com/yunionio/host
+        username: ${{ secrets.DOCKER_USERNAME }}
+        password: ${{ secrets.DOCKER_PASSWORD }}
+        registry: registry.cn-beijing.aliyuncs.com
+        snapshot: true
+        dockerfile: build/docker/Dockerfile.host
+
+    - name: Image host-deployer
+      uses: elgohr/Publish-Docker-Github-Action@master
+      with:
+        name: registry.cn-beijing.aliyuncs.com/yunionio/host-deployer
+        username: ${{ secrets.DOCKER_USERNAME }}
+        password: ${{ secrets.DOCKER_PASSWORD }}
+        registry: registry.cn-beijing.aliyuncs.com
+        snapshot: true
+        dockerfile: build/docker/Dockerfile.host-deployer
+
    - name: Image cloudevent
      uses: elgohr/Publish-Docker-Github-Action@master
      with:
@@ -211,16 +211,6 @@ jobs:
        snapshot: true
        dockerfile: build/docker/Dockerfile.cloudevent

-    - name: Image cloudid
-      uses: elgohr/Publish-Docker-Github-Action@master
-      with:
-        name: registry.cn-beijing.aliyuncs.com/yunionio/cloudid
-        username: ${{ secrets.DOCKER_USERNAME }}
-        password: ${{ secrets.DOCKER_PASSWORD }}
-        registry: registry.cn-beijing.aliyuncs.com
-        snapshot: true
-        dockerfile: build/docker/Dockerfile.cloudid
-
    - name: Image devtool
      uses: elgohr/Publish-Docker-Github-Action@master
      with:
--- a/.gitignore
+++ b/.gitignore
@@ -30,10 +30,7 @@ _output
 /relay
 /structargtest
 .idea
-.vscode
 GPATH
 GRTAGS
 GTAGS
 pkg/generated
-
-*.orig
--- a/22
+++ b/22
@@ -10,7 +10,7 @@ BUILD_SCRIPT := $(ROOT_DIR)/build/build.sh
 ifeq ($(ONECLOUD_CI_BUILD),)
 	GIT_COMMIT := $(shell git rev-parse --short HEAD)
 	GIT_BRANCH := $(shell git name-rev --name-only HEAD)
-	GIT_VERSION := $(shell git describe --always --tags --abbrev=14 $(GIT_COMMIT)^{commit})
+	GIT_VERSION := $(shell git describe --tags --abbrev=14 $(GIT_COMMIT)^{commit})
 	GIT_TREE_STATE := $(shell s=`git status --porcelain 2>/dev/null`; if [ -z "$$s" ]; then echo "clean"; else echo "dirty"; fi)
 	BUILD_DATE := $(shell date -u +'%Y-%m-%dT%H:%M:%SZ')
 else
@@ -51,7 +51,6 @@ ifdef LIBQEMUIO_PATH
 		X_CGO_LDFLAGS := ${CGO_LDFLAGS_ENV} -laio -lqemuio -lpthread  -L ${LIBQEMUIO_PATH}/src
 endif

-export GOOS ?= linux
 export GO111MODULE:=on
 export CGO_CFLAGS = ${X_CGO_CFLAGS}
 export CGO_LDFLAGS = ${X_CGO_LDFLAGS}
@@ -76,7 +75,7 @@ install: prepare_dir


 gencopyright:
-	@bash scripts/gencopyright.sh pkg cmd
+	@sh scripts/gencopyright.sh pkg cmd

 test:
 	@go test $(GO_BUILD_FLAGS) $(shell go list ./... | egrep -v 'host-image|hostimage')
@@ -163,15 +162,9 @@ goimports-check:
 	fi
 .PHONY: goimports-check

-vet-check:
-	./scripts/vet.sh gen
-	./scripts/vet.sh chk
-.PHONY: vet-check
-
 check: fmt-check
 check: gendocgo-check
 check: goimports-check
-check: vet-check
 .PHONY: check


@@ -194,7 +187,7 @@ dep:
 	@$(MAKE) mod

 mod:
-	go get -d $(patsubst %,%@master,$(shell GO111MODULE=on go mod edit -print  | sed -n -e 's|.*\(yunion.io/x/[a-z].*\) v.*|\1|p'))
+	go get $(patsubst %,%@master,$(shell GO111MODULE=on go mod edit -print  | sed -n -e 's|.*\(yunion.io/x/[a-z].*\) v.*|\1|p'))
 	go mod tidy
 	go mod vendor -v

@@ -297,15 +290,10 @@ gen-swagger: gen-swagger-check
 	$(ROOT_DIR)/scripts/codegen.py swagger-code
 	$(ROOT_DIR)/scripts/codegen.py swagger-yaml

-swagger-serve-only:
+swagger-serve: gen-swagger
 	$(ROOT_DIR)/scripts/codegen.py swagger-serve

-swagger-serve: gen-model-api gen-swagger swagger-serve-only
-
-swagger-site: gen-model-api gen-swagger
-	$(ROOT_DIR)/scripts/codegen.py swagger-site
-
-.PHONY: gen-model-api-check gen-model-api gen-swagger-check gen-swagger swagger-serve swagger-site
+.PHONY: gen-model-api-check gen-model-api gen-swagger-check gen-swagger swagger-serve

 REGISTRY ?= "registry.cn-beijing.aliyuncs.com/yunionio"
 VERSION ?= $(shell git describe --exact-match 2> /dev/null || \
--- a/README.md
+++ b/README.md
@@ -1,7 +1,7 @@
 # Yunion OneCloud

 [![CircleCI](https://circleci.com/gh/yunionio/onecloud.svg?style=svg)](https://circleci.com/gh/yunionio/onecloud) 
-[![Build Status](https://travis-ci.com/yunionio/onecloud.svg?branch=master)](https://travis-ci.com/yunionio/onecloud/branches) 
+[![Build Status](https://travis-ci.com/yunionio/onecloud.svg?branch=master)](https://travis-ci.org/yunionio/onecloud) 
 [![Go Report Card](https://goreportcard.com/badge/github.com/yunionio/onecloud)](https://goreportcard.com/report/github.com/yunionio/onecloud) 

 ### [README in Chinese](./README-CN.md)
@@ -40,7 +40,7 @@ Please refers to [install & deployment](https://docs.yunion.io/docs/setup/) (cur

 - [Yunion OneCloud Documents](https://docs.yunion.io/)

- [Swagger API](https://docs.yunion.cn/apiv2/)
+- [Swagger API](https://docs.yunion.cn/api/)

 ## Architecture

--- a/build/cloudid/vars
+++ b/build/cloudid/vars
@@ -1 +0,0 @@
-DESCRIPTION="Yunion CloudId"
--- a/build/docker/Dockerfile.ansibleserver
+++ b/build/docker/Dockerfile.ansibleserver
@@ -1,3 +1,3 @@
-FROM registry.cn-beijing.aliyuncs.com/yunionio/ansibleserver-base:1.0.0
+FROM registry.cn-beijing.aliyuncs.com/yunionio/ansibleserver-base:latest

-ADD ./_output/alpine-build/bin/ansibleserver /opt/yunion/bin/ansibleserver
+ADD ./_output/bin/ansibleserver /opt/yunion/bin/ansibleserver
--- a/build/docker/Dockerfile.ansibleserver-base
+++ b/build/docker/Dockerfile.ansibleserver-base
@@ -1,12 +1,13 @@
-FROM alpine:3.11.6
+FROM frolvlad/alpine-glibc:glibc-2.28

-MAINTAINER "Yousong Zhou <zhouyousong@yunionyun.com>"
+MAINTAINER "Zexi Li <lizexi@yunionyun.com>"

-# openssh-client, for ansible ssh connection
-# git, ca-certificates, for fetching ansible roles
-RUN set -x \
-	&& apk update \
-	&& apk add openssh-client \
-	&& apk add ansible \
-	&& apk add tzdata git ca-certificates \
-	&& rm -rf /var/cache/apk/*
+ENV TZ Asia/Shanghai
+
+RUN mkdir -p /opt/yunion/bin
+
+RUN apk update && \
+    apk add --no-cache tzdata ansible ca-certificates && \
+    rm -rf /var/cache/apk/*
+
+RUN cp /usr/share/zoneinfo/Asia/Shanghai /etc/localtime
--- a/build/docker/Dockerfile.cloudid
+++ b/build/docker/Dockerfile.cloudid
@@ -1,3 +0,0 @@
-FROM registry.cn-beijing.aliyuncs.com/yunionio/onecloud-base:v0.1
-
-ADD ./_output/bin/cloudid /opt/yunion/bin/cloudid
--- a/build/docker/Dockerfile.host
+++ b/build/docker/Dockerfile.host
@@ -1,4 +1,4 @@
-FROM registry.cn-beijing.aliyuncs.com/yunionio/host-base:v0.2.0
+FROM registry.cn-beijing.aliyuncs.com/yunionio/host-base:v0.1.0

 MAINTAINER "Yaoqi Wan wanyaoqi@yunionyun.com"

--- a/build/docker/Dockerfile.host-deployer
+++ b/build/docker/Dockerfile.host-deployer
@@ -1,4 +1,4 @@
-FROM registry.cn-beijing.aliyuncs.com/yunionio/host-deployer-base:v0.0.3
+FROM registry.cn-beijing.aliyuncs.com/yunionio/host-deployer-base:v0.0.2

 MAINTAINER "Yaoqi Wan wanyaoqi@yunionyun.com"

--- a/build/docker/Dockerfile.vpcagent
+++ b/build/docker/Dockerfile.vpcagent
@@ -1,4 +1,4 @@
-FROM registry.cn-beijing.aliyuncs.com/yunionio/openvswitch:2.10.4-0
+FROM registry.cn-beijing.aliyuncs.com/yunionio/openvswitch:2.9.6-1

 MAINTAINER "Yousong Zhou <zhouyousong@yunion.cn>"

--- a/build/notify/root/opt/yunion/share/notify/email_verify_template
+++ b/build/notify/root/opt/yunion/share/notify/email_verify_template
--- a/build/notify/root/opt/yunion/share/template/content/IMAGE_ACTIVED
+++ b/build/notify/root/opt/yunion/share/template/content/IMAGE_ACTIVED
@@ -1 +0,0 @@
-{{.os_type}} 镜像 {{.name}} 上传完成
--- a/build/notify/root/opt/yunion/share/template/content/VERIFY.email
+++ b/build/notify/root/opt/yunion/share/template/content/VERIFY.email
@@ -1,66 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-<head>
-  <meta charset="UTF-8">
-  <title>Document</title>
-</head>
-<body>
-  <table style="width: 650px; margin-bottom: 20px;" border="0" cellpadding="0" cellspacing="0" align="center">
-  	<tr style="height: 50px; background: #333; overflow: hidden;">
-  		<td>
-  			<table style="margin-left: 20px;">
-  				<tr>
-  					<td>
-  						<img src="data:{{.login_logo_format}};base64,{{.login_logo}}" alt="" style="color: #fff; height: 32px; vertical-align: middle;">
-  					</td>
-  				</tr>
-  			</table>
-  		</td>
-  		<td>
-  			<table style="float: right;">
-  				<tr>
-  					<td style="padding-left: 20px; padding-right: 20px;">
-  						<a href="" style="text-decoration: none; color: #20A0FF; font-size: 12px; display: none;">产品</a>
-  					</td>
-  					<td style="padding-left: 20px; padding-right: 20px;">
-  						<a href="" style="text-decoration: none; color: #20A0FF; font-size: 12px; display: none;">解决方案</a>
-  					</td>
-  					<td style="padding-left: 20px; padding-right: 20px;">
-  						<a href="" style="text-decoration: none; color: #20A0FF; font-size: 12px; display: none;">服务</a>
-  					</td>
-  					<td style="padding-left: 20px; padding-right: 20px;">
-  						<a href="" style="text-decoration: none; color: #20A0FF; font-size: 12px; display: none;">帮助与文档</a>
-  					</td>
-  					<td style="padding-left: 20px; padding-right: 20px;">
-  						<a href="" style="text-decoration: none; color: #20A0FF; font-size: 12px; display: none;">关于我们</a>
-  					</td>
-  				</tr>
-  			</table>
-  		</td>
-  	</tr>
-  	<tr>
-  		<td style="width: 100%;" colspan="2">
-  			<table style="padding: 20px 10px; width: 100%;">
-	  			<tr>
-	  				<td style="padding-bottom: 20px;">
-                        尊敬的{{.name}}，您好：
-	  				</td>
-	  			</tr>
-	  			<tr>
-	  				<td style="padding-bottom: 10px;">请点击下面的链接，完成用户邮箱的激活：</td>
-	  			</tr>
-	  			<tr>
-                    <td style="padding-bottom: 30px;"><a href="{{.link}}">{{.link}}</a></td>
-	  			</tr>
-	  		</table>
-  		</td>
-  	</tr>
-  	<tr style="width: 96%;">
-		<td colspan="2" style="border-top: 1px dashed #ccc; color: #333; font-size: 12px; padding-bottom: 10px; font-weight: 100;">*为了确保您的帐号安全，该链接仅48小时内访问有效，请勿直接回复此邮件。</td>
-	</tr>
-	<tr>
-		<td colspan="2" style="background: #333; text-align: right; padding-right: 20px; font-size: 12px; color: #fff; height: 50px;">版权所有 © {{.copyright}} 保留一切权利</td>
-	</tr>
-  </table>
-</body>
-</html>
--- a/build/notify/root/opt/yunion/share/template/title/IMAGE_ACTIVED
+++ b/build/notify/root/opt/yunion/share/template/title/IMAGE_ACTIVED
@@ -1 +0,0 @@
-镜像 {{.name}} 上传完成
--- a/build/notify/root/opt/yunion/share/template/title/VERIFY.email
+++ b/build/notify/root/opt/yunion/share/template/title/VERIFY.email
@@ -1 +0,0 @@
-Yunion Verify
--- a/cmd/climc/entry/climc.go
+++ b/cmd/climc/entry/climc.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package entry
+package main

 import (
 	"context"
@@ -32,6 +32,10 @@ import (

 	"yunion.io/x/onecloud/cmd/climc/promputils"
 	"yunion.io/x/onecloud/cmd/climc/shell"
+	_ "yunion.io/x/onecloud/cmd/climc/shell/cloudnet"
+	_ "yunion.io/x/onecloud/cmd/climc/shell/etcd"
+	_ "yunion.io/x/onecloud/cmd/climc/shell/k8s"
+	_ "yunion.io/x/onecloud/cmd/climc/shell/monitor"
 	"yunion.io/x/onecloud/pkg/mcclient"
 )

@@ -45,7 +49,7 @@ type BaseOptions struct {
 	CertFile string `default:"$YUNION_CERT_FILE" help:"certificate file"`
 	KeyFile  string `default:"$YUNION_KEY_FILE" help:"private key file"`

-	Completion     string `default:"" help:"Generate climc auto complete script" choices:"bash|zsh"`
+	Completion     string `default:"" help:"Generate climc auto complete script" choices:"bash"`
 	UseCachedToken bool   `default:"$YUNION_USE_CACHED_TOKEN|false" help:"Use cached token"`

 	OsUsername string `default:"$OS_USERNAME" help:"Username, defaults to env[OS_USERNAME]"`
@@ -58,8 +62,6 @@ type BaseOptions struct {
 	OsAccessKey string `default:"$OS_ACCESS_KEY" help:"ak/sk access key, defaults to env[OS_ACCESS_KEY]"`
 	OsSecretKey string `default:"$OS_SECRET_KEY" help:"ak/s secret, defaults to env[OS_SECRET_KEY]"`

-	OsAuthToken string `default:"$OS_AUTH_TOKEN" help:"token authenticate, defaults to env[OS_AUTH_TOKEN]"`
-
 	OsAuthURL string `default:"$OS_AUTH_URL" help:"Defaults to env[OS_AUTH_URL]"`

 	OsRegionName   string `default:"$OS_REGION_NAME" help:"Defaults to env[OS_REGION_NAME]"`
@@ -71,13 +73,9 @@ type BaseOptions struct {
 }

 func getSubcommandsParser() (*structarg.ArgumentParser, error) {
-	var (
-		prog = "climc"
-		desc = `Command-line interface to the API server.`
-	)
 	parse, e := structarg.NewArgumentParser(&BaseOptions{},
-		prog,
-		desc,
+		"climc",
+		`Command-line interface to the API server.`,
 		`See "climc help COMMAND" for help on a specific command.`)
 	if e != nil {
 		return nil, e
@@ -98,28 +96,26 @@ func getSubcommandsParser() (*structarg.ArgumentParser, error) {
 			return nil
 		}
 	})
-
-	promptRootCmd := promputils.InitRootCmd(prog, desc, parse.GetOptArgs(), parse.GetPosArgs())
 	for _, v := range shell.CommandTable {
 		_par, e := subcmd.AddSubParser(v.Options, v.Command, v.Desc, v.Callback)

 		if e != nil {
 			return nil, e
 		}
-		promputils.AppendCommand(promptRootCmd, v.Command, v.Desc)
+		promputils.AppendCommand(v.Command, v.Desc)
 		cmd := v.Command

 		for _, v := range _par.GetOptArgs() {
 			text := v.String()
 			text = strings.TrimLeft(text, "[<")
 			text = strings.TrimRight(text, "]>")
-			promputils.AppendOpt(cmd, text, v.HelpString(""), v)
+			promputils.AppendOpt(cmd, text, v.HelpString(""))
 		}
 		for _, v := range _par.GetPosArgs() {
 			text := v.String()
 			text = strings.TrimLeft(text, "[<")
 			text = strings.TrimRight(text, "]>")
-			promputils.AppendPos(cmd, text, v.HelpString(""), v)
+			promputils.AppendPos(cmd, text, v.HelpString(""))
 		}
 	}
 	return parse, nil
@@ -135,8 +131,8 @@ func newClientSession(options *BaseOptions) (*mcclient.ClientSession, error) {
 	if len(options.OsAuthURL) == 0 {
 		return nil, fmt.Errorf("Missing OS_AUTH_URL")
 	}
-	if len(options.OsUsername) == 0 && len(options.OsAccessKey) == 0 && len(options.OsAuthToken) == 0 {
-		return nil, fmt.Errorf("Missing OS_USERNAME or OS_ACCESS_KEY or OS_AUTH_TOKEN")
+	if len(options.OsUsername) == 0 && len(options.OsAccessKey) == 0 {
+		return nil, fmt.Errorf("Missing OS_USERNAME or OS_ACCESS_KEY")
 	}
 	if len(options.OsUsername) > 0 && len(options.OsPassword) == 0 {
 		return nil, fmt.Errorf("Missing OS_PASSWORD")
@@ -144,6 +140,9 @@ func newClientSession(options *BaseOptions) (*mcclient.ClientSession, error) {
 	if len(options.OsAccessKey) > 0 && len(options.OsSecretKey) == 0 {
 		return nil, fmt.Errorf("Missing OS_SECRET_KEY")
 	}
+	if len(options.OsRegionName) == 0 {
+		return nil, fmt.Errorf("Missing OS_REGION_NAME")
+	}

 	logLevel := "info"
 	if options.Debug {
@@ -186,11 +185,7 @@ func newClientSession(options *BaseOptions) (*mcclient.ClientSession, error) {
 	if cacheToken == nil {
 		var token mcclient.TokenCredential
 		var err error
-		if len(options.OsAuthToken) > 0 {
-			token, err = client.AuthenticateToken(options.OsAuthToken, options.OsProjectName,
-				options.OsProjectDomain,
-				mcclient.AuthSourceCli)
-		} else if len(options.OsAccessKey) > 0 {
+		if len(options.OsAccessKey) > 0 {
 			token, err = client.AuthenticateByAccessKey(options.OsAccessKey,
 				options.OsSecretKey, mcclient.AuthSourceCli)
 		} else {
@@ -266,7 +261,7 @@ func executeSubcommand(
 	}
 }

-func ClimcMain() {
+func main() {
 	parser, e := getSubcommandsParser()
 	if e != nil {
 		showErrorAndExit(e)
@@ -275,7 +270,7 @@ func ClimcMain() {
 	options := parser.Options().(*BaseOptions)

 	if len(options.Completion) > 0 {
-		completeScript := promputils.GenerateAutoCompleteCmds(promputils.GetRootCmd(), options.Completion)
+		completeScript := promputils.GenerateAutoCompleteCmds(options.Completion)
 		if len(completeScript) > 0 {
 			fmt.Printf("%s", completeScript)
 		}
--- a/cmd/climc/main.go
+++ b/cmd/climc/main.go
@@ -1,42 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package main
-
-import (
-	"yunion.io/x/onecloud/cmd/climc/entry"
-	_ "yunion.io/x/onecloud/cmd/climc/shell"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/ansible"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/cloudevent"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/cloudid"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/cloudnet"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/compute"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/etcd"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/events"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/identity"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/image"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/itsm"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/k8s"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/logger"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/meter"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/misc"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/monitor"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/notify"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/servicetree"
-	_ "yunion.io/x/onecloud/cmd/climc/shell/yunionconf"
-)
-
-func main() {
-	entry.ClimcMain()
-}
--- a/cmd/climc/promputils/complete_zsh.go
+++ b/cmd/climc/promputils/complete_zsh.go
@@ -1,178 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package promputils
-
-import (
-	"fmt"
-	"io"
-	"strings"
-	"text/template"
-
-	"yunion.io/x/structarg"
-)
-
-// ref: https://github.com/spf13/cobra/blob/master/zsh_completions.go
-
-var (
-	zshCompFuncMap = template.FuncMap{
-		"genZshFuncName":              zshCompGenFuncName,
-		"extractFlags":                zshCompExtractFlag,
-		"genFlagEntryForZshArguments": zshCompGenFlagEntryForArguments,
-		"extractArgsCompletions":      zshCompExtractArgumentCompletionHintsForRendering,
-	}
-	zshCompletionText = `
-{{/* should accept Command (that contains subcommands) as parameter */}}
-{{define "argumentsC" -}}
-{{ $cmdPath := genZshFuncName .}}
-function {{$cmdPath}} {
-	local -a commands
-	_arguments -C \{{- range extractFlags .}}
-	  {{genFlagEntryForZshArguments .}} \{{- end}}
-	  "1: :->cmnds" \
-	  "*::arg:->args"
-
-	case $state in
-	cmnds)
-	  commands=({{range .SubCmds}}
-		"{{.Name}}:{{.Desc}}"{{end}}
-	  )
-	  _describe "command" commands
-	  ;;
-	esac
-
-	case "$words[1]" in {{- range .SubCmds}}
-	{{.Name}})
-	  {{$cmdPath}}_{{.Name}}
-	  ;;{{end}}
-	esac
-}
-{{range .SubCmds}}
-{{template "selectCmdTemplate" .}}
-{{- end}}
-{{- end}}
-
-{{/* should accept Command without subcommands as parameter */}}
-{{define "arguments" -}}
-function {{genZshFuncName .}} {
-{{"  _arguments"}}{{range extractFlags .}} \
-     {{genFlagEntryForZshArguments . -}}
-{{end}}{{range extractArgsCompletions .}} \
-     {{.}}{{end}}
-}
-{{end}}
-
-{{/* dispatcher for commands with or without subcommands */}}
-{{define "selectCmdTemplate" -}}
-{{if .SubCmds}}{{template "argumentsC" .}}{{else}}{{template "arguments" .}}{{end}}
-{{- end}}
-
-{{/* template entry point */}}
-{{define "Main" -}}
-#compdef _{{.Name}} {{.Name}}
-
-{{template "selectCmdTemplate" .}}
-compdef _{{.Name}} {{.Name}}
-{{end}}
-`
-)
-
-func zshCompGenFuncName(c *Cmd) string {
-	if c.ParentCmd != nil {
-		return zshCompGenFuncName(c.ParentCmd) + "_" + c.Name
-	}
-	return "_" + c.Name
-}
-
-func zshCompExtractFlag(c *Cmd) []structarg.Argument {
-	return c.GetArguments()
-}
-
-func zshCompGenFlagEntryForArguments(f structarg.Argument) string {
-	// not process positional argument and single command
-	if f.IsPositional() {
-		return ""
-	}
-	if f.ShortToken() == "" {
-		return zshCompGenFlagEntryForSingleOptionFlag(f)
-	}
-	return zshCompGenFlagEntryForMultiOptionFlag(f)
-}
-
-func zshCompGenFlagEntryForSingleOptionFlag(f structarg.Argument) string {
-	var option, multiMark, extras string
-	if f.IsMulti() {
-		multiMark = "*"
-	}
-
-	option = "--" + f.Token()
-	extras = zshCompGenFlagEntryExtras(f)
-	return fmt.Sprintf(`'%s%s[%s]%s'`, multiMark, option, zshCompQuoteFlagDescription(f.HelpString("")), extras)
-}
-
-func zshCompGenFlagEntryForMultiOptionFlag(f structarg.Argument) string {
-	var options, parenMultiMark, curlyMultiMark, extras string
-	if f.IsMulti() {
-		parenMultiMark = "*"
-		curlyMultiMark = "\\*"
-	}
-
-	options = fmt.Sprintf(`'(%s-%s %s--%s)'{%s-%s,%s--%s}`,
-		parenMultiMark, f.ShortToken(), parenMultiMark, f.Token(), curlyMultiMark, f.ShortToken(), curlyMultiMark, f.Token())
-	extras = zshCompGenFlagEntryExtras(f)
-
-	return fmt.Sprintf(`%s'[%s]%s'`, options, zshCompQuoteFlagDescription(f.HelpString("")), extras)
-}
-
-func zshCompGenFlagEntryExtras(f structarg.Argument) string {
-	if !f.NeedData() {
-		return ""
-	}
-	// allow options for flag
-	extras := ":"
-
-	type iChoices interface {
-		Choices() []string
-	}
-	if hasChoices, ok := f.(iChoices); ok {
-		// process choices
-		var words []string
-		for _, w := range hasChoices.Choices() {
-			words = append(words, fmt.Sprintf("%q", w))
-		}
-		if len(words) != 0 {
-			extras = fmt.Sprintf("%s :(%s)", extras, strings.Join(words, " "))
-		}
-	}
-	return extras
-}
-
-func zshCompQuoteFlagDescription(s string) string {
-	s = strings.Replace(s, "'", `'\''`, -1)
-	s = strings.Replace(s, "[", `\[`, -1)
-	s = strings.Replace(s, "]", `\]`, -1)
-	return s
-}
-
-func zshCompExtractArgumentCompletionHintsForRendering(c *Cmd) []string {
-	return nil
-}
-
-func (c *Cmd) GenZshCompletion(w io.Writer) error {
-	tmpl, err := template.New("Main").Funcs(zshCompFuncMap).Parse(zshCompletionText)
-	if err != nil {
-		return fmt.Errorf("error creating zsh completion template: %v", err)
-	}
-	return tmpl.Execute(w, c.Root())
-}
--- a/cmd/climc/promputils/completescript.go
+++ b/cmd/climc/promputils/completescript.go
@@ -75,3 +75,33 @@ _climc(){
    do
        line="${arr[$i]}"
        allopts[count]=${line%s`
+
+const ZSH_COMPLETE_SCRIPT_2 = `%"#"*}
+        subopts[count]=${line#*"#"}
+        ((count++))
+    done
+    _arguments \
+            '1: :->commands'\
+            '*: :->options'
+
+    case $state in 
+        commands)
+        _arguments '1:Comm:(${allopts[*]})'
+        ;;
+        *)
+            item=1
+            for key in ${allopts[*]}
+            do
+                case $words[2] in
+                    $key)
+                    compadd "$@"` + "`echo $subopts[item]`" + `
+                    ;;
+                    *)
+                    ;;
+                esac
+                ((item++))
+            done
+    esac    
+}
+_climc "$@"
+`
--- a/cmd/climc/promputils/completor.go
+++ b/cmd/climc/promputils/completor.go
@@ -15,118 +15,26 @@
 package promputils

 import (
-	"bytes"
 	"fmt"
 	"regexp"
 	"strings"

 	prompt "github.com/c-bata/go-prompt"
-
-	"yunion.io/x/structarg"
 )

-var (
-	subcmds = make(map[string]*Cmd)
-	rootCmd *Cmd
-)
+type Cmd struct {
+	desc    string
+	optArgs []prompt.Suggest
+	posArgs []prompt.Suggest
+}
+
+var cmds = make(map[string]*Cmd)

 var optionHelp = []prompt.Suggest{
 	{Text: "-h"},
 	{Text: "--help"},
 }

-type Cmd struct {
-	Name      string
-	Desc      string
-	optArgs   []CmdArgument
-	posArgs   []CmdArgument
-	ParentCmd *Cmd
-	SubCmds   []*Cmd
-}
-
-func InitRootCmd(name, desc string, optArgs, posArgs []structarg.Argument) *Cmd {
-	rootCmd = NewCmd(name, desc)
-	for _, optA := range optArgs {
-		rootCmd.AddOptArgument(optA, optA.Token(), optA.HelpString(""))
-	}
-	for _, posA := range posArgs {
-		rootCmd.AddPosArgument(posA, posA.Token(), posA.HelpString(""))
-	}
-	return rootCmd
-}
-
-func GetRootCmd() *Cmd {
-	return rootCmd
-}
-
-func NewCmd(name, desc string) *Cmd {
-	return &Cmd{
-		Name:    name,
-		Desc:    desc,
-		optArgs: make([]CmdArgument, 0),
-		posArgs: make([]CmdArgument, 0),
-		SubCmds: make([]*Cmd, 0),
-	}
-}
-
-func (c Cmd) getPromptSuggests(args []CmdArgument) []prompt.Suggest {
-	ret := make([]prompt.Suggest, 0)
-	for _, arg := range args {
-		ret = append(ret, arg.Suggest)
-	}
-	return ret
-}
-
-func (c *Cmd) Root() *Cmd {
-	if c.ParentCmd == nil {
-		return c
-	}
-	return c.ParentCmd.Root()
-}
-
-func (c Cmd) GetName() string {
-	return c.Name
-}
-
-func (c *Cmd) AddCmd(cmd *Cmd) {
-	c.SubCmds = append(c.SubCmds, cmd)
-}
-
-func (c Cmd) GetPromptOptSuggests() []prompt.Suggest {
-	return c.getPromptSuggests(c.optArgs)
-}
-
-func (c Cmd) GetPromptPosSuggests() []prompt.Suggest {
-	return c.getPromptSuggests(c.posArgs)
-}
-
-func (c Cmd) GetOptArguments() []structarg.Argument {
-	return c.getArguments(c.optArgs)
-}
-
-func (c Cmd) GetArguments() []structarg.Argument {
-	ret := c.GetOptArguments()
-	ret = append(ret, c.GetPosArguments()...)
-	return ret
-}
-
-func (c Cmd) GetPosArguments() []structarg.Argument {
-	return c.getArguments(c.posArgs)
-}
-
-func (c Cmd) getArguments(args []CmdArgument) []structarg.Argument {
-	ret := make([]structarg.Argument, 0)
-	for _, a := range args {
-		ret = append(ret, a.Argument)
-	}
-	return ret
-}
-
-type CmdArgument struct {
-	Suggest  prompt.Suggest
-	Argument structarg.Argument
-}
-
 func optionCompleter(args []string, long bool) []prompt.Suggest {
 	l := len(args)
 	if l == 0 {
@@ -139,11 +47,11 @@ func optionCompleter(args []string, long bool) []prompt.Suggest {
 		return optionHelp
 	}

-	if subcmds[args[0]] == nil {
+	if cmds[args[0]] == nil {
 		return []prompt.Suggest{}
 	}

-	_cmd := subcmds[args[0]].GetPromptOptSuggests()
+	_cmd := cmds[args[0]].optArgs
 	return prompt.FilterContains(_cmd, strings.TrimLeft(args[l-1], "-"), true)
 }

@@ -192,7 +100,7 @@ func argumentsCompleter(args []string) []prompt.Suggest {
 	if len(args) == 1 {
 		return prompt.FilterHasPrefix(commands, args[0], true)
 	}
-	_cmd, ok := subcmds[args[0]]
+	_cmd, ok := cmds[args[0]]
 	if !ok {
 		return []prompt.Suggest{}
 	}
@@ -203,75 +111,48 @@ func argumentsCompleter(args []string) []prompt.Suggest {

 	prm := _cmd.posArgs[len(args)-2]
 	subcommands := []prompt.Suggest{
-		prm.Suggest,
+		prm,
 	}

 	return prompt.FilterHasPrefix(subcommands, args[len(args)-1], true)
 }

-func AppendCommand(parentCmd *Cmd, text, desc string) {
+func AppendCommand(text, desc string) {
 	commands = append(commands, prompt.Suggest{Text: text, Description: desc})
-	cmd := &Cmd{
-		Name:      text,
-		Desc:      desc,
-		posArgs:   make([]CmdArgument, 0),
-		optArgs:   make([]CmdArgument, 0),
-		ParentCmd: parentCmd,
-	}
-	subcmds[text] = cmd
-	parentCmd.AddCmd(cmd)
+	cmds[text] = &Cmd{}
+	cmds[text].desc = desc
+	cmds[text].posArgs = []prompt.Suggest{}
+	cmds[text].optArgs = []prompt.Suggest{}
 }

-func (c *Cmd) addArgument(target *[]CmdArgument, arg structarg.Argument, argStr string, desc string) {
-	*target = append(*target, CmdArgument{
-		Suggest: prompt.Suggest{
-			Text:        argStr,
-			Description: desc,
-		},
-		Argument: arg,
-	})
+func AppendPos(text, cmd, desc string) {
+	var v = []prompt.Suggest{{Text: cmd, Description: desc}}
+	cmds[text].posArgs = append(cmds[text].posArgs, v...)
 }

-func (c *Cmd) AddPosArgument(arg structarg.Argument, argStr string, desc string) {
-	c.addArgument(&c.posArgs, arg, argStr, desc)
+func AppendOpt(text, cmd, desc string) {
+	var v = []prompt.Suggest{{Text: cmd, Description: desc}}
+	cmds[text].optArgs = append(cmds[text].optArgs, v...)
 }

-func (c *Cmd) AddOptArgument(arg structarg.Argument, argStr string, desc string) {
-	c.addArgument(&c.optArgs, arg, argStr, desc)
-}
-
-func AppendPos(text, cmd, desc string, arg structarg.Argument) {
-	cmdObj := subcmds[text]
-	cmdObj.AddPosArgument(arg, cmd, desc)
-}
-
-func AppendOpt(text, cmd, desc string, arg structarg.Argument) {
-	cmdObj := subcmds[text]
-	cmdObj.AddOptArgument(arg, cmd, desc)
-}
-
-func GenerateAutoCompleteCmds(rootCmd *Cmd, shell string) string {
+func GenerateAutoCompleteCmds(shell string) string {
 	var ret = []string{}
 	var i = 0
 	if strings.ToLower(shell) == "zsh" {
-		out := bytes.NewBufferString("")
-		if err := rootCmd.GenZshCompletion(out); err != nil {
-			panic(err)
-		}
-		return out.String()
+		i = 1
 	}
-	for _, cmd := range subcmds {
+	for cmd, options := range cmds {
 		var (
 			strPosArgs = []string{}
 			strOptArgs = []string{}
 		)
-		for _, posArg := range cmd.GetPromptPosSuggests() {
+		for _, posArg := range options.posArgs {
 			strPosArgs = append(strPosArgs, posArg.Text)
 		}
-		for _, optArg := range cmd.GetPromptOptSuggests() {
+		for _, optArg := range options.optArgs {
 			strOptArgs = append(strOptArgs, strings.Split(optArg.Text, " ")[0])
 		}
-		ret = append(ret, fmt.Sprintf(`arr[%d]="%s# %s %s"`, i, cmd.Name,
+		ret = append(ret, fmt.Sprintf(`arr[%d]="%s# %s %s"`, i, cmd,
 			strings.Join(strPosArgs, " "), strings.Join(strOptArgs, " ")))
 		i += 1
 	}
--- a/cmd/climc/shell/meter/account_balances.go
+++ b/cmd/climc/shell/meter/account_balances.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/logger/actions.go
+++ b/cmd/climc/shell/logger/actions.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package logger
+package shell

 import (
 	"fmt"
--- a/cmd/climc/shell/servicetree/alarmevents.go
+++ b/cmd/climc/shell/servicetree/alarmevents.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package servicetree
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/servicetree/alarmlogs.go
+++ b/cmd/climc/shell/servicetree/alarmlogs.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package servicetree
+package shell

 import (
 	"fmt"
--- a/cmd/climc/shell/servicetree/alarms.go
+++ b/cmd/climc/shell/servicetree/alarms.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package servicetree
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/servicetree/alarmtemplates.go
+++ b/cmd/climc/shell/servicetree/alarmtemplates.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package servicetree
+package shell

 import (
 	"strings"
--- a/cmd/climc/shell/ansible/ansible.go
+++ b/cmd/climc/shell/ansible/ansible.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package ansible
+package shell

 import (
 	"fmt"
--- a/cmd/climc/shell/ansible/common.go
+++ b/cmd/climc/shell/ansible/common.go
@@ -1,32 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package ansible
-
-import (
-	"yunion.io/x/onecloud/cmd/climc/shell"
-	"yunion.io/x/onecloud/pkg/util/printutils"
-)
-
-var (
-	R                 = shell.R
-	printList         = printutils.PrintJSONList
-	printObject       = printutils.PrintJSONObject
-	printBatchResults = printutils.PrintJSONBatchResults
-
-	InvalidUpdateError   = shell.InvalidUpdateError
-	printObjectRecursive = printutils.PrintJSONObjectRecursive
-
-	exportList = shell.ExportList
-)
--- a/cmd/climc/shell/ansible/ansibleplaybooks.go
+++ b/cmd/climc/shell/ansible/ansibleplaybooks.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package ansible
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/ansible/ansibleplaybooks_v2.go
+++ b/cmd/climc/shell/ansible/ansibleplaybooks_v2.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package ansible
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
@@ -49,34 +49,4 @@ func init() {
 		printList(apbs, modules.AnsiblePlaybooksV2.GetColumns(s))
 		return nil
 	})
-	R(&AnsiblePlaybookV2IdOptions{}, "ansibleplaybookv2-run", "Run ansible playbook",
-		func(s *mcclient.ClientSession, opts *AnsiblePlaybookV2IdOptions) error {
-			apb, err := modules.AnsiblePlaybooksV2.PerformAction(s, opts.ID, "run", nil)
-			if err != nil {
-				return err
-			}
-			printObject(apb)
-			return nil
-		},
-	)
-	R(&AnsiblePlaybookV2IdOptions{}, "ansibleplaybookv2-stop", "Stop ansible playbook",
-		func(s *mcclient.ClientSession, opts *AnsiblePlaybookV2IdOptions) error {
-			apb, err := modules.AnsiblePlaybooksV2.PerformAction(s, opts.ID, "stop", nil)
-			if err != nil {
-				return err
-			}
-			printObject(apb)
-			return nil
-		},
-	)
-	R(&AnsiblePlaybookV2IdOptions{}, "ansibleplaybookv2-delete", "Delete ansible playbook",
-		func(s *mcclient.ClientSession, opts *AnsiblePlaybookV2IdOptions) error {
-			apb, err := modules.AnsiblePlaybooksV2.Delete(s, opts.ID, nil)
-			if err != nil {
-				return err
-			}
-			printObject(apb)
-			return nil
-		},
-	)
 }
--- a/cmd/climc/shell/meter/associated_bills.go
+++ b/cmd/climc/shell/meter/associated_bills.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/compute/baremetalagents.go
+++ b/cmd/climc/shell/compute/baremetalagents.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/logger/baremetalevents.go
+++ b/cmd/climc/shell/logger/baremetalevents.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package logger
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/compute/baremetalnetworks.go
+++ b/cmd/climc/shell/compute/baremetalnetworks.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/compute/baremetalstorages.go
+++ b/cmd/climc/shell/compute/baremetalstorages.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/meter/bill_analysises.go
+++ b/cmd/climc/shell/meter/bill_analysises.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/meter/bill_balances.go
+++ b/cmd/climc/shell/meter/bill_balances.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/meter/bill_cloudchecks.go
+++ b/cmd/climc/shell/meter/bill_cloudchecks.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/meter/bill_conditions.go
+++ b/cmd/climc/shell/meter/bill_conditions.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/meter/bill_details.go
+++ b/cmd/climc/shell/meter/bill_details.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/meter/bill_resources.go
+++ b/cmd/climc/shell/meter/bill_resources.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/meter/bucket_options.go
+++ b/cmd/climc/shell/meter/bucket_options.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/compute/buckets.go
+++ b/cmd/climc/shell/compute/buckets.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"fmt"
@@ -26,28 +26,18 @@ import (
 	"yunion.io/x/onecloud/pkg/mcclient/modulebase"
 	"yunion.io/x/onecloud/pkg/mcclient/modules"
 	"yunion.io/x/onecloud/pkg/mcclient/options"
-	"yunion.io/x/onecloud/pkg/multicloud/objectstore"
+	objectshell "yunion.io/x/onecloud/pkg/multicloud/objectstore"
 )

 func init() {
 	type BucketListOptions struct {
 		options.BaseListOptions
-		DistinctField string `help:"query specified distinct field"`
 	}
 	R(&BucketListOptions{}, "bucket-list", "List all buckets", func(s *mcclient.ClientSession, args *BucketListOptions) error {
 		params, err := options.ListStructToParams(args)
 		if err != nil {
 			return err
 		}
-		if len(args.DistinctField) > 0 {
-			params.Add(jsonutils.NewString(args.DistinctField), "extra_field")
-			result, err := modules.Buckets.Get(s, "distinct-field", params)
-			if err != nil {
-				return err
-			}
-			fmt.Println(result)
-			return nil
-		}
 		result, err := modules.Buckets.List(s, params)
 		if err != nil {
 			return err
@@ -56,10 +46,10 @@ func init() {
 		return nil
 	})

-	type BucketIdOptions struct {
+	type BucketShowOptions struct {
 		ID string `help:"ID or name of bucket"`
 	}
-	R(&BucketIdOptions{}, "bucket-show", "Id details of bucket", func(s *mcclient.ClientSession, args *BucketIdOptions) error {
+	R(&BucketShowOptions{}, "bucket-show", "Show details of bucket", func(s *mcclient.ClientSession, args *BucketShowOptions) error {
 		result, err := modules.Buckets.Get(s, args.ID, nil)
 		if err != nil {
 			return err
@@ -68,15 +58,6 @@ func init() {
 		return nil
 	})

-	R(&BucketIdOptions{}, "bucket-syncstatus", "Sync bucket statust", func(s *mcclient.ClientSession, args *BucketIdOptions) error {
-		result, err := modules.Buckets.PerformAction(s, args.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
 	type BucketUpdateOptions struct {
 		ID   string `help:"ID or name of bucket" json:"-"`
 		Name string `help:"new name of bucket" json:"name"`
@@ -188,7 +169,7 @@ func init() {
 		StorageClass  string `help:"storage CLass"`
 		Acl           string `help:"object acl." choices:"private|public-read|public-read-write"`

-		objectstore.ObjectHeaderOptions
+		objectshell.ObjectHeaderOptions
 	}
 	R(&BucketUploadObjectsOptions{}, "bucket-object-upload", "Upload an object into a bucket", func(s *mcclient.ClientSession, args *BucketUploadObjectsOptions) error {
 		var body io.Reader
@@ -325,7 +306,7 @@ func init() {

 		Key []string `help:"Optional object key" json:"key"`

-		objectstore.ObjectHeaderOptions
+		objectshell.ObjectHeaderOptions
 	}
 	R(&BucketSetMetadataOptions{}, "bucket-set-metadata", "Set metadata of object", func(s *mcclient.ClientSession, args *BucketSetMetadataOptions) error {
 		input := api.BucketMetadataInput{}
--- a/cmd/climc/shell/compute/cachedimages.go
+++ b/cmd/climc/shell/compute/cachedimages.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/compute/capabilities.go
+++ b/cmd/climc/shell/compute/capabilities.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/meter/cloud_sku_rates.go
+++ b/cmd/climc/shell/meter/cloud_sku_rates.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package meter
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/compute/cloudaccounts.go
+++ b/cmd/climc/shell/compute/cloudaccounts.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"fmt"
@@ -53,9 +53,6 @@ func init() {
 	type CloudaccountListOptions struct {
 		options.BaseListOptions
 		Capability []string `help:"capability filter" choices:"project|compute|network|loadbalancer|objectstore|rds|cache|event"`
-
-		DistinctField string `help:"distinct field"`
-		ProxySetting  string `help:"Proxy setting id or name"`
 	}
 	R(&CloudaccountListOptions{}, "cloud-account-list", "List cloud accounts", func(s *mcclient.ClientSession, args *CloudaccountListOptions) error {
 		var params *jsonutils.JSONDict
@@ -68,18 +65,6 @@ func init() {
 			if len(args.Capability) > 0 {
 				params.Add(jsonutils.NewStringArray(args.Capability), "capability")
 			}
-			if len(args.ProxySetting) > 0 {
-				params.Add(jsonutils.NewString(args.ProxySetting), "proxy_setting")
-			}
-		}
-		if len(args.DistinctField) > 0 {
-			params.Add(jsonutils.NewString(args.DistinctField), "extra_field")
-			result, err := modules.Cloudaccounts.Get(s, "distinct-field", params)
-			if err != nil {
-				return err
-			}
-			fmt.Println(result)
-			return nil
 		}
 		result, err := modules.Cloudaccounts.List(s, params)
 		if err != nil {
@@ -125,22 +110,6 @@ func init() {
 		printObject(result)
 		return nil
 	})
-	type SVMwareCloudAccountPrepareNetsOptions struct {
-		Project       string `help:"project for this account"`
-		ProjectDomain string `help:"domain for this account"`
-		NAME          string `help:"name for this account"`
-		options.SVMwareCredentialWithEnvironment
-	}
-	R(&SVMwareCloudAccountPrepareNetsOptions{}, "cloud-account-preparenets-vmware", "Prepare networks for a VMware cloud account before create this accout", func(s *mcclient.ClientSession, args *SVMwareCloudAccountPrepareNetsOptions) error {
-		params := jsonutils.Marshal(args)
-		params.(*jsonutils.JSONDict).Add(jsonutils.NewString("VMware"), "provider")
-		result, err := modules.Cloudaccounts.PerformClassAction(s, "prepare-nets", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})

 	R(&options.SAliyunCloudAccountCreateOptions{}, "cloud-account-create-aliyun", "Create an Aliyun cloud account", func(s *mcclient.ClientSession, args *options.SAliyunCloudAccountCreateOptions) error {
 		params := jsonutils.Marshal(args)
@@ -769,129 +738,6 @@ func init() {
 		return nil
 	})

-	R(&options.SVMwareCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-vmware", "Update credential of a VMware cloud account", func(s *mcclient.ClientSession, args *options.SVMwareCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SAliyunCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-aliyun", "Update credential of an Aliyun cloud account", func(s *mcclient.ClientSession, args *options.SAliyunCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SAzureCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-azure", "Update credential of an Azure cloud account", func(s *mcclient.ClientSession, args *options.SAzureCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SQcloudCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-qcloud", "Update credential of a Qcloud cloud account", func(s *mcclient.ClientSession, args *options.SQcloudCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SAWSCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-aws", "Update credential of an AWS cloud account", func(s *mcclient.ClientSession, args *options.SAWSCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SOpenStackCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-openstack", "Update credential of an OpenStack cloud account", func(s *mcclient.ClientSession, args *options.SOpenStackCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SHuaweiCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-huawei", "Update credential of an Huawei cloud account", func(s *mcclient.ClientSession, args *options.SHuaweiCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SUcloudCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-ucloud", "Update credential of a Ucloud cloud account", func(s *mcclient.ClientSession, args *options.SUcloudCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SZStackCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-zstack", "Update credential of a ZStack cloud account", func(s *mcclient.ClientSession, args *options.SZStackCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SGoogleCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-google", "Update credential of a Google cloud account", func(s *mcclient.ClientSession, args *options.SGoogleCloudAccountUpdateCredentialOptions) error {
-		params, err := parseGcpCredential(args.GoogleJsonFile)
-		if err != nil {
-			return err
-		}
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SS3CloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-s3", "Update credential of a generic S3 cloud account", func(s *mcclient.ClientSession, args *options.SS3CloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&options.SCtyunCloudAccountUpdateCredentialOptions{}, "cloud-account-test-connectivity-ctyun", "Update credential of an Ctyun cloud account", func(s *mcclient.ClientSession, args *options.SCtyunCloudAccountUpdateCredentialOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "test-connectivity", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
 	type CloudaccountSyncOptions struct {
 		ID       string   `help:"ID or Name of cloud account"`
 		Force    bool     `help:"Force sync no matter what"`
@@ -955,14 +801,10 @@ func init() {
 	})

 	type CloudaccountPublicOptions struct {
-		ID            string   `help:"ID or name of cloud account" json:"-"`
-		Scope         string   `help:"public_sccope" choices:"domain|system" json:"scope"`
-		SharedDomains []string `help:"shared domains" json:"shared_domains"`
-		ShareMode     string   `help:"share_mode" choices:"account_domain|provider_domain|system"`
+		ID string `help:"ID or name of cloud account"`
 	}
 	R(&CloudaccountPublicOptions{}, "cloud-account-public", "Mark this cloud account public ", func(s *mcclient.ClientSession, args *CloudaccountPublicOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "public", params)
+		result, err := modules.Cloudaccounts.PerformAction(s, args.ID, "public", nil)
 		if err != nil {
 			return err
 		}
@@ -1022,47 +864,4 @@ func init() {
 		printObject(result)
 		return nil
 	})
-
-	type ClouaccountChangeOwnerOptions struct {
-		ID            string `help:"ID or name of cloudaccount" json:"-"`
-		ProjectDomain string `json:"project_domain" help:"target domain"`
-	}
-	R(&ClouaccountChangeOwnerOptions{}, "cloud-account-change-owner", "Change owner domain of cloudaccount", func(s *mcclient.ClientSession, args *ClouaccountChangeOwnerOptions) error {
-		if len(args.ProjectDomain) == 0 {
-			return fmt.Errorf("empty project_domain")
-		}
-		params := jsonutils.Marshal(args)
-		ret, err := modules.Cloudaccounts.PerformAction(s, args.ID, "change-owner", params)
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-
-	type ClouaccountChangeProjectOptions struct {
-		ID      string `help:"ID or name of cloudaccount" json:"-"`
-		PROJECT string `json:"project" help:"target domain"`
-	}
-	R(&ClouaccountChangeProjectOptions{}, "cloud-account-change-project", "Change domain/project of cloudaccount", func(s *mcclient.ClientSession, args *ClouaccountChangeProjectOptions) error {
-		if len(args.PROJECT) == 0 {
-			return fmt.Errorf("empty project")
-		}
-		params := jsonutils.Marshal(args)
-		ret, err := modules.Cloudaccounts.PerformAction(s, args.ID, "change-project", params)
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-
-	R(&CloudaccountShowOptions{}, "cloud-account-change-owner-candidate-domains", "Show candiate domains of a cloud account changing project", func(s *mcclient.ClientSession, args *CloudaccountShowOptions) error {
-		result, err := modules.Cloudaccounts.GetSpecific(s, args.ID, "change-owner-candidate-domains", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
 }
--- a/cmd/climc/shell/cloudevent/common.go
+++ b/cmd/climc/shell/cloudevent/common.go
@@ -1,25 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudevent
-
-import (
-	"yunion.io/x/onecloud/cmd/climc/shell"
-	"yunion.io/x/onecloud/pkg/util/printutils"
-)
-
-var (
-	R         = shell.R
-	printList = printutils.PrintJSONList
-)
--- a/cmd/climc/shell/cloudevent/cloudevents.go
+++ b/cmd/climc/shell/cloudevent/cloudevents.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package cloudevent
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/cloudid/cloudgroup.go
+++ b/cmd/climc/shell/cloudid/cloudgroup.go
@@ -1,168 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type CloudgroupListOptions struct {
-		options.BaseListOptions
-
-		ClouduserId   string `json:"clouduser_id"`
-		CloudpolicyId string `json:"cloudpolicy_id"`
-		Usable        bool   `json:"usable"`
-	}
-	R(&CloudgroupListOptions{}, "cloud-group-list", "List cloud groups", func(s *mcclient.ClientSession, opts *CloudgroupListOptions) error {
-		params, err := options.ListStructToParams(opts)
-		if err != nil {
-			return err
-		}
-		result, err := modules.Cloudgroups.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Cloudgroups.GetColumns(s))
-		return nil
-	})
-
-	type CloudgroupCreateOptions struct {
-		NAME           string   `json:"name"`
-		PROVIDER       string   `json:"provider" choices:"Google|Aliyun|Aws|Huawei|Qcloud"`
-		CloudpolicyIds []string `json:"cloudpolicy_ids"`
-		Desc           string   `json:"description"`
-	}
-
-	R(&CloudgroupCreateOptions{}, "cloud-group-create", "Create cloud group", func(s *mcclient.ClientSession, opts *CloudgroupCreateOptions) error {
-		params := jsonutils.Marshal(opts)
-		result, err := modules.Cloudgroups.Create(s, params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type CloudgroupIdOptions struct {
-		ID string `help:"Cloudgroup Id"`
-	}
-
-	R(&CloudgroupIdOptions{}, "cloud-group-delete", "Delete cloud group", func(s *mcclient.ClientSession, opts *CloudgroupIdOptions) error {
-		result, err := modules.Cloudgroups.Delete(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&CloudgroupIdOptions{}, "cloud-group-show", "Show cloud group", func(s *mcclient.ClientSession, opts *CloudgroupIdOptions) error {
-		result, err := modules.Cloudgroups.Get(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&CloudgroupIdOptions{}, "cloud-group-syncstatus", "Sync cloud group status", func(s *mcclient.ClientSession, opts *CloudgroupIdOptions) error {
-		result, err := modules.Cloudgroups.PerformAction(s, opts.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type CloudgroupPolicyOptions struct {
-		ID             string `help:"Cloudgroup Id"`
-		CLOUDPOLICY_ID string `help:"Cloudpolicy Id"`
-	}
-
-	R(&CloudgroupPolicyOptions{}, "cloud-group-attach-policy", "Attach policy for cloud group", func(s *mcclient.ClientSession, opts *CloudgroupPolicyOptions) error {
-		result, err := modules.Cloudgroups.PerformAction(s, opts.ID, "attach-policy", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&CloudgroupPolicyOptions{}, "cloud-group-detach-policy", "Detach policy from cloud group", func(s *mcclient.ClientSession, opts *CloudgroupPolicyOptions) error {
-		result, err := modules.Cloudgroups.PerformAction(s, opts.ID, "detach-policy", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type CloudgroupUserOptions struct {
-		ID           string `help:"Cloudgroup Id"`
-		CLOUDUSER_ID string `help:"Clouduser Id"`
-	}
-
-	R(&CloudgroupUserOptions{}, "cloud-group-add-user", "Add user to cloud group", func(s *mcclient.ClientSession, opts *CloudgroupUserOptions) error {
-		result, err := modules.Cloudgroups.PerformAction(s, opts.ID, "add-user", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&CloudgroupUserOptions{}, "cloud-group-remove-user", "Remove user from cloud group", func(s *mcclient.ClientSession, opts *CloudgroupUserOptions) error {
-		result, err := modules.Cloudgroups.PerformAction(s, opts.ID, "remove-user", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type CloudgroupPoliciesOptions struct {
-		ID             string   `help:"Cloudgroup Id"`
-		CloudpolicyIds []string `json:"cloudpolicy_ids"`
-	}
-
-	R(&CloudgroupPoliciesOptions{}, "cloud-group-set-policies", "Set cloudpolicies for cloud group", func(s *mcclient.ClientSession, opts *CloudgroupPoliciesOptions) error {
-		result, err := modules.Cloudgroups.PerformAction(s, opts.ID, "set-policies", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type CloudgroupUsersOptions struct {
-		ID           string   `help:"Cloudgroup Id"`
-		ClouduserIds []string `json:"clouduser_ids"`
-	}
-
-	R(&CloudgroupUsersOptions{}, "cloud-group-set-users", "Set users for cloud group", func(s *mcclient.ClientSession, opts *CloudgroupUsersOptions) error {
-		result, err := modules.Cloudgroups.PerformAction(s, opts.ID, "set-users", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/cloudid/cloudgroupcache.go
+++ b/cmd/climc/shell/cloudid/cloudgroupcache.go
@@ -1,73 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type CloudgroupcacheListOptions struct {
-		options.BaseListOptions
-
-		CloudgroupId string `json:"cloudgroup_id"`
-	}
-	R(&CloudgroupcacheListOptions{}, "cloud-group-cache-list", "List cloud group caches", func(s *mcclient.ClientSession, opts *CloudgroupcacheListOptions) error {
-		params, err := options.ListStructToParams(opts)
-		if err != nil {
-			return err
-		}
-		result, err := modules.Cloudgroupcaches.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Cloudgroupcaches.GetColumns(s))
-		return nil
-	})
-
-	type CloudgroupcacheIdOption struct {
-		ID string `help:"Cloudgroup Id"`
-	}
-
-	R(&CloudgroupcacheIdOption{}, "cloud-group-cache-show", "Show cloud groupcache details", func(s *mcclient.ClientSession, opts *CloudgroupcacheIdOption) error {
-		result, err := modules.Cloudgroupcaches.Get(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&CloudgroupcacheIdOption{}, "cloud-group-cache-syncstatus", "Sync cloudgroupcache", func(s *mcclient.ClientSession, opts *CloudgroupcacheIdOption) error {
-		result, err := modules.Cloudgroupcaches.PerformAction(s, opts.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&CloudgroupcacheIdOption{}, "cloud-group-cache-delete", "Delete cloudgroupcache", func(s *mcclient.ClientSession, opts *CloudgroupcacheIdOption) error {
-		result, err := modules.Cloudgroupcaches.Delete(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/cloudid/cloudgrouppolicies.go
+++ b/cmd/climc/shell/cloudid/cloudgrouppolicies.go
@@ -1,72 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modulebase"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type CloudgroupPolicyListOptions struct {
-		options.BaseListOptions
-		Cloudgroup  string `help:"ID or Name of Cloudgroup"`
-		Cloudpolicy string `help:"Policy ID or name"`
-	}
-	R(&CloudgroupPolicyListOptions{}, "cloud-group-policy-list", "List cloudgroup cloudpolicy pairs", func(s *mcclient.ClientSession, args *CloudgroupPolicyListOptions) error {
-		var params *jsonutils.JSONDict
-		{
-			var err error
-			params, err = args.BaseListOptions.Params()
-			if err != nil {
-				return err
-
-			}
-		}
-		var result *modulebase.ListResult
-		var err error
-		if len(args.Cloudgroup) > 0 {
-			result, err = modules.Cloudgrouppolicies.ListDescendent(s, args.Cloudgroup, params)
-		} else if len(args.Cloudpolicy) > 0 {
-			result, err = modules.Cloudgrouppolicies.ListDescendent2(s, args.Cloudpolicy, params)
-		} else {
-			result, err = modules.Cloudgrouppolicies.List(s, params)
-		}
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Cloudgrouppolicies.GetColumns(s))
-		return nil
-	})
-
-	type CloudgroupPolicyDetailOptions struct {
-		CLOUDUSER   string `help:"ID or Name of Cloudgroup"`
-		CLOUDPOLICY string `help:"ID or Name of Cloudpolicy"`
-	}
-	R(&CloudgroupPolicyDetailOptions{}, "cloud-group-policy-show", "Show cloudgrouppolicy details", func(s *mcclient.ClientSession, args *CloudgroupPolicyDetailOptions) error {
-		query := jsonutils.NewDict()
-		result, err := modules.Cloudgrouppolicies.Get(s, args.CLOUDUSER, args.CLOUDPOLICY, query)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/cloudid/cloudgroupusers.go
+++ b/cmd/climc/shell/cloudid/cloudgroupusers.go
@@ -1,72 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modulebase"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type CloudgroupUserListOptions struct {
-		options.BaseListOptions
-		Cloudgroup string `help:"ID or Name of Cloudgroup"`
-		Clouduser  string `help:"User ID or name"`
-	}
-	R(&CloudgroupUserListOptions{}, "cloud-group-user-list", "List cloudgroup clouduser pairs", func(s *mcclient.ClientSession, args *CloudgroupUserListOptions) error {
-		var params *jsonutils.JSONDict
-		{
-			var err error
-			params, err = args.BaseListOptions.Params()
-			if err != nil {
-				return err
-
-			}
-		}
-		var result *modulebase.ListResult
-		var err error
-		if len(args.Cloudgroup) > 0 {
-			result, err = modules.Cloudgroupusers.ListDescendent(s, args.Cloudgroup, params)
-		} else if len(args.Clouduser) > 0 {
-			result, err = modules.Cloudgroupusers.ListDescendent2(s, args.Clouduser, params)
-		} else {
-			result, err = modules.Cloudgroupusers.List(s, params)
-		}
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Cloudgroupusers.GetColumns(s))
-		return nil
-	})
-
-	type CloudgroupUserDetailOptions struct {
-		CLOUDUSER   string `help:"ID or Name of Cloudgroup"`
-		CLOUDPOLICY string `help:"ID or Name of Clouduser"`
-	}
-	R(&CloudgroupUserDetailOptions{}, "cloud-group-user-show", "Show cloudgroupuser details", func(s *mcclient.ClientSession, args *CloudgroupUserDetailOptions) error {
-		query := jsonutils.NewDict()
-		result, err := modules.Cloudgroupusers.Get(s, args.CLOUDUSER, args.CLOUDPOLICY, query)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/cloudid/cloudpolicy.go
+++ b/cmd/climc/shell/cloudid/cloudpolicy.go
@@ -1,81 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type CloudpolicyListOptions struct {
-		options.BaseListOptions
-
-		ClouduserId  string `json:"clouduser_id"`
-		CloudgroupId string `json:"cloudgroup_id"`
-	}
-	R(&CloudpolicyListOptions{}, "cloud-policy-list", "List cloud policies", func(s *mcclient.ClientSession, opts *CloudpolicyListOptions) error {
-		params, err := options.ListStructToParams(opts)
-		if err != nil {
-			return err
-		}
-		result, err := modules.Cloudpolicies.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Cloudpolicies.GetColumns(s))
-		return nil
-	})
-
-	type CloudpolicyIdOptions struct {
-		ID string `help:"Cloudpolicy Id"`
-	}
-
-	R(&CloudpolicyIdOptions{}, "cloud-policy-show", "Sow cloud policiy details", func(s *mcclient.ClientSession, opts *CloudpolicyIdOptions) error {
-		result, err := modules.Cloudpolicies.Get(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type CloudpolicyGroupOptions struct {
-		ID            string `help:"Cloudpolicy Id"`
-		CLOUDGROUP_ID string `help:"Cloudgroup Id" json:"cloudgroup_id"`
-	}
-
-	R(&CloudpolicyGroupOptions{}, "cloud-policy-assign-group", "Assign cloud policiy to group", func(s *mcclient.ClientSession, opts *CloudpolicyGroupOptions) error {
-		result, err := modules.Cloudpolicies.PerformAction(s, opts.ID, "assign-group", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&CloudpolicyGroupOptions{}, "cloud-policy-revoke-group", "Revoke cloud policiy from group", func(s *mcclient.ClientSession, opts *CloudpolicyGroupOptions) error {
-		result, err := modules.Cloudpolicies.PerformAction(s, opts.ID, "revoke-group", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/cloudid/clouduserpolicies.go
+++ b/cmd/climc/shell/cloudid/clouduserpolicies.go
@@ -1,72 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modulebase"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type ClouduserPolicyListOptions struct {
-		options.BaseListOptions
-		Clouduser   string `help:"ID or Name of Clouduser"`
-		Cloudpolicy string `help:"Policy ID or name"`
-	}
-	R(&ClouduserPolicyListOptions{}, "cloud-user-policy-list", "List clouduser cloudpolicy pairs", func(s *mcclient.ClientSession, args *ClouduserPolicyListOptions) error {
-		var params *jsonutils.JSONDict
-		{
-			var err error
-			params, err = args.BaseListOptions.Params()
-			if err != nil {
-				return err
-
-			}
-		}
-		var result *modulebase.ListResult
-		var err error
-		if len(args.Clouduser) > 0 {
-			result, err = modules.Clouduserpolicies.ListDescendent(s, args.Clouduser, params)
-		} else if len(args.Cloudpolicy) > 0 {
-			result, err = modules.Clouduserpolicies.ListDescendent2(s, args.Cloudpolicy, params)
-		} else {
-			result, err = modules.Clouduserpolicies.List(s, params)
-		}
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Clouduserpolicies.GetColumns(s))
-		return nil
-	})
-
-	type ClouduserPolicyDetailOptions struct {
-		CLOUDUSER   string `help:"ID or Name of Clouduser"`
-		CLOUDPOLICY string `help:"ID or Name of Cloudpolicy"`
-	}
-	R(&ClouduserPolicyDetailOptions{}, "cloud-user-policy-show", "Show clouduserpolicy details", func(s *mcclient.ClientSession, args *ClouduserPolicyDetailOptions) error {
-		query := jsonutils.NewDict()
-		result, err := modules.Clouduserpolicies.Get(s, args.CLOUDUSER, args.CLOUDPOLICY, query)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/cloudid/cloudusers.go
+++ b/cmd/climc/shell/cloudid/cloudusers.go
@@ -1,197 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type ClouduserListOptions struct {
-		options.BaseListOptions
-		CloudaccountId  string `help:"Cloudaccount Id"`
-		CloudproviderId string `help:"Cloudprovider Id"`
-		CloudpolicyId   string `help:"filter cloudusers by cloudpolicy"`
-		CloudgroupId    string `help:"filter cloudusers by cloudgroup"`
-	}
-	R(&ClouduserListOptions{}, "cloud-user-list", "List cloud users", func(s *mcclient.ClientSession, opts *ClouduserListOptions) error {
-		params, err := options.ListStructToParams(opts)
-		if err != nil {
-			return err
-		}
-		result, err := modules.Cloudusers.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Cloudusers.GetColumns(s))
-		return nil
-	})
-
-	type ClouduserCreateOptions struct {
-		Name            string
-		CloudaccountId  string   `help:"Cloudaccount Id"`
-		CloudproviderId string   `help:"Cloudprovider Id"`
-		OwnerId         string   `help:"Owner Id"`
-		CloudpolicyIds  []string `help:"cloudpolicy ids"`
-		CloudgroupIds   []string `help:"cloudgroup ids"`
-		Email           string   `help:"email address"`
-		MobilePhone     string   `help:"phone number"`
-		IsConsoleLogin  *bool    `help:"is console login"`
-		Password        string   `help:"clouduser password"`
-	}
-
-	R(&ClouduserCreateOptions{}, "cloud-user-create", "Create cloud user", func(s *mcclient.ClientSession, opts *ClouduserCreateOptions) error {
-		params := jsonutils.Marshal(opts)
-		result, err := modules.Cloudusers.Create(s, params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type ClouduserIdOptions struct {
-		ID string `help:"Clouduser Id"`
-	}
-
-	R(&ClouduserIdOptions{}, "cloud-user-delete", "Delete cloud user", func(s *mcclient.ClientSession, opts *ClouduserIdOptions) error {
-		result, err := modules.Cloudusers.Delete(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&ClouduserIdOptions{}, "cloud-user-logininfo", "Show cloud user login info", func(s *mcclient.ClientSession, opts *ClouduserIdOptions) error {
-		result, err := modules.Cloudusers.GetLoginInfo(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&ClouduserIdOptions{}, "cloud-user-show", "Show cloud user", func(s *mcclient.ClientSession, opts *ClouduserIdOptions) error {
-		result, err := modules.Cloudusers.Get(s, opts.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type ClouduserSyncOptions struct {
-		ID         string `help:"Clouduser Id"`
-		PolicyOnly bool   `help:"Ony sync clouduser policies for cloud"`
-	}
-
-	R(&ClouduserSyncOptions{}, "cloud-user-sync", "Sync cloud user policies", func(s *mcclient.ClientSession, opts *ClouduserSyncOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "sync", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&ClouduserIdOptions{}, "cloud-user-syncstatus", "Sync cloud user status", func(s *mcclient.ClientSession, opts *ClouduserIdOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type ClouduserPolicyOptions struct {
-		ID             string `help:"Clouduser Id"`
-		CLOUDPOLICY_ID string `help:"cloudpolicy Id"`
-	}
-
-	R(&ClouduserPolicyOptions{}, "cloud-user-attach-policy", "Attach policy for cloud user", func(s *mcclient.ClientSession, opts *ClouduserPolicyOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "attach-policy", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&ClouduserPolicyOptions{}, "cloud-user-detach-policy", "Detach policy from cloud user", func(s *mcclient.ClientSession, opts *ClouduserPolicyOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "detach-policy", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type ClouduserPasswordOptions struct {
-		ID       string `help:"Clouduser Id"`
-		Password string `help:"clouduser password"`
-	}
-
-	R(&ClouduserPasswordOptions{}, "cloud-user-reset-password", "Reset clouduser password", func(s *mcclient.ClientSession, opts *ClouduserPasswordOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "reset-password", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type ClouduserChangeOwnerOptions struct {
-		ID      string `help:"clouduser id"`
-		USER_ID string `help:"local user id"`
-	}
-
-	R(&ClouduserChangeOwnerOptions{}, "cloud-user-change-owner", "Change clouduser owner", func(s *mcclient.ClientSession, opts *ClouduserChangeOwnerOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "change-owner", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type ClouduserGroupOptions struct {
-		ID            string `help:"clouduser id"`
-		CLOUDGROUP_ID string `help:"cloudgroup id" json:"cloudgroup_id"`
-	}
-
-	R(&ClouduserGroupOptions{}, "cloud-user-join-group", "Join user to cloudgroup", func(s *mcclient.ClientSession, opts *ClouduserGroupOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "join-group", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&ClouduserGroupOptions{}, "cloud-user-leave-group", "Leave from cloudgroup", func(s *mcclient.ClientSession, opts *ClouduserGroupOptions) error {
-		result, err := modules.Cloudusers.PerformAction(s, opts.ID, "leave-group", jsonutils.Marshal(opts))
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/cloudid/common.go
+++ b/cmd/climc/shell/cloudid/common.go
@@ -1,26 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package cloudid
-
-import (
-	"yunion.io/x/onecloud/cmd/climc/shell"
-	"yunion.io/x/onecloud/pkg/util/printutils"
-)
-
-var (
-	R           = shell.R
-	printList   = printutils.PrintJSONList
-	printObject = printutils.PrintJSONObject
-)
--- a/pkg/multicloud/zstack/shell/quota.go
+++ b/pkg/multicloud/zstack/shell/quota.go
@@ -15,19 +15,16 @@
 package shell

 import (
-	"yunion.io/x/onecloud/pkg/multicloud/zstack"
-	"yunion.io/x/onecloud/pkg/util/shellutils"
+	"yunion.io/x/onecloud/pkg/mcclient"
 )

 func init() {
-	type QuotaListOptions struct {
+	type CloudmetaOptions struct {
+		PROVIDER_ID string `help:"provider_id"`
+		REGION_ID   string `help:"region_id"`
+		ZONE_ID     string `help:"zone_id"`
 	}
-	shellutils.R(&QuotaListOptions{}, "quota-list", "List quotas", func(cli *zstack.SRegion, args *QuotaListOptions) error {
-		quotas, err := cli.GetQuotas()
-		if err != nil {
-			return err
-		}
-		printList(quotas, 0, 0, 0, nil)
+	R(&CloudmetaOptions{}, "instance-type-list", "query backend service for its version", func(s *mcclient.ClientSession, args *CloudmetaOptions) error {
 		return nil
 	})
 }
--- a/cmd/climc/shell/compute/cloudproviderregions.go
+++ b/cmd/climc/shell/compute/cloudproviderregions.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -27,8 +27,6 @@ func init() {
 	type CloudproviderRegionListOptions struct {
 		options.BaseListOptions
 		Region string `help:"ID or Name of Host"`
-
-		Capability []string `help:"capability filter" choices:"project|compute|network|loadbalancer|objectstore|rds|cache|event"`
 	}
 	R(&CloudproviderRegionListOptions{}, "cloud-provider-region-list", "List cloudprovider region synchronization status", func(s *mcclient.ClientSession, args *CloudproviderRegionListOptions) error {
 		var params *jsonutils.JSONDict
--- a/cmd/climc/shell/compute/cloudproviders.go
+++ b/cmd/climc/shell/compute/cloudproviders.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"fmt"
@@ -34,7 +34,6 @@ func init() {
 		HasObjectStorage bool     `help:"filter cloudproviders that has object storage"`
 		NoObjectStorage  bool     `help:"filter cloudproviders that has no object storage"`
 		Capability       []string `help:"capability filter" choices:"project|compute|network|loadbalancer|objectstore|rds|cache|event"`
-		Cloudregion      string   `help:"filter cloudproviders by cloudregion"`
 	}
 	R(&CloudproviderListOptions{}, "cloud-provider-list", "List cloud providers", func(s *mcclient.ClientSession, args *CloudproviderListOptions) error {
 		var params *jsonutils.JSONDict
@@ -58,10 +57,6 @@ func init() {
 			if len(args.Capability) > 0 {
 				params.Add(jsonutils.NewStringArray(args.Capability), "capability")
 			}
-
-			if len(args.Cloudregion) > 0 {
-				params.Add(jsonutils.NewString(args.Cloudregion), "cloudregion")
-			}
 		}
 		result, err := modules.Cloudproviders.List(s, params)
 		if err != nil {
@@ -250,13 +245,4 @@ func init() {
 		printObject(result)
 		return nil
 	})
-
-	R(&CloudproviderShowOptions{}, "cloud-provider-change-owner-candidate-domains", "Show candiate domains of a cloud provider  changing owner", func(s *mcclient.ClientSession, args *CloudproviderShowOptions) error {
-		result, err := modules.Cloudproviders.GetSpecific(s, args.ID, "change-owner-candidate-domains", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
 }
--- a/cmd/climc/shell/compute/cloudregions.go
+++ b/cmd/climc/shell/compute/cloudregions.go
@@ -12,11 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
-	"fmt"
-
 	"yunion.io/x/jsonutils"

 	"yunion.io/x/onecloud/pkg/mcclient"
@@ -34,25 +32,12 @@ func init() {
 		Service   string `help:"List regions which service has available skus" choices:"dbinstances|servers"`

 		City string `help:"List regions in the specified city"`
-
-		Capability []string `help:"capability filter" choices:"project|compute|network|loadbalancer|objectstore|rds|cache|event"`
-
-		DistinctField string `help:"list the specified distinct field, e.g. city, region"`
 	}
 	R(&CloudregionListOptions{}, "cloud-region-list", "List cloud regions", func(s *mcclient.ClientSession, opts *CloudregionListOptions) error {
 		params, err := options.ListStructToParams(opts)
 		if err != nil {
 			return err
 		}
-		if len(opts.DistinctField) > 0 {
-			params.Add(jsonutils.NewString(opts.DistinctField), "extra_field")
-			result, err := modules.Cloudregions.Get(s, "distinct-field", params)
-			if err != nil {
-				return err
-			}
-			fmt.Println(result)
-			return nil
-		}
 		result, err := modules.Cloudregions.List(s, params)
 		if err != nil {
 			return err
--- a/cmd/climc/shell/compute/cloudprovider_quota.go
+++ b/cmd/climc/shell/compute/cloudprovider_quota.go
@@ -1,60 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type CloudproviderQuotaListOptions struct {
-		options.BaseListOptions
-		Cloudregion string `help:"ID or Name of Cloudregion"`
-		QuotaType   string `help:"Quota type"`
-		QuotaRange  string `help:"Quota range" choices:"cloudregion|cloudprovider"`
-	}
-
-	R(&CloudproviderQuotaListOptions{}, "cloud-provider-quota-list", "List cloudprovider quota", func(s *mcclient.ClientSession, args *CloudproviderQuotaListOptions) error {
-		params, err := args.Params()
-		if err != nil {
-			return err
-		}
-
-		if len(args.QuotaType) > 0 {
-			params.Add(jsonutils.NewString(args.QuotaType), "quota_type")
-		}
-
-		if len(args.QuotaRange) > 0 {
-			params.Add(jsonutils.NewString(args.QuotaRange), "quota_range")
-		}
-
-		if len(args.Cloudregion) > 0 {
-			params.Add(jsonutils.NewString(args.Cloudregion), "cloudregion")
-		}
-
-		result, err := modules.CloudproviderQuotas.List(s, params)
-		if err != nil {
-			return err
-		}
-
-		printList(result, modules.CloudproviderQuotas.GetColumns(s))
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/compute/common.go
+++ b/cmd/climc/shell/compute/common.go
@@ -1,34 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"yunion.io/x/onecloud/cmd/climc/shell"
-	"yunion.io/x/onecloud/pkg/util/printutils"
-)
-
-var (
-	R                 = shell.R
-	printList         = printutils.PrintJSONList
-	printObject       = printutils.PrintJSONObject
-	printBatchResults = printutils.PrintJSONBatchResults
-
-	InvalidUpdateError   = shell.InvalidUpdateError
-	printObjectRecursive = printutils.PrintJSONObjectRecursive
-
-	exportList = shell.ExportList
-
-	printObjectRecursiveEx = printutils.PrintJSONObjectRecursiveEx
-)
--- a/cmd/climc/shell/compute/globalvpcs.go
+++ b/cmd/climc/shell/compute/globalvpcs.go
@@ -1,109 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"fmt"
-
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type GlobalVpcListOptions struct {
-		options.BaseListOptions
-	}
-	R(&GlobalVpcListOptions{}, "global-vpc-list", "List global vpcs", func(s *mcclient.ClientSession, args *GlobalVpcListOptions) error {
-		params, err := options.ListStructToParams(args)
-		if err != nil {
-			return err
-		}
-		result, err := modules.GlobalVpcs.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(result, modules.GlobalVpcs.GetColumns(s))
-		return nil
-	})
-
-	type GlobalVpcShowOptions struct {
-		ID string `help:"ID or Name of globalvpc"`
-	}
-	R(&GlobalVpcShowOptions{}, "global-vpc-show", "Show details of a global vpc", func(s *mcclient.ClientSession, args *GlobalVpcShowOptions) error {
-		result, err := modules.GlobalVpcs.GetById(s, args.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type GlobalVpcPublicOptions struct {
-		ID            string   `help:"ID or name of global vpc" json:"-"`
-		Scope         string   `help:"sharing scope" choices:"system|domain"`
-		SharedDomains []string `help:"share to domains"`
-	}
-	R(&GlobalVpcPublicOptions{}, "global-vpc-public", "Make global vpc public", func(s *mcclient.ClientSession, args *GlobalVpcPublicOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.GlobalVpcs.PerformAction(s, args.ID, "public", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type GlobalVpcPrivateOptions struct {
-		ID string `help:"ID or name of global vpc" json:"-"`
-	}
-	R(&GlobalVpcPrivateOptions{}, "global-vpc-private", "Make global vpc private", func(s *mcclient.ClientSession, args *GlobalVpcPrivateOptions) error {
-		params := jsonutils.Marshal(args)
-		result, err := modules.GlobalVpcs.PerformAction(s, args.ID, "private", params)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&GlobalVpcShowOptions{}, "global-vpc-change-owner-candidate-domains", "Show candiate domains of a global vpc for changing owner", func(s *mcclient.ClientSession, args *GlobalVpcShowOptions) error {
-		result, err := modules.GlobalVpcs.GetSpecific(s, args.ID, "change-owner-candidate-domains", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type GlobalVpcChangeOwnerOptions struct {
-		ID            string `help:"ID or name of vpc" json:"-"`
-		ProjectDomain string `json:"project_domain" help:"target domain"`
-	}
-	R(&GlobalVpcChangeOwnerOptions{}, "global-vpc-change-owner", "Change owner domain of a global vpc", func(s *mcclient.ClientSession, args *GlobalVpcChangeOwnerOptions) error {
-		if len(args.ProjectDomain) == 0 {
-			return fmt.Errorf("empty project_domain")
-		}
-		params := jsonutils.Marshal(args)
-		ret, err := modules.GlobalVpcs.PerformAction(s, args.ID, "change-owner", params)
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-}
--- a/cmd/climc/shell/compute/policy_definition.go
+++ b/cmd/climc/shell/compute/policy_definition.go
@@ -1,62 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type PolicyListOptions struct {
-		options.BaseListOptions
-	}
-	R(&PolicyListOptions{}, "policy-definition-list", "List policy definitions", func(s *mcclient.ClientSession, args *PolicyListOptions) error {
-		params, err := args.Params()
-		if err != nil {
-			return err
-		}
-		result, err := modules.PolicyDefinition.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(result, modules.PolicyDefinition.GetColumns(s))
-		return nil
-	})
-
-	type PolicyIdOptions struct {
-		ID string `help:"policy definition name or id"`
-	}
-
-	R(&PolicyIdOptions{}, "policy-definition-show", "Show policy definition", func(s *mcclient.ClientSession, args *PolicyIdOptions) error {
-		result, err := modules.PolicyDefinition.Get(s, args.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&PolicyIdOptions{}, "policy-definition-syncstatus", "Sync policy definition status", func(s *mcclient.ClientSession, args *PolicyIdOptions) error {
-		result, err := modules.PolicyDefinition.PerformAction(s, args.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/compute/scaling_activity.go
+++ b/cmd/climc/shell/compute/scaling_activity.go
@@ -1,41 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type ScalingActivityListOptions struct {
-		options.BaseListOptions
-	}
-	R(&ScalingActivityListOptions{}, "scaling-activity-list", "List Scaling Activity",
-		func(s *mcclient.ClientSession, args *ScalingActivityListOptions) error {
-			params, err := options.ListStructToParams(args)
-			if err != nil {
-				return err
-			}
-			list, err := modules.ScalingActivity.List(s, params)
-			if err != nil {
-				return err
-			}
-			printList(list, modules.ScalingActivity.GetColumns(s))
-			return nil
-		},
-	)
-}
--- a/cmd/climc/shell/compute/scaling_group.go
+++ b/cmd/climc/shell/compute/scaling_group.go
@@ -1,145 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type ScalingGroupListOptions struct {
-		options.BaseListOptions
-		Hypervisor  string `help:"Hypervisor"`
-		Cloudregion string `help:"Cloudregion"`
-		Network     string `help:"network"`
-	}
-	R(&ScalingGroupListOptions{}, "scaling-group-list", "List scaling group", func(s *mcclient.ClientSession, args *ScalingGroupListOptions) error {
-		params, err := options.ListStructToParams(args)
-		if err != nil {
-			return err
-		}
-		result, err := modules.ScalingGroup.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(result, modules.ScalingGroup.GetColumns(s))
-		return nil
-	})
-
-	type ScalingGroupShowOptions struct {
-		ID string
-	}
-	R(&ScalingGroupShowOptions{}, "scaling-group-show", "Show scaling group", func(s *mcclient.ClientSession,
-		args *ScalingGroupShowOptions) error {
-		result, err := modules.ScalingGroup.Get(s, args.ID, nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	type ScalingGroupCreateOptions struct {
-		NAME string
-
-		Hypervisor           string
-		Cloudregion          string
-		Network              string
-		GuestTemplate        string
-		MinInstanceNumber    string
-		MaxInstanceNumber    string
-		DesireInstanceNumber string
-		Loadbalance          string
-	}
-	R(&ScalingGroupCreateOptions{}, "scaling-group-create", "Create scaling group", func(s *mcclient.ClientSession, args *ScalingGroupCreateOptions) error {
-		params := jsonutils.Marshal(args).(*jsonutils.JSONDict)
-		scalingGroup, err := modules.ScalingGroup.Create(s, params)
-		if err != nil {
-			return err
-		}
-		printObject(scalingGroup)
-		return nil
-	})
-	type ScalingGroupDeleteOptions struct {
-		ID string `help:"ScalingGroup ID or Name"`
-	}
-	R(&ScalingGroupDeleteOptions{}, "scaling-group-delete", "Delete Scaling Group", func(s *mcclient.ClientSession,
-		args *ScalingGroupDeleteOptions) error {
-		scalingGroup, err := modules.ScalingGroup.Delete(s, args.ID, jsonutils.NewDict())
-		if err != nil {
-			return err
-		}
-		printObject(scalingGroup)
-		return nil
-	})
-	type ScalingGroupListInstanceOptions struct {
-		ID string `help:"ScalingGroup ID or Name"`
-	}
-	R(&ScalingGroupListInstanceOptions{}, "scaling-group-list-instance", "List all instance of Scaling Group",
-		func(s *mcclient.ClientSession, args *ScalingGroupListInstanceOptions) error {
-			params := jsonutils.NewDict()
-			params.Set("scaling_group", jsonutils.NewString(args.ID))
-			servers, err := modules.Servers.List(s, params)
-			if err != nil {
-				return err
-			}
-			printList(servers, modules.Servers.GetColumns(s))
-			return nil
-		},
-	)
-	type ScalingGroupRemoveInstanceOptions struct {
-		ID     string `help:"ScalingGroup ID or Name"`
-		SERVER string `help:"Server ID or Name"`
-		Delete bool   `help:"If delete the server"`
-	}
-	R(&ScalingGroupRemoveInstanceOptions{}, "scaling-group-remove-instance", "Remove instance of ScalingGroup",
-		func(s *mcclient.ClientSession, args *ScalingGroupRemoveInstanceOptions) error {
-			params := jsonutils.NewDict()
-			params.Set("scaling_group", jsonutils.NewString(args.ID))
-			params.Set("delete_server", jsonutils.NewBool(args.Delete))
-			server, err := modules.Servers.PerformAction(s, args.SERVER, "detach-scaling-group", params)
-			if err != nil {
-				return err
-			}
-			printObject(server)
-			return nil
-		},
-	)
-	type ScalingGroupEnableOptions struct {
-		ID string `help:"ScalingGroup ID or Name"`
-	}
-	R(&ScalingGroupEnableOptions{}, "scaling-group-enable", "Enable ScalingGroup", func(s *mcclient.ClientSession,
-		args *ScalingGroupEnableOptions) error {
-		ret, err := modules.ScalingGroup.PerformAction(s, args.ID, "enable", jsonutils.NewDict())
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-	R(&ScalingGroupEnableOptions{}, "scaling-group-disable", "Disable ScalingGroup", func(s *mcclient.ClientSession,
-		args *ScalingGroupEnableOptions) error {
-		ret, err := modules.ScalingGroup.PerformAction(s, args.ID, "disable", jsonutils.NewDict())
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-}
--- a/cmd/climc/shell/compute/scaling_policy.go
+++ b/cmd/climc/shell/compute/scaling_policy.go
@@ -1,202 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"fmt"
-	"time"
-
-	"yunion.io/x/jsonutils"
-
-	api "yunion.io/x/onecloud/pkg/apis/compute"
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-type Timer struct {
-	TimingExecTime string `help:"Exectime for 'timing' type trigger, format:'2006-01-02 15:04:05'" json:"exec_time"`
-}
-
-type CycleTimer struct {
-	CycleCycleType string `help:"Cycle type for cycle timer" json:"cycle_type" choices:"day|week|month"`
-	CycleMinute    int    `help:"Minute of cycle timer" json:"minute"`
-	CycleHour      int    `help:"Hour of cycle timer" json:"hour"`
-	CycleWeekdays  []int  `help:"Weekdays for cycle timer" json:"weekdays"`
-	CycleMonthDays []int  `help:"Month days for cycle timer" json:"month_days"`
-	CycleStartTime string `help:"Start time for cycle timer, format:'2006-01-02 15:04:05'" json:"start_time"`
-	CycleEndTime   string `help:"End time for cycle timer, format:'2006-01-02 15:04:05'" json:"end_time"`
-}
-
-func init() {
-	type ScalingPolicyListOptions struct {
-		options.BaseListOptions
-		ScalingGroup string `help:"ScalingGroup ID or Name"`
-		TriggerType  string `help:"Trigger type" choices:"alarm|timing|cycle"`
-	}
-	R(&ScalingPolicyListOptions{}, "scaling-policy-list", "List Scaling Policy", func(s *mcclient.ClientSession,
-		args *ScalingPolicyListOptions) error {
-		params, err := options.ListStructToParams(args)
-		policies, err := modules.ScalingPolicy.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(policies, modules.ScalingPolicy.GetColumns(s))
-		return nil
-	})
-
-	type ScalingPolicyShowOptions struct {
-		ID string `help:"ScalingPolicy ID or Name"`
-	}
-	R(&ScalingPolicyShowOptions{}, "scaling-policy-show", "Show Scaling Policy", func(s *mcclient.ClientSession,
-		args *ScalingPolicyShowOptions) error {
-		params := jsonutils.NewDict()
-		params.Set("details", jsonutils.JSONTrue)
-		policy, err := modules.ScalingPolicy.Get(s, args.ID, params)
-		if err != nil {
-			return err
-		}
-		printObject(policy)
-		return nil
-	})
-
-	type ScalingAlarm struct {
-		AlarmCumulate  int     `help:"Cumulate times alarm will trigger, for 'alarm' trigger" json:"cumulate"`
-		AlarmCycle     int     `help:"Monitoring cycle for indicators, for 'alarm' trigger" json:"cycle"`
-		AlarmIndicator string  `help:"Indicator for 'alarm' trigger" json:"indicator"`
-		AlarmWrapper   string  `help:"Wrapper for Indicators" choices:"max|min|average" json:"wrapper"`
-		AlarmOperator  string  `help:"Operator between Indicator and Operator" json:"operator"`
-		AlarmValue     float64 `help:"Value of Indicator" json:"value"`
-	}
-
-	type ScalingPolicyCreateOptions struct {
-		NAME         string `help:"ScalingPolicy Name" json:"name"`
-		ScalingGroup string `help:"ScalingGroup ID or Name" json:"scaling_group"`
-		TriggerType  string `help:"Trigger type" choices:"alarm|timing|cycle" json:"trigger_type"`
-
-		Timer
-		CycleTimer
-		ScalingAlarm
-
-		Action      string `help:"Action for scaling policy" choices:"add|remove|set" json:"action"`
-		Number      int    `help:"Instance number for action" json:"number"`
-		Unit        string `help:"Unit for Number" choices:"s|%" json:"unit"`
-		CoolingTime int    `help:"Cooling time, unit: s" json:"cooling_time"`
-	}
-
-	R(&ScalingPolicyCreateOptions{}, "scaling-policy-create", "Create Scaling Policy",
-		func(s *mcclient.ClientSession, args *ScalingPolicyCreateOptions) error {
-			formatStr := "2006-01-02 15:04:05"
-			timingExecTime, err := time.Parse(formatStr, args.TimingExecTime)
-			if err != nil {
-				return fmt.Errorf("invalid time format for 'exec_time'")
-			}
-			cycleStarTime, err := time.Parse(formatStr, args.CycleStartTime)
-			if err != nil {
-				return fmt.Errorf("invalid time format for 'start_time'")
-			}
-			cycleEndTime, err := time.Parse(formatStr, args.CycleEndTime)
-			if err != nil {
-				return fmt.Errorf("invalid time format for 'end_time'")
-			}
-			spCreateInput := api.ScalingPolicyCreateInput{
-				ScalingGroup: args.ScalingGroup,
-				TriggerType:  args.TriggerType,
-				Timer: api.TimerCreateInput{
-					ExecTime: timingExecTime,
-				},
-				CycleTimer: api.CycleTimerCreateInput{
-					CycleType: args.CycleCycleType,
-					Minute:    args.CycleMinute,
-					Hour:      args.CycleHour,
-					WeekDays:  args.CycleWeekdays,
-					MonthDays: args.CycleMonthDays,
-					StartTime: cycleStarTime,
-					EndTime:   cycleEndTime,
-				},
-				Alarm: api.ScalingAlarmCreateInput{
-					Cumulate:  args.AlarmCumulate,
-					Cycle:     args.AlarmCycle,
-					Indicator: args.AlarmIndicator,
-					Wrapper:   args.AlarmWrapper,
-					Operator:  args.AlarmOperator,
-					Value:     args.AlarmValue,
-				},
-				Action:      args.Action,
-				Number:      args.Number,
-				Unit:        args.Unit,
-				CoolingTime: args.CoolingTime,
-			}
-			spCreateInput.Name = args.NAME
-			ret, err := modules.ScalingPolicy.Create(s, jsonutils.Marshal(spCreateInput))
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-
-	type ScalingPolicyEnableOptions struct {
-		ID string `help:"ScalingPolicy ID or Name"`
-	}
-	R(&ScalingPolicyEnableOptions{}, "scaling-policy-enable", "Enable ScalingPolicy", func(s *mcclient.ClientSession,
-		args *ScalingPolicyEnableOptions) error {
-		ret, err := modules.ScalingPolicy.PerformAction(s, args.ID, "enable", jsonutils.NewDict())
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-	R(&ScalingPolicyEnableOptions{}, "scaling-policy-disable", "Disable ScalingPolicy",
-		func(s *mcclient.ClientSession, args *ScalingPolicyEnableOptions) error {
-			ret, err := modules.ScalingPolicy.PerformAction(s, args.ID, "disable", jsonutils.NewDict())
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-
-	type ScalingPolicyTriggerOptions struct {
-		ID string `help:"ScalingPolicy ID or Name"`
-	}
-	R(&ScalingPolicyTriggerOptions{}, "scaling-policy-trigger", "Trigger ScalingPolicy's action",
-		func(s *mcclient.ClientSession, args *ScalingPolicyTriggerOptions) error {
-			ret, err := modules.ScalingPolicy.PerformAction(s, args.ID, "trigger", jsonutils.NewDict())
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-
-	type ScalingPolicyDeleteOptions struct {
-		ID string `help:"ScalingPolicy ID or Name"`
-	}
-	R(&ScalingPolicyDeleteOptions{}, "scaling-policy-delete", "Delete SclaingPolicy",
-		func(s *mcclient.ClientSession, args *ScalingPolicyDeleteOptions) error {
-			ret, err := modules.ScalingPolicy.Delete(s, args.ID, jsonutils.NewDict())
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-}
--- a/cmd/climc/shell/compute/scheduled_task.go
+++ b/cmd/climc/shell/compute/scheduled_task.go
@@ -1,211 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"fmt"
-	"time"
-
-	"yunion.io/x/jsonutils"
-
-	apis "yunion.io/x/onecloud/pkg/apis/compute"
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type ScheduledTaskListOptions struct {
-		options.BaseListOptions
-
-		ScheduledType string `help:"scheduled type" choices:"timing|cycle"`
-		ResourceType  string `help:"resource type"`
-		Operation     string `help:"operation"`
-	}
-	R(&ScheduledTaskListOptions{}, "scheduledtask-list", "list Scheduled Task", func(s *mcclient.ClientSession, args *ScheduledTaskListOptions) error {
-		params, err := options.ListStructToParams(args)
-		tasks, err := modules.ScheduledTask.List(s, params)
-		if err != nil {
-			return err
-		}
-		printList(tasks, modules.ScheduledTask.GetColumns(s))
-		return nil
-	})
-
-	type ScheduledTaskShowOptions struct {
-		ID string `help:"ScheduledTask ID or Name"`
-	}
-	R(&ScheduledTaskShowOptions{}, "scheudled-task-show", "Show Scheduled Task", func(s *mcclient.ClientSession,
-		args *ScheduledTaskShowOptions) error {
-		params := jsonutils.NewDict()
-		params.Set("details", jsonutils.JSONTrue)
-		task, err := modules.ScheduledTask.Get(s, args.ID, params)
-		if err != nil {
-			return err
-		}
-		printObject(task)
-		return nil
-	})
-
-	type ScheduledTaskCreateOptions struct {
-		NAME          string `help:"ScheduledTask Name" json:"name"`
-		ScheduledType string `help:"Scheudled Type" choices:"timing|cycle" json:"scheduled_type"`
-
-		Timer
-		CycleTimer
-
-		ResourceType string   `help:"resource type"`
-		Operation    string   `help:"operation"`
-		LabelType    string   `help:"label type"`
-		Labels       []string `help:"labels"`
-	}
-	R(&ScheduledTaskCreateOptions{}, "scheduledtask-create", "Create Scheduled Task", func(s *mcclient.ClientSession, args *ScheduledTaskCreateOptions) error {
-		formatStr := "2006-01-02 15:04:05"
-		var exectime, starttime, endtime time.Time
-		var err error
-		if len(args.TimingExecTime) > 0 {
-			exectime, err = time.Parse(formatStr, args.TimingExecTime)
-			if err != nil {
-				return fmt.Errorf("invalid time format for 'exec_time'")
-			}
-		}
-		if len(args.CycleStartTime) > 0 {
-			starttime, err = time.Parse(formatStr, args.CycleStartTime)
-			if err != nil {
-				return fmt.Errorf("invalid time format for 'start_time'")
-			}
-		}
-		if len(args.CycleEndTime) > 0 {
-			endtime, err = time.Parse(formatStr, args.CycleEndTime)
-			if err != nil {
-				return fmt.Errorf("invalid time format for 'end_time'")
-			}
-		}
-		stCreateInput := apis.ScheduledTaskCreateInput{
-			ScheduledType: args.ScheduledType,
-			Timer: apis.TimerCreateInput{
-				ExecTime: exectime,
-			},
-			CycleTimer: apis.CycleTimerCreateInput{
-				CycleType: args.CycleCycleType,
-				Minute:    args.CycleMinute,
-				Hour:      args.CycleHour,
-				WeekDays:  args.CycleWeekdays,
-				MonthDays: args.CycleMonthDays,
-				StartTime: starttime,
-				EndTime:   endtime,
-			},
-			ResourceType: args.ResourceType,
-			Operation:    args.Operation,
-			LabelType:    args.LabelType,
-			Labels:       args.Labels,
-		}
-		stCreateInput.Name = args.NAME
-		ret, err := modules.ScheduledTask.Create(s, jsonutils.Marshal(stCreateInput))
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-
-	type ScheduledTaskEnableOptions struct {
-		ID string `help:"ScheduledTask ID or Name"`
-	}
-	R(&ScheduledTaskEnableOptions{}, "scheduledtask-enable", "Enable ScheduledTask", func(s *mcclient.ClientSession,
-		args *ScheduledTaskEnableOptions) error {
-		ret, err := modules.ScheduledTask.PerformAction(s, args.ID, "enable", jsonutils.NewDict())
-		if err != nil {
-			return err
-		}
-		printObject(ret)
-		return nil
-	})
-	R(&ScheduledTaskEnableOptions{}, "scheduledtask-disable", "Disable ScheduledTask",
-		func(s *mcclient.ClientSession, args *ScheduledTaskEnableOptions) error {
-			ret, err := modules.ScheduledTask.PerformAction(s, args.ID, "disable", jsonutils.NewDict())
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-
-	type ScheduledTaskSetLabelsOptions struct {
-		ID     string   `help:"ScheduledTask ID or Name"`
-		Labels []string `help:"Label"`
-	}
-	R(&ScheduledTaskSetLabelsOptions{}, "scheduledtask-setlabels", "Trigger ScheduledTask's action",
-		func(s *mcclient.ClientSession, args *ScheduledTaskSetLabelsOptions) error {
-			params := jsonutils.NewDict()
-			params.Set("labels", jsonutils.Marshal(args.Labels))
-			ret, err := modules.ScheduledTask.PerformAction(s, args.ID, "set-labels", params)
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-
-	type ScheduledTaskTriggerOptions struct {
-		ID string `help:"ScheduledTask ID or Name"`
-	}
-	R(&ScheduledTaskTriggerOptions{}, "scheduledtask-trigger", "Trigger ScheduledTask",
-		func(s *mcclient.ClientSession, args *ScheduledTaskTriggerOptions) error {
-			params := jsonutils.NewDict()
-			ret, err := modules.ScheduledTask.PerformAction(s, args.ID, "trigger", params)
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-
-	type ScheduledTaskDeleteOptions struct {
-		ID string `help:"ScheduledTask ID or Name"`
-	}
-	R(&ScheduledTaskDeleteOptions{}, "scheduledtask-delete", "Delete ScheduledTask",
-		func(s *mcclient.ClientSession, args *ScheduledTaskDeleteOptions) error {
-			ret, err := modules.ScheduledTask.Delete(s, args.ID, jsonutils.NewDict())
-			if err != nil {
-				return err
-			}
-			printObject(ret)
-			return nil
-		},
-	)
-
-	type ScheduledTaskAvtivityListOptions struct {
-		options.BaseListOptions
-		ScheduledTask string `help:"Scheduled Task" json:"scheduled_task"`
-	}
-	R(&ScheduledTaskAvtivityListOptions{}, "scheduledtask-activity-list", "List Scheduled Task Activity",
-		func(s *mcclient.ClientSession, args *ScheduledTaskAvtivityListOptions) error {
-			params, err := options.ListStructToParams(args)
-			if err != nil {
-				return err
-			}
-			list, err := modules.ScheduledTaskActivity.List(s, params)
-			if err != nil {
-				return err
-			}
-			printList(list, modules.ScheduledTaskActivity.GetColumns(s))
-			return nil
-		},
-	)
-}
--- a/cmd/climc/shell/compute/serversecgroups.go
+++ b/cmd/climc/shell/compute/serversecgroups.go
@@ -1,72 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package compute
-
-import (
-	"yunion.io/x/jsonutils"
-
-	"yunion.io/x/onecloud/pkg/mcclient"
-	"yunion.io/x/onecloud/pkg/mcclient/modulebase"
-	"yunion.io/x/onecloud/pkg/mcclient/modules"
-	"yunion.io/x/onecloud/pkg/mcclient/options"
-)
-
-func init() {
-	type ServerSecgroupListOptions struct {
-		options.BaseListOptions
-		Server   string `help:"ID or Name of Server"`
-		Secgroup string `help:"Secgroup ID or name"`
-	}
-	R(&ServerSecgroupListOptions{}, "server-secgroup-list", "List server secgroup pairs", func(s *mcclient.ClientSession, args *ServerSecgroupListOptions) error {
-		var params *jsonutils.JSONDict
-		{
-			var err error
-			params, err = args.BaseListOptions.Params()
-			if err != nil {
-				return err
-
-			}
-		}
-		var result *modulebase.ListResult
-		var err error
-		if len(args.Server) > 0 {
-			result, err = modules.Serversecgroups.ListDescendent(s, args.Server, params)
-		} else if len(args.Secgroup) > 0 {
-			result, err = modules.Serversecgroups.ListDescendent2(s, args.Secgroup, params)
-		} else {
-			result, err = modules.Serversecgroups.List(s, params)
-		}
-		if err != nil {
-			return err
-		}
-		printList(result, modules.Serversecgroups.GetColumns(s))
-		return nil
-	})
-
-	type ServerSecgroupDetailOptions struct {
-		SERVER   string `help:"ID or Name of Server"`
-		SECGROUP string `help:"ID or Name of Security Group"`
-	}
-	R(&ServerSecgroupDetailOptions{}, "server-secgroup-show", "Show server security group details", func(s *mcclient.ClientSession, args *ServerSecgroupDetailOptions) error {
-		query := jsonutils.NewDict()
-		result, err := modules.Serversecgroups.Get(s, args.SERVER, args.SECGROUP, query)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-}
--- a/cmd/climc/shell/notify/contactgroups.go
+++ b/cmd/climc/shell/notify/contactgroups.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package notify
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/notify/contacts.go
+++ b/cmd/climc/shell/notify/contacts.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package notify
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/misc/copyright.go
+++ b/cmd/climc/shell/misc/copyright.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package misc
+package shell

 import (
 	"bufio"
@@ -35,7 +35,6 @@ func init() {
 		Copyright string  `help:"The copyright"`
 		Email     string  `help:"The Email"`
 		Docs      *string `help:"the Docs website address"`
-		License   *string `help:"the license generator website address"`
 	}

 	R(&CopyrightUpdateOptions{}, "copyright-update", "update copyright", func(s *mcclient.ClientSession, args *CopyrightUpdateOptions) error {
@@ -48,10 +47,6 @@ func init() {
 			params.Add(jsonutils.NewString(*args.Docs), "docs")
 		}

-		if args.License != nil {
-			params.Add(jsonutils.NewString(*args.License), "license")
-		}
-
 		if len(args.Copyright) > 0 {
 			params.Add(jsonutils.NewString(args.Copyright), "copyright")
 		}
--- a/cmd/climc/shell/identity/credentials.go
+++ b/cmd/climc/shell/identity/credentials.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package identity
+package shell

 import (
 	"fmt"
--- a/cmd/climc/shell/misc/current.go
+++ b/cmd/climc/shell/misc/current.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package misc
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/compute/dbinstance_accounts.go
+++ b/cmd/climc/shell/compute/dbinstance_accounts.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/compute/dbinstance_backups.go
+++ b/cmd/climc/shell/compute/dbinstance_backups.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -38,11 +38,7 @@ func init() {
 		if err != nil {
 			return err
 		}
-		if len(opts.ExportFile) > 0 {
-			exportList(result, opts.ExportFile, opts.ExportKeys, opts.ExportTexts, modules.DBInstanceBackups.GetColumns(s))
-		} else {
-			printList(result, modules.DBInstanceBackups.GetColumns(s))
-		}
+		printList(result, modules.DBInstanceBackups.GetColumns(s))
 		return nil
 	})

@@ -84,15 +80,6 @@ func init() {
 		return nil
 	})

-	R(&DBInstanceBackupIdOptions{}, "dbinstance-backup-syncstatus", "Sync DB instance backup status", func(s *mcclient.ClientSession, opts *DBInstanceBackupIdOptions) error {
-		result, err := modules.DBInstanceBackups.PerformAction(s, opts.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
 	R(&DBInstanceBackupIdOptions{}, "dbinstance-backup-delete", "Delete DB instance backup", func(s *mcclient.ClientSession, opts *DBInstanceBackupIdOptions) error {
 		result, err := modules.DBInstanceBackups.Delete(s, opts.ID, nil)
 		if err != nil {
--- a/cmd/climc/shell/compute/dbinstance_databases.go
+++ b/cmd/climc/shell/compute/dbinstance_databases.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/compute/dbinstance_skus.go
+++ b/cmd/climc/shell/compute/dbinstance_skus.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/compute/dbinstances.go
+++ b/cmd/climc/shell/compute/dbinstances.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"fmt"
@@ -41,12 +41,7 @@ func init() {
 		if err != nil {
 			return err
 		}
-
-		if len(opts.ExportFile) > 0 {
-			exportList(result, opts.ExportFile, opts.ExportKeys, opts.ExportTexts, modules.DBInstance.GetColumns(s))
-		} else {
-			printList(result, modules.DBInstance.GetColumns(s))
-		}
+		printList(result, modules.DBInstance.GetColumns(s))
 		return nil
 	})

@@ -263,24 +258,6 @@ func init() {
 		return nil
 	})

-	R(&DBInstanceIdOptions{}, "dbinstance-syncstatus", "Sync status for DB instance", func(s *mcclient.ClientSession, opts *DBInstanceIdOptions) error {
-		result, err := modules.DBInstance.PerformAction(s, opts.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
-	R(&DBInstanceIdOptions{}, "dbinstance-sync", "Sync conf from cloud dbinstance", func(s *mcclient.ClientSession, opts *DBInstanceIdOptions) error {
-		result, err := modules.DBInstance.PerformAction(s, opts.ID, "sync", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
 	R(&DBInstanceIdOptions{}, "dbinstance-sync-status", "Sync status for DB instance", func(s *mcclient.ClientSession, opts *DBInstanceIdOptions) error {
 		result, err := modules.DBInstance.PerformAction(s, opts.ID, "sync-status", nil)
 		if err != nil {
--- a/cmd/climc/shell/ansible/devtoolcronjob.go
+++ b/cmd/climc/shell/ansible/devtoolcronjob.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package ansible
+package shell

 import (
 	"fmt"
--- a/cmd/climc/shell/ansible/devtooltemplate.go
+++ b/cmd/climc/shell/ansible/devtooltemplate.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package ansible
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -107,7 +107,7 @@ func init() {
 	R(
 		&options.DevtoolTemplateBindingOptions{},
 		"devtooltemplate-unbind",
-		"UnBinding devtool template to a host/vm",
+		"Binding devtool template to a host/vm",
 		func(s *mcclient.ClientSession, opts *options.DevtoolTemplateBindingOptions) error {
 			params := jsonutils.NewDict()
 			params.Set("server_id", jsonutils.NewString(opts.ServerID))
--- a/cmd/climc/shell/compute/disks.go
+++ b/cmd/climc/shell/compute/disks.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -133,15 +133,6 @@ func init() {
 		return nil
 	})

-	R(&DiskDetailOptions{}, "disk-syncstatus", "Sync status for disk", func(s *mcclient.ClientSession, args *DiskDetailOptions) error {
-		ret, e := modules.Disks.PerformAction(s, args.ID, "syncstatus", nil)
-		if e != nil {
-			return e
-		}
-		printObject(ret)
-		return nil
-	})
-
 	type DiskUpdateOptions struct {
 		ID           string `help:"ID or name of disk"`
 		Name         string `help:"New name of disk"`
@@ -291,14 +282,4 @@ func init() {
 		printObject(srv)
 		return nil
 	})
-
-	R(&DiskDetailOptions{}, "disk-change-owner-candidate-domains", "Get change owner candidate domain list", func(s *mcclient.ClientSession, args *DiskDetailOptions) error {
-		result, err := modules.Disks.GetSpecific(s, args.ID, "change-owner-candidate-domains", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
 }
--- a/cmd/climc/shell/compute/dnsrecords.go
+++ b/cmd/climc/shell/compute/dnsrecords.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/onecloud/pkg/mcclient"
--- a/cmd/climc/shell/identity/domains.go
+++ b/cmd/climc/shell/identity/domains.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package identity
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -83,8 +83,6 @@ func init() {
 		Desc     string `help:"Description"`
 		Enabled  bool   `help:"Set the domain enabled"`
 		Disabled bool   `help:"Set the domain disabled"`
-
-		Displayname string `help:"display name"`
 	}
 	R(&DomainCreateOptions{}, "domain-create", "Create a new domain", func(s *mcclient.ClientSession, args *DomainCreateOptions) error {
 		params := jsonutils.NewDict()
@@ -97,9 +95,6 @@ func init() {
 		} else if !args.Enabled && args.Disabled {
 			params.Add(jsonutils.JSONFalse, "enabled")
 		}
-		if len(args.Displayname) > 0 {
-			params.Add(jsonutils.NewString(args.Displayname), "displayname")
-		}
 		result, err := modules.Domains.Create(s, params)
 		if err != nil {
 			return err
@@ -115,8 +110,6 @@ func init() {
 		Enabled  bool   `help:"Set the domain enabled"`
 		Disabled bool   `help:"Set the domain disabled"`
 		Driver   string `help:"Set the domain Driver"`
-
-		Displayname string `help:"display name"`
 	}
 	R(&DomainUpdateOptions{}, "domain-update", "Update a domain", func(s *mcclient.ClientSession, args *DomainUpdateOptions) error {
 		obj, err := modules.Domains.Get(s, args.ID, nil)
@@ -143,9 +136,6 @@ func init() {
 		} else if !args.Enabled && args.Disabled {
 			params.Add(jsonutils.JSONFalse, "enabled")
 		}
-		if len(args.Displayname) > 0 {
-			params.Add(jsonutils.NewString(args.Displayname), "displayname")
-		}
 		result, err := modules.Domains.Patch(s, objId, params)
 		if err != nil {
 			return err
--- a/cmd/climc/shell/compute/dynamicschedtags.go
+++ b/cmd/climc/shell/compute/dynamicschedtags.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/compute/elasticcache.go
+++ b/cmd/climc/shell/compute/elasticcache.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"fmt"
@@ -75,16 +75,6 @@ func init() {
 		return nil
 	})

-	R(&options.ElasticCacheIdOptions{}, "elastic-cache-syncstatus", "Sync elastisc cache instance status", func(s *mcclient.ClientSession, opts *options.ElasticCacheIdOptions) error {
-		result, err := modules.ElasticCache.PerformAction(s, opts.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-
-		printObject(result)
-		return nil
-	})
-
 	R(&options.ElasticCacheIdOptions{}, "elastic-cache-delete", "Delete elastisc cache instance", func(s *mcclient.ClientSession, opts *options.ElasticCacheIdOptions) error {
 		result, err := modules.ElasticCache.Delete(s, opts.ID, nil)
 		if err != nil {
--- a/cmd/climc/shell/compute/elasticips.go
+++ b/cmd/climc/shell/compute/elasticips.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -46,9 +46,9 @@ func init() {
 	})

 	type EipCreateOptions struct {
+		MANAGER    string `help:"cloud provider"`
+		REGION     string `help:"cloud region in which EIP is allocated"`
 		NAME       string `help:"name of the EIP"`
-		Manager    string `help:"cloud provider"`
-		Region     string `help:"cloud region in which EIP is allocated"`
 		Bandwidth  int    `help:"Bandwidth in Mbps"`
 		Ip         string `help:"IP address of the EIP"`
 		Network    string `help:"Network of the EIP"`
@@ -56,13 +56,9 @@ func init() {
 	}
 	R(&EipCreateOptions{}, "eip-create", "Create an EIP", func(s *mcclient.ClientSession, args *EipCreateOptions) error {
 		params := jsonutils.NewDict()
+		params.Add(jsonutils.NewString(args.MANAGER), "manager")
+		params.Add(jsonutils.NewString(args.REGION), "region")
 		params.Add(jsonutils.NewString(args.NAME), "name")
-		if args.Region != "" {
-			params.Add(jsonutils.NewString(args.Region), "region")
-		}
-		if args.Manager != "" {
-			params.Add(jsonutils.NewString(args.Manager), "manager")
-		}
 		if args.Bandwidth != 0 {
 			params.Add(jsonutils.NewInt(int64(args.Bandwidth)), "bandwidth")
 		}
@@ -174,15 +170,6 @@ func init() {
 		return nil
 	})

-	R(&EipSingleOptions{}, "eip-syncstatus", "Synchronize status of an EIP", func(s *mcclient.ClientSession, args *EipSingleOptions) error {
-		result, err := modules.Elasticips.PerformAction(s, args.ID, "syncstatus", nil)
-		if err != nil {
-			return err
-		}
-		printObject(result)
-		return nil
-	})
-
 	type ServerCreateEipOptions struct {
 		ID         string `help:"server ID or name"`
 		BW         int    `help:"EIP bandwidth in Mbps"`
--- a/cmd/climc/shell/notify/email_config.go
+++ b/cmd/climc/shell/notify/email_config.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package notify
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/identity/endpoints.go
+++ b/cmd/climc/shell/identity/endpoints.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package identity
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -64,15 +64,14 @@ func init() {
 	})

 	type EndpointCreateOptions struct {
-		SERVICE            string `help:"Service ID or Name"`
-		REGION             string `help:"Region"`
-		INTERFACE          string `help:"Interface types" choices:"internal|public|admin|console"`
-		URL                string `help:"URL"`
-		Zone               string `help:"Zone"`
-		Name               string `help:"Name"`
-		Enabled            bool   `help:"Enabled"`
-		Disabled           bool   `help:"Disabled"`
-		ServiceCertificate string `help:"Service certificate id or name"`
+		SERVICE   string `help:"Service ID or Name"`
+		REGION    string `help:"Region"`
+		INTERFACE string `help:"Interface types" choices:"internal|public|admin|console"`
+		URL       string `help:"URL"`
+		Zone      string `help:"Zone"`
+		Name      string `help:"Name"`
+		Enabled   bool   `help:"Enabled"`
+		Disabled  bool   `help:"Disabled"`
 	}
 	R(&EndpointCreateOptions{}, "endpoint-create", "Create endpoint", func(s *mcclient.ClientSession, args *EndpointCreateOptions) error {
 		params := jsonutils.NewDict()
@@ -96,9 +95,6 @@ func init() {
 		} else if !args.Enabled && args.Disabled {
 			params.Add(jsonutils.JSONFalse, "enabled")
 		}
-		if len(args.ServiceCertificate) > 0 {
-			params.Add(jsonutils.NewString(args.ServiceCertificate), "service_certificate")
-		}
 		ep, err := modules.EndpointsV3.Create(s, params)
 		if err != nil {
 			return err
@@ -108,12 +104,11 @@ func init() {
 	})

 	type EndpointUpdateOptions struct {
-		ID                 string `help:"ID or name of endpoint"`
-		Url                string `help:"URL"`
-		Name               string `help:"Name"`
-		Enabled            bool   `help:"Enabled"`
-		Disabled           bool   `help:"Disabled"`
-		ServiceCertificate string `help:"Service certificate id or name"`
+		ID       string `help:"ID or name of endpoint"`
+		Url      string `help:"URL"`
+		Name     string `help:"Name"`
+		Enabled  bool   `help:"Enabled"`
+		Disabled bool   `help:"Disabled"`
 	}
 	R(&EndpointUpdateOptions{}, "endpoint-update", "Update a endpoint", func(s *mcclient.ClientSession, args *EndpointUpdateOptions) error {
 		params := jsonutils.NewDict()
@@ -128,9 +123,6 @@ func init() {
 		} else if !args.Enabled && args.Disabled {
 			params.Add(jsonutils.JSONFalse, "enabled")
 		}
-		if len(args.ServiceCertificate) > 0 {
-			params.Add(jsonutils.NewString(args.ServiceCertificate), "service_certificate")
-		}
 		ep, err := modules.EndpointsV3.Patch(s, args.ID, params)
 		if err != nil {
 			return err
--- a/cmd/climc/shell/events/events.go
+++ b/cmd/climc/shell/events/events.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package events
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -34,11 +34,8 @@ type BaseEventListOptions struct {
 	OrderBy    string   `help:"order by specific field"`
 	Action     []string `help:"Log action"`

-	User    string `help:"filter by operator user"`
-	Project string `help:"filter by operator user's project"`
-
-	OwnerProjectIds []string `help:"filter by owner project ids"`
-	OwnerDomainIds  []string `help:"filter by owner domain ids"`
+	User    []string `help:"filter by operator user"`
+	Project []string `help:"filter by owner project"`

 	PagingMarker string `help:"marker for pagination"`
 }
@@ -102,20 +99,14 @@ func doEventList(man modulebase.ResourceManager, s *mcclient.ClientSession, args
 		params.Add(jsonutils.NewStringArray(args.Action), "action")
 	}
 	if len(args.User) > 0 {
-		params.Add(jsonutils.NewString(args.User), "user")
+		params.Add(jsonutils.NewStringArray(args.User), "user")
 	}
 	if len(args.Project) > 0 {
-		params.Add(jsonutils.NewString(args.Project), "project")
+		params.Add(jsonutils.NewStringArray(args.Project), "project")
 	}
 	if len(args.Scope) > 0 {
 		params.Add(jsonutils.NewString(args.Scope), "scope")
 	}
-	if len(args.OwnerProjectIds) > 0 {
-		params.Add(jsonutils.NewStringArray(args.OwnerProjectIds), "owner_project_ids")
-	}
-	if len(args.OwnerDomainIds) > 0 {
-		params.Add(jsonutils.NewStringArray(args.OwnerDomainIds), "owner_domain_ids")
-	}
 	if len(args.PagingMarker) > 0 {
 		params.Add(jsonutils.NewString(args.PagingMarker), "paging_marker")
 	}
@@ -190,6 +181,11 @@ func init() {
 		return doComputeEventList(s, &nargs)
 	})

+	R(&TypeEventListOptions{}, "kube-cluster-event", "Show operation event logs of kubernetes cluster", func(s *mcclient.ClientSession, args *TypeEventListOptions) error {
+		nargs := EventListOptions{BaseEventListOptions: args.BaseEventListOptions, Id: args.ID, Type: []string{"kube_cluster"}}
+		return doK8sEventList(s, &nargs)
+	})
+
 	R(&TypeEventListOptions{}, "kubecluster-event", "Show operation event logs of kubernetes cluster", func(s *mcclient.ClientSession, args *TypeEventListOptions) error {
 		nargs := EventListOptions{BaseEventListOptions: args.BaseEventListOptions, Id: args.ID, Type: []string{"kubecluster"}}
 		return doK8sEventList(s, &nargs)
@@ -220,11 +216,6 @@ func init() {
 		return doIdentityEventList(s, &nargs)
 	})

-	R(&TypeEventListOptions{}, "idp-event", "Show operation event logs of keystone identity provider", func(s *mcclient.ClientSession, args *TypeEventListOptions) error {
-		nargs := EventListOptions{BaseEventListOptions: args.BaseEventListOptions, Id: args.ID, Type: []string{"identity_provider"}}
-		return doIdentityEventList(s, &nargs)
-	})
-
 	R(&TypeEventListOptions{}, "project-event", "Show operation event logs of keystone projects", func(s *mcclient.ClientSession, args *TypeEventListOptions) error {
 		nargs := EventListOptions{BaseEventListOptions: args.BaseEventListOptions, Id: args.ID, Type: []string{"project"}}
 		return doIdentityEventList(s, &nargs)
--- a/cmd/climc/shell/events/common.go
+++ b/cmd/climc/shell/events/common.go
@@ -1,30 +0,0 @@
-// Copyright 2019 Yunion
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package events
-
-import (
-	"yunion.io/x/onecloud/cmd/climc/shell"
-	"yunion.io/x/onecloud/pkg/util/printutils"
-)
-
-var (
-	R                 = shell.R
-	printList         = printutils.PrintJSONList
-	printObject       = printutils.PrintJSONObject
-	printBatchResults = printutils.PrintJSONBatchResults
-
-	InvalidUpdateError   = shell.InvalidUpdateError
-	printObjectRecursive = printutils.PrintJSONObjectRecursive
-)
--- a/cmd/climc/shell/compute/external_projects.go
+++ b/cmd/climc/shell/compute/external_projects.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/cmd/climc/shell/compute/policy_assignment.go
+++ b/cmd/climc/shell/compute/policy_assignment.go
@@ -12,48 +12,40 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
-	"yunion.io/x/jsonutils"
-
 	"yunion.io/x/onecloud/pkg/mcclient"
 	"yunion.io/x/onecloud/pkg/mcclient/modules"
 	"yunion.io/x/onecloud/pkg/mcclient/options"
 )

 func init() {
-	type PolicyListOptions struct {
+	type GlobalVpcListOptions struct {
 		options.BaseListOptions
-		Policydefinition string `help:"filter by policydefinition"`
 	}
-	R(&PolicyListOptions{}, "policy-assignment-list", "List policy assignments", func(s *mcclient.ClientSession, args *PolicyListOptions) error {
-		params, err := args.Params()
+	R(&GlobalVpcListOptions{}, "global-vpc-list", "List global vpcs", func(s *mcclient.ClientSession, args *GlobalVpcListOptions) error {
+		params, err := options.ListStructToParams(args)
 		if err != nil {
 			return err
 		}
-		if len(args.Policydefinition) > 0 {
-			params.Add(jsonutils.NewString(args.Policydefinition), "policydefinition")
-		}
-		result, err := modules.PolicyAssignment.List(s, params)
+		result, err := modules.GlobalVpcs.List(s, params)
 		if err != nil {
 			return err
 		}
-		printList(result, modules.PolicyAssignment.GetColumns(s))
+		printList(result, modules.GlobalVpcs.GetColumns(s))
 		return nil
 	})

-	type PolicyIdOptions struct {
-		ID string `help:"policy assignment id or name"`
+	type GlobalVpcShowOptions struct {
+		ID string `help:"ID or Name of globalvpc"`
 	}
-
-	R(&PolicyIdOptions{}, "policy-assignment-show", "Show policy assignment details", func(s *mcclient.ClientSession, args *PolicyIdOptions) error {
-		result, err := modules.PolicyAssignment.Get(s, args.ID, nil)
+	R(&GlobalVpcShowOptions{}, "global-vpc-show", "Show details of a global vpc", func(s *mcclient.ClientSession, args *GlobalVpcShowOptions) error {
+		result, err := modules.GlobalVpcs.GetById(s, args.ID, nil)
 		if err != nil {
 			return err
 		}
 		printObject(result)
 		return nil
-
 	})
 }
--- a/cmd/climc/shell/identity/groups.go
+++ b/cmd/climc/shell/identity/groups.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package identity
+package shell

 import (
 	"yunion.io/x/jsonutils"
@@ -224,29 +224,4 @@ func init() {
 			printObject(sp)
 			return nil
 		})
-
-	type GroupAddUsersOptions struct {
-		GROUP string   `help:"group Id or Name" json:"-"`
-		USER  []string `help:"list of user id or name" json:"user"`
-	}
-	R(&GroupAddUsersOptions{}, "group-add-users", "Add users into group",
-		func(s *mcclient.ClientSession, args *GroupAddUsersOptions) error {
-			params := jsonutils.Marshal(args)
-			result, err := modules.Groups.PerformAction(s, args.GROUP, "add-users", params)
-			if err != nil {
-				return err
-			}
-			printObject(result)
-			return nil
-		})
-	R(&GroupAddUsersOptions{}, "group-remove-users", "Add users into group",
-		func(s *mcclient.ClientSession, args *GroupAddUsersOptions) error {
-			params := jsonutils.Marshal(args)
-			result, err := modules.Groups.PerformAction(s, args.GROUP, "remove-users", params)
-			if err != nil {
-				return err
-			}
-			printObject(result)
-			return nil
-		})
 }
--- a/cmd/climc/shell/compute/guest_template.go
+++ b/cmd/climc/shell/compute/guest_template.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"fmt"
@@ -145,15 +145,6 @@ func init() {
 		},
 	)

-	R(&GuestTemplateOptions{}, "server-template-inspect", "Inspect server template", func(s *mcclient.ClientSession, opts *GuestTemplateOptions) error {
-		tem, err := modules.GuestTemplate.PerformAction(s, opts.ID, "inspect", jsonutils.JSONNull)
-		if err != nil {
-			return err
-		}
-		printObject(tem)
-		return nil
-	})
-
 	type GuestTemplatePublicOptions struct {
 		ID          string `help:"ID or Name of server template"`
 		PublicScope string `help:"public scope"`
--- a/cmd/climc/shell/compute/hostcachedimages.go
+++ b/cmd/climc/shell/compute/hostcachedimages.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.

-package compute
+package shell

 import (
 	"yunion.io/x/jsonutils"
--- a/Show More
+++ b/Show More