opensource/cloudpods
3
0
Fork 0
mirror of https://github.com/yunionio/cloudpods.git synced 2026-09-24 16:03:43 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #18787 from swordqiu/hotfix/qj-cloud-account-update-aliyun-billing-scope

Browse Source 
fix: cloud account update aliyun billing scope
This commit is contained in:
Zexi Li
2023-11-27 19:06:37 +08:00
committed by GitHub
parent 19b4400137 11cd6ec3b0
commit 8ee1e7c07c
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/mcclient/options/cloudaccounts.go
+8
View File
@@ -684,6 +684,8 @@ type SAliyunCloudAccountUpdateOptions struct {
RemoveOptionsBillingBucketAccount bool `help:"remove id of account that can access bucket, blank if this account can access" json:"-"`
OptionsBillingFilePrefix string `help:"update prefix of billing file name" json:"-"`
RemoveOptionsBillingFilePrefix bool `help:"remove prefix of billing file name" json:"-"`
OptionsBillingScope string `help:"update billing scope" choices:"all|managed" json:"-"`
RemoveOptionsBillingScope bool `help:"remove billing scope" json:"-"`
}
func (opts *SAliyunCloudAccountUpdateOptions) Params() (jsonutils.JSONObject, error) {
@@ -699,6 +701,9 @@ func (opts *SAliyunCloudAccountUpdateOptions) Params() (jsonutils.JSONObject, er
if len(opts.OptionsBillingFilePrefix) > 0 {
options.Add(jsonutils.NewString(opts.OptionsBillingFilePrefix), "billing_file_prefix")
}
if len(opts.OptionsBillingScope) > 0 {
options.Add(jsonutils.NewString(opts.OptionsBillingScope), "billing_scope")
}
if options.Size() > 0 {
params.Add(options, "options")
}
@@ -712,6 +717,9 @@ func (opts *SAliyunCloudAccountUpdateOptions) Params() (jsonutils.JSONObject, er
if opts.RemoveOptionsBillingFilePrefix {
removeOptions = append(removeOptions, "billing_file_prefix")
}
if opts.RemoveOptionsBillingScope {
removeOptions = append(removeOptions, "billing_scope")
}
if len(removeOptions) > 0 {
params.Add(jsonutils.NewStringArray(removeOptions), "remove_options")
}