[GitHub] cloudstack pull request: Fix Sync of template.properties in Swift

[jburwell]

Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1331#issuecomment-215620806
  
@rafaelweingartner I apologize -- I did not understand that you were 
focused on ``final`` not ``final static``.  You are correct, ``final`` has 
nothing to do with garbage collection -- it is about thread safety.  There are 
very few (and rare) circumstances where a ``static`` reference should be 
mutable because it accessible across thread boundarys.  When they need to be 
mutable, special care must be taken to ensure exclusive access (e.g. 
``synchronized`` blocks, ``ReentrantLock``, ``AtomicReference``, etc).  
Therefore, I always put the two together.  In this case, the logger is a 
constant value and should be declared ``final``.  Sacrificing thread-safety for 
unit test is not an appropriate trade-off in my opinion.

Stepping back a bit, why are we investing so much time and effort to test a 
``DEBUG`` log message?  ``DEBUG`` level messages should only be for developers 
during development of the system.  Not only is a low value item to test, it 
makes the test case brittle as there should never be any guarantees about log 
messages at ```DEBUG`` and ``TRACE`` log levels.  In my opinion, @syed should 
remove this part of the test entirely.


---
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: CLOUDSTACK-8818: Use MySQL native connect...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215615196
  
CI has passed.  The only issue we see is a connectivity issue I sometimes 
see in my environment.


---
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: CLOUDSTACK-8818: Use MySQL native connect...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215615124
  


### CI RESULTS

```
Tests Run: 85
  Skipped: 0
   Failed: 1
   Errors: 0
```

**Summary of the problem(s):**
```
FAIL: test_03_vpc_privategw_restart_vpc_cleanup 
(integration.smoke.test_privategw_acl.TestPrivateGwACL)
--
Traceback (most recent call last):
  File 
"/data/git/cs1/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
265, in test_03_vpc_privategw_restart_vpc_cleanup
self.performVPCTests(vpc_off, True)
  File 
"/data/git/cs1/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
331, in performVPCTests
self.check_pvt_gw_connectivity(vm1, public_ip_1, vm2.nic[0].ipaddress)
  File 
"/data/git/cs1/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
559, in check_pvt_gw_connectivity
"Ping to outside world from VM should be successful"
AssertionError: Ping to outside world from VM should be successful
--
Additional details in: /tmp/MarvinLogs/test_network_G9N9AW/results.txt
```



**Associated Uploads**

**`/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_22_47_49_Z93Y1T:`**
* 
[dc_entries.obj](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_22_47_49_Z93Y1T/dc_entries.obj)
* 
[failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_22_47_49_Z93Y1T/failed_plus_exceptions.txt)
* 
[runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_22_47_49_Z93Y1T/runinfo.txt)

**`/tmp/MarvinLogs/test_network_G9N9AW:`**
* 
[failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/test_network_G9N9AW/failed_plus_exceptions.txt)
* 
[results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/test_network_G9N9AW/results.txt)
* 
[runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/test_network_G9N9AW/runinfo.txt)

**`/tmp/MarvinLogs/test_vpc_routers_V1YYOR:`**
* 
[failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/test_vpc_routers_V1YYOR/failed_plus_exceptions.txt)
* 
[results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/test_vpc_routers_V1YYOR/results.txt)
* 
[runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1054/tmp/MarvinLogs/test_vpc_routers_V1YYOR/runinfo.txt)


Uploads will be available until `2016-06-29 02:00:00 +0200 CEST`

*Comment created by [`upr comment`](https://github.com/cloudops/upr).*



---
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: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

[Slair1]

Github user Slair1 commented on the pull request:

https://github.com/apache/cloudstack/pull/1433#issuecomment-215615141
  
That's ok, thanks!!


---
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: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1433#issuecomment-215614954
  
None of the failures above are related to this PR, they are all issues I 
see periodically in my environment.  All of the supported usage tests passed, 
so CI has passed for this PR.  Sorry the output makes that hard to understand. 


---
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: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1433#issuecomment-215614748
  


### CI RESULTS

```
Tests Run: 103
  Skipped: 0
   Failed: 3
   Errors: 2
```

**Summary of the problem(s):**
```
ERROR: Test Create/Delete a manual snap shot and verify
--
Traceback (most recent call last):
  File 
"/data/git/cs2/cloudstack/test/integration/component/test_project_usage.py", 
line 1376, in test_01_snapshot_usage
snapshot = Snapshot.create(self.apiclient, volumes[0].id)
  File "/usr/lib/python2.7/site-packages/marvin/lib/base.py", line 1018, in 
create
return Snapshot(apiclient.createSnapshot(cmd).__dict__)
  File 
"/usr/lib/python2.7/site-packages/marvin/cloudstackAPI/cloudstackAPIClient.py", 
line 797, in createSnapshot
response = self.connection.marvinRequest(command, 
response_type=response, method=method)
  File "/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", 
line 379, in marvinRequest
raise e
CloudstackAPIException: Execute cmd: createsnapshot failed, due to: 
errorCode: 530, errorText:KVM Snapshot is not supported: 1
--
Additional details in: /tmp/MarvinLogs/test_usage_A3LTTO/results.txt
```

```
ERROR: Test Create/Delete a manual snap shot and verify
--
Traceback (most recent call last):
  File "/data/git/cs2/cloudstack/test/integration/component/test_usage.py", 
line 1300, in test_01_snapshot_usage
snapshot = Snapshot.create(self.apiclient, volumes[0].id)
  File "/usr/lib/python2.7/site-packages/marvin/lib/base.py", line 1018, in 
create
return Snapshot(apiclient.createSnapshot(cmd).__dict__)
  File 
"/usr/lib/python2.7/site-packages/marvin/cloudstackAPI/cloudstackAPIClient.py", 
line 797, in createSnapshot
response = self.connection.marvinRequest(command, 
response_type=response, method=method)
  File "/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", 
line 379, in marvinRequest
raise e
CloudstackAPIException: Execute cmd: createsnapshot failed, due to: 
errorCode: 530, errorText:KVM Snapshot is not supported: 1
--
Additional details in: /tmp/MarvinLogs/test_usage_A3LTTO/results.txt
```

```
FAIL: Test redundant router internals
--
Traceback (most recent call last):
  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_routers_network_ops.py", 
line 290, in test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true
"Attempt to retrieve google.com index page should be successful!"
AssertionError: Attempt to retrieve google.com index page should be 
successful!
--
Additional details in: /tmp/MarvinLogs/test_network_4ISAIJ/results.txt
```

```
FAIL: test_02_vpc_privategw_static_routes 
(integration.smoke.test_privategw_acl.TestPrivateGwACL)
--
Traceback (most recent call last):
  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
253, in test_02_vpc_privategw_static_routes
self.performVPCTests(vpc_off)
  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
324, in performVPCTests
self.check_pvt_gw_connectivity(vm1, public_ip_1, vm2.nic[0].ipaddress)
  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
559, in check_pvt_gw_connectivity
"Ping to outside world from VM should be successful"
AssertionError: Ping to outside world from VM should be successful
--
Additional details in: /tmp/MarvinLogs/test_network_4ISAIJ/results.txt
```

```
FAIL: test_03_vpc_privategw_restart_vpc_cleanup 
(integration.smoke.test_privategw_acl.TestPrivateGwACL)
--
Traceback (most recent call last):
  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
265, in test_03_vpc_privategw_restart_vpc_cleanup
self.performVPCTests(vpc_off, True)
  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
324, in performVPCTests
self.check_pvt_gw_connectivity(vm1, public_ip_1, vm2.nic[0].ipaddress)
  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 
559, in check_pvt_gw_connectivity

[GitHub] cloudstack pull request: CLOUDSTACK-9340: General DB Optimization

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215586085
  
OK. I will just do a standard CI run then. Thx. 


---
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: CLOUDSTACK-9340: General DB Optimization

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215583632
  
If you run your CI tests, it should detect if it breaks something. But more 
importantly would be to check if the performance was improved. But, I am sure 
@nvazquez and @serg38 have checked and re-checked that (given the work I have 
seen them doing here).


---
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: CLOUDSTACK-9340: General DB Optimization

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215581686
  
should I be running some sort of CI or validation for this change?  How do 
I validate this works and more importantly, not break anything?  I also need 
another LGTM.  Thx.  :)  @rhtyd have you had a chance to review this?


---
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: Build failed in Jenkins: build-master-jdk18 #509

[Daan Hoogland]

the timout on this job was 60, (or 400% of the last job) I upped it to 120
minuted

On Thu, Apr 28, 2016 at 11:58 PM, Will Stevens 
wrote:

