Re: Virtual Router execute python slowly

2016-08-19 Thread Gust
A port forward rule operation spent:

not  optimized  5min
optimized 3min
optmized+cProfile8min

top 30 :

Fri Aug 19 11:27:50 2016d:\temp\py_pro.data

 614887355 function calls (614884838 primitive calls) in 548.846 seconds

   Ordered by: cumulative time
   List reduced from 404 to 30 due to restriction <30>

   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
10.0000.000  548.846  548.846 :1()
10.0000.000  548.846  548.846 update_config.py:114(mainp)
10.0000.000  548.846  548.846 update_config.py:50(process_file)
10.0010.001  548.837  548.837 update_config.py:44(finish_config)
10.0000.000  548.836  548.836 configure.py:903(main)
10.0000.000  546.228  546.228 configure.py:717(process)
 1088   53.9360.050  536.6650.493 CsAddress.py:145(ip_in_subnet)
   160.0000.000  364.540   22.784 
configure.py:766(processForwardRule)
   160.0010.000  364.540   22.784 configure.py:772(forward_vr)
   325.8690.183  351.378   10.981 configure.py:747(getNetworkByIp)
 68170816   65.4350.000  342.4270.000 __init__.py:1682(iter_iprange)
 68177344  182.9250.000  277.0510.000 __init__.py:248(__init__)
   483.6850.077  194.8454.059 configure.py:741(getDeviceByIp)
   160.0010.000  181.689   11.356 
configure.py:882(processStaticNatRule)
 68170816   54.6550.000  140.1860.000 __init__.py:63(__hash__)
 68170976   48.5770.000   69.1120.000 __init__.py:439(key)
 68187121   42.9760.000   42.9760.000 {hasattr}
 68188576   26.2020.000   26.2050.000 {isinstance}
 68178432   24.9450.000   24.9450.000 __init__.py:34(__init__)
 68177504   20.5370.000   20.5370.000 __init__.py:232(version)
 68170816   16.4190.000   16.4190.000 {hash}
10.0200.0201.5221.522 CsNetfilter.py:134(compare)
  3290.0070.0001.5130.005 CsHelper.py:182(execute)
  3290.0020.0001.0600.003 subprocess.py:768(communicate)
  3290.0040.0001.0570.003 subprocess.py:1400(_communicate)
  3290.0120.0001.0460.003 
subprocess.py:1433(_communicate_with_poll)
 23521.0210.0001.0210.000 {built-in method poll}
10.0010.0011.0051.005 CsAddress.py:99(process)
   220.0010.0000.9890.045 CsAddress.py:271(configure)
   220.0010.0000.9370.043 CsAddress.py:285(post_configure)



> 在 2016年8月20日,09:41,ilya  写道:
> 
> Hi Gust
> 
> Are you proposing solution "optimized CsAddress.py in VR" or just
> analysis on where time is being spent?
> 
> Regards
> ilya
> On 8/18/16 9:34 PM, Gust wrote:
>> 
>> Hi,all 
>> 
>> we constructed a advance network in cloudstack , it work fine early , but 
>> when I added some VR rules ,it execute slow more and more.
>> 
>> About total 100 rules already exist ,  I add a port forward rule , it would 
>> execute about 3 minutes , that we changed the agent source  set more seconds 
>> timeout ,else agent will timeout at 120s,  and optimized CsAddress.py in VR.
>> 
>> So I login to VR vm , and  modify the update_config.pywith  cProfile , 
>> and print out execute result . because add cProfile , python execute more 
>> than 8 min.
>> 
>> It shows too many netaddr  object init invoked.
>> 
>> vr hypervisor hardware :   E3 1230v2  3.4GHz   , vr alloc 3GHz
>> 
>> --
>> Optimized CsAddress.py , execute 3 min, else 5 min
>> 
>> CsAddress.py (line 145)
>>   def ip_in_subnet(self, ip):
>>   ipo = IPAddress(ip) 
>>   net = IPNetwork("%s/%s" % (self.get_ip(), self.get_size()))
>>   aset = set(net)
>>   return  ipo in aset
>> 
>> --
>> def mainp():
>>   if not (os.path.isfile(jsonCmdConfigPath) and os.access(jsonCmdConfigPath, 
>> os.R_OK)):
>>   print "[ERROR] update_config.py :: You are telling me to process %s, 
>> but i can't access it" % jsonCmdConfigPath
>>   sys.exit(1)
>> 
>>   # If the command line json file is unprocessed process it
>>   # This is important or, the control interfaces will get deleted!
>>   if os.path.isfile(jsonPath % "cmd_line.json"):
>>   qf = QueueFile()
>>   qf.setFile("cmd_line.json")
>>   qf.load(None)
>> 
>>   # If the guest network is already configured and have the same IP, do not 
>> try to configure it again otherwise it will break
>>   if sys.argv[1] == "guest_network.json":
>>   if os.path.isfile(currentGuestNetConfig):
>>   file = open(currentGuestNetConfig)
>>   guestnet_dict = json.load(file)
>> 
>>   if not is_guestnet_configured(guestnet_dict, ['eth1', 'eth2', 
>> 'eth3', 'eth4', 'eth5', 'eth6', 'eth7', 'eth8', 'eth9']):
>>   

