Skip to content

Commit 5504b05

Browse files
Merge branch '4.20' into 4.22
2 parents d160731 + d53b6db commit 5504b05

File tree

3 files changed

+16
-4
lines changed

3 files changed

+16
-4
lines changed

api/src/main/java/org/apache/cloudstack/api/command/admin/storage/UpdateStoragePoolCmd.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,8 @@ public void execute() {
153153
if (ObjectUtils.anyNotNull(name, capacityIops, capacityBytes, url, isTagARule, tags) ||
154154
MapUtils.isNotEmpty(details)) {
155155
result = _storageService.updateStoragePool(this);
156+
} else {
157+
result = _storageService.getStoragePool(getId());
156158
}
157159

158160
if (enabled != null) {

test/integration/smoke/test_direct_download.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,10 @@ def getCurrentStoragePoolTags(self, poolId):
251251
self.apiclient,
252252
id=poolId
253253
)
254-
return local_pool[0].tags
254+
if local_pool[0].tags:
255+
return local_pool[0].tags
256+
else:
257+
return ""
255258

256259
def updateStoragePoolTags(self, poolId, tags):
257260
StoragePool.update(

ui/src/views/compute/DeployVM.vue

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1232,6 +1232,7 @@ export default {
12321232
return _.map(this.affinityGroups, 'id')
12331233
},
12341234
params () {
1235+
const listAll = isAdmin()
12351236
return {
12361237
serviceOfferings: {
12371238
list: 'listServiceOfferings',
@@ -1279,25 +1280,31 @@ export default {
12791280
domainid: this.owner.domainid,
12801281
projectid: this.owner.projectid,
12811282
keyword: undefined,
1282-
listall: false
1283+
listall: listAll
12831284
}
12841285
},
12851286
sshKeyPairs: {
12861287
list: 'listSSHKeyPairs',
12871288
options: {
12881289
page: 1,
12891290
pageSize: 10,
1291+
account: this.owner.account,
1292+
domainid: this.owner.domainid,
1293+
projectid: this.owner.projectid,
12901294
keyword: undefined,
1291-
listall: false
1295+
listall: listAll
12921296
}
12931297
},
12941298
userDatas: {
12951299
list: 'listUserData',
12961300
options: {
12971301
page: 1,
12981302
pageSize: 10,
1303+
account: this.owner.account,
1304+
domainid: this.owner.domainid,
1305+
projectid: this.owner.projectid,
12991306
keyword: undefined,
1300-
listall: false
1307+
listall: listAll
13011308
}
13021309
},
13031310
networks: {

0 commit comments

Comments
 (0)