> Some of the tests that were added can take some time, so is it possible to
> bump the timeout value on Jenkins?  If we were close before, we are going
> to be over now.
>
> *Will STEVENS*
> Lead Developer
>
> *CloudOps* *| *Cloud Solutions Experts
> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> w cloudops.com *|* tw @CloudOps_
>
> On Thu, Apr 28, 2016 at 5:46 PM,  wrote:
>
>> See 
>>
>> Changes:
>>
>> [anshul.gangwar] CLOUDSTACK-9194:  Making the console popup window
>> resizable in IE to
>>
>> [Daan Hoogland] CLOUDSTACK-9142 Migrate VM changes xmlDesc in a safe way
>>
>> [David Amorim Faria] CLOUDSTACK-9283: add pid to java arguments in
>>
>> [dmabry] Let hypervisor type KVM and Simulator detach root volumes.
>> Updated
>>
>> [dmabry] Modified test_volumes.py to include a hypervisor test for root
>>
>> [dmabry] Added required_hardware="false" attr above
>>
>> [Rohit Yadav] maven: Upgrade dependency versions
>>
>> [Rohit Yadav] framework/quota: fix checkstyle issue
>>
>> [Rohit Yadav] further maven dependency updates from Daan
>>
>> [Rohit Yadav] maven: fix dependency version support by JDK7
>>
>> [Rohit Yadav] engine/schema: fix upgrade path to work with MySQL 5.7
>>
>> [abhinandan.prateek] CLOUDSTACK-9323: Fix Cancel maintenance so that if
>> maintenance is
>>
>> [dmabry] Removed sleeps and used validateList as requested.
>>
>> [njohnson] Log responses from asynchronous api commands
>>
>> --
>> [...truncated 37087 lines...]
>> 2459/2764 KB
>> 2463/2764 KB
>> 2467/2764 KB
>> 2471/2764 KB
>> 2475/2764 KB
>> 2479/2764 KB
>> 2483/2764 KB
>> 2487/2764 KB
>> 2491/2764 KB
>> 2495/2764 KB
>> 2499/2764 KB
>> 2503/2764 KB
>> 2507/2764 KB
>> 2511/2764 KB
>> 2515/2764 KB
>> 2519/2764 KB
>> 2523/2764 KB
>> 2527/2764 KB
>> 2531/2764 KB
>> 2535/2764 KB
>> 2539/2764 KB
>> 2543/2764 KB
>> 2547/2764 KB
>> 2551/2764 KB
>> 2555/2764 KB
>> 2559/2764 KB
>> 2563/2764 KB
>> 2567/2764 KB
>> 2571/2764 KB
>> 2575/2764 KB
>> 2579/2764 KB
>> 2583/2764 KB
>> 2587/2764 KB
>> 2591/2764 KB
>> 2595/2764 KB
>> 2599/2764 KB
>> 2603/2764 KB
>> 2607/2764 KB
>> 2611/2764 KB
>> 2615/2764 KB
>> 2619/2764 KB
>> 2623/2764 KB
>> 2627/2764 KB
>> 2631/2764 KB
>> 2635/2764 KB
>> 2639/2764 KB
>> 2643/2764 KB
>> 2647/2764 KB
>> 2651/2764 KB
>> 2655/2764 KB
>> 2659/2764 KB
>> 2663/2764 KB
>> 2667/2764 KB
>> 2671/2764 KB
>> 2675/2764 KB
>> 2679/2764 KB
>> 2683/2764 KB
>> 2687/2764 KB
>> 2691/2764 KB
>> 2695/2764 KB
>> 2699/2764 KB
>> 2703/2764 KB
>> 2707/2764 KB
>> 2711/2764 KB
>> 2715/2764 KB
>> 2719/2764 KB
>> 2723/2764 KB
>> 2727/2764 KB
>> 2731/2764 KB
>> 2735/2764 KB
>> 2739/2764 KB
>> 2743/2764 KB
>> 2747/2764 KB
>> 2751/2764 KB
>> 2755/2764 KB
>> 2759/2764 KB
>> 2763/2764 KB
>> 2764/2764 KB
>>
>> Downloaded:
>> http://repo.maven.apache.org/maven2/org/apache/tomcat/embed/tomcat-embed-core/8.0.30/tomcat-embed-core-8.0.30.jar
>> (2764 KB at 9031.6 KB/sec)
>> [INFO]
>> [INFO] --- maven-clean-plugin:2.5:clean (default-clean) @
>> cloudstack-service-console-proxy-rdpclient ---
>> [INFO] Deleting <
>> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole/target>
>> (includes = [**/*], excludes = [])
>> [INFO] Deleting <
>> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole>
>> (includes = [target, dist], excludes = [])
>> [INFO]
>> [INFO] --- maven-checkstyle-plugin:2.17:check (cloudstack-checkstyle) @
>> cloudstack-service-console-proxy-rdpclient ---
>> [INFO] Starting audit...
>> Audit done.
>> [INFO]
>> [INFO] --- maven-remote-resources-plugin:1.3:process (default) @
>> cloudstack-service-console-proxy-rdpclient ---
>> [INFO]
>> [INFO] --- maven-resources-plugin:2.5:resources (default-resources) @
>> cloudstack-service-console-proxy-rdpclient ---
>> [debug] execute contextualize
>> [INFO] Using 'UTF-8' encoding to copy filtered resources.
>> [INFO] Copying 1 resource
>> [INFO] Copying 3 resources
>> [INFO]
>> [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @
>> cloudstack-service-console-proxy-rdpclient ---
>> [INFO] Changes detected - recompiling the module!
>> [INFO] Compiling 146 source files to <
>> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole/target/classes
>> >
>> [INFO]
>> [INFO] --- maven-resources-plugin:2.5:testResources
>> (default-testResources) @ cloudstack-service-console-proxy-rdpclient ---
>> [debug] execute contextualize
>> [INFO] Using 'UTF-8' encoding to copy filtered resources.
>> [INFO] skip non existing resourceDirectory <
>> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole/src/test/resources
>> >
>> [INFO] Copying 3 resources
>> [INFO]
>> 

Re: Build failed in Jenkins: build-master-jdk18 #509

[Will Stevens]

Some of the tests that were added can take some time, so is it possible to
bump the timeout value on Jenkins?  If we were close before, we are going
to be over now.

*Will STEVENS*
Lead Developer

*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_

On Thu, Apr 28, 2016 at 5:46 PM,  wrote:

> See 
>
> Changes:
>
> [anshul.gangwar] CLOUDSTACK-9194:  Making the console popup window
> resizable in IE to
>
> [Daan Hoogland] CLOUDSTACK-9142 Migrate VM changes xmlDesc in a safe way
>
> [David Amorim Faria] CLOUDSTACK-9283: add pid to java arguments in
>
> [dmabry] Let hypervisor type KVM and Simulator detach root volumes. Updated
>
> [dmabry] Modified test_volumes.py to include a hypervisor test for root
>
> [dmabry] Added required_hardware="false" attr above
>
> [Rohit Yadav] maven: Upgrade dependency versions
>
> [Rohit Yadav] framework/quota: fix checkstyle issue
>
> [Rohit Yadav] further maven dependency updates from Daan
>
> [Rohit Yadav] maven: fix dependency version support by JDK7
>
> [Rohit Yadav] engine/schema: fix upgrade path to work with MySQL 5.7
>
> [abhinandan.prateek] CLOUDSTACK-9323: Fix Cancel maintenance so that if
> maintenance is
>
> [dmabry] Removed sleeps and used validateList as requested.
>
> [njohnson] Log responses from asynchronous api commands
>
> --
> [...truncated 37087 lines...]
> 2459/2764 KB
> 2463/2764 KB
> 2467/2764 KB
> 2471/2764 KB
> 2475/2764 KB
> 2479/2764 KB
> 2483/2764 KB
> 2487/2764 KB
> 2491/2764 KB
> 2495/2764 KB
> 2499/2764 KB
> 2503/2764 KB
> 2507/2764 KB
> 2511/2764 KB
> 2515/2764 KB
> 2519/2764 KB
> 2523/2764 KB
> 2527/2764 KB
> 2531/2764 KB
> 2535/2764 KB
> 2539/2764 KB
> 2543/2764 KB
> 2547/2764 KB
> 2551/2764 KB
> 2555/2764 KB
> 2559/2764 KB
> 2563/2764 KB
> 2567/2764 KB
> 2571/2764 KB
> 2575/2764 KB
> 2579/2764 KB
> 2583/2764 KB
> 2587/2764 KB
> 2591/2764 KB
> 2595/2764 KB
> 2599/2764 KB
> 2603/2764 KB
> 2607/2764 KB
> 2611/2764 KB
> 2615/2764 KB
> 2619/2764 KB
> 2623/2764 KB
> 2627/2764 KB
> 2631/2764 KB
> 2635/2764 KB
> 2639/2764 KB
> 2643/2764 KB
> 2647/2764 KB
> 2651/2764 KB
> 2655/2764 KB
> 2659/2764 KB
> 2663/2764 KB
> 2667/2764 KB
> 2671/2764 KB
> 2675/2764 KB
> 2679/2764 KB
> 2683/2764 KB
> 2687/2764 KB
> 2691/2764 KB
> 2695/2764 KB
> 2699/2764 KB
> 2703/2764 KB
> 2707/2764 KB
> 2711/2764 KB
> 2715/2764 KB
> 2719/2764 KB
> 2723/2764 KB
> 2727/2764 KB
> 2731/2764 KB
> 2735/2764 KB
> 2739/2764 KB
> 2743/2764 KB
> 2747/2764 KB
> 2751/2764 KB
> 2755/2764 KB
> 2759/2764 KB
> 2763/2764 KB
> 2764/2764 KB
>
> Downloaded:
> http://repo.maven.apache.org/maven2/org/apache/tomcat/embed/tomcat-embed-core/8.0.30/tomcat-embed-core-8.0.30.jar
> (2764 KB at 9031.6 KB/sec)
> [INFO]
> [INFO] --- maven-clean-plugin:2.5:clean (default-clean) @
> cloudstack-service-console-proxy-rdpclient ---
> [INFO] Deleting <
> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole/target>
> (includes = [**/*], excludes = [])
> [INFO] Deleting <
> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole>
> (includes = [target, dist], excludes = [])
> [INFO]
> [INFO] --- maven-checkstyle-plugin:2.17:check (cloudstack-checkstyle) @
> cloudstack-service-console-proxy-rdpclient ---
> [INFO] Starting audit...
> Audit done.
> [INFO]
> [INFO] --- maven-remote-resources-plugin:1.3:process (default) @
> cloudstack-service-console-proxy-rdpclient ---
> [INFO]
> [INFO] --- maven-resources-plugin:2.5:resources (default-resources) @
> cloudstack-service-console-proxy-rdpclient ---
> [debug] execute contextualize
> [INFO] Using 'UTF-8' encoding to copy filtered resources.
> [INFO] Copying 1 resource
> [INFO] Copying 3 resources
> [INFO]
> [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @
> cloudstack-service-console-proxy-rdpclient ---
> [INFO] Changes detected - recompiling the module!
> [INFO] Compiling 146 source files to <
> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole/target/classes
> >
> [INFO]
> [INFO] --- maven-resources-plugin:2.5:testResources
> (default-testResources) @ cloudstack-service-console-proxy-rdpclient ---
> [debug] execute contextualize
> [INFO] Using 'UTF-8' encoding to copy filtered resources.
> [INFO] skip non existing resourceDirectory <
> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole/src/test/resources
> >
> [INFO] Copying 3 resources
> [INFO]
> [INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @
> cloudstack-service-console-proxy-rdpclient ---
> [INFO] Changes detected - recompiling the module!
> [INFO] Compiling 4 source files to <
> http://jenkins.buildacloud.org/job/build-master-jdk18/ws/services/console-proxy-rdp/rdpconsole/target/test-classes
> >
> [INFO]
> 

Build failed in Jenkins: build-master-jdk18 #509

[jenkins]

See 

Changes:

[anshul.gangwar] CLOUDSTACK-9194:  Making the console popup window resizable in 
IE to

[Daan Hoogland] CLOUDSTACK-9142 Migrate VM changes xmlDesc in a safe way

[David Amorim Faria] CLOUDSTACK-9283: add pid to java arguments in

[dmabry] Let hypervisor type KVM and Simulator detach root volumes. Updated

[dmabry] Modified test_volumes.py to include a hypervisor test for root

[dmabry] Added required_hardware="false" attr above

[Rohit Yadav] maven: Upgrade dependency versions

[Rohit Yadav] framework/quota: fix checkstyle issue

[Rohit Yadav] further maven dependency updates from Daan

[Rohit Yadav] maven: fix dependency version support by JDK7

[Rohit Yadav] engine/schema: fix upgrade path to work with MySQL 5.7

[abhinandan.prateek] CLOUDSTACK-9323: Fix Cancel maintenance so that if 
maintenance is

[dmabry] Removed sleeps and used validateList as requested.

[njohnson] Log responses from asynchronous api commands

--
[...truncated 37087 lines...]
2459/2764 KB   
2463/2764 KB   
2467/2764 KB   
2471/2764 KB   
2475/2764 KB   
2479/2764 KB   
2483/2764 KB   
2487/2764 KB   
2491/2764 KB   
2495/2764 KB   
2499/2764 KB   
2503/2764 KB   
2507/2764 KB   
2511/2764 KB   
2515/2764 KB   
2519/2764 KB   
2523/2764 KB   
2527/2764 KB   
2531/2764 KB   
2535/2764 KB   
2539/2764 KB   
2543/2764 KB   
2547/2764 KB   
2551/2764 KB   
2555/2764 KB   
2559/2764 KB   
2563/2764 KB   
2567/2764 KB   
2571/2764 KB   
2575/2764 KB   
2579/2764 KB   
2583/2764 KB   
2587/2764 KB   
2591/2764 KB   
2595/2764 KB   
2599/2764 KB   
2603/2764 KB   
2607/2764 KB   
2611/2764 KB   
2615/2764 KB   
2619/2764 KB   
2623/2764 KB   
2627/2764 KB   
2631/2764 KB   
2635/2764 KB   
2639/2764 KB   
2643/2764 KB   
2647/2764 KB   
2651/2764 KB   
2655/2764 KB   
2659/2764 KB   
2663/2764 KB   
2667/2764 KB   
2671/2764 KB   
2675/2764 KB   
2679/2764 KB   
2683/2764 KB   
2687/2764 KB   
2691/2764 KB   
2695/2764 KB   
2699/2764 KB   
2703/2764 KB   
2707/2764 KB   
2711/2764 KB   
2715/2764 KB   
2719/2764 KB   
2723/2764 KB   
2727/2764 KB   
2731/2764 KB   
2735/2764 KB   
2739/2764 KB   
2743/2764 KB   
2747/2764 KB   
2751/2764 KB   
2755/2764 KB   
2759/2764 KB   
2763/2764 KB   
2764/2764 KB   
   
Downloaded: 
http://repo.maven.apache.org/maven2/org/apache/tomcat/embed/tomcat-embed-core/8.0.30/tomcat-embed-core-8.0.30.jar
 (2764 KB at 9031.6 KB/sec)
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloudstack-service-console-proxy-rdpclient ---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.17:check (cloudstack-checkstyle) @ 
cloudstack-service-console-proxy-rdpclient ---
[INFO] Starting audit...
Audit done.
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloudstack-service-console-proxy-rdpclient ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloudstack-service-console-proxy-rdpclient ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloudstack-service-console-proxy-rdpclient ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 146 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloudstack-service-console-proxy-rdpclient ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloudstack-service-console-proxy-rdpclient ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 4 source files to 

[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ 
cloudstack-service-console-proxy-rdpclient ---
[INFO] Surefire report directory: 


---
 T E S T S
---
Build timed out (after 105 minutes). Marking the 

[GitHub] cloudstack pull request: CLOUDSTACK-9340: General DB Optimization

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215572438
  
off course ;)
I was only waiting for that
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 pull request: CLOUDSTACK-9340: General DB Optimization

[serg38]

Github user serg38 commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215572026
  
With this change PR passed Travis and Jenkins. @rafaelweingartner  and 
@rhtyd will you be able to give your blessing?


---
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: [ANNOUNCE] New committer: Simon Weller

[Simon Weller]

Thanks everyone! I look forward to continuing to help out where I can as we 
march towards 4.9.

- Si

From: Pierre-Luc Dion 
Sent: Thursday, April 28, 2016 4:20 PM
To: dev@cloudstack.apache.org
Subject: Re: [ANNOUNCE] New committer: Simon Weller

Congrats Simon!

On Thu, Apr 28, 2016 at 3:52 PM, Rafael Weingärtner <
rafaelweingart...@gmail.com> wrote:

> Congratulations Simon,
>
> Let’s keep up with the good work ;)
>
> On Thu, Apr 28, 2016 at 8:34 AM, Daan Hoogland 
> wrote:
>
> > welcome, kiwi. Good work and keep going.
> >
> > On Thu, Apr 28, 2016 at 11:25 AM, Nick LIVENS <
> > nick.liv...@nuagenetworks.net
> > > wrote:
> >
> > > Congrats Simon!
> > >
> > > Kind regards,
> > > Nick Livens
> > >
> > > On Thu, Apr 28, 2016 at 10:15 AM, Nux!  wrote:
> > >
> > > > Congrats! :-)
> > > >
> > > > --
> > > > Sent from the Delta quadrant using Borg technology!
> > > >
> > > > Nux!
> > > > www.nux.ro
> > > >
> > > > - Original Message -
> > > > > From: "Paul Angus" 
> > > > > To: dev@cloudstack.apache.org
> > > > > Sent: Thursday, 28 April, 2016 08:44:36
> > > > > Subject: RE: [ANNOUNCE] New committer: Simon Weller
> > > >
> > > > > Congratulations Simon!
> > > > >
> > > > >
> > > > >
> > > > > Kind regards,
> > > > >
> > > > > Paul Angus
> > > > >
> > > > > Regards,
> > > > >
> > > > > Paul Angus
> > > > >
> > > > > paul.an...@shapeblue.com
> > > > > www.shapeblue.com
> > > > > 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> > > > > @shapeblue
> > > > >
> > > > > -Original Message-
> > > > > From: Erik Weber [mailto:terbol...@gmail.com]
> > > > > Sent: 28 April 2016 08:24
> > > > > To: dev 
> > > > > Subject: [ANNOUNCE] New committer: Simon Weller
> > > > >
> > > > > The Project Management Committee (PMC) for Apache CloudStack has
> > asked
> > > > Simon
> > > > > Weller to become a committer and we are pleased to announce that
> they
> > > > have
> > > > > accepted.
> > > > >
> > > > >
> > > > > Being a committer allows many contributors to contribute more
> > > > autonomously. For
> > > > > developers, it makes it easier to submit changes and eliminates the
> > > need
> > > > to
> > > > > have contributions reviewed via the patch submission process.
> Whether
> > > > > contributions are development-related or otherwise, it is a
> > recognition
> > > > of a
> > > > > contributor's participation in the project and commitment to the
> > > project
> > > > and
> > > > > the Apache Way.
> > > > >
> > > > > Please join me in congratulating Simon
> > > > >
> > > > > --Erik
> > > > > on behalf of the CloudStack PMC
> > > >
> > >
> >
> >
> >
> > --
> > Daan
> >
>
>
>
> --
> Rafael Weingärtner
>