Re: Virtual Router execute python slowly

2016-08-19 Thread ilya
Hi Gust

Are you proposing solution "optimized CsAddress.py in VR" or just
analysis on where time is being spent?

Regards
ilya
On 8/18/16 9:34 PM, Gust wrote:
> 
> Hi,all 
> 
> we constructed a advance network in cloudstack , it work fine early , but 
> when I added some VR rules ,it execute slow more and more.
> 
> About total 100 rules already exist ,  I add a port forward rule , it would 
> execute about 3 minutes , that we changed the agent source  set more seconds 
> timeout ,else agent will timeout at 120s,  and optimized CsAddress.py in VR.
> 
> So I login to VR vm , and  modify the update_config.pywith  cProfile , 
> and print out execute result . because add cProfile , python execute more 
> than 8 min.
> 
> It shows too many netaddr  object init invoked.
> 
> vr hypervisor hardware :   E3 1230v2  3.4GHz   , vr alloc 3GHz
> 
> --
> Optimized CsAddress.py , execute 3 min, else 5 min
> 
> CsAddress.py (line 145)
>def ip_in_subnet(self, ip):
>ipo = IPAddress(ip) 
>net = IPNetwork("%s/%s" % (self.get_ip(), self.get_size()))
>aset = set(net)
>return  ipo in aset
> 
> --
> def mainp():
>if not (os.path.isfile(jsonCmdConfigPath) and os.access(jsonCmdConfigPath, 
> os.R_OK)):
>print "[ERROR] update_config.py :: You are telling me to process %s, 
> but i can't access it" % jsonCmdConfigPath
>sys.exit(1)
> 
># If the command line json file is unprocessed process it
># This is important or, the control interfaces will get deleted!
>if os.path.isfile(jsonPath % "cmd_line.json"):
>qf = QueueFile()
>qf.setFile("cmd_line.json")
>qf.load(None)
> 
># If the guest network is already configured and have the same IP, do not 
> try to configure it again otherwise it will break
>if sys.argv[1] == "guest_network.json":
>if os.path.isfile(currentGuestNetConfig):
>file = open(currentGuestNetConfig)
>guestnet_dict = json.load(file)
> 
>if not is_guestnet_configured(guestnet_dict, ['eth1', 'eth2', 
> 'eth3', 'eth4', 'eth5', 'eth6', 'eth7', 'eth8', 'eth9']):
>print "[INFO] update_config.py :: Processing Guest Network."
>process_file()
>else:
>print "[INFO] update_config.py :: No need to process Guest 
> Network."
>finish_config()
>else:
>print "[INFO] update_config.py :: No GuestNetwork configured yet. 
> Configuring first one now."
>process_file()
>else:
>print "[INFO] update_config.py :: Processing incoming file => %s" % 
> sys.argv[1]
>process_file()
> 
> cProfile.run("mainp()", "/var/log/py_pro.data”)
> 
> --
> 
> Fri Aug 19 11:27:50 2016d:\temp\py_pro.data
> 
> 614887355 function calls (614884838 primitive calls) in 548.846 
> seconds
> 
>   Ordered by: internal time
> 
>   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
> 68177344  182.9250.000  277.0510.000 __init__.py:248(__init__)
> 68170816   65.4350.000  342.4270.000 __init__.py:1682(iter_iprange)
> 68170816   54.6550.000  140.1860.000 __init__.py:63(__hash__)
> 1088   53.9360.050  536.6650.493 CsAddress.py:145(ip_in_subnet)
> 68170976   48.5770.000   69.1120.000 __init__.py:439(key)
> 68187121   42.9760.000   42.9760.000 {hasattr}
> 68188576   26.2020.000   26.2050.000 {isinstance}
> 68178432   24.9450.000   24.9450.000 __init__.py:34(__init__)
> 68177504   20.5370.000   20.5370.000 __init__.py:232(version)
> 68170816   16.4190.000   16.4190.000 {hash}
>   325.8690.183  351.378   10.981 configure.py:747(getNetworkByIp)
>   483.6850.077  194.8454.059 configure.py:741(getDeviceByIp)
> 23521.0210.0001.0210.000 {built-in method poll}
>   2454410.3520.0000.5100.000 CsNetfilter.py:296(__eq__)
> 21490.2510.0000.2510.000 {posix.read}
>  7980.1110.0000.6080.001 CsNetfilter.py:116(has_rule)
>   4935850.0980.0000.0980.000 CsNetfilter.py:258(get_table)
>  3290.0770.0000.0770.000 {posix.fork}
> 24440.0590.0000.0590.000 {method 'flush' of 'file' 
> objects}
> 24440.0390.0000.0740.000 __init__.py:243(__init__)
>   1139500.0300.0000.0300.000 CsNetfilter.py:270(get_chain)
>  3290.0280.0000.3670.001 
> subprocess.py:1195(_execute_child)
> 24440.0210.0000.0780.000 __init__.py:458(format)
> 20690.0210.0000.0390.000 
> CsNetfilter.py:236(__convert_to_dict)
>10.0200.0201.5221.522 CsNetfilter.py:134(compare)
> 2444  

RE: Migration of secondary storage templates to another secondary storage location

2016-08-19 Thread Suresh Sadhu
Oh great , you can check the storage-pool,storage_pool_details  table at 
backend db and remove the corresponding entries.

Make sure any existing storage resource are not point to existing storage ID

Regards
Sadhu


-Original Message-
From: Marty Godsey [mailto:ma...@gonsource.com]
Sent: Saturday, August 20, 2016 1:53 AM
To: dev@cloudstack.apache.org
Subject: RE: Migration of secondary storage templates to another secondary 
storage location

Thank you Sadhu. I got it figured out.

I do have a question about primary storage. I tried removing primary storage 
from ACS and even though it said it had volumes attached it did not so I 
forced.. I had another primary on a different server so my VMs are fine.. 
Question I have it now when I go to add back the new primary that was jjust 
reloaded with the same IP, I get that something already exists at this resource 
location though it is not in the GUI..

Where in the DB do I need to delete this?


Regards,
Marty Godsey

-Original Message-
From: Suresh Sadhu [mailto:suresh.sa...@accelerite.com]
Sent: Friday, August 19, 2016 3:19 PM
To: dev@cloudstack.apache.org
Subject: RE: Migration of secondary storage templates to another secondary 
storage location

I don't think we have api for that ,public templates will sync automatically  
and  private template we have  to copy manually  and update the storage id of 
those templates  with new storage Id.

Regards
sadhu

-Original Message-
From: Marty Godsey [mailto:ma...@gonsource.com]
Sent: Friday, August 19, 2016 8:21 PM
To: dev@cloudstack.apache.org
Subject: Migration of secondary storage templates to another secondary storage 
location

Hello,

If I have a secondary storage with templates and ssvms only and I want to 
migrate those to a new secondary storage, is there an API command for that?

Regards,
Marty Godsey




DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.



DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.


RE: Migration of secondary storage templates to another secondary storage location

2016-08-19 Thread Marty Godsey
Thank you Sadhu. I got it figured out.

I do have a question about primary storage. I tried removing primary storage 
from ACS and even though it said it had volumes attached it did not so I 
forced.. I had another primary on a different server so my VMs are fine.. 
Question I have it now when I go to add back the new primary that was jjust 
reloaded with the same IP, I get that something already exists at this resource 
location though it is not in the GUI..

Where in the DB do I need to delete this?


Regards,
Marty Godsey

-Original Message-
From: Suresh Sadhu [mailto:suresh.sa...@accelerite.com] 
Sent: Friday, August 19, 2016 3:19 PM
To: dev@cloudstack.apache.org
Subject: RE: Migration of secondary storage templates to another secondary 
storage location

I don't think we have api for that ,public templates will sync automatically  
and  private template we have  to copy manually  and update the storage id of 
those templates  with new storage Id.

Regards
sadhu

-Original Message-
From: Marty Godsey [mailto:ma...@gonsource.com] 
Sent: Friday, August 19, 2016 8:21 PM
To: dev@cloudstack.apache.org
Subject: Migration of secondary storage templates to another secondary storage 
location

Hello,

If I have a secondary storage with templates and ssvms only and I want to 
migrate those to a new secondary storage, is there an API command for that?

Regards,
Marty Godsey




DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.


[GitHub] cloudstack-docs-admin pull request #40: index.po and ui.po translated in FR

2016-08-19 Thread lmrv
GitHub user lmrv opened a pull request:

https://github.com/apache/cloudstack-docs-admin/pull/40

index.po and ui.po translated in FR

I started working on a french translation of the admin doc.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/lmrv/cloudstack-docs-admin dev

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack-docs-admin/pull/40.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #40


commit 028ffc9f2d7c1727b346d395eadffdd54914fe54
Author: Antoine Le Morvan 
Date:   2016-08-19T22:03:28Z

Index.po and UI.po translation in FR




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


RE: Migration of secondary storage templates to another secondary storage location

2016-08-19 Thread Suresh Sadhu
I don't think we have api for that ,public templates will sync automatically  
and  private template we have  to copy manually  and update the storage id of 
those templates  with new storage Id.

Regards
sadhu

-Original Message-
From: Marty Godsey [mailto:ma...@gonsource.com]
Sent: Friday, August 19, 2016 8:21 PM
To: dev@cloudstack.apache.org
Subject: Migration of secondary storage templates to another secondary storage 
location

Hello,

If I have a secondary storage with templates and ssvms only and I want to 
migrate those to a new secondary storage, is there an API command for that?

Regards,
Marty Godsey




DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.


[GitHub] cloudstack pull request #1652: Marvin Tests: Fix VPC network offering select...

2016-08-19 Thread PaulAngus
GitHub user PaulAngus opened a pull request:

https://github.com/apache/cloudstack/pull/1652

Marvin Tests: Fix VPC network offering selection .

It appears that original code did not expect there to be an alternate 
default VPC offering (Nuage) .
VPC selection uses isdefault=True  which selects the Nuage Network 
offering, stopping the test from being run.
Fix was to look for name="Default VPC Offering" instead.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shapeblue/cloudstack vpcselectionfix

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1652.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1652


commit f1b64a3b509ed59ac0061a4e11dbff34d7ce1ba0
Author: Paul Angus 
Date:   2016-08-19T15:29:22Z

VPC fix network offering.
orginal code did not expect there to be an alternate default (Nuage) there 
when the isdefault=True is used a Nuage Network offering is returned stopping 
the test from being run.
Fix was to look for name="Default VPC Offering"




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1651: Marvin Tests: fix expected return string for ...

2016-08-19 Thread PaulAngus
GitHub user PaulAngus opened a pull request:

https://github.com/apache/cloudstack/pull/1651

Marvin Tests: fix expected return string for successful pings.

fix expected return string for successful pings.  these pings are from the 
test instance inside the isolated networks. The original response was searched 
for was that which would be seen on a Mac.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shapeblue/cloudstack pingfixes

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1651.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1651


commit 686b87d68051d4e3f0a9619d346a29d84d82d5ac
Author: Paul Angus 
Date:   2016-08-19T15:25:57Z

fix expected return string for successful pings.  these pings are from the 
test instance inside the isolated networks. The original response was taken 
from a Mac.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1650: fix macchinina template specificied for vmwar...

2016-08-19 Thread PaulAngus
GitHub user PaulAngus opened a pull request:

https://github.com/apache/cloudstack/pull/1650

fix macchinina template specificied for vmware in Marvin tests

fix macchinina template specificied for vmware - .vmdk.bz2 is not valid… 
for CloudStack. edit to use .ova

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shapeblue/cloudstack templatefix

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1650.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1650


commit 4d22367760ab1c605bbc31e9c749ebb544c4c9ca
Author: Paul Angus 
Date:   2016-08-19T15:23:26Z

fix macchinina template speficied for vmware - .vmdk.bz2 is not valid for 
CloudStack. edit to use .ova




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Migration of secondary storage templates to another secondary storage location

2016-08-19 Thread Marty Godsey
Hello,

If I have a secondary storage with templates and ssvms only and I want to 
migrate those to a new secondary storage, is there an API command for that?

Regards,
Marty Godsey



[GitHub] cloudstack issue #1649: CLOUDSTACK-9463: Fix dynamic-roles migrate script fo...

2016-08-19 Thread PaulAngus
Github user PaulAngus commented on the issue:

https://github.com/apache/cloudstack/pull/1649
  
LGTM tested against a client's customised commands.properties file.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1649: CLOUDSTACK-9463: Fix dynamic-roles migrate script fo...

2016-08-19 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1649
  
Packaging result: ✔centos6 ✔centos7 ✔debian repo: 
http://packages.shapeblue.com/cloudstack/pr/1649
Job ID-93


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

2016-08-19 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1647
  
@wido given it's a pure packaging fix, and as per RMs we've enough LGTMs 
but we lack a test LGTM -- I can help merge this after some tests wrt usage and 
kvm agent (on ubuntu 16.04) over the weekend or next week.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1649: CLOUDSTACK-9463: Fix dynamic-roles migrate script fo...

2016-08-19 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1649
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1649: CLOUDSTACK-9463: Fix dynamic-roles migrate script fo...

2016-08-19 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1649
  
@blueorangutan package


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1649: CLOUDSTACK-9463: Fix dynamic-roles migrate sc...

2016-08-19 Thread rhtyd
GitHub user rhtyd opened a pull request:

https://github.com/apache/cloudstack/pull/1649

CLOUDSTACK-9463: Fix dynamic-roles migrate script for old format

The old commands.properties format included the full class name such as:

createAccount=com.cloud.api.commands.CreateAccountCmd;1

The migration script did not consider this format and fails. With this fix
the migration script will process both the formats, including processing a
commands.properties file with mixed format, for example:

$ cat commands.properties
### Account commands
createAccount=1
deleteAccount=2

markDefaultZoneForAccount=com.cloud.api.commands.MarkDefaultZoneForAccountCmd;3

$ python scripts/util/migrate-dynamicroles.py -d -f commands.properties
Apache CloudStack Role Permission Migration Tool
(c) Apache CloudStack Authors and the ASF, under the Apache License, 
Version 2.0

Running this migration tool will remove any default-role permissions 
from cloud.role_permissions. Do you want to continue? [y/N]y
The commands.properties file has been deprecated and moved at: 
commands.properties.deprecated
Running SQL query: DELETE FROM `cloud`.`role_permissions` WHERE 
`role_id` in (1,2,3,4);
Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, 
`role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 1, '*', 'ALLOW', 
0);
Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, 
`role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 2, 
'deleteAccount', 'ALLOW', 0);
Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, 
`role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 2, 
'markDefaultZoneForAccount', 'ALLOW', 1);
Static role permissions from commands.properties have been migrated 
into the db
Running SQL query: UPDATE `cloud`.`configuration` SET value='true' 
where name='dynamic.apichecker.enabled'
Dynamic role based API checker has been enabled!


/cc @jburwell @karuturi @PaulAngus 

Since we don't have any upgrade/marvin tests for this, the changes have 
been verified with above test as the script works against a 
commands.properties. A manual verification by anyone else would be required to 
validate the changes.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shapeblue/cloudstack 4.9-upgraderolescriptfix

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1649.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1649


commit f699fd4a15c8e58c534e98d8436beedf3b54fdc5
Author: Rohit Yadav 
Date:   2016-08-19T11:57:20Z

CLOUDSTACK-9463: Fix dynamic-roles migrate script for old format

The old commands.properties format included the full class name such as:

createAccount=com.cloud.api.commands.CreateAccountCmd;1

The migration script did not consider this format and fails. With this fix
the migration script will process both the formats, including processing a
commands.properties file with mixed format, for example:

$ cat commands.properties
### Account commands
createAccount=1
deleteAccount=2

markDefaultZoneForAccount=com.cloud.api.commands.MarkDefaultZoneForAccountCmd;3

$ python scripts/util/migrate-dynamicroles.py -d -f commands.properties
Apache CloudStack Role Permission Migration Tool
(c) Apache CloudStack Authors and the ASF, under the Apache License, 
Version 2.0

Running this migration tool will remove any default-role permissions 
from cloud.role_permissions. Do you want to continue? [y/N]y
The commands.properties file has been deprecated and moved at: 
commands.properties.deprecated
Running SQL query: DELETE FROM `cloud`.`role_permissions` WHERE 
`role_id` in (1,2,3,4);
Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, 
`role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 1, '*', 'ALLOW', 
0);
Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, 
`role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 2, 
'deleteAccount', 'ALLOW', 0);
Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, 
`role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 2, 
'markDefaultZoneForAccount', 'ALLOW', 1);
Static role permissions from commands.properties have been migrated 
into the db
Running SQL query: UPDATE `cloud`.`configuration` SET value='true' 
where name='dynamic.apichecker.enabled'
Dynamic role based API checker has been enabled!

