Skip to content

Commit 80703ca

Browse files
committed
Merge pull request apache#1325 from remibergsma/vpc_ui_fix_47
CLOUDSTACK-9221 Allow admin to see user VMs on port forwarding pageOn commit a902443 the 'listAll=true' is removed. On some places the domainid and accountid are added but not on these. I added them now. It's either doing this, or readding listAll is true. I've seeing other folks doing that so let's see what performs best. * pr/1325: Admin cannot see VMs on port forwarding page Signed-off-by: Remi Bergsma <github@remi.nl>
2 parents 4b974c5 + f186181 commit 80703ca

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

ui/scripts/network.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3526,6 +3526,8 @@
35263526
if ('vpc' in args.context) {
35273527
var data = {
35283528
//listAll: true, //do not pass listAll to listNetworks under VPC
3529+
domainid: args.context.vpc[0].domainid,
3530+
account: args.context.vpc[0].account,
35293531
supportedservices: 'Lb'
35303532
};
35313533
if (args.context.ipAddresses[0].associatednetworkid == null) {
@@ -4158,6 +4160,8 @@
41584160
if ('vpc' in args.context) {
41594161
var data = {
41604162
//listAll: true, //do not pass listAll to listNetworks under VPC
4163+
domainid: args.context.vpc[0].domainid,
4164+
account: args.context.vpc[0].account,
41614165
supportedservices: 'PortForwarding'
41624166
};
41634167
if (args.context.ipAddresses[0].associatednetworkid == null) {

0 commit comments

Comments
 (0)