Re: [ANNOUNCE] New committer: Simon Weller

[Pierre-Luc Dion]

Congrats Simon!

On Thu, Apr 28, 2016 at 3:52 PM, Rafael Weingärtner <
rafaelweingart...@gmail.com> wrote:

> Congratulations Simon,
>
> Let’s keep up with the good work ;)
>
> On Thu, Apr 28, 2016 at 8:34 AM, Daan Hoogland 
> wrote:
>
> > welcome, kiwi. Good work and keep going.
> >
> > On Thu, Apr 28, 2016 at 11:25 AM, Nick LIVENS <
> > nick.liv...@nuagenetworks.net
> > > wrote:
> >
> > > Congrats Simon!
> > >
> > > Kind regards,
> > > Nick Livens
> > >
> > > On Thu, Apr 28, 2016 at 10:15 AM, Nux!  wrote:
> > >
> > > > Congrats! :-)
> > > >
> > > > --
> > > > Sent from the Delta quadrant using Borg technology!
> > > >
> > > > Nux!
> > > > www.nux.ro
> > > >
> > > > - Original Message -
> > > > > From: "Paul Angus" 
> > > > > To: dev@cloudstack.apache.org
> > > > > Sent: Thursday, 28 April, 2016 08:44:36
> > > > > Subject: RE: [ANNOUNCE] New committer: Simon Weller
> > > >
> > > > > Congratulations Simon!
> > > > >
> > > > >
> > > > >
> > > > > Kind regards,
> > > > >
> > > > > Paul Angus
> > > > >
> > > > > Regards,
> > > > >
> > > > > Paul Angus
> > > > >
> > > > > paul.an...@shapeblue.com
> > > > > www.shapeblue.com
> > > > > 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> > > > > @shapeblue
> > > > >
> > > > > -Original Message-
> > > > > From: Erik Weber [mailto:terbol...@gmail.com]
> > > > > Sent: 28 April 2016 08:24
> > > > > To: dev 
> > > > > Subject: [ANNOUNCE] New committer: Simon Weller
> > > > >
> > > > > The Project Management Committee (PMC) for Apache CloudStack has
> > asked
> > > > Simon
> > > > > Weller to become a committer and we are pleased to announce that
> they
> > > > have
> > > > > accepted.
> > > > >
> > > > >
> > > > > Being a committer allows many contributors to contribute more
> > > > autonomously. For
> > > > > developers, it makes it easier to submit changes and eliminates the
> > > need
> > > > to
> > > > > have contributions reviewed via the patch submission process.
> Whether
> > > > > contributions are development-related or otherwise, it is a
> > recognition
> > > > of a
> > > > > contributor's participation in the project and commitment to the
> > > project
> > > > and
> > > > > the Apache Way.
> > > > >
> > > > > Please join me in congratulating Simon
> > > > >
> > > > > --Erik
> > > > > on behalf of the CloudStack PMC
> > > >
> > >
> >
> >
> >
> > --
> > Daan
> >
>
>
>
> --
> Rafael Weingärtner
>

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1320#issuecomment-215554347
  
@nitin-maharana please rebase as we currently have merge conflicts.  
Thanks...


---
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: CLOUDSTACK-8800 : Improved the listVirtua...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1444#issuecomment-215554170
  
@rafaelweingartner please rebase as we currently have merge conflicts.  
Thanks...


---
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: Restore iptables at once using iptables-r...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1482#issuecomment-215553431
  
@remibergsma please rebase as we currently have merge conflicts with 
master.  Thanks...


---
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: Remodeling of Nuage VSP Plugin + CLOUDSTA...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1494#issuecomment-215553058
  
@nlivens please rebase as we have merge conflicts with master now.  
Thanks...


---
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: CLOUDSTACK-9350: KVM-HA- Fix CheckOnHost ...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1496#issuecomment-215552813
  
@abhinandanprateek please rebase as this currently has merge conflicts with 
master.  Thanks...


---
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: Kvm plugin restructure

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1521#issuecomment-215552234
  
@DaanHoogland please rebase as master has caused merge conflicts...


---
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: CLOUDSTACK-8818: Use MySQL native connect...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215551291
  
Running CI now.  Will have results later tonight or tomorrow...


---
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: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1433#issuecomment-215551223
  