Signed-off-by: Rohit Yadav 

[GitHub] cloudstack pull request #1648: test/integration: fix tearDown order in list_...

2016-08-19 Thread rhtyd
GitHub user rhtyd opened a pull request:

https://github.com/apache/cloudstack/pull/1648

test/integration: fix tearDown order in list_acl_ tests

This fixes tearDown issues because of which few Travis tests have failed
such as:

https://travis-ci.org/apache/cloudstack/jobs/152610967
https://travis-ci.org/apache/cloudstack/jobs/152610968

/cc @jburwell @karuturi 

The fix is specific to tests that run 'only' on simulator with Travis. A 
passing Travis run should be enough to validate the changes.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shapeblue/cloudstack 4.9-fixmarvintest

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1648.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1648


commit f9d3345e8f3989d3cd8774b17e8c875817dad0fb
Author: Rohit Yadav 
Date:   2016-08-19T11:42:09Z

test/integration: fix tearDown order in list_acl_ tests

This fixes tearDown issues because of which few Travis tests have failed
such as:

https://travis-ci.org/apache/cloudstack/jobs/152610967
https://travis-ci.org/apache/cloudstack/jobs/152610968

Signed-off-by: Rohit Yadav 




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

2016-08-19 Thread wido
Github user wido commented on the issue:

