[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
changed the title of commits.


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
Thanks @ustcweizhou please also meld them into a single commit


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
squashed. no code change compared with last commit.
git diff 9d4d2683ff7eaed27e15e7db91aadcba05736cc8



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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

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


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread blueorangutan
Github user blueorangutan commented on the issue:

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


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-449


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
@blueorangutan test


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


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


[GitHub] cloudstack issue #1888: CLOUDSTACK-9710: Switch to JRE1.8

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1888
  
With `cs.jdk.version` set to 1.8 for both as source and target versions, 
generated jars can only be run against JRE1.8. Forward compatibility is not 
supported (build with 1.8, run on 1.7), we can force the maven compiler plugin 
to target for 1.7 while set source to 1.8, but I read that it may not work 
sometimes and I would like to avoid that given 1.7 has EOL-ed.

Without having package/version dependencies fixated on jdk/jre1.8, 
sysadmins will only find out when the agents/usage server or mgmt server fails 
to start with an error message such as `Unsupported major.minor version 52.0`


---
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 #1897: CLOUDSTACK-9733: Concurrent volume snapshots ...

2017-01-09 Thread sureshanaparti
GitHub user sureshanaparti opened a pull request:

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

CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and 
are not limited per host as per the global configuration parameter 
"concurrent.snapshots.threshold.perhost".

- User can take snapshots of the volumes attached to a VM simultaneously 
(when the underlying hypervisor supports). Only subsequent snapshot operations 
are in parallel.
- User wants to take snapshots of the same volume at the same time 
(either recurring mode or manually). Only subsequent snapshot operations are 
in parallel.
- User wants to limit the no. of concurrent snapshots per host in the 
cluster/cloud using the config paramter 
"concurrent.snapshots.threshold.perhost" (at the cluster level).

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

$ git pull https://github.com/Accelerite/cloudstack CLOUDSTACK-9733

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

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

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

This closes #1897


commit 46d79f1678399cbd884973358504a2773a61abaf
Author: Suresh Kumar Anaparti 
Date:   2017-01-09T11:59:48Z

CLOUDSTACK-9733: Concurrent volume snapshots of a VM are not allowed and 
are not limited per host as per the global configuration parameter 
"concurrent.snapshots.threshold.perhost".




---
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 #1898: CLOUDSTACK-8862: Lock on volume before attemp...

2017-01-09 Thread ProjectMoon
GitHub user ProjectMoon opened a pull request:

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

CLOUDSTACK-8862: Lock on volume before attempting to attach it.

It is possible to attach a single volume to multiple instances by running 
the commands close to one another. In KVM, this can be verified by checking the 
`virsh dumpxml` for the instances once the volume attaches are complete. This 
PR adds a lock on the volume ID in the orchestration step to make sure the 
method is only executed one at a time for any given volume ID. So in the 
scenario of someone sending simultaneous volume attach requests, one should 
succeed and the others will fail after hitting the check added in 
8aa34d048ab3e459de6d254049f3a31dde42b50d.

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

$ git pull https://github.com/greenqloud/cloudstack pr-volume-sync-fix

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

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

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

This closes #1898


commit c7d174ceee5e87d5423741923e4f87086d042bd5
Author: jeff 
Date:   2017-01-09T16:28:33Z

CLOUDSTACK-8862: Lock on volume before attempting to attach it.

This commit updates the volume orchestration to lock on the volume ID
before sending the attach commands. While there is already validation
for checking if a VM is already attached to an instance in both the
API command and the volume orchestration framework, they aren't of
much use if there's no synchronization.

With this commit, a single volume being attached to multiple instances
will successfully attach to one and return the error added in commit
8aa34d048ab3e459de6d254049f3a31dde42b50d for the other requests.




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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread serg38
Github user serg38 commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
@rhtyd @ustcweizhou We need to enable smoke tests for vmsnaphsots for KVM
in test_vm_snapshots.py 
if cls.hypervisor.lower() in (KVM.lower(), "hyperv", "lxc"):
cls.unsupportedHypervisor = True
return



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


CS Collab Conf Brazil page is down

2017-01-09 Thread Nicolás Vázquez
Hi,

I noticed http://cloudstack.usp.br/ is down, is it another URL for this
event?

Thanks,
Nicolas


Re: CS Collab Conf Brazil page is down

2017-01-09 Thread Will Stevens
I don't think that CCC is actually going to happen.  My understanding is
that they had problems getting the event off the ground.  We have tried to
reach out to them a few times and no one seems to be able to get a response
(or at least a straight answer) about what is going on.

I am assuming that event is not going to happen at this point.

I currently manage http://cloudstackcollab.org/.  Should I remove it from
this site?

*Will STEVENS*
Lead Developer



On Mon, Jan 9, 2017 at 1:52 PM, Nicolás Vázquez 
wrote:

> Hi,
>
> I noticed http://cloudstack.usp.br/ is down, is it another URL for this
> event?
>
> Thanks,
> Nicolas
>


Re: CS Collab Conf Brazil page is down

2017-01-09 Thread Rajesh Ramchandani
If that event is not happening in Brazil, I propose we have on in the bay area?


From: williamstev...@gmail.com  on behalf of Will 
Stevens 
Sent: Monday, January 9, 2017 10:57:29 AM
To: dev@cloudstack.apache.org
Subject: Re: CS Collab Conf Brazil page is down

I don't think that CCC is actually going to happen.  My understanding is
that they had problems getting the event off the ground.  We have tried to
reach out to them a few times and no one seems to be able to get a response
(or at least a straight answer) about what is going on.

I am assuming that event is not going to happen at this point.

I currently manage http://cloudstackcollab.org/.  Should I remove it from
this site?

*Will STEVENS*
Lead Developer



On Mon, Jan 9, 2017 at 1:52 PM, Nicolás Vázquez 
wrote:

> Hi,
>
> I noticed http://cloudstack.usp.br/ is down, is it another URL for this
> event?
>
> Thanks,
> Nicolas
>



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 issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
Trillian test result (tid-772)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 48642 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr977-t772-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermitten failure detected: /marvin/tests/smoke/test_iso.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: 
/marvin/tests/smoke/test_routers_network_ops.py
Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py
Intermitten failure detected: /marvin/tests/smoke/test_templates.py
Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Test completed. 41 look ok, 11 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_04_extract_template | `Failure` | 5.17 | test_templates.py
test_04_rvpc_privategw_static_routes | `Failure` | 340.87 | 
test_privategw_acl.py
test_04_extract_Iso | `Failure` | 5.10 | test_iso.py
ContextSuite context=TestVpcSite2SiteVpn>:setup | `Error` | 0.00 | 
test_vpc_vpn.py
ContextSuite context=TestVpcRemoteAccessVpn>:setup | `Error` | 0.00 | 
test_vpc_vpn.py
ContextSuite context=TestRVPCSite2SiteVpn>:setup | `Error` | 0.00 | 
test_vpc_vpn.py
ContextSuite context=TestVPCRedundancy>:setup | `Error` | 0.00 | 
test_vpc_redundant.py
test_01_create_volume | `Error` | 279.69 | test_volumes.py
ContextSuite context=TestSnapshotRootDisk>:setup | `Error` | 0.00 | 
test_snapshots.py
ContextSuite context=TestInternalLb>:setup | `Error` | 0.00 | 
test_internal_lb.py
test_02_VPC_default_routes | Success | 448.19 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 824.92 | test_vpc_router_nics.py
test_09_delete_detached_volume | Success | 156.67 | test_volumes.py
test_08_resize_volume | Success | 156.47 | test_volumes.py
test_07_resize_fail | Success | 156.46 | test_volumes.py
test_06_download_detached_volume | Success | 156.30 | test_volumes.py
test_05_detach_volume | Success | 241.30 | test_volumes.py
test_04_delete_attached_volume | Success | 146.22 | test_volumes.py
test_03_download_attached_volume | Success | 151.26 | test_volumes.py
test_02_attach_volume | Success | 214.74 | test_volumes.py
test_deploy_vm_multiple | Success | 343.24 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.72 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.26 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 40.98 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.13 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 130.86 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 125.95 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.20 | test_vm_life_cycle.py
test_01_stop_vm | Success | 125.87 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 80.72 | test_templates.py
test_08_list_system_templates | Success | 0.03 | test_templates.py
test_07_list_public_templates | Success | 0.04 | test_templates.py
test_05_template_permissions | Success | 0.05 | test_templates.py
test_03_delete_template | Success | 5.11 | test_templates.py
test_02_edit_template | Success | 90.13 | test_templates.py
test_01_create_template | Success | 40.47 | test_templates.py
test_10_destroy_cpvm | Success | 161.69 | test_ssvm.py
test_09_destroy_ssvm | Success | 168.59 | test_ssvm.py
test_08_reboot_cpvm | Success | 131.76 | test_ssvm.py
test_07_reboot_ssvm | Success | 133.58 | test_ssvm.py
test_06_stop_cpvm | Success | 131.77 | test_ssvm.py
test_05_stop_ssvm | Success | 138.82 | test_ssvm.py
test_04_cpvm_internals | Success | 1.20 | test_ssvm.py
test_03_ssvm_internals | Success | 3.47 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.12 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.13 | test_ssvm.py
test_04_change_offering_small | Success | 239.61 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.08 | test_service_offerings.py
test_01_create_service_offering | Success | 0.11 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.13 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.18 | test_secondary_storage.py
test_09_reboot_router | Success | 35.35 | test_routers.py
test_08_

[GitHub] cloudstack issue #1884: CLOUDSTACK-9699: Add global setting for enable/disab...

2017-01-09 Thread rashmidixit
Github user rashmidixit commented on the issue:

https://github.com/apache/cloudstack/pull/1884
  
Thank you @jburwell. I will incorporate the comments in a couple of days 
and resubmit.


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


[ANNOUNCE][CLOUDSTACK] Apache CloudStack 4.9.2.0

2017-01-09 Thread Rohit Yadav
# Apache CloudStack LTS Maintenance Release 4.9.2.0

The Apache CloudStack project is pleased to announce the release of the
CloudStack 4.9.2.0 as part of its LTS 4.9.x releases. The CloudStack
4.9.2.0 release contains more than 80 bug-fixes and improvements since the
CloudStack 4.9.0 release. The LTS branches are supported for 20 months, and
will receive updates for first 14 months and only security updates in its
last 6 months. The 4.9 LTS branch is supported till 1 June 2018.

Apache CloudStack is an integrated Infrastructure-as-a-Service (IaaS)
software platform that allows users to build feature-rich public and
private cloud environments. CloudStack includes an intuitive user interface
and rich API for managing the compute, networking, software, and storage
resources. The project became an Apache top level project in March, 2013.

More information about Apache CloudStack can be found at:
http://cloudstack.apache.org/

# Documentation

What's new in CloudStack 4.9.2.0:
http://docs.cloudstack.apache.org/projects/cloudstack-release-notes/en/4.9.2.0/about.html

The 4.9.2.0 release notes include a full list of corrected issues, as well
as upgrade instructions from previous versions of Apache CloudStack, and
can be found at:
http://docs.cloudstack.apache.org/projects/cloudstack-release-notes/en/4.9.2.0

The official installation, administration and API documentation for
each release are available on our Documentation Page:
http://docs.cloudstack.apache.org/

# Downloads

The official source code for the 4.9.2.0 release can be downloaded from our
downloads page:
http://cloudstack.apache.org/downloads.html

## Downloads

The official source code release can be downloaded from:

http://cloudstack.apache.org/downloads.html

In addition to the official source code release, individual contributors
have also made convenience binaries available on the Apache CloudStack
download page, and can be found at:

http://www.shapeblue.com/packages/
http://cloudstack.apt-get.eu/ubuntu/dists/ (packages to be published soon)
http://cloudstack.apt-get.eu/centos/6/ (packages to be published soon)
http://cloudstack.apt-get.eu/centos/7/ (packages to be published soon)

-- 
Regards.


Re: [DISCUSS] Moving for JDK8, this time for real!

2017-01-09 Thread Rohit Yadav
All,


Thank you for your feedback.


We've no outstanding objections to the PR, at the time of the 4.10 release 
we'll document the new requirements around JRE1.8 and new systemvmtemplate in 
our installation/upgrade notes.


I'll proceed with the merge later this week.


Regards.


From: Erik Weber 
Sent: 06 January 2017 02:50:36
To: dev
Subject: Re: [DISCUSS] Moving for JDK8, this time for real!

Yes, I understood that your proposal didn't change the debian version, I
suggested that it should do, since when we do update the debian version we
have to test everything over again anyway.

That said, I won't oppose your proposal, I'm +1 of upgrading to jdk 1.8

--
Erik


tor. 5. jan. 2017 kl. 15.46 skrev Rohit Yadav :

> Erik,
>
>
>
>
>
> The PR I've proposed does not change the base Debian distro, it's still
> Debian 7 (wheezy) and not Jessie. I would like to avoid making too many
> changes in the same PR, especially moving to Jessie which won't be straight
> forward as it would introduce several changes, major package version
> changes, systemd usage etc.
>
>
>
>
>
> What I'm proposing is simple -- moving to jdk8 for building CloudStack
> (this works flawlessly now), and using jre-8 on systemvm template.
>
>
>
>
>
> Regards.
>
>
>
> 
>
> From: Erik Weber 
>
> Sent: 05 January 2017 18:25:38
>
> To: dev
>
> Subject: Re: [DISCUSS] Moving for JDK8, this time for real!
>
>
>
>
>
> rohit.ya...@shapeblue.com
>
> www.shapeblue.com
>
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
>
> @shapeblue
>
>
>
>
>
>
>
> On Thu, Jan 5, 2017 at 1:11 PM, Rohit Yadav 
> wrote:
>
> > All,
>
> >
>
> >
>
> > I've been working on moving our codebase and runtime-environment to Java
> 8, and since this is a much needed big change I want your feedback and
> blessings on:
>
> >
>
> > https://github.com/apache/cloudstack/pull/1888
>
> >
>
> >
>
> > In order for this to work, I had to make changes in our veewee-based
> systemvmtemplate build scripts to install a third party opensource
> openjdk-8-jre distribution (xulu-8 is a openjdk-8 distribution from Azul,
> please review and advise wrt their terms of usage here:
> http://www.azul.com/products/zulu/zulu-terms-of-use/), since
>
> >
>
> > Debian 7 (wheezy) repositories don't have openjdk-8-jre and installing
> openjdk-8-jre from testing/sid updates several system packages such as libc
> that I wanted to avoid. Accepting this change on master/4.10 would require
> us to have a new systemvmtemplate for 4.10+ releases.
>
> >
>
> >
>
> > With these changes, I've ran smoke tests across KVM, VMware and
> XenServer in the following PR and all of the tests are passing (ignoring
> known intermittent failures in vpc/rvr related tests).
>
> >
>
> >
>
> > Thoughts, feedback?
>
>
>
> Wheezy has roughly 1 year left (until May 2018) before it is EOL.
>
> Since 4.10 is not LTS and the fact that both Java8 and Debian Jessie
>
> could have hard to find issues I would suggest that you update the
>
> systemvm now, and work through bugs and quirks throught the 4.10.x
>
> version.
>
> I believe there are other changes, like strongswan, as well that could
>
> make 4.10 a potentially unstable version?
>
>
>
> --
>
> Erik
>
>

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



[GitHub] cloudstack issue #1896: [CLOUDSTACK-9732] Update L10N resource files with 4....

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1896
  
LGTM, only text/translation changes. Thanks @milamberspace 
Travis test LGTM, trillian smoke tests are not necessary.


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


[GitHub] cloudstack issue #1888: CLOUDSTACK-9710: Switch to JRE1.8

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1888
  
We've enough LGTMs and test results on this PR. I'll keep this PR open till 
end of the week to receive any other feedback, suggestions and test results. 
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 issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
added a commit to enable the smoke test test_vm_snapshots.py on KVM.
thanks @serg38 
@rhtyd could you please test it again?


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
@ustcweizhou can you squash your changes, 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 issue #1888: CLOUDSTACK-9710: Switch to JRE1.8

2017-01-09 Thread milamberspace
Github user milamberspace commented on the issue:

https://github.com/apache/cloudstack/pull/1888
  
I'm not agree with this PR because it's break the support of Ubuntu 14.04 
with the default JRE (java7) provide by Ubuntu and the support of Ubuntu 16.04 
(java8) the is currently partial for the cloudstack-management.
If you don't know that you need to install a Java 8 via a PPA or manually, 
you cannot run CS 4.10 on Ubuntu 14.04...
I would not block the PR, but I think at least the installation guide must 
be update to mention theses points for Ubuntu 14 (need a PPA or manual 
installation of Java8) and 16 (no support for management)

(Or remove the Ubuntu support 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 issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
squashed and rebased with latest master.
However, this commit still changes the setup/db/db/schema-4920to41000.sql, 
not schema-4930to41000.sql which does not exist. @rhtyd 


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

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


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/977
  
@ustcweizhou we can work on the db schema paths soon; assuming 4.9.3.0 
won't bring in any db changes; we may not change the file names.


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


[GitHub] cloudstack issue #977: [4.10] CLOUDSTACK-8746: VM Snapshotting implementatio...

2017-01-09 Thread blueorangutan
Github user blueorangutan commented on the issue:

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


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


[GitHub] cloudstack issue #1888: CLOUDSTACK-9710: Switch to JRE1.8

2017-01-09 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1888
  
@milamberspace thanks, we can document this in the upgrade notes for Ubuntu 
14.04 users to use the widely known/used PPA.


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