Running CI now.  Will have results later tonight or tomorrow...


---
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: CLOUDSTACK-9340: General DB Optimization

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215549970
  
I confess that without looking the code, I thought the same as @nvazquez.
First we would execute schema-XXXtoYYY.sql, schemaXXXtoYYY-cleanup.sql and 
then schema-XXX+1toYYY+1.sql, schemaXXX+1toYYY+1-cleanup.sql.
It is good that we have clarified that ;)



---
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: CLOUDSTACK-9362: Skip VXLANs when rewriti...

[kiwiflyer]

Github user kiwiflyer commented on the pull request:

https://github.com/apache/cloudstack/pull/1513#issuecomment-215549283
  
We'll pull this in for testing.


---
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: [Bug-Id: CLOUDSTACK-9306]Replace testdata...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1436#issuecomment-215548871
  
@sanju1010 please review the CI run results when you have a second.  
Thanks...


---
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: CLOUDSTACK-9340: General DB Optimization

[rhtyd]

Github user rhtyd commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215548664
  
@nvazquez that's by design -- all upgrade paths (migration code and the 
schema files) are run first and after that only the cleanup scripts are ran


---
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: CLOUDSTACK-9340: General DB Optimization

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215548103
  
@rafaelweingartner exactly, I meant to explain that I realized that first 
all `schema-XXXtoYYY.sql` were executed and after they finished, all 
`schema-XXXtoYYY-cleanup.sql` were executed, that's why I moved to cleanup sql 
:)


---
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: CLOUDSTACK-9283: add pid to java argument...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: engine/schema: fix upgrade path to work w...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: CLOUDSTACK-9323: Fix cancel host maintena...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: CLOUDSTACK-9340: General DB Optimization

[rafaelweingartner]

Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215546030
  
@nvazquez, 
What does that mean?

First we are executing all of the `schema-XXXtoYYY.sql`, then all of the 
`schemaXXXtoYYY-cleanup.sql`? Instead of` schema-XXXtoYYY.sql, 
schemaXXXtoYYY-cleanup.sql` and then `schema-XXX+1toYYY+1.sql, 
schemaXXX+1toYYY+1-cleanup.sql`

I have just looked at the code at 
"com.cloud.upgrade.DatabaseUpgradeChecker.upgrade(String, String)", that is 
exactly what happens.
First we are executing all of the `schema-XXXtoYYY.sql`, then all of the 
`schemaXXXtoYYY-cleanup.sql`

So, I am ok with your changes to create the "schema-410to420-cleanup.sql"

So, I am ok with you moving some scripts to 



---
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: CLOUDSTACK-9349: Enable root disk detach ...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: CLOUDSTACK-9194: Making the console popup...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: Log asynchronous responses in the api log

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: 4.9 mvn version safeupgradeonly

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: CLOUDSTACK-9142 Migrate VM changes xmlDes...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: [ANNOUNCE] New committer: Simon Weller

[Rafael Weingärtner]

Congratulations Simon,

Let’s keep up with the good work ;)

On Thu, Apr 28, 2016 at 8:34 AM, Daan Hoogland 
wrote:

> welcome, kiwi. Good work and keep going.
>
> On Thu, Apr 28, 2016 at 11:25 AM, Nick LIVENS <
> nick.liv...@nuagenetworks.net
> > wrote:
>
> > Congrats Simon!
> >
> > Kind regards,
> > Nick Livens
> >
> > On Thu, Apr 28, 2016 at 10:15 AM, Nux!  wrote:
> >
> > > Congrats! :-)
> > >
> > > --
> > > Sent from the Delta quadrant using Borg technology!
> > >
> > > Nux!
> > > www.nux.ro
> > >
> > > - Original Message -
> > > > From: "Paul Angus" 
> > > > To: dev@cloudstack.apache.org
> > > > Sent: Thursday, 28 April, 2016 08:44:36
> > > > Subject: RE: [ANNOUNCE] New committer: Simon Weller
> > >
> > > > Congratulations Simon!
> > > >
> > > >
> > > >
> > > > Kind regards,
> > > >
> > > > Paul Angus
> > > >
> > > > Regards,
> > > >
> > > > Paul Angus
> > > >
> > > > paul.an...@shapeblue.com
> > > > www.shapeblue.com
> > > > 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> > > > @shapeblue
> > > >
> > > > -Original Message-
> > > > From: Erik Weber [mailto:terbol...@gmail.com]
> > > > Sent: 28 April 2016 08:24
> > > > To: dev 
> > > > Subject: [ANNOUNCE] New committer: Simon Weller
> > > >
> > > > The Project Management Committee (PMC) for Apache CloudStack has
> asked
> > > Simon
> > > > Weller to become a committer and we are pleased to announce that they
> > > have
> > > > accepted.
> > > >
> > > >
> > > > Being a committer allows many contributors to contribute more
> > > autonomously. For
> > > > developers, it makes it easier to submit changes and eliminates the
> > need
> > > to
> > > > have contributions reviewed via the patch submission process. Whether
> > > > contributions are development-related or otherwise, it is a
> recognition
> > > of a
> > > > contributor's participation in the project and commitment to the
> > project
> > > and
> > > > the Apache Way.
> > > >
> > > > Please join me in congratulating Simon
> > > >
> > > > --Erik
> > > > on behalf of the CloudStack PMC
> > >
> >
>
>
>
> --
> Daan
>



-- 
Rafael Weingärtner

Re: Rafael Weingartner joining the PMC

[Rafael Weingärtner]

Thank you all for the warm welcome. It is an honor to be part of the Apache
CloudStack (ACS) committee. I will try to work as much as I can to help the
PMC, committers, and contributors on improving the ACS. Sometimes I might
not be as present as others (as it is happening now), but that is due to my
Ph. D. tasks.

As soon as I get things back on tracks I make myself more present :)


On Thu, Apr 28, 2016 at 10:55 AM, Patrick Dube 
wrote:

> Congratulations!
>
> On Thu, Apr 28, 2016 at 8:19 AM, Simon Weller  wrote:
>
> > Congratulations Rafael!
> >
> > - Si
> >
> > 
> > From: Nux! 
> > Sent: Thursday, April 28, 2016 7:06 AM
> > To: dev@cloudstack.apache.org
> > Subject: Re: Rafael Weingartner joining the PMC
> >
> > Congrats Rafael :)
> >
> > --
> > Sent from the Delta quadrant using Borg technology!
> >
> > Nux!
> > www.nux.ro
> >
> > - Original Message -
> > > From: "Daan Hoogland" 
> > > To: "dev" 
> > > Sent: Thursday, 28 April, 2016 11:59:28
> > > Subject: Rafael Weingartner joining the PMC
> >
> > > People,
> > >
> > > The PMC has ask Rafael to join them to oversee the project and he has
> > > gracefully accepted. Please join me in congratulating Rafael and wish
> him
> > > wisdom in his new task.
> > >
> > > ​on behalve of the PMC,​
> > > --
> > > Daan
> >
>



-- 
Rafael Weingärtner

[GitHub] cloudstack pull request: Bump ssh retries to prevent false positiv...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1473#issuecomment-215531806
  
@remibergsma can you force push again so we can try to get it all green?  
Thanks...


---
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: CLOUDSTACK-9340: General DB Optimization

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1466#issuecomment-215529568
  
@rhtyd @rafaelweingartner I could discover the problem by deleting `cloud` 
and `cloud_usage` database, then `cloudstack-databases-setup` to create them 
blank and restarting CS to create schemas. Error was whis:

2016-04-27 08:35:19,850 ERROR [utils.db.ScriptRunner] (main:null) Error 
executing: ALTER TABLE `cloud`.`remote_access_vpn` DROP INDEX `id`
2016-04-27 08:35:19,851 ERROR [utils.db.ScriptRunner] (main:null) 
com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: Incorrect table 
definition; there can be only one auto column and it must be defined as a key

That shouldn't be a problem because in `schema-410to420-cleanup.sql` it was:

ALTER TABLE `cloud`.`remote_access_vpn` DROP primary key;
ALTER TABLE `cloud`.`remote_access_vpn` ADD primary key (`id`);

However, it was noticed that `schemaXXXtoYYY-cleanup.sql` files hadn't been 
executed before the failure, so it was decided to move some sentences from 
`schema-481to490.sql` to `schema-481to490-cleanup.sql` so they can be executed 
after all `schemaXXXtoYYY.sql` files, it succeeded.


---
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: [CLOUDSTACK-9328]: Fix vlan issues from t...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1455#issuecomment-215512619
  
@sanju1010 though the code looks alright I don't like the integration test 
should be snooping under the hood. I would rather see them behave as black-box 
tests. I am sure this kind of validating happens elsewhere but still I would 
like some reason for it, can you explain?


---
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.
---

[ASF PERKS] JetBrains products - free to committers

[ilya]

Just found a cool perk many would benefit from on this list.

JetBrains is a company behind many development tools.

AppCode
CLion
DataGrip
dotCover
dotMemory
dotPeek
dotTrace
Hub
IntelliJ IDEA
Kotlin
MPS
PhpStorm
PyCharm
ReSharper
RubyMine
TeamCity
Upsource
WebStorm
YouTrack



If you are ASF committer with valid @apache.org email, you can get
access to JetBrains products free of charge (assuming products will be
used to better open source project within ASF umbrella).

Follow this link to request your license:

https://www.jetbrains.com/shop/eform/apache?product=I

Regards
ilya

[GitHub] cloudstack pull request: [4.7] vmware: improve support for disks

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1365#issuecomment-215506015
  
These CI results basically show that the standard VM workflow in KVM is not 
affected.  I don't have the ability to test this on VMware, but I know you guys 
are already using this, yes?

I think I am pending one LGTM and this is ready...


---
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: [4.7] vmware: improve support for disks

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1365#issuecomment-215505081
  


### CI RESULTS

```
Tests Run: 85
  Skipped: 0
   Failed: 0
   Errors: 0
```



**Associated Uploads**

**`/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_06_35_56_T0QPTY:`**
* 
[dc_entries.obj](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_06_35_56_T0QPTY/dc_entries.obj)
* 
[failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_06_35_56_T0QPTY/failed_plus_exceptions.txt)
* 
[runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/DeployDataCenter__Apr_28_2016_06_35_56_T0QPTY/runinfo.txt)

**`/tmp/MarvinLogs/test_network_IQO9KO:`**
* 
[failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/test_network_IQO9KO/failed_plus_exceptions.txt)
* 
[results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/test_network_IQO9KO/results.txt)
* 
[runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/test_network_IQO9KO/runinfo.txt)

**`/tmp/MarvinLogs/test_vpc_routers_WQUMTV:`**
* 
[failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/test_vpc_routers_WQUMTV/failed_plus_exceptions.txt)
* 
[results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/test_vpc_routers_WQUMTV/results.txt)
* 
[runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1365/tmp/MarvinLogs/test_vpc_routers_WQUMTV/runinfo.txt)