https://github.com/apache/cloudstack/pull/1647
  
@rhtyd No, it doesn't do the management server.

Let's just merge this PR and make a new one for that. I don't want to delay 
merging this any longer.

Thanks @vincentbernat for the comment, that's something for a next PR.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-19 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1635
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

2016-08-19 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1647
  
Packaging result: ✔centos6 ✔centos7 ✔debian repo: 
http://packages.shapeblue.com/cloudstack/pr/1647
Job ID-92


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1647: [lts] CLOUDSTACK-9462: Systemd support for Ubuntu 16...

2016-08-19 Thread vincentbernat
Github user vincentbernat commented on the issue:

https://github.com/apache/cloudstack/pull/1647
  
LGTM for the Ubuntu part. You could share `/etc/sysconfig/*` and 
`/etc/default/*` files between RHEL/Ubuntu (same files, just installed at 
different locations). Same for the service files:

```
[service]
EnvironmentFile=-/etc/default/cloudstack-agent
EnvironmentFile=-/etc/sysconfig/cloudstack-agent
```



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1541: Systemd packaging for Ubuntu 16.04

2016-08-19 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1541
  
@wido I've used your PR to create this new branch on asf remote (so we both 
and anyone else can collaborate) and this PR: #1647 -- let's close this PR and 
continue on the new one against 4.9 branch?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request #1647: CLOUDSTACK-9462: Systemd support for Ubuntu 1...