Uploads will be available until `2016-06-28 02:00:00 +0200 CEST`

*Comment created by [`upr comment`](https://github.com/cloudops/upr).*



---
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: CLOUDSTACK-8818: Use MySQL native connect...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215504839
  
I see this in the output when Marvin is installed.
```
Requirement already up-to-date: mysql-connector-python>=1.1.6
```

If I check the version I have it is: 
```
# pip freeze | grep mysql-connector-python
mysql-connector-python==1.1.6
```

Is that appropriate or should we be updating the Marvin requirements?


---
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: CLOUDSTACK-8818: Use MySQL native connect...

[rhtyd]

Github user rhtyd commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215502330
  
@swill for the CI you can install using pip with: (this is also in 
tools/travis/install.sh)
pip install --upgrade --egg 
http://cdn.mysql.com/Downloads/Connector-Python/mysql-connector-python-2.0.4.zip#md5=3df394d89300db95163f17c843ef49df


---
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: CLOUDSTACK-8562: Dynamic Role-Based API C...

[rhtyd]

Github user rhtyd commented on the pull request:

https://github.com/apache/cloudstack/pull/1489#issuecomment-215501683
  
@swill existing users won't be automatically migrated to use this feature, 
see the admin-doc PR that documents the upgrade/migrate procedure; though new 
installations will use dynamic-checker by default.

Will, while the PR is ready to be merged -- I think I can improve the UI/UX 
around how rules are added and evaluated (for example, we may drag/drop rules, 
make it more intuitive etc) and I would also like to improve the docs on using 
the dynamic-checker wrt rules (API name and wildcard rules)


---
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: Handle private gateways more reliably

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1474#issuecomment-215494066
  
cool thanks. :)


---
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: Log asynchronous responses in the api log

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1522#issuecomment-215493805
  
its added to my merge queue.  I need to get through this queue today, it is 
getting long now.  :)


---
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: Handle private gateways more reliably

[kiwiflyer]

Github user kiwiflyer commented on the pull request:

https://github.com/apache/cloudstack/pull/1474#issuecomment-215492977
  
We're pulling this in for testing.


---
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: Log asynchronous responses in the api log

[kiwiflyer]

Github user kiwiflyer commented on the pull request:

https://github.com/apache/cloudstack/pull/1522#issuecomment-215486826
  
@swill This is Ready to Merge


---
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: Log asynchronous responses in the api log

[kiwiflyer]

Github user kiwiflyer commented on the pull request:

https://github.com/apache/cloudstack/pull/1522#issuecomment-215486284
  
Code is straight forward and makes sense. I validated it works in the lab.

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 pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1455#issuecomment-215481337
  
Yes, I think this one is ready...  Can I get one more LGTM code review on 
this one?  

@pdube @kiwiflyer @DaanHoogland @rafaelweingartner 


---
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: Cloudstack 9339: Virtual Routers don't ha...

[dsclose]

Github user dsclose commented on the pull request:

https://github.com/apache/cloudstack/pull/1519#issuecomment-215480948
  
@ustcweizhou How would you recommend I separate this? I can imagine 
separating the issues broadly into two parts:

1. **Routing tables and iptables rules should be properly configured for 
multiple public NICs.** This PR should allow all features to work on standalone 
virtual routers. To achieve this, we'd need commits 9ee1eb6, ad9d72f, 8db879e, 
788b1be and b19e8aa.
2. **Public Interfaces should be down on backup RvRs.** This PR would also 
need to ensure that the transition to master restores the proper routing and 
tables and iptables rules. Therefore it would need commits d582358 and bf285e1.

I'm reticent to do this because the first PR would not allow multiple 
subnets to work on RvR setups. Do you agree with this separation and if so how 
should it be handled?


---
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: CLOUDSTACK-8562: Dynamic Role-Based API C...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1489#issuecomment-215479369
  
In reading this, is it the case that if someone upgrades from a previous 
version to 4.9 (with this code included) they will not be able to take 
advantage of this functionality?  Maybe I did not understand those details 
correctly.  This is just a clarification so I know what to expect (I have 
customers who are interested in this functionality).

@koushik-das Regarding: "Regarding writing a plugin, I meant that if 
someone really needed a functionality like this, they could have easily 
developed it outside of Cloudstack as well (like some kind of portal calling 
Cloudstack APIs)."  Given that I work a company who has done exactly that, I 
can assure you it is not "easy" and takes a huge development effort to recreate 
all the functionality provided by the ACS UI.  This is not really a viable 
option for people looking for this feature.

@koushik-das I also don't understand this "Any blocker (including security 
bugs) in this will make the 4.9 release unusable for new installs as this the 
only authorisation mechanism for entering into the system.".  Can you explain 
what you mean?  How does this feature and a 'blocker bug' render 4.9 unusable?  
I am not quite following this.  Maybe someone else can also chime in to help 
explain his concerns.

Thx...


---
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: CLOUDSTACK-8818: Use MySQL native connect...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215472985
  
So are we happy with the state of this?  How should I adjust my CI to 
validate this fix?  

I currently have: 
```
yum -y install maven tomcat mkisofs python-paramiko 
jakarta-commons-daemon-jsvc jsvc ws-commons-util genisoimage gcc python 
MySQL-python openssh-clients wget git python-ecdsa bzip2 python-setuptools 
mariadb-server mariadb python-devel vim nfs-utils screen setroubleshoot 
openssh-askpass java-1.7.0-openjdk-devel.x86_64 rpm-build rubygems nc
yum -y install 
http://mirror.karneval.cz/pub/linux/fedora/epel/epel-release-latest-7.noarch.rpm
yum --enablerepo=epel -y install sshpass mariadb mysql-connector-python
```

and when marvin is installed:

```
pip install --upgrade tools/marvin/dist/Marvin-*.tar.gz --allow-external 
mysql-connector-python
```

But `--allow-external` has been deprecated, so I don't actually think that 
does anything anymore...


---
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: Log asynchronous responses in the api log

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1522#issuecomment-215463060
  
This change is pretty straight forward and it has been manually verified by 
@koushik-das in addition to the output originally posted by @nathanejohnson.  I 
think this is ready to merge pending one more LGTM.  @kiwiflyer can you review 
this for me?


---
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: CLOUDSTACK-9200: Fixed failed to delete s...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/1282#issuecomment-215460320
  
@anshul1886 I see in some other places in that file when 
`snapshotDao.remove(snapshotId);` is called there is a `return true` right 
after it.  Can you help me understand why we would or would not do that in this 
case?  Thanks...


---
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: CLOUDSTACK-8970 Centos 6.{1,2,3,4,5} gues...

[swill]

Github user swill commented on the pull request:

https://github.com/apache/cloudstack/pull/956#issuecomment-215451916
  
Do we have any verification that this does what it is supposed to?  It 
would be good if we could get some form of verification posted that this 
behaves as expected.


---
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: [LOGISTICS] Move 'apache/cloudstack' to 'apache-cloudstack/cloudstack'

[Will Stevens]

Thanks Bertrand,
I can put a report together for the next board meeting if that will be
helpful.  Would you prefer that I remove the board@ list from this thread
and only keep the ASF Infra team CC'ed?

Cheers,

Will

On Thu, Apr 28, 2016 at 3:58 AM, Bertrand Delacretaz  wrote:

> Hi Will,
>
> On Wed, Apr 27, 2016 at 8:01 PM, Will Stevens 
> wrote:
> > ...I have CC'ed in both the ACS dev@ list and the ASF board@ list to
> try to
> > keep everyone on the same page...
>
> Thanks - with my board member hat on what's important is that relevant
> changes are mentioned in your next report to the board.
>
> You're also welcome to send an out-of-schedule report if that makes
> sense, the board meetings are on the third Wednesday of each month.
>
> -Bertrand
>

[GitHub] cloudstack pull request: CLOUDSTACK-9365 : updateVirtualMachine wi...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1523#issuecomment-215447324
  
@nlivens I don't care for more scripts, just for a test showing the problem 
so we no it doesn't regress. Whatever meets the goal easiest.


---
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: Kvm plugin restructure

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1521#issuecomment-215446492
  
Enjoying the discussion I would like to -1 for the entire refactor all at 
once. I changed this project because it was feasible and convenient. I can see 
myself doing more then just this one but why refactorring unsupported projects 
that are laying around waiting for a quiet or banging death out there. Let's do 
only those things that are actively being maintained.


---
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: Kvm plugin restructure

[jburwell]

Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1521#issuecomment-215438021
  
I would love to dump Maven for Gradle.  However, I agree with @wido -- we 
currently have much higher priorities to address.  For me, it is fun to banter 
a bit about it in a PR thread. ;)

Changing the Maven structure has other benefits.  Not only could we rid 
ourselves of the additional code to override the structure, conforming wider 
Java idioms and standards it easier for new contributors to understand the 
codebase.  @wido I agree with your suggestion to make one bulk change.  It 
would require coordination across the community as it would require a lot of 
rebasing once it is merged, but it seems like we would get an economy of scale 
on the effort.


---
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: IPv6 progress in Basic Networking

[Wido den Hollander]

> Op 28 april 2016 om 14:22 schreef Nux! :
> 
> 
> Ok Wido, thanks for the clarification, it sounds good.
> 
> From my point of view, if there is a clear relation between IPs and VMs, then 
> as an operator I am happy.
> 

Yes, there will. So for a POD VLAN you will assing *ONE* IPv6 /64. That gives 
you roughly 18 trillion addresses to hand out in that POD. Not something will 
will run out quickly.

With the MAC Address of the Instance you can calculate which address it will 
obtain. The API can return that address towards any caller.

So yes, IPv6 address maps to Instance.

Wido

> Lucian
> 
> --
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro
> 
> - Original Message -
> > From: "Wido den Hollander" 
> > To: "Nux!" , dev@cloudstack.apache.org
> > Sent: Thursday, 28 April, 2016 11:41:30
> > Subject: Re: IPv6 progress in Basic Networking
> 
> >> Op 28 april 2016 om 10:50 schreef Nux! :
> >> 
> >> 
> >> Hi, see in-line
> >> 
> >> --
> >> Sent from the Delta quadrant using Borg technology!
> >> 
> >> Nux!
> >> www.nux.ro
> >> 
> >> >>Since we know the MAC address and the /64 prefix we can *calculate* the 
> >> >>address
> >> >>a Instance will take. With that we do not have to store the address in a
> >> >>database.
> >> > 
> >> > Doing bit of research on this makes me believe that this is a preferred 
> >> > way of
> >> > getting ipv6 address for the instance. Was wondering why we cannot do 
> >> > the same
> >> > for ipV4, but there I guess we will run into uniqueness issue.
> >> 
> >> Please feel free to correct me, but SLAAC obtained IPv6 addresses will not
> >> always be determinable based on MAC address.
> >> Last I looked at this - because we wanted to somehow use IPv6 in SG zone 
> >> and
> >> failed :> - some Linux distros and Windows versions can enable privacy
> >> extensions (net.ipv6.conf.all.use_tempaddr).
> >> 
> > 
> > Yes, so a few things are required for the Instances/Templates running. Just 
> > like
> > a Instance now has to use DHCP for IPv4 it will have to use SLAAC for IPv6.
> > 
> > All installers right now already support SLAAC. Keep in mind that even if 
> > you
> > use Privacy Extensions your system will *ALWAYS* generate the SLAAC address
> > based on the MAC.
> > 
> > Take my desktop here for example:
> > 
> > wido@wido-desktop:~$ ip addr show dev eth0
> > 2: eth0:  mtu 1500 qdisc pfifo_fast state 
> > UP
> > group default qlen 1000
> >link/ether c0:3f:d5:68:28:08 brd ff:ff:ff:ff:ff:ff
> >inet 192.168.5.70/24 brd 192.168.5.255 scope global eth0
> >   valid_lft forever preferred_lft forever
> >inet6 2001:980:XXX:0:f1ac:3224:c00d:47e4/64 scope global temporary 
> > dynamic
> >   valid_lft 7197sec preferred_lft 7197sec
> >inet6 2001:980:XXX:0:c23f:d5ff:fe68:2808/64 scope global dynamic
> >   valid_lft 7197sec preferred_lft 7197sec
> >inet6 fe80::c23f:d5ff:fe68:2808/64 scope link
> >   valid_lft forever preferred_lft forever
> > wido@wido-desktop:~$
> > 
> > It generates the temporary address, but next to that it also generates the
> > static address.
> > 
> > For Windows it is easy as well:
> > 
> > netsh interface ipv6 set privacy state=disabled store=persistent
> > netsh interface ipv6 set global randomizeidentifiers=disabled 
> > store=persistent
> > 
> >> So for the scenario to work, we need to make sure privacy extensions are
> >> switched off - but who knows for how long this will be doable, not to 
> >> mention
> >> some ISO, PCI etc policies might require it on.
> >> 
> > 
> > As CloudStack we do not have 100% control of the Instances. Same story with 
> > IPv4
> > right now. Our docs should just say how to configure a Instance.
> > 
> > But with IPv6 you should also let go of the single address. My proposal is 
> > also
> > to enable Prefix Delegation:
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+Basic+Networking
> > 
> > Next to the /128 (Single Address) a Instance can get a /60 where it can 
> > pick as
> > many random addresses from as it wants.
> > 
> >> Some random reading from the interwebz, he touches on the subject:
> >> https://major.io/2016/04/17/enable-ipv6-privacy-networkmanager/
> >> 
> >> 
> >> IMHO - without knowing all the ugly details behind DHCPv6 and so on - I 
> >> think
> >> going for DHCP might be the better option in the long run.
> >> 
> > 
> > DHCPv6 is ugly and not properly supported. If we go down that road we 
> > suddenly
> > have to:
> > - Store a IPv6 address in a database
> > - Provision a DHCPv6 server
> > 
> > That's additional steps which can fail and make stuff more complex. KISS is 
> > want
> > you want.
> > 
> > SLAAC is supported in Linux, *BSD and Windows with EUI-64 based addresses, 
> > that
> > is doable.
> > 
> > Again, with IPv4 we currently also require various settings inside the 
> > Instance.
> > That will not be different with IPv6.
> 

Re: Rafael Weingartner joining the PMC

[Patrick Dube]

Congratulations!

On Thu, Apr 28, 2016 at 8:19 AM, Simon Weller  wrote:

> Congratulations Rafael!
>
> - Si
>
> 
> From: Nux! 
> Sent: Thursday, April 28, 2016 7:06 AM
> To: dev@cloudstack.apache.org
> Subject: Re: Rafael Weingartner joining the PMC
>
> Congrats Rafael :)
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
> - Original Message -
> > From: "Daan Hoogland" 
> > To: "dev" 
> > Sent: Thursday, 28 April, 2016 11:59:28
> > Subject: Rafael Weingartner joining the PMC
>
> > People,
> >
> > The PMC has ask Rafael to join them to oversee the project and he has
> > gracefully accepted. Please join me in congratulating Rafael and wish him
> > wisdom in his new task.
> >
> > ​on behalve of the PMC,​
> > --
> > Daan
>

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

[sanju1010]

Github user sanju1010 commented on the pull request:

https://github.com/apache/cloudstack/pull/1455#issuecomment-215427658
  
@swill , it is all green now. Can you please merge this 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 pull request: CLOUDSTACK-9365 : updateVirtualMachine wi...

[nlivens]

Github user nlivens commented on the pull request:

https://github.com/apache/cloudstack/pull/1523#issuecomment-215421778
  
@DaanHoogland, you already have a UserData test script, do you want us to 
add this test to the same script, or create a new one for it?


---
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: CLOUDSTACK-8818: Use MySQL native connect...

[wido]

Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215419241
  
@rhtyd I grepped *after* my modifications :)

Yes, the API between MySQLdb and mysql.connector is the same. The only 
difference is the import and the connection creation.

Afterwards you can get a cursor and execute queries.

When you do more advanced/complex things there are API differences, but 
that's not being used by CloudStack.


---
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: CLOUDSTACK-9349: Enable root disk detach ...

[kiwiflyer]

Github user kiwiflyer commented on the pull request:

https://github.com/apache/cloudstack/pull/1500#issuecomment-215410772
  
We'll take a look at this in a separate PR. We don't use the UI.


---
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: CLOUDSTACK-9365 : updateVirtualMachine wi...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1523#issuecomment-215410690
  
thought the code looks sane, could you be bothered to show the error in a 
marvin test, Nick. It seems a good candidate to enter the regression suite.


---
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: Incorrect system vm templates for master build.

[Will Stevens]

4.9 will require a new system vm template when we add the strongswan
support and ospf, but I don't think I have added anything yet which will
require a new system vm (yet).

Rohit started a dev@ thread about a week or so ago on this topic for us.
On Apr 28, 2016 7:38 AM, "Rajani Karuturi"  wrote:

> ah..makes sense. Thanks Erik. I will revert the config change.
>
> ~Rajani
>
> On Thu, Apr 28, 2016 at 4:11 PM, Erik Weber  wrote:
>
> > AFAIK it hasn't been necessary to update the systemvm template yet, that
> is
> > probably why it still shows as 4.6.0.
> >
> > --
> > Erik
> >
> > On Thu, Apr 28, 2016 at 12:04 PM, Bharat Kumar <
> > bharat.ku...@accelerite.com>
> > wrote:
> >
> > > Hi All,
> > >
> > > looks like the systemvm templates are not getting built correctly in
> > > http://jenkins.buildacloud.org/job/build-systemvm64-master.
> > >
> > > The build version is in correctly set to 4.6,  Im not sure but i think
> > the
> > > build version in
> > > /etc/cloudstck-release is also getting wrongly set inside the template.
> > >
> > > Can any one please fix this. I do not have access to jenkins.
> > >
> > > Thanks,
> > > Bharat.
> > >
> > >
> > >
> > >
> > >
> > >
> > > 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: CLOUDSTACK-9283: add pid to java argument...

[kiwiflyer]

Github user kiwiflyer commented on the pull request:

https://github.com/apache/cloudstack/pull/1409#issuecomment-215409715
  
Tested this in a 4.8 lab. I agree that there is little point in running CI 
against this, as it's a config change.

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 pull request: CLOUDSTACK-9349: Enable root disk detach ...

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1500#issuecomment-215408560
  
is there any UI change possible?


---
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: Log asynchronous responses in the api log

[koushik-das]

Github user koushik-das commented on the pull request:

https://github.com/apache/cloudstack/pull/1522#issuecomment-215407693
  
LGTM. Verified in api.log that job id is getting appended.


---
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: IPv6 progress in Basic Networking

[Nux!]

Ok Wido, thanks for the clarification, it sounds good.

>From my point of view, if there is a clear relation between IPs and VMs, then 
>as an operator I am happy.

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Wido den Hollander" 
> To: "Nux!" , dev@cloudstack.apache.org
> Sent: Thursday, 28 April, 2016 11:41:30
> Subject: Re: IPv6 progress in Basic Networking

>> Op 28 april 2016 om 10:50 schreef Nux! :
>> 
>> 
>> Hi, see in-line
>> 
>> --
>> Sent from the Delta quadrant using Borg technology!
>> 
>> Nux!
>> www.nux.ro
>> 
>> >>Since we know the MAC address and the /64 prefix we can *calculate* the 
>> >>address
>> >>a Instance will take. With that we do not have to store the address in a
>> >>database.
>> > 
>> > Doing bit of research on this makes me believe that this is a preferred 
>> > way of
>> > getting ipv6 address for the instance. Was wondering why we cannot do the 
>> > same
>> > for ipV4, but there I guess we will run into uniqueness issue.
>> 
>> Please feel free to correct me, but SLAAC obtained IPv6 addresses will not
>> always be determinable based on MAC address.
>> Last I looked at this - because we wanted to somehow use IPv6 in SG zone and
>> failed :> - some Linux distros and Windows versions can enable privacy
>> extensions (net.ipv6.conf.all.use_tempaddr).
>> 
> 
> Yes, so a few things are required for the Instances/Templates running. Just 
> like
> a Instance now has to use DHCP for IPv4 it will have to use SLAAC for IPv6.
> 
> All installers right now already support SLAAC. Keep in mind that even if you
> use Privacy Extensions your system will *ALWAYS* generate the SLAAC address
> based on the MAC.
> 
> Take my desktop here for example:
> 
> wido@wido-desktop:~$ ip addr show dev eth0
> 2: eth0:  mtu 1500 qdisc pfifo_fast state UP
> group default qlen 1000
>link/ether c0:3f:d5:68:28:08 brd ff:ff:ff:ff:ff:ff
>inet 192.168.5.70/24 brd 192.168.5.255 scope global eth0
>   valid_lft forever preferred_lft forever
>inet6 2001:980:XXX:0:f1ac:3224:c00d:47e4/64 scope global temporary dynamic
>   valid_lft 7197sec preferred_lft 7197sec
>inet6 2001:980:XXX:0:c23f:d5ff:fe68:2808/64 scope global dynamic
>   valid_lft 7197sec preferred_lft 7197sec
>inet6 fe80::c23f:d5ff:fe68:2808/64 scope link
>   valid_lft forever preferred_lft forever
> wido@wido-desktop:~$
> 
> It generates the temporary address, but next to that it also generates the
> static address.
> 
> For Windows it is easy as well:
> 
> netsh interface ipv6 set privacy state=disabled store=persistent
> netsh interface ipv6 set global randomizeidentifiers=disabled store=persistent
> 
>> So for the scenario to work, we need to make sure privacy extensions are
>> switched off - but who knows for how long this will be doable, not to mention
>> some ISO, PCI etc policies might require it on.
>> 
> 
> As CloudStack we do not have 100% control of the Instances. Same story with 
> IPv4
> right now. Our docs should just say how to configure a Instance.
> 
> But with IPv6 you should also let go of the single address. My proposal is 
> also
> to enable Prefix Delegation:
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+Basic+Networking
> 
> Next to the /128 (Single Address) a Instance can get a /60 where it can pick 
> as
> many random addresses from as it wants.
> 
>> Some random reading from the interwebz, he touches on the subject:
>> https://major.io/2016/04/17/enable-ipv6-privacy-networkmanager/
>> 
>> 
>> IMHO - without knowing all the ugly details behind DHCPv6 and so on - I think
>> going for DHCP might be the better option in the long run.
>> 
> 
> DHCPv6 is ugly and not properly supported. If we go down that road we suddenly
> have to:
> - Store a IPv6 address in a database
> - Provision a DHCPv6 server
> 
> That's additional steps which can fail and make stuff more complex. KISS is 
> want
> you want.
> 
> SLAAC is supported in Linux, *BSD and Windows with EUI-64 based addresses, 
> that
> is doable.
> 
> Again, with IPv4 we currently also require various settings inside the 
> Instance.
> That will not be different with IPv6.
> 
> Wido
> 
> > Lucian