2016-08-19 Thread rhtyd
GitHub user rhtyd opened a pull request:

https://github.com/apache/cloudstack/pull/1647

CLOUDSTACK-9462: Systemd support for Ubuntu 16.04

Created this based on @wido 's origin PR #1541 .

Requesting for review and testing -- @jburwell @karuturi @wido 
@vincentbernat 
@wido I think this change only brings systemd support to agent and usage 
packages, or does cloudstack-management pkg has systemd support too?

@blueorangutan package

- systemd: Add a /etc/sysconfig/cloudstack-* file

  This allows users to easily override variables passed to Java when
  starting up.

  It also creates a foundation for sharing the systemd service profile
  between CentOS and Ubuntu since it only requires the environment file
  to be changed.

- deb: Drop Ubuntu 12.04 support

  The libvirt and Qemu versions (1.2.2 and 2.0) in Ubuntu 12.04 are
  to old to support for CloudStack going forward.

  Ubuntu 14.04 and 16.04 provide the support required to provide new
  features and more stability.

- deb: Add Ubuntu 16.04 support

  Ubuntu 16.04 differs from Ubuntu 14.04 in a few ways:
  - systemd instead of sysvinit / upstart
  - Java 8 support

  The packaging now detects on which distribution it is being
  build and based on that it installs different files in the
  packages, but it also changes the Dependencies.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/apache/cloudstack 4.9-systemdubuntupkging

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1647.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1647