Re: Rafael Weingartner joining the PMC

[Simon Weller]

Congratulations Rafael!

- Si


From: Nux! 
Sent: Thursday, April 28, 2016 7:06 AM
To: dev@cloudstack.apache.org
Subject: Re: Rafael Weingartner joining the PMC

Congrats Rafael :)

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Daan Hoogland" 
> To: "dev" 
> Sent: Thursday, 28 April, 2016 11:59:28
> Subject: Rafael Weingartner joining the PMC

> People,
>
> The PMC has ask Rafael to join them to oversee the project and he has
> gracefully accepted. Please join me in congratulating Rafael and wish him
> wisdom in his new task.
>
> ​on behalve of the PMC,​
> --
> Daan

Re: CaaS in CloudStack with Kubernetes

[Simon Weller]

Very exciting Sebastien. Congratulations!

We're a heavy user of containers, so we'll definitely be taking a look at this 
when it is released.

- Si


From: Sanjeev Neelarapu 
Sent: Thursday, April 28, 2016 6:41 AM
To: dev@cloudstack.apache.org
Cc: market...@cloudstack.apache.org
Subject: RE: CaaS in CloudStack with Kubernetes

Congratulations Sebastien !!

Best Regards,
Sanjeev N
Chief Product Engineer, Accelerite
Off: +91 40 6722 9368 | EMail: sanjeev.neelar...@accelerite.com



-Original Message-
From: Sebastien Goasguen [mailto:run...@gmail.com]
Sent: Wednesday, April 27, 2016 11:41 PM
To: dev@cloudstack.apache.org
Cc: market...@cloudstack.apache.org
Subject: Re: CaaS in CloudStack with Kubernetes

let me reply on dev@ to avoid cross posting.

> On Apr 27, 2016, at 4:06 PM, Nux!  wrote:
>
> Hello,
>
> Very nice, but I am somewhat surprised! I thought we were not going after 
> docker/containers/etc and let others (CoreOS etc) do it?
> What has changed?
>
> Lucian
>
> --
> Sent from the Delta quadrant using Borg technology!
>
> Nux!
> www.nux.ro
>
> - Original Message -
>> From: "Sebastien Goasguen" 
>> To: market...@cloudstack.apache.org
>> Cc: dev@cloudstack.apache.org
>> Sent: Friday, 22 April, 2016 17:02:22
>> Subject: CaaS in CloudStack with Kubernetes
>
>> Hi folks,
>>
>> I have been quite silent this past few weeks because aside from
>> voting in a new VP (soon to be announced), I  have been hard at work
>> for my new endeavors with Skippbox.
>>
>> I am quite excited that today we could not wait and Skippbox together
>> with Shapeblue announced a preview of a Kubernetes/CloudStack integration:
>> http://www.benzinga.com/pressreleases/16/04/g7872516/shapeblue-harnes
>> s-the-power-of-kubernetes-and-apache-cloudstack-to-give
>>
>> This is currently still a by request preview, but once we are done
>> with much better testing and even tighter integration, the code will
>> be open sourced as a CloudStack plugin.
>>
>> I wouldn’t mind couple supportive tweets and if you want a demo
>> before you can run it yourself, ping me.
>>
>> Cheers,
>>
>> -Sebastien




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: Rafael Weingartner joining the PMC

[Nux!]

Congrats Rafael :)

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Daan Hoogland" 
> To: "dev" 
> Sent: Thursday, 28 April, 2016 11:59:28
> Subject: Rafael Weingartner joining the PMC

> People,
> 
> The PMC has ask Rafael to join them to oversee the project and he has
> gracefully accepted. Please join me in congratulating Rafael and wish him
> wisdom in his new task.
> 
> ​on behalve of the PMC,​
> --
> Daan

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

[kiwiflyer]

Github user kiwiflyer commented on the pull request:

https://github.com/apache/cloudstack/pull/1500#issuecomment-215401063
  
@swill CI Test good, 2+ reviews. Ready to Merge.


---
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: CaaS in CloudStack with Kubernetes

[Sanjeev Neelarapu]

Congratulations Sebastien !!

Best Regards,
Sanjeev N
Chief Product Engineer, Accelerite
Off: +91 40 6722 9368 | EMail: sanjeev.neelar...@accelerite.com 



-Original Message-
From: Sebastien Goasguen [mailto:run...@gmail.com] 
Sent: Wednesday, April 27, 2016 11:41 PM
To: dev@cloudstack.apache.org
Cc: market...@cloudstack.apache.org
Subject: Re: CaaS in CloudStack with Kubernetes

let me reply on dev@ to avoid cross posting.

> On Apr 27, 2016, at 4:06 PM, Nux!  wrote:
> 
> Hello,
> 
> Very nice, but I am somewhat surprised! I thought we were not going after 
> docker/containers/etc and let others (CoreOS etc) do it?
> What has changed?
> 
> Lucian
> 
> --
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro
> 
> - Original Message -
>> From: "Sebastien Goasguen" 
>> To: market...@cloudstack.apache.org
>> Cc: dev@cloudstack.apache.org
>> Sent: Friday, 22 April, 2016 17:02:22
>> Subject: CaaS in CloudStack with Kubernetes
> 
>> Hi folks,
>> 
>> I have been quite silent this past few weeks because aside from 
>> voting in a new VP (soon to be announced), I  have been hard at work 
>> for my new endeavors with Skippbox.
>> 
>> I am quite excited that today we could not wait and Skippbox together 
>> with Shapeblue announced a preview of a Kubernetes/CloudStack integration:
>> http://www.benzinga.com/pressreleases/16/04/g7872516/shapeblue-harnes
>> s-the-power-of-kubernetes-and-apache-cloudstack-to-give
>> 
>> This is currently still a by request preview, but once we are done 
>> with much better testing and even tighter integration, the code will 
>> be open sourced as a CloudStack plugin.
>> 
>> I wouldn’t mind couple supportive tweets and if you want a demo 
>> before you can run it yourself, ping me.
>> 
>> Cheers,
>> 
>> -Sebastien




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: Incorrect system vm templates for master build.

[Rajani Karuturi]

ah..makes sense. Thanks Erik. I will revert the config change.

~Rajani

On Thu, Apr 28, 2016 at 4:11 PM, Erik Weber  wrote:

> AFAIK it hasn't been necessary to update the systemvm template yet, that is
> probably why it still shows as 4.6.0.
>
> --
> Erik
>
> On Thu, Apr 28, 2016 at 12:04 PM, Bharat Kumar <
> bharat.ku...@accelerite.com>
> wrote:
>
> > Hi All,
> >
> > looks like the systemvm templates are not getting built correctly in
> > http://jenkins.buildacloud.org/job/build-systemvm64-master.
> >
> > The build version is in correctly set to 4.6,  Im not sure but i think
> the
> > build version in
> > /etc/cloudstck-release is also getting wrongly set inside the template.
> >
> > Can any one please fix this. I do not have access to jenkins.
> >
> > Thanks,
> > Bharat.
> >
> >
> >
> >
> >
> >
> > 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: [ANNOUNCE] New committer: Simon Weller

[Daan Hoogland]

welcome, kiwi. Good work and keep going.

On Thu, Apr 28, 2016 at 11:25 AM, Nick LIVENS  wrote:

> Congrats Simon!
>
> Kind regards,
> Nick Livens
>
> On Thu, Apr 28, 2016 at 10:15 AM, Nux!  wrote:
>
> > Congrats! :-)
> >
> > --
> > Sent from the Delta quadrant using Borg technology!
> >
> > Nux!
> > www.nux.ro
> >
> > - Original Message -
> > > From: "Paul Angus" 
> > > To: dev@cloudstack.apache.org
> > > Sent: Thursday, 28 April, 2016 08:44:36
> > > Subject: RE: [ANNOUNCE] New committer: Simon Weller
> >
> > > Congratulations Simon!
> > >
> > >
> > >
> > > Kind regards,
> > >
> > > Paul Angus
> > >
> > > Regards,
> > >
> > > Paul Angus
> > >
> > > paul.an...@shapeblue.com
> > > www.shapeblue.com
> > > 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> > > @shapeblue
> > >
> > > -Original Message-
> > > From: Erik Weber [mailto:terbol...@gmail.com]
> > > Sent: 28 April 2016 08:24
> > > To: dev 
> > > Subject: [ANNOUNCE] New committer: Simon Weller
> > >
> > > The Project Management Committee (PMC) for Apache CloudStack has asked
> > Simon
> > > Weller to become a committer and we are pleased to announce that they
> > have
> > > accepted.
> > >
> > >
> > > Being a committer allows many contributors to contribute more
> > autonomously. For
> > > developers, it makes it easier to submit changes and eliminates the
> need
> > to
> > > have contributions reviewed via the patch submission process. Whether
> > > contributions are development-related or otherwise, it is a recognition
> > of a
> > > contributor's participation in the project and commitment to the
> project
> > and
> > > the Apache Way.
> > >
> > > Please join me in congratulating Simon
> > >
> > > --Erik
> > > on behalf of the CloudStack PMC
> >
>



-- 
Daan

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

[rhtyd]

Github user rhtyd commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215392929
  
@wido that is true, so will simply fixing the imports and usage work?

I found using `git grep MySQLdb`:
```
client/bindir/cloud-update-xenserver-licenses.in:import MySQLdb
client/bindir/cloud-update-xenserver-licenses.in:   conn = 
MySQLdb.connect(host=host,user=username,passwd=password)
python/lib/cloudutils/db.py:import MySQLdb
python/lib/cloudutils/db.py:txn = 
MySQLdb.Connect(host=self.host, user=self.username,
python/lib/cloudutils/db.py:txn = 
MySQLdb.Connect(host=self.host, user=self.username,
python/lib/cloudutils/db.py:db = 
MySQLdb.Connect(host=self.host, user=self.username,
python/lib/cloudutils/db.py:db = 
MySQLdb.Connect(host=self.host, user=self.username,
setup/bindir/cloud-migrate-databases.in:import MySQLdb
setup/bindir/cloud-migrate-databases.in:self.conn = 
MySQLdb.connect(host=self.host,
```


---
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: CLOUDSTACK-8818: Use MySQL native connect...

[wido]

Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1054#issuecomment-215391701
  
@swill @rhtyd: I looked through all the code and fixed it. Is was just two 
more scripts.

wido@wido-desktop:~/repos/cloudstack$ grep -R "MySQLdb" *
wido@wido-desktop:~/repos/cloudstack$

API-wise the syntax is the same between MySQLdb and this native connector.


---
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: Rafael Weingartner joining the PMC

[Sebastien Goasguen]

Congrats Rafael,


> On Apr 28, 2016, at 12:59 PM, Daan Hoogland  wrote:
> 
> People,
> 
> The PMC has ask Rafael to join them to oversee the project and he has
> gracefully accepted. Please join me in congratulating Rafael and wish him
> wisdom in his new task.
> 
> ​on behalve of the PMC,​
> -- 
> Daan

[ANNOUNCE] Rafael Weingartner joining the PMC

[Daan Hoogland]

Guess I should have facilitated everybody's mail filters

On Thu, Apr 28, 2016 at 12:59 PM, Daan Hoogland 
wrote:

> People,
>
> The PMC has ask Rafael to join them to oversee the project and he has
> gracefully accepted. Please join me in congratulating Rafael and wish him
> wisdom in his new task.
>
> ​on behalve of the PMC,​
> --
> Daan
>



-- 
Daan

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

[koushik-das]

Github user koushik-das commented on the pull request:

https://github.com/apache/cloudstack/pull/1489#issuecomment-215389142
  
+1 on the feature. Based on testing done on the simulator with root admin 
and normal user for a few APIs
+0 on the immediate removal of command.properties/static checker for new 
installs and making dynamic checker as the only option for authorization.
Reason: Any blocker (including security bugs) in this will make the 4.9 
release unusable for new installs as this the only authorisation mechanism for 
entering into the system. It can still be used for pilot/test purposes though.

@rhtyd Who is saying to stop innovating and improving? I am only talking 
about stability and choice.
@jburwell Hopefully the above reason explains how this is different from 
other features (which impacts only a certain part of the product). Regarding 
writing a plugin, I meant that if someone really needed a functionality like 
this, they could have easily developed it outside of Cloudstack as well (like 
some kind of portal calling Cloudstack APIs).



---
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.
---

Rafael Weingartner joining the PMC

[Daan Hoogland]

People,

The PMC has ask Rafael to join them to oversee the project and he has
gracefully accepted. Please join me in congratulating Rafael and wish him
wisdom in his new task.

​on behalve of the PMC,​
-- 
Daan

Re: Incorrect system vm templates for master build.

[Erik Weber]

AFAIK it hasn't been necessary to update the systemvm template yet, that is
probably why it still shows as 4.6.0.

-- 
Erik

On Thu, Apr 28, 2016 at 12:04 PM, Bharat Kumar 
wrote:

> Hi All,
>
> looks like the systemvm templates are not getting built correctly in
> http://jenkins.buildacloud.org/job/build-systemvm64-master.
>
> The build version is in correctly set to 4.6,  Im not sure but i think the
> build version in
> /etc/cloudstck-release is also getting wrongly set inside the template.
>
> Can any one please fix this. I do not have access to jenkins.
>
> Thanks,
> Bharat.
>
>
>
>
>
>
> 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: IPv6 progress in Basic Networking

[Wido den Hollander]

> Op 28 april 2016 om 10:50 schreef Nux! :
> 
> 
> Hi, see in-line
> 
> --
> Sent from the Delta quadrant using Borg technology!
> 
> Nux!
> www.nux.ro
> 
> >>Since we know the MAC address and the /64 prefix we can *calculate* the 
> >>address
> >>a Instance will take. With that we do not have to store the address in a
> >>database.
> > 
> > Doing bit of research on this makes me believe that this is a preferred way 
> > of
> > getting ipv6 address for the instance. Was wondering why we cannot do the 
> > same
> > for ipV4, but there I guess we will run into uniqueness issue.
> 
> Please feel free to correct me, but SLAAC obtained IPv6 addresses will not 
> always be determinable based on MAC address.
> Last I looked at this - because we wanted to somehow use IPv6 in SG zone and 
> failed :> - some Linux distros and Windows versions can enable privacy 
> extensions (net.ipv6.conf.all.use_tempaddr).
> 

Yes, so a few things are required for the Instances/Templates running. Just 
like a Instance now has to use DHCP for IPv4 it will have to use SLAAC for IPv6.

All installers right now already support SLAAC. Keep in mind that even if you 
use Privacy Extensions your system will *ALWAYS* generate the SLAAC address 
based on the MAC.

Take my desktop here for example:

wido@wido-desktop:~$ ip addr show dev eth0
2: eth0:  mtu 1500 qdisc pfifo_fast state UP 
group default qlen 1000
link/ether c0:3f:d5:68:28:08 brd ff:ff:ff:ff:ff:ff
inet 192.168.5.70/24 brd 192.168.5.255 scope global eth0
   valid_lft forever preferred_lft forever
inet6 2001:980:XXX:0:f1ac:3224:c00d:47e4/64 scope global temporary dynamic 
   valid_lft 7197sec preferred_lft 7197sec
inet6 2001:980:XXX:0:c23f:d5ff:fe68:2808/64 scope global dynamic 
   valid_lft 7197sec preferred_lft 7197sec
inet6 fe80::c23f:d5ff:fe68:2808/64 scope link 
   valid_lft forever preferred_lft forever
wido@wido-desktop:~$

It generates the temporary address, but next to that it also generates the 
static address.

For Windows it is easy as well:

netsh interface ipv6 set privacy state=disabled store=persistent
netsh interface ipv6 set global randomizeidentifiers=disabled store=persistent

> So for the scenario to work, we need to make sure privacy extensions are 
> switched off - but who knows for how long this will be doable, not to mention 
> some ISO, PCI etc policies might require it on.
> 

As CloudStack we do not have 100% control of the Instances. Same story with 
IPv4 right now. Our docs should just say how to configure a Instance.

But with IPv6 you should also let go of the single address. My proposal is also 
to enable Prefix Delegation: 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+Basic+Networking

Next to the /128 (Single Address) a Instance can get a /60 where it can pick as 
many random addresses from as it wants.

> Some random reading from the interwebz, he touches on the subject: 
> https://major.io/2016/04/17/enable-ipv6-privacy-networkmanager/
> 
> 
> IMHO - without knowing all the ugly details behind DHCPv6 and so on - I think 
> going for DHCP might be the better option in the long run.
> 

DHCPv6 is ugly and not properly supported. If we go down that road we suddenly 
have to:
- Store a IPv6 address in a database
- Provision a DHCPv6 server

That's additional steps which can fail and make stuff more complex. KISS is 
want you want.

SLAAC is supported in Linux, *BSD and Windows with EUI-64 based addresses, that 
is doable.

Again, with IPv4 we currently also require various settings inside the 
Instance. That will not be different with IPv6.

Wido

> Lucian

Re: Incorrect system vm templates for master build.

[Rajani Karuturi]

In the job configuration, I see the version as 4.6.0.
I am not sure if that is where it reads from. I changed it to 4.9.0

- export version=4.6.0
+ export version=4.9.0

I will keep an eye on the job. Let me know if there are any issues with
this change.


~Rajani

On Thu, Apr 28, 2016 at 3:34 PM, Bharat Kumar 
wrote:

> Hi All,
>
> looks like the systemvm templates are not getting built correctly in
> http://jenkins.buildacloud.org/job/build-systemvm64-master.
>
> The build version is in correctly set to 4.6,  Im not sure but i think the
> build version in
> /etc/cloudstck-release is also getting wrongly set inside the template.
>
> Can any one please fix this. I do not have access to jenkins.
>
> Thanks,
> Bharat.
>
>
>
>
>
>
> 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.
>

Incorrect system vm templates for master build.

[Bharat Kumar]

Hi All,

looks like the systemvm templates are not getting built correctly in 
http://jenkins.buildacloud.org/job/build-systemvm64-master.

The build version is in correctly set to 4.6,  Im not sure but i think the 
build version in
/etc/cloudstck-release is also getting wrongly set inside the template.

Can any one please fix this. I do not have access to jenkins.

Thanks,
Bharat.






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: CLOUDSTACK-9200: Fixed failed to delete s...

[anshul1886]

Github user anshul1886 commented on the pull request:

https://github.com/apache/cloudstack/pull/1282#issuecomment-215375588
  
@koushik-das All hypervisor uses XenServerSnapshotStrategy so it should be 
applicable on all hypervisors.


---
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: CLOUDSTACK-9349: Enable root disk detach ...

[koushik-das]

Github user koushik-das commented on the pull request:

https://github.com/apache/cloudstack/pull/1500#issuecomment-215372191
  
LGTM. Had already given one earlier based on simulator testing.


---
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: CLOUDSTACK-9194: Making the console popup...

[anshul1886]

Github user anshul1886 commented on the pull request:

https://github.com/apache/cloudstack/pull/1270#issuecomment-215368331
  
@koushik-das Other browsers except IE stopped supporting that attribute. So 
it will not have impact. This can also be verified by seeing in other browsers 
console window is resizable.


---
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: [4.7] secure and hidden config values are...

[koushik-das]

Github user koushik-das commented on the pull request:

https://github.com/apache/cloudstack/pull/927#issuecomment-215366546
  
@karuturi Any updates?


---
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: [4.7] CLOUDSTACK-8950 Hypervisor Paramete...

[koushik-das]

Github user koushik-das commented on the pull request:

https://github.com/apache/cloudstack/pull/928#issuecomment-215366187
  
@karuturi Any updates on this?


---
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: CLOUDSTACK-8969 VPN customer gateway can'...

[koushik-das]

Github user koushik-das commented on the pull request:

https://github.com/apache/cloudstack/pull/955#issuecomment-215365287
  
@SudharmaJain Any updates?


---
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.
---