commit a6defab4b2df923c07be63c584a5b84be8926f7d
Author: Wido den Hollander 
Date:   2016-04-28T18:59:30Z

CLOUDSTACK-9462: Systemd support for Ubuntu 16.04

- systemd: Add a /etc/sysconfig/cloudstack-* file

  This allows users to easily override variables passed to Java when
  starting up.

  It also creates a foundation for sharing the systemd service profile
  between CentOS and Ubuntu since it only requires the environment file
  to be changed.

- deb: Drop Ubuntu 12.04 support

  The libvirt and Qemu versions (1.2.2 and 2.0) in Ubuntu 12.04 are
  to old to support for CloudStack going forward.

  Ubuntu 14.04 and 16.04 provide the support required to provide new
  features and more stability.

- deb: Add Ubuntu 16.04 support

  Ubuntu 16.04 differs from Ubuntu 14.04 in a few ways:
  - systemd instead of sysvinit / upstart
  - Java 8 support

  The packaging now detects on which distribution it is being
  build and based on that it installs different files in the
  packages, but it also changes the Dependencies.

  Packages for Ubuntu 16.04 will require Java 8 as a JRE

Signed-off-by: Rohit Yadav 




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1647: CLOUDSTACK-9462: Systemd support for Ubuntu 16.04

2016-08-19 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1647
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Virtual Router : Marvin test gap analysis

2016-08-19 Thread Raja Pullela
Hi Murali, 

Great initiative… and VR is one area that could help greatly if we have more 
automation.   we have done high-level analysis on VR functionality/automation 
recently and have posted our findings to the wiki under “high-level Automation 
analysis” section.  Please take a look, 

[1] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Virtual+Router%3A+Smoke+and+Integration+tests+gap+analysis

Best,
Raja
Senior Manager, Product Development
Accelerite, www.accelerite.com, @accelerite
2055, Laurelwood Road,  Santa Clara, CA 95054, USA
Phone: 1-408-216-7010

On 8/19/16, 12:33 PM, "Murali Reddy"  wrote:

All,

We (at ShapeBlue) did a gap analysis to figure if current set of smoke and 
component tests sufficiently test the VR functionality for the regressions. I 
have posted the analysis at [1]. I went through the test suites, and listed 
down all the tests that touch virtual router functionality. There is listing of 
general observations on grey areas.

One particular area where there were no tests, was related multiple public IP’s 
from different public IP ranges associated with a network. From 4.6, all the 
way to master only IP’s from one public IP range (eth2 on the VR) is working, 
any network services  on the public IP’s on eth3, eth4 etc on VR are not 
functional. This is a common use case and is broken for last few releases. Bug 
in this area are reported [2] and PR is yet to be merged [3]. I will be work on 
the patch to get this fixed in LTS. I have also added Marvin tests [4] cover 
multiple public IP scenarios. 

Given the flexibility and rich set of network functionality is CloudStack, we 
could catch regression only if we have good test suite. If there are any other 
areas related to virtual router functionality that you see there are 
significant gaps, please chime in share your thoughts or add the the wiki.

[1] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Virtual+Router%3A+Smoke+and+Integration+tests+gap+analysis
[2] https://issues.apache.org/jira/browse/CLOUDSTACK-9339
[3] https://github.com/apache/cloudstack/pull/1519
[4] 
https://github.com/murali-reddy/cloudstack/commit/0b6fbc29fcadb39b08d0050ca473680a614dfab4







DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.


Re: AJAX video stream for VM console is broken in Chrome

2016-08-19 Thread Rohit Yadav

Hi Sergey,


Since the same URL open perfectly fine in a tab, there must be something in 
your installation of Chrome blocking elements in the pop-up. Can you try to 
upgrade to latest Chrome, and then enable pop-ups in Chrome (follow something 
like: http://www.wikihow.com/Allow-Pop-ups-on-Google-Chrome).


You also are using ssl enabled console proxy, but the certificate is valid as 
per Chrome so it should not block any of the iframe elements but in one of the 
screenshots the URL call to the https based console proxy URL did not load or 
was blocked in the popup (which is where you don't see the certificate in the 
security tab). Based on the screenshots, it looks like Chrome hit one error. 
Can you share what error was reported on the console. With the console popup 
opened, open chrome developer tools and click on 'Console' tab and share what 
error you got there.


Regards.




From: Sergey Levitskiy 
Sent: 19 August 2016 10:21:10
To: dev@cloudstack.apache.org
Subject: Re: AJAX video stream for VM console is broken in Chrome

Hi Rohit,
As it turned out ACS 4.9 is fine and issue is only in ACP/CCP 4.7 which I 
believe is pretty close to ACS 4.7. Interestingly enough if I copy console URL 
and open a tab in main Chrome window it works just fine. I took 3 more 
screenshots of Chrome Development tools to compare. It looks like Chrome for 
whatever reason block returned Ajax stream to the popup window after the very 
first frame.

https://autodesk.box.com/s/psudrtjq09ke3p6t08ft2700l96govkg
https://autodesk.box.com/s/gn8gd1ieqmsq9yx1fzdopj4dycp3gm0u
https://autodesk.box.com/s/5sklr6sli7pgy1ar26l3vps0n8pzoa60

Thanks,
Sergey



On 8/18/16, 1:37 AM, "Rohit Yadav"  wrote:

Hi Sergey,


Can you open console in Chrome and then click on Tools-> Developer Tools. 
It would open a separate window from where you can debug what's going wrong. In 
the developer tools window, head over to the console tab and share if you're 
seeing any errors. Head over to the network tab, where you can ajax? and 
ajaximg? url requests being made, if you're not seeing any such URL 
requests/activities perhaps there was an issue with loading resources from the 
embedded iframe. To debug this, head over to the 'Elements' tab, click on 
iframset and copy the iframe src url. Try to open this URL in Chrome in a 
separate tab. It might fail to load due to SSL/TLS certificate error or any 
other network configuration.


Can you also share your OS details, Chrome version and CloudStack version. 
Are you using SSL certificates with console proxy? If so can you check that 
they are valid and resolve to a valid hostname/ip for which the certificate was 
generated and/or fix as necessary. If you're using a custom CA, make sure to 
import the root CA certificate in Chrome.


Regards.

rohit.ya...@shapeblue.com
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue






rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 



[GitHub] cloudstack issue #1646: [4.9/LTS] Add upgrade path from 4.9.0 to 4.9.1, chan...

2016-08-19 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1646
  
Packaging result: ✔centos6 ✔centos7 ✔debian repo: 
http://packages.shapeblue.com/cloudstack/pr/1646
Job ID-91


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Consoleproxy FF Quicksearch

2016-08-19 Thread Stephan Seitz
As a quick fix, you could modify the systemvm.iso which is used to
initialize the console-proxy-vm.

Im not sure if it's 
/usr/share/cloudstack-common/vms/systemvm.iso
or
/usr/share/cloudstack-management/webapps/client/WEB-
INF/classes/vms/systemvm.iso

Inside the systemvm.iso you'll find a systemvm.zip which contains
js/ajaxviewer.js

Here's the patch https://issues.apache.org/jira/browse/CLOUDSTACK-9164

I think it's 10 minutes worth to modify the zip content and create a
new iso.

If you're not attempting to destroy your console-proxy-vm you could
also modify /usr/local/cloud/forgot-the-exact-path.../ajaxviewer.js 
inside the running instance.

Oh, and flushing the browsercache is also necessery.

cheers,

- Stephan



Am Montag, den 14.12.2015, 20:55 -0500 schrieb Pierre-Luc Dion:
> Yet that is very annoying, no chance it could make it in 4.7.0rc2 ?
> ...
> 
> Thanks Stephan!
> 
> 
> On Mon, Dec 14, 2015 at 10:39 AM, Nux!  wrote:
> 
> > 
> > Stephan,
> > 
> > Thanks for this. It was driving me nuts, I even complained to
> > Mozilla lol.
> > 
> > I ended up installing an extension that disabled the quicksearch on
> > "/"
> > press.
> > https://freeshell.de//~kaosmos/index-en.html#searchkeys
> > 
> > --
> > Sent from the Delta quadrant using Borg technology!
> > 
> > Nux!
> > www.nux.ro
> > 
> > - Original Message -
> > > 
> > > From: "Stephan Seitz" 
> > > To: dev@cloudstack.apache.org
> > > Sent: Monday, 14 December, 2015 15:36:33
> > > Subject: Consoleproxy FF Quicksearch
> > > 
> > > Hi devs!
> > > 
> > > Despite the fact I was too lazy that weekend cloning the repo and
> > > filing
> > > a pull request, I hope that small fix could make it upstream
> > > anyway.
> > > 
> > > --- ajaxviewer.js.orig 2015-12-13 11:10:47.851177577 +
> > > +++ ajaxviewer.js 2015-12-13 14:14:32.367382547 +
> > > @@ -649,6 +649,7 @@
> > > this.sendingEventInProgress = false;
> > > ajaxViewer.installMouseHook();
> > > ajaxViewer.installKeyboardHook();
> > > + ajaxViewer.panel.parent().focus();
> > > 
> > > $(window).bind("resize", function() {
> > > ajaxViewer.onWindowResize();
> > > @@ -1259,6 +1260,7 @@
> > > case 38 : // UP
> > > case 39 : // RIGHT
> > > case 40 : // DOWN
> > > + case 47 : // slash (prevent quick search)
> > > return false;
> > > }
> > > 
> > > 
> > > 
> > > I've already filed this as bug
> > > https://issues.apache.org/jira/browse/CLOUDSTACK-9164
> > > 
> > > These two lines prevent Firefox from starting quicksearch which
> > > is
> > > really annoying.
> > > 
> > > - Stephan


[GitHub] cloudstack issue #1541: Systemd packaging for Ubuntu 16.04

2016-08-19 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1541
  
@wido if this helps you I can port your PR to 4.9 and push on asf remote so 
we can work on it together? Cherry-picking to 4.9 would against cause 
conflicts, and nice we've a fwd-merging based workflow, it would mess up the 
git-history/graph the next time someone fwd-merged 4.9 to master after this is 
cherry-picked to 4.9.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1646: [4.9/LTS] Add upgrade path from 4.9.0 to 4.9.1, chan...

2016-08-19 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1646
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1646: [4.9/LTS] Add upgrade path from 4.9.0 to 4.9.1, chan...

2016-08-19 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1646
  
@blueorangutan package


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1646: [4.9/LTS] Add upgrade path from 4.9.0 to 4.9.1

2016-08-19 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1646
  
@jburwell @karuturi fixed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack issue #1599: Marvin: Fix codegenerator to work with API discovery

2016-08-19 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1599
  
@jburwell fixed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Virtual Router : Marvin test gap analysis

2016-08-19 Thread Murali Reddy
All,

We (at ShapeBlue) did a gap analysis to figure if current set of smoke and 
component tests sufficiently test the VR functionality for the regressions. I 
have posted the analysis at [1]. I went through the test suites, and listed 
down all the tests that touch virtual router functionality. There is listing of 
general observations on grey areas.

One particular area where there were no tests, was related multiple public IP’s 
from different public IP ranges associated with a network. From 4.6, all the 
way to master only IP’s from one public IP range (eth2 on the VR) is working, 
any network services  on the public IP’s on eth3, eth4 etc on VR are not 
functional. This is a common use case and is broken for last few releases. Bug 
in this area are reported [2] and PR is yet to be merged [3]. I will be work on 
the patch to get this fixed in LTS. I have also added Marvin tests [4] cover 
multiple public IP scenarios. 

Given the flexibility and rich set of network functionality is CloudStack, we 
could catch regression only if we have good test suite. If there are any other 
areas related to virtual router functionality that you see there are 
significant gaps, please chime in share your thoughts or add the the wiki.

[1] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Virtual+Router%3A+Smoke+and+Integration+tests+gap+analysis
[2] https://issues.apache.org/jira/browse/CLOUDSTACK-9339
[3] https://github.com/apache/cloudstack/pull/1519
[4] 
https://github.com/murali-reddy/cloudstack/commit/0b6fbc29fcadb39b08d0050ca473680a614dfab4




Re: AJAX video stream for VM console is broken in Chrome

2016-08-19 Thread Wei ZHOU
Hi Sergey,

Could you check if disk is full in console proxy vm?

-Wei

2016-08-17 22:42 GMT+02:00 Sergey Levitskiy :

> Has anyone observed issues in using Chrome for VM console sessions?  Since
> recently we have had reports from many but not all users seeing white blank
> space where VM screen output goes. Other parts of the window e.g.
> Ctrl-Alt-Del button works just fine so it seems that AJAX stream is broken
> in Chrome although no other sites were reported with the same issues.
>
> I attached a screenshot of console window. Has anyone seen this issue or
> knows if there is a workaround except using a different browser?
>
>
>
>
>
>
>


Re: Jira permissions

2016-08-19 Thread Rajani Karuturi
done

~ Rajani
http://cloudplatform.accelerite.com/

On August 18, 2016 at 10:22 PM, Nathan Johnson (njohn...@ena.com)
wrote:
Is there any way I could be added to the list of people who can
be assigned
bugs in Jira? My username is nathanejohnson .

Thanks in advance,
Nathan